The issue is that after we updated the sonarqube version we lost all the Azure DevOps integration settings for all (more hundred) SonarQube projects which means that we can’t use the SonarQube/quality gate on our Pull requests anymore in VSTS.
Also in our VSTS builds for Publish Quality Gate Result task we can see the following log:
##[warning]No analyses found in this build! Please check your build configuration.
We are using the 5.13.0 (latest) Azure Devops extension.
sonarqube version: 10.0.0.68432
how is SonarQube deployed: zip
So we want to have those settings filled automatically by SonarQube as it was the case before the update.
A workaround is that we manually set the Azure DevOps integration settings but that’s not really an option since as mentioned we have more hundreds of projects where we should do this setting.
With regards to the Azure DevOps integration details, take a look here.
With regards to this:
That’s definitely odd, as this should be fixed in v5.13 of the extension. Are you sure all your build agents are using the updated extension (nothing being cached)?
Does SonarSube really understand the impact of his on their clients using Azure DevOps ?(?).
We are force to reconfigure 70 (!) projects. I read that we are lucky as there are client that need to take actions on 500 projects.
I already planned some time next week for these 70 projects but now I got the feedback that a lot of decisions that were made previously have to be re-takane again.
For us we are talking about noise generated on our pull request that represent 75K (!N) thousands issue.
Why this decisision was taken is unclear to me but I think it was an optimisation on SonarQube side to make their lives easier.
Sorry, excuse me.
This is adressed by an update to the release notes. Are we an edge case? I don’t understand what you are doing SonarQube.
I agree with you. Recently we had some discussions on this toipc and I’m happy to tell you that we will be reenabling the old parameters before the next LTA version of SonarQube (scheduled early Q1) while we find a better solution to handle binding, without a lot of manual configuration.
Well, you are welcome to configure the 70 projects so that you can continue working as expected. Until we reenable those parameters, there’s no other solution for v10.x versions.
I think we can consider that these issues are completely unrelated.
I am going to split off this other issue into a new thread.
To the others who are upgrading
These were the changes necessary to get our integration with the PR timeline going again
(We still have an open issue (129673) with scannermode=cli tasks)
Explicitly setting the sonar.pullrequest properties were never necessary when using the Azure DevOps integration (they were auto-configured behind the scenes). For now, they’ll be ignored, but by the next LTA, they will be working again. You should not need to explicitly set them at that point.
So on my end, right now, nothing that needs to be explicitly documented other than the already referenced [upgrade note](Release upgrade notes].
Explicitly setting the sonar.pullrequest properties were never necessary when using the Azure DevOps integration (they were auto-configured behind the scenes).
For now, they’ll be ignored, but by the next LTA, they will be working again. You should not need to explicitly set them at that point.
Ok, but something isn’t clear to me. So we now have to set repo and project information on the SonarQube server since V9>V10 (we 70, other 500 manual changes).
But the sonar.pullrequest.* properties were always ignored by the Azure DevOps integration?
Not so right, as in V9 information was send to SQ Server so we didn’t have to manually configure it there (as we do now)
This is just for clarification as I may be somehting missing. This topic can be closed as the problem at hand is solved and we are on @7 for scannermode=datnet
(the other isse scannermode=cli is still open)
These properties are getting auto-configured (see here), and in SonarQUbe v9.9 that are being used to execute the PR decoration. Now they are still getting set, but they are being ignored by SonarQube 10. We will stop ignoring them (revert the behavior) for the next LTA version.
This is all unrelated to the change in task version.