-
-
Notifications
You must be signed in to change notification settings - Fork 528
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
[QUESTION] Any experience with 10.5 yet? #900
Comments
I've not properly tested the plugin against 10.5 yet. The release is binary compatible with 10.4 for the classes we depend on so the plugin compiles and starts with the UI showing branches, but I've not tried running any scans yet. |
just saw this on the upgrade notes for 10.5. not sure if this has any impact on this plugin. https://docs.sonarsource.com/sonarqube/latest/setup-and-upgrade/release-upgrade-notes/ |
we upgraded now and so far everything seems to work as expected. we are using GitHub only. |
I ran into an incompatiblity when setting up a the 'DevOps Platform Integration' on a newly created SonarQube project. This is the exception:
Full stacktrace
I guess the cause is commit 99bf48a for Line 58 in 3353612
However, I just recompiled the 1.19.0 tag of this plugin but referenced sonarqube 10.5 in
This solved the issue. |
Hi, are you using 'DevOps Platform Integration' feature ? Need to upgrade to 10.5 to fix https://sonarsource.atlassian.net/browse/SONAR-21600, but as we are heavily relying on this feature (which is not easy to test on a sandbox platform) i would prefer to confirm this point... Change #904 made sense, but got rejected so situation is not clear |
I tested this on our installation. Same behavior. Looks like SonarQube started adding this headers. |
Might be related to https://sonarsource.atlassian.net/browse/SONAR-21780 Can probably we solved using https://github.com/mc1arke/sonarqube-community-branch-plugin?tab=readme-ov-file#serving-images-for-pr-decoration |
Good spot. Maybe they will address this issue with https://sonarsource.atlassian.net/browse/SONAR-22149 |
Had the same error with 10.5 helm install with the 1.19.0 plugin. It wasn't until I added a new project that I realized it couldn't be linked in the DevOps Platform Integration settings. |
Same error. Falling when trying update settings on existing project. Sonar 10.5.1, plugin 1.19.0 |
For people having similar issue, can you test version available in #904 (comment) and provide feedback? |
I'm having the issue:
and
Per my investigations, i think the problem is related to: jgit PS: Although there have been some errors, but still the PR decoration still works fine. |
Hello @azun-c
For the |
Thanks @gjabouley-invn |
now that 10.5 is out.
https://www.sonarsource.com/products/sonarqube/whats-new/sonarqube-10-5/
anyone tried the current plugin with 10.5 yet? did it work?
The text was updated successfully, but these errors were encountered: