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

Mercurial changeset numbers no longer work with newer versions of email-ext-plugin

    Details

    • Type: Bug
    • Status: Resolved (View Workflow)
    • Priority: Major
    • Resolution: Fixed
    • Component/s: mercurial-plugin
    • Labels:
      None
    • Environment:
      Jenkins version 1.591
      Mercurial Plugin version 1.51
      Email Extension Plugin version 2.40.5 (upgraded from 2.34)
    • Similar Issues:

      Description

      Starting with version 2.40 of the Email Extension Plugin, it uses 'getCommitId' on the ChangeLogSet.Entry object to get the commit id. However, the Mercurial Plugin only provides this information through the MercurialChangeset.getRevision method, which the Email Extension Plugin used to access through reflection.

      This is breaking e-mail configurations that used to work with Email Extension Plugin version 2.34. Specifically, the problem is that the %r (revision) formatting option for the ${CHANGES} option no longer works and will always show as 'null'.

      I suspect that the %d (date) option might be broken for a similar reason, though I have not tested that: In this commit: https://github.com/jenkinsci/email-ext-plugin/commit/6453a56cdcc82d2d7f085e8dba9c2b7032cb112a the Email Extension Plugin seems to have been modified to use getTimeStamp instead of getDate, which is also not yet implemented by the MercurialChangeSet.

      I've put the priority on 'major' because the only workaround is to use an older version of the Email Extension Plugin, which is undesirable because a different functionality of older versions of the Email Extension Plugin is not working under Jenkins 1.591 (the ability to put information about failed unit tests in the e-mail).

        Attachments

          Issue Links

            Activity

            siriah Iris Hupkens created issue -
            slide_o_mix Alex Earl made changes -
            Field Original Value New Value
            Assignee Alex Earl [ slide_o_mix ]
            slide_o_mix Alex Earl made changes -
            Component/s email-ext-plugin [ 15538 ]
            rtyler R. Tyler Croy made changes -
            Workflow JNJira [ 165389 ] JNJira + In-Review [ 181949 ]
            jglick Jesse Glick made changes -
            Remote Link This issue links to "PR 74 (Web Link)" [ 14941 ]
            jglick Jesse Glick made changes -
            Assignee Iris Hupkens [ siriah ]
            jglick Jesse Glick made changes -
            Status Open [ 1 ] In Progress [ 3 ]
            jglick Jesse Glick made changes -
            Status In Progress [ 3 ] In Review [ 10005 ]
            jglick Jesse Glick made changes -
            Status In Review [ 10005 ] Resolved [ 5 ]
            Resolution Fixed [ 1 ]

              People

              • Assignee:
                siriah Iris Hupkens
                Reporter:
                siriah Iris Hupkens
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: