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

BUILD_USER_ID is not set in Maven build execution triggered by release:perform

    Details

    • Similar Issues:

      Description

      When doing a release for a Maven build project (using the m2release plugin), the build user vars gets screwed up in the Maven build during the Maven build execution triggered by release:perform.
      BUILD_USER is set to the user id ('kaha14' for example). BUILD_USER_ID is not set. BUILD_USER_FIRST_NAME is set to the same as BUILD_USER and BUILD_USER_LAST_NAME is not set (probably because BUILD_USER is just one word).

        Attachments

          Issue Links

            Activity

            Hide
            ahammar Anders Hammar added a comment - - edited

            Refreshing my memory I verified this for the m2release-plugin and JENKINS-20791 solves it and v0.14.0 is verified to work.
            The other plugin (release-plugin) I don't know.

            Show
            ahammar Anders Hammar added a comment - - edited Refreshing my memory I verified this for the m2release-plugin and JENKINS-20791 solves it and v0.14.0 is verified to work. The other plugin (release-plugin) I don't know.
            Hide
            ahammar Anders Hammar added a comment -

            dan tran What PR is that? I can't find one.

            Show
            ahammar Anders Hammar added a comment - dan tran What PR is that? I can't find one.
            Hide
            dantran dan tran added a comment -

            I see a pull request for this issue, but merge not yet happen since 2013. No more active committer?

            Show
            dantran dan tran added a comment - I see a pull request for this issue, but merge not yet happen since 2013. No more active committer?
            Hide
            lshatzer Larry Shatzer, Jr. added a comment -

            I've created JENKINS-20797 for the Release plugin.

            Show
            lshatzer Larry Shatzer, Jr. added a comment - I've created JENKINS-20797 for the Release plugin.
            Hide
            ahammar Anders Hammar added a comment -

            I've filed JENKINS-20791 specifically for the m2release plugin and I'm working on a fix for that. The idea is that no change should be needed in build-user-vars.

            Show
            ahammar Anders Hammar added a comment - I've filed JENKINS-20791 specifically for the m2release plugin and I'm working on a fix for that. The idea is that no change should be needed in build-user-vars.
            Hide
            lshatzer Larry Shatzer, Jr. added a comment -

            For the regular Release Plugin, it uses Cause.UserCause, while the build-user-vars-plugin looks for Cause.UserIdCause.

            Not sure what should change, release plugin to use Cause.UserIdCause, or build-user-vars-plugin to also look for Cause.UserCause, this might get the m2release plugin too.

            Show
            lshatzer Larry Shatzer, Jr. added a comment - For the regular Release Plugin, it uses Cause.UserCause, while the build-user-vars-plugin looks for Cause.UserIdCause. Not sure what should change, release plugin to use Cause.UserIdCause, or build-user-vars-plugin to also look for Cause.UserCause, this might get the m2release plugin too.
            Hide
            lshatzer Larry Shatzer, Jr. added a comment -

            The same thing for the regular release plugin too.

            Show
            lshatzer Larry Shatzer, Jr. added a comment - The same thing for the regular release plugin too.
            Hide
            ahammar Anders Hammar added a comment -

            Doing some digging, the problem is that the m2release plugin has created a ReleaseClause that extends UserCause. And it overrides getUserName().

            Show
            ahammar Anders Hammar added a comment - Doing some digging, the problem is that the m2release plugin has created a ReleaseClause that extends UserCause. And it overrides getUserName().

              People

              • Assignee:
                Unassigned
                Reporter:
                ahammar Anders Hammar
              • Votes:
                2 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated: