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

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

XMLWordPrintable

      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!

          [JENKINS-20319] Configuration slicing resets send separate emails... for Email notification

          Torbjörn Axelsson created issue -
          Torbjörn Axelsson made changes -
          Summary Original: Configuration slicing resets """ for Email notification New: Configuration slicing resets send separate emails... for Email notification

          S.B. added a comment -

          I don't think this is a Minor bug.

          S.B. added a comment - I don't think this is a Minor bug.

          Martin Kutter added a comment -

          Pull request created: https://github.com/jenkinsci/configurationslicing-plugin/pull/14

          Martin Kutter added a comment - Pull request created: https://github.com/jenkinsci/configurationslicing-plugin/pull/14

          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.

          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.

          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

          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

          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

          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

          Ninian added a comment -

          fixed in v1.41

          Ninian added a comment - fixed in v1.41
          Ninian made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: Open [ 1 ] New: Resolved [ 5 ]

          Torbjörn Axelsson added a comment -

          Thanks.

          Torbjörn Axelsson added a comment - Thanks.
          Torbjörn Axelsson made changes -
          Status Original: Resolved [ 5 ] New: Closed [ 6 ]

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

              Created:
              Updated:
              Resolved: