• Icon: Bug Bug
    • Resolution: Unresolved
    • Icon: Major Major
    • envinject-plugin
    • None
    • Jenkins 1.475 on RHEL5 x86_64, Windows XP slave

      I check 'Prepare an environment for the run' and provide the following script content:

      echo %PATH%
      

      I saw that PATH had additional Unix paths I provided at global configuration (my mistake). However, after removing PATH configuration from the global settings, all following builds still show these Unix paths appended. It looks like the plugin did not notice new global configuration and still injects old PATH settings.

          [JENKINS-14623] EnvInject ignores changes in system configuration

          Global configuration is only taken in account when the Jenkins instance is restarted?
          Did you restart after changing your global configuration?

          Gregory Boissinot added a comment - Global configuration is only taken in account when the Jenkins instance is restarted? Did you restart after changing your global configuration?

          I eventually did, but I find this behavior unfriendly and not intuitive. In this particular situation I could not restart Jenkins at once, since there were other jobs building, some of them run for many hours. So in case I could not restart Jenkins instance, I also could not build another job which relied on global configuration. Built-in variable setting does not work that way and does not require Jenkins restart after changing global configuration.

          Krzysztof Malinowski added a comment - I eventually did, but I find this behavior unfriendly and not intuitive. In this particular situation I could not restart Jenkins at once, since there were other jobs building, some of them run for many hours. So in case I could not restart Jenkins instance, I also could not build another job which relied on global configuration. Built-in variable setting does not work that way and does not require Jenkins restart after changing global configuration.

          Gregory Boissinot added a comment - - edited

          I'm afraid I can't fix your issue. With the current Jenkins infrastructure, there is a need to restart for doing these EnvInject features.
          Therefore, could we close the issue?
          Thanks

          Gregory Boissinot added a comment - - edited I'm afraid I can't fix your issue. With the current Jenkins infrastructure, there is a need to restart for doing these EnvInject features. Therefore, could we close the issue? Thanks

          Any update?
          What is your opinion?

          Gregory Boissinot added a comment - Any update? What is your opinion?

          Is it possible to help me?

          Gregory Boissinot added a comment - Is it possible to help me?

          If this is blocked by Jenkins core, I think it would be better to retarget this issue to core. Restarting Jenkins instance is an expensive operation and until new configuration is applied some of the builds are failing. I find such behavior unacceptable.

          Krzysztof Malinowski added a comment - If this is blocked by Jenkins core, I think it would be better to retarget this issue to core. Restarting Jenkins instance is an expensive operation and until new configuration is applied some of the builds are failing. I find such behavior unacceptable.

          Re-target to core to find if there is a solution?

          Gregory Boissinot added a comment - Re-target to core to find if there is a solution?

          evernat added a comment -

          There is no description in this issue, about what doesn't work or what should be improved in core.
          So this issue is much incomplete.
          Reassigning to envinject for that reason.

          evernat added a comment - There is no description in this issue, about what doesn't work or what should be improved in core. So this issue is much incomplete. Reassigning to envinject for that reason.

            gbois Gregory Boissinot
            raspy Krzysztof Malinowski
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated: