sonar-gerrit-plugin icon indicating copy to clipboard operation
sonar-gerrit-plugin copied to clipboard

Undetected issue on new code - change gerrit

Open finitha opened this issue 6 years ago • 8 comments

Hi,

I have a problem with the parameter GERRIT_COMMENT_NEW_ISSUES_ONLY set to true.

If I change some lines in a file, sometimes sonar reports +1 to Gerrit change. But it should be -1, because it breaks rules.

For exemple if I set the parameter to "false", sonar will report -1 and list all the issues.

I checked in SonarQube and there aren't new modifications stored in database.

Do you know why ?

Thanks

finitha avatar May 28 '19 13:05 finitha

Hi,

Even new issues could be squelched by the threshold settings. Are these new issues over the threshold ?

mildis avatar May 29 '19 06:05 mildis

Thanks for your response.

New issues are "MAJOR" and the parameter Key: GERRIT_THRESHOLD is INFO. So the new issues are over the threshold.


Apparemment, vous êtes Français (ou du moins votre entreprise), comme moi. Cela sera peut etre plus simple pour répondre ?

finitha avatar May 29 '19 07:05 finitha

Bonjour,

Pourriez vous nous donner la version SonarQube, la version du plugin que vous avez déployé ? Pourriez vous également activer le mode verbeux si vous ne l'avez pas fait et nous mettre à disposition le log qui a été généré ?

Merci,

greg7859 avatar May 29 '19 07:05 greg7859

Bonjour,

Pour SonarQube il s'agit de la version 6.5 et pour le plugin : sonar-gerrit-plugin-2.3.0.jar Le mode verbeux dans SonarQube directement ?

Merci

finitha avatar May 29 '19 07:05 finitha

Le mode verbeux s'active avec l'option sonar.verbose=true.

Elle doit être insérée dans la ligne qui exécute l'analyse ou dans le fichier de configuration en fonction du langage que vous analysez.

greg7859 avatar May 31 '19 07:05 greg7859

D'accord merci, je vais test avec cette option et je vous envoie le résultat.

finitha avatar Jun 03 '19 07:06 finitha

Il faut ajouter -X aussi pour déclencher le mode debug. sonar.verbose=true en ajoute une couche mais ne déclare pas l'execution en debug.

mildis avatar Jun 03 '19 13:06 mildis

Vus les logs, la liste de problèmes dans SQ est vide. L'analyse ayant lieu coté SQ, c'est directement le moteur qui nous retourne en interne cette liste vide. C'est SonarLint qui vous remonte un problème dans le code changé qui devrait aussi être remonté par SQ ?

mildis avatar Jun 06 '19 17:06 mildis