Continuum
  1. Continuum
  2. CONTINUUM-2626

Email Notification not sent when Distributed Build enabled

    Details

    • Type: Bug Bug
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.3.7
    • Fix Version/s: 1.3.8, 1.4.1
    • Labels:
      None
    • Environment:
      Linux, Windows, Jdk 1.6.0_17
    • Complexity:
      Intermediate
    • Patch Submitted:
      Yes
    • Number of attachments :
      4

      Description

      I am using Continuum 1.3.7 and Build Agent 1.3.7. When I am disabling the build agent, the email notifications are being sent when there is any change of state. When I enable distributed build, the build is completing successfully, but no email notification is being sent. The Notifier section under Project Information page is also not showing any of the configured email ids.

      If I add any Project group notifier, the notification is being sent to that email id irrespective of the distributed build state.

      The ciManagement section of the pom.xml file is like below;

      <ciManagement>
      <system>continuum</system>
      <url>http://server.url/continuum</url>
      <notifiers>
      <notifier>
      <type>mail</type>
      <sendOnError>true</sendOnError>
      <sendOnFailure>true</sendOnFailure>
      <sendOnSuccess>true</sendOnSuccess>
      <sendOnWarning>true</sendOnWarning>
      <configuration>
      <address>email.id@server.com</address>
      </configuration>
      </notifier>
      </notifiers>
      </ciManagement>

      Please let me know if this is bug or not. If not, what I need to do to send email notification when distributed build is enabled.

      P.S.: I have tested this with continuum 1.3.4 and 1.4.0 and both in Windows and Linux environment with no luck.

      Thanks,
      Bodhayan.

      1. patch.diff
        1 kB
        Bodhayan Chakraborty
      1. Notifier_gone_after_Build_Ended.png
        39 kB
      2. Notifier_present_Before_Build_Started.png
        32 kB

        Activity

        Hide
        Bodhayan Chakraborty added a comment -

        I am able to fix this issue. Patch for this issue;

        The getProjectNotifiers( List<ProjectNotifier> notifiers ) in AbstractBuildExecutor.java under continuum-buildagent-core has been changed. PFA the patch.diff file and continuum-buildagent-core-1.3.7.jar file.

        Show
        Bodhayan Chakraborty added a comment - I am able to fix this issue. Patch for this issue; The getProjectNotifiers( List<ProjectNotifier> notifiers ) in AbstractBuildExecutor.java under continuum-buildagent-core has been changed. PFA the patch.diff file and continuum-buildagent-core-1.3.7.jar file.
        Hide
        Bodhayan Chakraborty added a comment -

        The continuum-buildagent-core-1.3.7.jar contains patched AbstractBuildExecutor class.

        Show
        Bodhayan Chakraborty added a comment - The continuum-buildagent-core-1.3.7.jar contains patched AbstractBuildExecutor class.
        Hide
        Brett Porter added a comment -

        thanks for the patch!

        Show
        Brett Porter added a comment - thanks for the patch!
        Hide
        Maria Catherine Tan added a comment -

        fixed in r1159490 of 1.3.x branch and r1159492 of trunk.

        • Applied patch of Bodhayan Chakraborty. Thanks!
        • fixed how project notifiers created by users is overwritten by notifiers created by project (POM)
        Show
        Maria Catherine Tan added a comment - fixed in r1159490 of 1.3.x branch and r1159492 of trunk. Applied patch of Bodhayan Chakraborty. Thanks! fixed how project notifiers created by users is overwritten by notifiers created by project (POM)

          People

          • Assignee:
            Maria Catherine Tan
            Reporter:
            Bodhayan Chakraborty
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: