Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-31013

Add parameter to do not notify developer when posting comments

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      As soon as comments are sent as separate review from Gerrit Trigger verification, Gerrit sends more mails then probably necessary. Would be great if plugin could be run in silence mode and no additional mails will come.

      I one project I have three parallel sonar review of different components, and as result I have 3 mails from 'sonar' review and one from gerrit trigger review. It's too much spam.

      -My proposal is checkbox 'Keep silence' (like in main Gerrit trigger). Checkbox must be checked by default. Help text is 'In case if checked, notification mails about review and sonar violations would not be sent by Gerrit'. -

      Add 2 fields:
      1) Who will be notified if some Sonar violations are found (Owner by default)
      2) Who will be notified if there are no Sonar violations found (None by default)

        Attachments

          Activity

          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Tatiana Didik
          Path:
          src/main/java/org/jenkinsci/plugins/sonargerrit/SonarToGerritBuilder.java
          src/main/resources/messages.properties
          src/main/resources/org/jenkinsci/plugins/sonargerrit/SonarToGerritBuilder/config.jelly
          src/main/resources/org/jenkinsci/plugins/sonargerrit/SonarToGerritBuilder/config.properties
          src/main/resources/org/jenkinsci/plugins/sonargerrit/SonarToGerritBuilder/help-issuesNotification.html
          src/main/resources/org/jenkinsci/plugins/sonargerrit/SonarToGerritBuilder/help-noIssuesNotification.html
          src/test/java/org/jenkinsci/plugins/sonargerrit/SonarToGerritBuilderTest.java
          http://jenkins-ci.org/commit/sonar-gerrit-plugin/f9ad3a18fdcb6e38324f477e7e55bd4e09aaf1f1
          Log:
          JENKINS-31013 Notification settings

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Tatiana Didik Path: src/main/java/org/jenkinsci/plugins/sonargerrit/SonarToGerritBuilder.java src/main/resources/messages.properties src/main/resources/org/jenkinsci/plugins/sonargerrit/SonarToGerritBuilder/config.jelly src/main/resources/org/jenkinsci/plugins/sonargerrit/SonarToGerritBuilder/config.properties src/main/resources/org/jenkinsci/plugins/sonargerrit/SonarToGerritBuilder/help-issuesNotification.html src/main/resources/org/jenkinsci/plugins/sonargerrit/SonarToGerritBuilder/help-noIssuesNotification.html src/test/java/org/jenkinsci/plugins/sonargerrit/SonarToGerritBuilderTest.java http://jenkins-ci.org/commit/sonar-gerrit-plugin/f9ad3a18fdcb6e38324f477e7e55bd4e09aaf1f1 Log: JENKINS-31013 Notification settings
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Tatiana Didik
          Path:
          src/main/java/org/jenkinsci/plugins/sonargerrit/SonarToGerritBuilder.java
          src/main/resources/messages.properties
          src/main/resources/org/jenkinsci/plugins/sonargerrit/SonarToGerritBuilder/config.jelly
          src/main/resources/org/jenkinsci/plugins/sonargerrit/SonarToGerritBuilder/config.properties
          src/main/resources/org/jenkinsci/plugins/sonargerrit/SonarToGerritBuilder/help-issuesNotification.html
          src/main/resources/org/jenkinsci/plugins/sonargerrit/SonarToGerritBuilder/help-noIssuesNotification.html
          src/test/java/org/jenkinsci/plugins/sonargerrit/SonarToGerritBuilderTest.java
          http://jenkins-ci.org/commit/sonar-gerrit-plugin/40a6624ccb426f5bc98db597717d6ef7b5426840
          Log:
          Merge with JENKINS-31013

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Tatiana Didik Path: src/main/java/org/jenkinsci/plugins/sonargerrit/SonarToGerritBuilder.java src/main/resources/messages.properties src/main/resources/org/jenkinsci/plugins/sonargerrit/SonarToGerritBuilder/config.jelly src/main/resources/org/jenkinsci/plugins/sonargerrit/SonarToGerritBuilder/config.properties src/main/resources/org/jenkinsci/plugins/sonargerrit/SonarToGerritBuilder/help-issuesNotification.html src/main/resources/org/jenkinsci/plugins/sonargerrit/SonarToGerritBuilder/help-noIssuesNotification.html src/test/java/org/jenkinsci/plugins/sonargerrit/SonarToGerritBuilderTest.java http://jenkins-ci.org/commit/sonar-gerrit-plugin/40a6624ccb426f5bc98db597717d6ef7b5426840 Log: Merge with JENKINS-31013
          Hide
          aquarellian Tatiana Didik added a comment -

          1.0.4

          Show
          aquarellian Tatiana Didik added a comment - 1.0.4

            People

            Assignee:
            aquarellian Tatiana Didik
            Reporter:
            adidik Aleksey Didik
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved: