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

Configuration slicing resets send separate emails... for Email notification

    Details

    • Similar Issues:

      Description

      When using configuration slicing to configure "Email notification" (default mailer) it only uses the recipients list.

      We are using an empty recipient list with "Send separate e-mails to individuals who broke the build" active.

      Configuration slicing will disable "Email notification" if the recipient list is empty.

      Support for the two parameters "Send separate e-mails to individuals who broke the build" and "Send email for every unstable build" is needed to use configuration slicing on this option.

      Thanks for an otherwise very handy and time saving plugin!

        Attachments

          Activity

          torax_68 Torbjörn Axelsson created issue -
          torax_68 Torbjörn Axelsson made changes -
          Field Original Value New Value
          Summary Configuration slicing resets """ for Email notification Configuration slicing resets send separate emails... for Email notification
          Hide
          steboe S.B. added a comment -

          I don't think this is a Minor bug.

          Show
          steboe S.B. added a comment - I don't think this is a Minor bug.
          Show
          martinkutter Martin Kutter added a comment - Pull request created: https://github.com/jenkinsci/configurationslicing-plugin/pull/14
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: unknown
          Path:
          src/main/java/configurationslicing/email/AbstractEmailSliceSpec.java
          src/main/java/configurationslicing/email/CoreEmailSlicer.java
          src/main/java/configurationslicing/email/ExtEmailSlicer.java
          src/main/java/configurationslicing/email/ProjectHandler.java
          src/test/java/configurationslicing/EmailSlicerTest.java
          http://jenkins-ci.org/commit/configurationslicing-plugin/49f3ca78926d9cf3b3f74978655f614f27a3e2d7
          Log:
          Partial fix for JENKINS-20319: Configuration slicing resets send separate emails... for Email notification
          The configurationslicing plugin now only disables core mailers, if

          • the recipient list is set to empty, and the notify individuals... checkbox is unchecked.
          • the recipient list is explicitly set to "(Disabled)" (case-insensitive) via the configuration slicer.

          Email notifications using the email-ext plugin still suffer from this error.

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: unknown Path: src/main/java/configurationslicing/email/AbstractEmailSliceSpec.java src/main/java/configurationslicing/email/CoreEmailSlicer.java src/main/java/configurationslicing/email/ExtEmailSlicer.java src/main/java/configurationslicing/email/ProjectHandler.java src/test/java/configurationslicing/EmailSlicerTest.java http://jenkins-ci.org/commit/configurationslicing-plugin/49f3ca78926d9cf3b3f74978655f614f27a3e2d7 Log: Partial fix for JENKINS-20319 : Configuration slicing resets send separate emails... for Email notification The configurationslicing plugin now only disables core mailers, if the recipient list is set to empty, and the notify individuals... checkbox is unchecked. the recipient list is explicitly set to "(Disabled)" (case-insensitive) via the configuration slicer. Email notifications using the email-ext plugin still suffer from this error.
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: mkutter
          Path:
          src/main/java/configurationslicing/email/ExtEmailSlicer.java
          src/test/java/configurationslicing/EmailSlicerTest.java
          http://jenkins-ci.org/commit/configurationslicing-plugin/2097bff8152a1753b0ba5c7835a0d0a81159d2e4
          Log:
          Fixes JENKINS-20319 for email-ext plugin

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: mkutter Path: src/main/java/configurationslicing/email/ExtEmailSlicer.java src/test/java/configurationslicing/EmailSlicerTest.java http://jenkins-ci.org/commit/configurationslicing-plugin/2097bff8152a1753b0ba5c7835a0d0a81159d2e4 Log: Fixes JENKINS-20319 for email-ext plugin
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: ninian
          Path:
          pom.xml
          src/main/java/configurationslicing/email/AbstractEmailSliceSpec.java
          src/main/java/configurationslicing/email/CoreEmailSlicer.java
          src/main/java/configurationslicing/email/ExtEmailSlicer.java
          src/main/java/configurationslicing/email/ProjectHandler.java
          src/test/java/configurationslicing/EmailSlicerTest.java
          http://jenkins-ci.org/commit/configurationslicing-plugin/c0d1e56794acb0de5928c56a1b3c3a6606df89ca
          Log:
          Merge pull request #14 from mkutter/master

          Fix for JENKINS-20319

          Compare: https://github.com/jenkinsci/configurationslicing-plugin/compare/cf502530591d...c0d1e56794ac

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: ninian Path: pom.xml src/main/java/configurationslicing/email/AbstractEmailSliceSpec.java src/main/java/configurationslicing/email/CoreEmailSlicer.java src/main/java/configurationslicing/email/ExtEmailSlicer.java src/main/java/configurationslicing/email/ProjectHandler.java src/test/java/configurationslicing/EmailSlicerTest.java http://jenkins-ci.org/commit/configurationslicing-plugin/c0d1e56794acb0de5928c56a1b3c3a6606df89ca Log: Merge pull request #14 from mkutter/master Fix for JENKINS-20319 Compare: https://github.com/jenkinsci/configurationslicing-plugin/compare/cf502530591d...c0d1e56794ac
          Hide
          ninian Ninian added a comment -

          fixed in v1.41

          Show
          ninian Ninian added a comment - fixed in v1.41
          ninian Ninian made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Hide
          torax_68 Torbjörn Axelsson added a comment -

          Thanks.

          Show
          torax_68 Torbjörn Axelsson added a comment - Thanks.
          torax_68 Torbjörn Axelsson made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          rtyler R. Tyler Croy made changes -
          Workflow JNJira [ 151843 ] JNJira + In-Review [ 207094 ]

            People

            • Assignee:
              mdonohue mdonohue
              Reporter:
              torax_68 Torbjörn Axelsson
            • Votes:
              1 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: