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

For a maven 2 project, send mail to those defined in the notifiers field in the ciManagement section in the pom

    • Icon: Improvement Improvement
    • Resolution: Fixed
    • Icon: Major Major
    • maven-plugin
    • None
    • Platform: All, OS: All

      For a maven 2 project, it would be very nice if hudson parsed the ciManagement
      section in the pom, and sent mail to those defined in the notifier section on
      failure.

      This is a very nice functionality to have using maven, with many modules and
      many developers to avoid "spam".

          [JENKINS-1201] For a maven 2 project, send mail to those defined in the notifiers field in the ciManagement section in the pom

          How would Hudson handle the conflicting rules of email notification between the
          parent pom.xml and the child modules pom.xml. Are the settings merged between
          the parent and child for the child module?

          If settings are just in the parent pom.xml I would hope they would apply to all
          the child pom.xmls.

          If the user edits the email information in the hudson form then the pom.xml is
          updated with new information will the new email information in the pom.xml
          overwrite the users configuration applied in the form?

          headofyellowhue added a comment - How would Hudson handle the conflicting rules of email notification between the parent pom.xml and the child modules pom.xml. Are the settings merged between the parent and child for the child module? If settings are just in the parent pom.xml I would hope they would apply to all the child pom.xmls. If the user edits the email information in the hudson form then the pom.xml is updated with new information will the new email information in the pom.xml overwrite the users configuration applied in the form?

          Kohsuke Kawaguchi added a comment - Recent recurrence of this at users@hudson: http://www.nabble.com/ciManagement-notifications-for-a-maven-2-build-tp15006168p15006168.html

          kenliu added a comment -

          add cc

          kenliu added a comment - add cc

          Jesse Glick added a comment -

          Just adding reference URL.

          Jesse Glick added a comment - Just adding reference URL.

          kenliu added a comment -

          assigning to self to work on this issue.

          kenliu added a comment - assigning to self to work on this issue.

          kutzi added a comment -

          kenliu are you still working on this?

          kutzi added a comment - kenliu are you still working on this?

          kenliu added a comment -

          not actively, no - someone else can volunteer to take it over

          kenliu added a comment - not actively, no - someone else can volunteer to take it over

          Olamy, are you still working on this issue? I'm interested in helping.

          Marcelo Rebasti added a comment - Olamy, are you still working on this issue? I'm interested in helping.

          Marcelo Rebasti added a comment - - edited

          Hi, I create the pull request https://github.com/jenkinsci/jenkins/pull/950 for this issue. Any comments are welcome!

          Marcelo Rebasti added a comment - - edited Hi, I create the pull request https://github.com/jenkinsci/jenkins/pull/950 for this issue. Any comments are welcome!

          Code changed in jenkins
          User: Marcelo Rebasti
          Path:
          src/main/java/hudson/maven/MavenModule.java
          src/main/java/hudson/maven/MavenModuleSetBuild.java
          src/main/java/hudson/maven/reporters/MavenMailer.java
          src/test/java/hudson/maven/MavenModuleTest.java
          src/test/java/hudson/maven/reporters/MavenMailerTest.java
          src/test/resources/hudson/maven/JENKINS-1201-module-defined.zip
          src/test/resources/hudson/maven/JENKINS-1201-parent-defined.zip
          http://jenkins-ci.org/commit/maven-plugin/2f924d838f4ad49334531950cb3e7187ed147b69
          Log:
          Support for ciManagement maven notifications.

          In order to enable ciManagement maven notifications you have to check the
          option "E-mail Notification" into the section "Build Settings" from the
          job, and add a ciManagement section to the pom.xml with the following
          settings:

          ciManagement

          – system=hudson (you have to put "hudson" here )
          – ur=http://serverurl/
          – notifiers
          ----- notifier
          -------- type=mail (you have to put "mail" here)
          -------- sendOnError=true
          -------- sendOnFailure=true
          -------- sendOnSuccess>false
          -------- sendOnWarning>false
          -------- configuration
          ------------ recipients=some.email@domain.org (recipients have the same
          format than the same field in the Job configuration page)

          You can override this configuration in any module of the maven proyect.

          [FIXED JENKINS-1201]
          [FIXED JENKINS-6421]

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Marcelo Rebasti Path: src/main/java/hudson/maven/MavenModule.java src/main/java/hudson/maven/MavenModuleSetBuild.java src/main/java/hudson/maven/reporters/MavenMailer.java src/test/java/hudson/maven/MavenModuleTest.java src/test/java/hudson/maven/reporters/MavenMailerTest.java src/test/resources/hudson/maven/ JENKINS-1201 -module-defined.zip src/test/resources/hudson/maven/ JENKINS-1201 -parent-defined.zip http://jenkins-ci.org/commit/maven-plugin/2f924d838f4ad49334531950cb3e7187ed147b69 Log: Support for ciManagement maven notifications. In order to enable ciManagement maven notifications you have to check the option "E-mail Notification" into the section "Build Settings" from the job, and add a ciManagement section to the pom.xml with the following settings: ciManagement – system=hudson (you have to put "hudson" here ) – ur= http://serverurl/ – notifiers ----- notifier -------- type=mail (you have to put "mail" here) -------- sendOnError=true -------- sendOnFailure=true -------- sendOnSuccess>false -------- sendOnWarning>false -------- configuration ------------ recipients=some.email@domain.org (recipients have the same format than the same field in the Job configuration page) You can override this configuration in any module of the maven proyect. [FIXED JENKINS-1201] [FIXED JENKINS-6421]

          Code changed in jenkins
          User: Olivier Lamy
          Path:
          src/main/java/hudson/maven/MavenModule.java
          src/main/java/hudson/maven/MavenModuleSetBuild.java
          src/main/java/hudson/maven/reporters/MavenMailer.java
          src/test/java/hudson/maven/MavenModuleTest.java
          src/test/java/hudson/maven/reporters/MavenMailerTest.java
          src/test/resources/hudson/maven/JENKINS-1201-module-defined.zip
          src/test/resources/hudson/maven/JENKINS-1201-parent-defined.zip
          http://jenkins-ci.org/commit/maven-plugin/605614f6d73920642ed6f4cee945cd026724203a
          Log:
          Merge pull request #1 from mrebasti/jenkins-1201

          [FIXED JENKINS-1201] Support for ciManagement maven notifications.
          Thanks

          Compare: https://github.com/jenkinsci/maven-plugin/compare/0a5edb60d3fb...605614f6d739

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Olivier Lamy Path: src/main/java/hudson/maven/MavenModule.java src/main/java/hudson/maven/MavenModuleSetBuild.java src/main/java/hudson/maven/reporters/MavenMailer.java src/test/java/hudson/maven/MavenModuleTest.java src/test/java/hudson/maven/reporters/MavenMailerTest.java src/test/resources/hudson/maven/ JENKINS-1201 -module-defined.zip src/test/resources/hudson/maven/ JENKINS-1201 -parent-defined.zip http://jenkins-ci.org/commit/maven-plugin/605614f6d73920642ed6f4cee945cd026724203a Log: Merge pull request #1 from mrebasti/jenkins-1201 [FIXED JENKINS-1201] Support for ciManagement maven notifications. Thanks Compare: https://github.com/jenkinsci/maven-plugin/compare/0a5edb60d3fb...605614f6d739

            olamy Olivier Lamy
            rohnny rohnny
            Votes:
            26 Vote for this issue
            Watchers:
            26 Start watching this issue

              Created:
              Updated:
              Resolved: