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

EnvInject Plugin 1.26 does not substitute ${WORKSPACE} correctly

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      This problem appeared when I upgraded the EnvInject Plugin from 1.25 to 1.26, so the problem was probably introduced with the fix for JENKINS-12704.

      I have a job with "Inject environment variables to the build process" selected, and the following line in "Properties Content":
      materialize_workspace_path=${WORKSPACE}/materialize_workspace

      In 1.25, this environment variable was correctly given the value
      /scratch/jenkins_workspace/GDA_master.tests.RH5_64/materialize_workspace

      In 1.26, this environment variable was WRONGLY given the value
      /scratch/jenkins_slave//scratch/jenkins_workspace/GDA_master.tests.RH5_64/materialize_workspace

        Attachments

          Issue Links

            Activity

            Hide
            gbois Gregory Boissinot added a comment -

            Could you test with envinject 1.28?

            Show
            gbois Gregory Boissinot added a comment - Could you test with envinject 1.28?
            Hide
            mwebber Matthew Webber added a comment -

            The problem is fixed in 1.28 - thanks for your prompt attention.
            Should a test be added for this case - we've had a couple of releases lately which have broken stuff.

            Show
            mwebber Matthew Webber added a comment - The problem is fixed in 1.28 - thanks for your prompt attention. Should a test be added for this case - we've had a couple of releases lately which have broken stuff.
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Gregory Boissinot
            Path:
            src/test/java/org/jenkinsci/plugins/envinject/EnvInjectBuildWrapperTest.java
            http://jenkins-ci.org/commit/envinject-plugin/eb3cb491dcede8409b26bc8fe3cfc1e0af99b1d9
            Log:
            Add an integration test to fix JENKINS-12841

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Gregory Boissinot Path: src/test/java/org/jenkinsci/plugins/envinject/EnvInjectBuildWrapperTest.java http://jenkins-ci.org/commit/envinject-plugin/eb3cb491dcede8409b26bc8fe3cfc1e0af99b1d9 Log: Add an integration test to fix JENKINS-12841
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Gregory Boissinot
            Path:
            src/test/java/org/jenkinsci/plugins/envinject/EnvInjectBuildWrapperTest.java
            http://jenkins-ci.org/commit/envinject-plugin/6fd8c4018572af8c7154cab49c7552da831f96f1
            Log:
            Add an integration test to fix JENKINS-12841

            Compare: https://github.com/jenkinsci/envinject-plugin/compare/db8e61f...6fd8c40

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Gregory Boissinot Path: src/test/java/org/jenkinsci/plugins/envinject/EnvInjectBuildWrapperTest.java http://jenkins-ci.org/commit/envinject-plugin/6fd8c4018572af8c7154cab49c7552da831f96f1 Log: Add an integration test to fix JENKINS-12841 Compare: https://github.com/jenkinsci/envinject-plugin/compare/db8e61f...6fd8c40
            Hide
            gbois Gregory Boissinot added a comment -

            Test added

            Show
            gbois Gregory Boissinot added a comment - Test added

              People

              • Assignee:
                gbois Gregory Boissinot
                Reporter:
                mwebber Matthew Webber
              • Votes:
                1 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: