Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Spike Sonarqube 9.9 issues #1348

Closed
kford-newrelic opened this issue Jun 22, 2023 · 3 comments · Fixed by #1522
Closed

Spike Sonarqube 9.9 issues #1348

kford-newrelic opened this issue Jun 22, 2023 · 3 comments · Fixed by #1522
Assignees
Labels
2 Story Point Estimate

Comments

@kford-newrelic
Copy link
Contributor

kford-newrelic commented Jun 22, 2023

Description

Investigate why Sonarqube 9.9 won't start up unless agent is run in lite_mode (further discussion on Sonarqube site here

  • They are able to run our Java APM agent on SonarQube version 9.7 with no problem
  • They upgraded their SonarQube version to 9.9 which makes use of Java 17 and our APM agent doesn’t function properly unless they run it in ‘lite_mode’
  • The ask is that we investigate why / what issues we are having with running our agent in normal mode

Acceptance Criteria

Agent is able to properly run with Sonarqube 9.9 and above.

Design Consideration/Limitations

N/A

Dependencies

N/A

Additional context

Support case and it looks like SonarQube isn't starting up (causes the app to crash)

@kford-newrelic kford-newrelic added the 2 Story Point Estimate label Jun 22, 2023
@workato-integration
Copy link

@kanderson250 kanderson250 self-assigned this Sep 8, 2023
@kanderson250
Copy link
Contributor

Verified that SQ Server 9.7 runs with the agent and SQ Server 9.9 breaks.

This was fixed by adding the following classes to excludes:

java/util/AbstractList$RandomAccessSpliterator
java/util/stream/MatchOps$MatchOp
java/util/stream/MatchOps$BooleanTerminalSink
javax/security/auth/Subject$SecureSet$1
  • These were not added to the default agent excludes.
  • The exclusions were tested against our PR suite with all tests passing.

@kanderson250
Copy link
Contributor

Update: The classes above were added to the default excludes file for the agent, so they will be excluded by default on the next release.

Until the next release, customers can manually exclude the classes in their newrelic.yml file:

class_transformer:
    excludes: 
      ^java/util/AbstractList\$RandomAccessSpliterator, 
      ^java/util/stream/MatchOps\$MatchOp,
      ^java/util/stream/MatchOps\$BooleanTerminalSink, 
      ^javax/security/auth/Subject\$SecureSet\$1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2 Story Point Estimate
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants