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

EnvInject/Parameterized trigger not handling properties file with logical lines

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      I've attached a jobs.properties file that defines jobslist, using multiple logical lines.
      When I add ${jobslist} as the list of jobs for the parameterized trigger, it only triggers the first job (apple) and stops. The config.xml file for my test job is attached.
      If I change the properties file to use a single line, it triggers all the jobs.
      In our productions job, we have over 20 jobs to trigger and the developers require using the logical lines.

        Attachments

          Issue Links

            Activity

            Hide
            gbois Gregory Boissinot added a comment -

            Firstly, it seems there is a bug in the envinject plguin.
            If you look at the jobslist variable in the Injected environment, you should have only one value. It is the current bug.
            Could you check in your environment?

            Show
            gbois Gregory Boissinot added a comment - Firstly, it seems there is a bug in the envinject plguin. If you look at the jobslist variable in the Injected environment, you should have only one value. It is the current bug. Could you check in your environment?
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Gregory Boissinot
            Path:
            src/main/java/org/jenkinsci/plugins/envinject/service/PropertiesLoader.java
            http://jenkins-ci.org/commit/envinject-plugin/3e05be8be28ed213858e97df30bd7db71aa7d789
            Log:
            Fix JENKINS-14232

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Gregory Boissinot Path: src/main/java/org/jenkinsci/plugins/envinject/service/PropertiesLoader.java http://jenkins-ci.org/commit/envinject-plugin/3e05be8be28ed213858e97df30bd7db71aa7d789 Log: Fix JENKINS-14232
            Hide
            gbois Gregory Boissinot added a comment -

            EnvInject 1.57 should solve the problem

            Show
            gbois Gregory Boissinot added a comment - EnvInject 1.57 should solve the problem
            Hide
            mharris Mike Harris added a comment -

            I have verified version 1.57 fixes the issue for me.

            Thanks.

            Show
            mharris Mike Harris added a comment - I have verified version 1.57 fixes the issue for me. Thanks.
            Hide
            gbois Gregory Boissinot added a comment -

            Please could you check also version 1.58 (1.58 fixes a regression introduced by the fix of this issue).
            Thanks

            Show
            gbois Gregory Boissinot added a comment - Please could you check also version 1.58 (1.58 fixes a regression introduced by the fix of this issue). Thanks
            Hide
            mharris Mike Harris added a comment -

            I verified version 1.59 works for me.

            Show
            mharris Mike Harris added a comment - I verified version 1.59 works for me.

              People

              • Assignee:
                gbois Gregory Boissinot
                Reporter:
                mharris Mike Harris
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: