docker-workflow-plugin icon indicating copy to clipboard operation
docker-workflow-plugin copied to clipboard

[JENKINS-52264] Avoid empty environment injection from configuration

Open Alex-Vol opened this issue 6 years ago • 9 comments

Fix empty environment issue in docker exec use case.

Alex-Vol avatar Jul 24 '18 01:07 Alex-Vol

I'm running in to the same issue that this pr is attempting to fix so I want to build it locally to deploy it to our jenkins server in the meantime. However, Im having issues:

[ERROR] java.nio.file.NoSuchFileException: projects/jenkinsci/docker-workflow-plugin/target/classes/META-INF/annotations/hudson.Extension
[INFO] 1 error

jchauncey avatar Aug 29 '18 16:08 jchauncey

Nevermind I just downloaded the hpi from the incrementals.

jchauncey avatar Aug 29 '18 17:08 jchauncey

Ok I'm still seeing the following:

13:06:43 invalid argument "=" for "-e, --env" flag: invalid environment variable: =
13:06:43 See 'docker exec --help'.

jchauncey avatar Aug 29 '18 17:08 jchauncey

Ok I figured out my issue and was able to fix it without this plugin.

Go to Manage Jenkins -> Configure System there will be an empty environment variable in the Global Properties section. If you remove that everything should work

jchauncey avatar Aug 30 '18 12:08 jchauncey

Why in the world is this not merged yet? Such a small and simple fix to eliminate a very confusing and annoying problem. Please merge this so it can get into the next release. Thanks

albertsj1 avatar Oct 16 '18 18:10 albertsj1

This fix is very much needed as e.g. our optional but empty jenkins parameters will be empty env variables -> trigger the problem

keijohyttinen avatar Mar 13 '19 14:03 keijohyttinen

Also waiting for this fix. Any chance this will be merged soon?

JungPhilipp avatar Nov 22 '19 09:11 JungPhilipp

Rebased to latest and cleaned up unintended whitespace changes.

Alex-Vol-SV avatar Nov 23 '19 16:11 Alex-Vol-SV

Could the maintainers of this plugin merge this simple pull request?

Alex-Vol-SV avatar Feb 27 '20 15:02 Alex-Vol-SV