Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
[FIXED JENKINS-16316] Do not inject master node property variables in…
…to EnvVars.masterEnvVars on slaves.
  • Loading branch information
jglick committed Jul 26, 2013
1 parent 96a5269 commit dedee19
Showing 1 changed file with 0 additions and 7 deletions.
Expand Up @@ -77,13 +77,6 @@ private EnvVars getNewSlaveEnvironmentVariables(Computer c, FilePath nodePath, T

Map<String, String> currentEnvVars = new HashMap<String, String>();

// -- Retrieve Environment variables from master
for (NodeProperty<?> nodeProperty : Hudson.getInstance().getGlobalNodeProperties()) {
if (nodeProperty instanceof EnvironmentVariablesNodeProperty) {
currentEnvVars.putAll(((EnvironmentVariablesNodeProperty) nodeProperty).getEnvVars());
}
}

EnvInjectLogger logger = new EnvInjectLogger(listener);
EnvInjectEnvVars envInjectEnvVarsService = new EnvInjectEnvVars(logger);

Expand Down

1 comment on commit dedee19

@sschuberth
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jglick, can this by any chance be the cause for this bug?

Please sign in to comment.