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

AzureDevops PR Decoration - comments in PR are not closed #270

Closed
bambuca opened this issue Nov 12, 2020 · 5 comments
Closed

AzureDevops PR Decoration - comments in PR are not closed #270

bambuca opened this issue Nov 12, 2020 · 5 comments
Labels
bug Something isn't working

Comments

@bambuca
Copy link

bambuca commented Nov 12, 2020

Describe the bug
Some comments remain active in AzureDevops PR, although they are marked as closed in the sonar.
These issues are in sonar twice.

To Reproduce
Steps to reproduce the behavior:
I'm not entirely sure, but I'd say the problem occurs when the pipeline starts in AzureDevops, but before it finish, the programmer make a push with some SQ issues fixes. This cancels the running pipeline and starts a new one. After that the fixed issues are not closed in PR.

  1. Create PR with issues
  2. Sonnar will create comments in PR
  3. Push commit to Azure
  4. Azure trigger PR validation pipeline
  5. Push commit to Azure with fix of some issues (maybe you have to wait until "Prepare Analysis Configuration" finish, but before "Run Code Analysis" task start.
  6. Azure cancels pipeline from step 4 and trigger new one
  7. Issue is closed in sonar but not in Azure

Expected behavior
I expect that corresponding thread in PR will be closed.

Screenshots
If applicable, add screenshots to help explain your problem.

Software Versions

  • SonarQube Version: 8.4.2.36762 in docker
  • Plugin Version: 1.5.0

Additional context
Add any other context about the problem here.

@bambuca bambuca added the bug Something isn't working label Nov 12, 2020
@bambuca
Copy link
Author

bambuca commented Nov 12, 2020

I tried to simulate the error again, but without success. So the problem still exists, but I don't know how to simulate it.
I'll leave the TRACE log enabled and when the problem reappears, I'll check the log and upload it here.

@bambuca
Copy link
Author

bambuca commented Nov 16, 2020

The comments remains active in PR when the source file is removed.
AzureDevops
image

SonarQube:
image

@mc1arke
Copy link
Owner

mc1arke commented Jul 25, 2021

I believe this is fixed by the second commit in #391 . Are you able to test out a snapshot of that build and provide feedback?

@mc1arke
Copy link
Owner

mc1arke commented Aug 14, 2021

Released in v1.9.0 of the plugin

@mc1arke mc1arke closed this as completed Aug 14, 2021
@mc1arke
Copy link
Owner

mc1arke commented Aug 23, 2021

Backported to 1.8.1 of the plugin for Sonaqube 8.9.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants