Continuum
  1. Continuum
  2. CONTINUUM-2102

Unable to configure subject line for email notifications

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: 1.3.1 (Alpha)
    • Fix Version/s: 1.3.2 (Beta)
    • Component/s: Notifier - Mail
    • Labels:
      None
    • Environment:
      1.3.1, also seen with trunk at r748719

      Description

      After upgrading to 1.3.x, I'm getting notification emails with the subject line:

      [continuum] PREPARE BUILD SUCCESSFUL: $

      {projectScmRoot.scmRootAddress}

      Other than the subject, the notification looks the same as we used to get for normal scheduled builds in 1.2.x.

      The subject line seems to be hard coded in MailContinuumNotifier.java, and was added in r709300 in the merge from the transient state branch.

      Previously, the subject was configurable in
      ./apps/continuum/WEB-INF/classes/META-INF/plexus/application.xml
      with
      <subject-format>[continuum] BUILD ${state}: ${project.projectGroup.name} - ${project.name} - ${build.buildDefinition.description}</subject-format>
      .

      From continuum.log:
      2009-02-27 20:25:48,106 [pool-1-thread-4] WARN velocityComponent - org.apache.velocity.runtime.exception.ReferenceException: reference : template = subjectPattern [line 1,column 37] : ${projectScmRoot.scmRootAddress}

      is not a valid reference.
      2009-02-27 20:25:48,106 [pool-1-thread-4] INFO org.apache.maven.continuum.notification.mail.MailContinuumNotifier - Message Subject: '[continuum] PREPARE BUILD SUCCESSFUL: $

      {projectScmRoot.scmRootAddress}

      '.

      I'm less concerned about the 'not a valid reference' warning, since what I really want is for the subject line to be

      • configurable in plain text
      • the same as it was in 1.2.x

      Related thread:
      http://mail-archives.apache.org/mod_mbox/continuum-dev/200902.mbox/%3Cadba96190902261025o5f0f35cfk70554c62cafda630@mail.gmail.com%3E

        Issue Links

          Activity

          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Closed Closed
          7d 11h 47m 1 Maria Catherine Tan 07/Mar/09 09:26
          Mark Thomas made changes -
          Workflow jira [ 12948497 ] Default workflow, editable Closed status [ 12984378 ]
          Mark Thomas made changes -
          Project Import Sun Apr 05 21:12:18 UTC 2015 [ 1428268338676 ]
          Mark Thomas made changes -
          Workflow jira [ 12710754 ] Default workflow, editable Closed status [ 12740422 ]
          Mark Thomas made changes -
          Project Import Sun Apr 05 08:36:01 UTC 2015 [ 1428222961749 ]
          Hide
          Wendy Smoak added a comment -

          Thanks, Marica! FYI I opened CONTINUUM-2117 for more work on this. The subject line is now configurable, but $

          {project.projectGroup.name}

          is not getting substituted (it comes through literally in the email subject).

          Show
          Wendy Smoak added a comment - Thanks, Marica! FYI I opened CONTINUUM-2117 for more work on this. The subject line is now configurable, but $ {project.projectGroup.name} is not getting substituted (it comes through literally in the email subject).
          Wendy Smoak made changes -
          Link This issue is related to CONTINUUM-2117 [ CONTINUUM-2117 ]
          Wendy Smoak made changes -
          Link This issue relates to CONTINUUM-1929 [ CONTINUUM-1929 ]
          Maria Catherine Tan made changes -
          Status Open [ 1 ] Closed [ 6 ]
          Resolution Fixed [ 1 ]
          Hide
          Maria Catherine Tan added a comment -

          Fixed in revision 751253

          Show
          Maria Catherine Tan added a comment - Fixed in revision 751253
          Wendy Smoak made changes -
          Affects Version/s 1.3.2 [ 14902 ]
          Assignee Maria Catherine Tan [ ctan ]
          Affects Version/s 1.3.1 [ 14741 ]
          Fix Version/s 1.3.2 [ 14902 ]
          Wendy Smoak made changes -
          Field Original Value New Value
          Affects Version/s 1.3.1 [ 14741 ]
          Affects Version/s 1.3.2 [ 14902 ]
          Hide
          Wendy Smoak added a comment -

          Thanks! I'd say go ahead and commit what you have, which makes the 'prepare build' subject line configurable... it sounds fine, it just needs to be documented.

          If the 'PREPARE BUILD SUCCESSFUL' notifications get too noisy or are considered redundant, we can consider removing them in a separate issue.

          Show
          Wendy Smoak added a comment - Thanks! I'd say go ahead and commit what you have, which makes the 'prepare build' subject line configurable... it sounds fine, it just needs to be documented. If the 'PREPARE BUILD SUCCESSFUL' notifications get too noisy or are considered redundant, we can consider removing them in a separate issue.
          Hide
          Maria Catherine Tan added a comment -

          For the successful 'prepare' notification, you can get it if alwaysSend is true or if the scmroot previous state while preparing is an Error. The Prepare Build Error notification is configurable in the ui.

          So if the prepare build failed because let's say the svn is down, and you configured to received notification when prepare build failed then you will receive a mail notification for that.

          I can remove the prepare build successful mail notification, since it seems like you'll get notification anyway during the build which also means the prepare build was successful.

          Show
          Maria Catherine Tan added a comment - For the successful 'prepare' notification, you can get it if alwaysSend is true or if the scmroot previous state while preparing is an Error. The Prepare Build Error notification is configurable in the ui. So if the prepare build failed because let's say the svn is down, and you configured to received notification when prepare build failed then you will receive a mail notification for that. I can remove the prepare build successful mail notification, since it seems like you'll get notification anyway during the build which also means the prepare build was successful.
          Hide
          Wendy Smoak added a comment -

          Okay, after playing with this a bit more it seems like I get either the 'prepare' notification or the normal one, but not both. Is that correct?

          For example, if the scm url in the pom is incorrect, I get a 'PREPARE BUILD ERROR' message. If I fix it and delete/re-add the project then I get only a 'BUILD SUCCESSFUL' message.

          Show
          Wendy Smoak added a comment - Okay, after playing with this a bit more it seems like I get either the 'prepare' notification or the normal one, but not both. Is that correct? For example, if the scm url in the pom is incorrect, I get a 'PREPARE BUILD ERROR' message. If I fix it and delete/re-add the project then I get only a 'BUILD SUCCESSFUL' message.
          Hide
          Wendy Smoak added a comment -

          I don't understand why there would be two notifications for a single build.

          I'll test it again, but it seemed to me the email I got was the normal build notification I'm used to, it just had this new hardcoded subject line. Are you saying there should be two?

          I looked at the transient state issues CONTINUUM-1864 and CONTINUUM-1829 and didn't see adding a second notification mentioned there or in the threads linked from the issues.

          Show
          Wendy Smoak added a comment - I don't understand why there would be two notifications for a single build. I'll test it again, but it seemed to me the email I got was the normal build notification I'm used to, it just had this new hardcoded subject line. Are you saying there should be two? I looked at the transient state issues CONTINUUM-1864 and CONTINUUM-1829 and didn't see adding a second notification mentioned there or in the threads linked from the issues.
          Hide
          Maria Catherine Tan added a comment -

          Here's what I did though I haven't committed it yet.

          Both in MailContinuumNotifier and application.xml
          (1) changed subjectFormat to buildSubjectFormat (use for actual build notification)
          (2) added prepareBuildSubjectFormat ( use for prepare build notification )

          WDYT?

          Show
          Maria Catherine Tan added a comment - Here's what I did though I haven't committed it yet. Both in MailContinuumNotifier and application.xml (1) changed subjectFormat to buildSubjectFormat (use for actual build notification) (2) added prepareBuildSubjectFormat ( use for prepare build notification ) WDYT?
          Hide
          Maria Catherine Tan added a comment -

          Only the subject line for the mail notification of "Prepare build" is not configurable.

          Preparation of build, which includes cleaning of directory, checkout and update, was separated from the actual build.
          Any suggestion with the subject format?

          Show
          Maria Catherine Tan added a comment - Only the subject line for the mail notification of "Prepare build" is not configurable. Preparation of build, which includes cleaning of directory, checkout and update, was separated from the actual build. Any suggestion with the subject format?
          Wendy Smoak created issue -

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development