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

EnvInject/Parameterized trigger not handling properties file with logical lines

XMLWordPrintable

      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.

        1. config.xml
          2 kB
        2. jobs.properties
          0.2 kB

          [JENKINS-14232] EnvInject/Parameterized trigger not handling properties file with logical lines

          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?

          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?

          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

          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

          Gregory Boissinot added a comment -

          EnvInject 1.57 should solve the problem

          Gregory Boissinot added a comment - EnvInject 1.57 should solve the problem

          Mike Harris added a comment -

          I have verified version 1.57 fixes the issue for me.

          Thanks.

          Mike Harris added a comment - I have verified version 1.57 fixes the issue for me. Thanks.

          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

          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

          Mike Harris added a comment -

          I verified version 1.59 works for me.

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

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

              Created:
              Updated:
              Resolved: