We see a major regression after updating from 7.9.1 LTS to 8.2
We use Sonarqube to analyze the master branch, feature branches derived from it and pull requests.
With 8.2 there is the problem that sonar issues that are introduced and closed on master appear in the feature branch analysis when the feature branch merges the actual master.
Same for Security Hotspot review.
Also the other direction does not work.
Closing Security Hotspot review in feature branch, but they reappear on master after merging.
This kills our whole branching workflow
1 Like
Colin
(Colin)
March 12, 2020, 10:15am
2
I’ve moved your post to a new topic.
ganncamp
(G Ann Campbell)
March 12, 2020, 12:51pm
3
Hi,
I’m confused. Could you give some more detail about the process here?
Ann
Here is the procedure we follow:
Development on new code in feature branch feature/A that introduces sonar issues
Run sonar analysis on feature/A and close issues in sonarqube
Create Pull Request and run sonar analysis on pull request merged with master
Problem1: Closed issues reappear in sonarqube, but this is the case with every sonarqube version.
Close issues again on PR and merge to master
Problem2: Issues reappear on master
Close issues on master again
Developer merges master into other feature branch feature/B
Problem3: Run sonar analysis on feature/B and issues appear again
ganncamp
(G Ann Campbell)
March 16, 2020, 1:54pm
6
Hi,
reitzmichnicht:
Close issues again on PR and merge to master
Problem2: Issues reappear on master
Close issues on master again
Developer merges master into other feature branch feature/B
Problem3: Run sonar analysis on feature/B and issues appear again
When you “Close” in #5 , I guess we’re talking about FP/WF?
And in #8 , is DeveloperB touching the same code that DeveloperA touched?
Ann
Hi
in 5# yes they are closed as False Positive or Wont Fix.
in 8# DeveloperB is not touching the same code but gets the issues from DeveloperA again.
Kind regards,
Michael
1 Like
Please let me know if you need further input for reproducing the issue. This is really a show stopper actually
1 Like