-
Notifications
You must be signed in to change notification settings - Fork 5
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
on arm64 run local sonarqube with latest LTS 9 #712
base: master
Are you sure you want to change the base?
Conversation
If this works well locally compared to the 9.7 image, I would suggest to merge this in. My understanding of the code change is that it would only change the version for arm64, not on CI. So I do not see an issue with being slightly ahead in terms of versions :) |
For CI the PR updates SONAR_SCANNER_VERSION from 4.6.2.* to 4.8.0.* and CNES_REPORT_VERSION from 4.0.0 to 4.2.0I. While ODS is still at 8.9. From https://community.sonarsource.com/t/sonarscanner-sonarqube-server-compatibility/85702/2 So it seems like it is low risk. I will finalize the PR then |
f3c74ca
to
8d967e6
Compare
Note that my claim that it works referred to run-sonarqube.sh succeeding. |
8d967e6
to
35e2ed8
Compare
With version 9.8 the default branch was changed to main so that this now fails with:
One possibility would be if one could configure this so that one can revert back, but at least using sonar.properties this appears not to be supported: Setting default main branch name in sonar.properties |
Closes #711
For the image used in CI the version should be aligned with ODS (see opendevstack/ods-core#1211.)
Tasks:
docs/design
directory or not applicabledocs
directory or not applicablemake test
) or not applicable