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

    Details

    • Type: Improvement
    • Status: Resolved (View Workflow)
    • Priority: Major
    • Resolution: Fixed
    • Component/s: maven-plugin
    • Labels:
      None
    • Environment:
      Platform: All, OS: All
    • Similar Issues:

      Description

      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".

        Attachments

          Issue Links

            Activity

            rohnny rohnny created issue -
            Hide
            headofyellowhue 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?

            Show
            headofyellowhue 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?
            Show
            kohsuke 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
            Hide
            kenliu kenliu added a comment -

            add cc

            Show
            kenliu kenliu added a comment - add cc
            Hide
            jglick Jesse Glick added a comment -

            Just adding reference URL.

            Show
            jglick Jesse Glick added a comment - Just adding reference URL.
            Hide
            kenliu kenliu added a comment -

            assigning to self to work on this issue.

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

            kenliu are you still working on this?

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

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

            Show
            kenliu kenliu added a comment - not actively, no - someone else can volunteer to take it over
            kutzi kutzi made changes -
            Field Original Value New Value
            Assignee kenliu [ kenliu ]
            olamy Olivier Lamy made changes -
            Assignee olamy [ olamy ]
            kutzi kutzi made changes -
            Link This issue is related to JENKINS-1763 [ JENKINS-1763 ]
            evernat evernat made changes -
            Link This issue is duplicated by JENKINS-6421 [ JENKINS-6421 ]
            Hide
            mrebasti Marcelo Rebasti added a comment -

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

            Show
            mrebasti Marcelo Rebasti added a comment - Olamy, are you still working on this issue? I'm interested in helping.
            Hide
            mrebasti 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!

            Show
            mrebasti 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!
            Hide
            scm_issue_link 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]

            Show
            scm_issue_link 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]
            Hide
            scm_issue_link 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

            Show
            scm_issue_link 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
            scm_issue_link SCM/JIRA link daemon made changes -
            Status Open [ 1 ] Resolved [ 5 ]
            Resolution Fixed [ 1 ]
            jglick Jesse Glick made changes -
            Link This issue depends on JENKINS-20209 [ JENKINS-20209 ]
            ircbot Jenkins IRC Bot made changes -
            Component/s maven-plugin [ 16033 ]
            Component/s maven2 [ 15487 ]
            rtyler R. Tyler Croy made changes -
            Workflow JNJira [ 131274 ] JNJira + In-Review [ 186439 ]

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: