Skip to content
This repository has been archived by the owner on Dec 15, 2021. It is now read-only.

Commit

Permalink
Merge pull request #34 from jenkinsci/JENKINS-26398-comment
Browse files Browse the repository at this point in the history
Linking to a filed issue
  • Loading branch information
jglick committed Jan 14, 2015
2 parents 375c1db + 73d09e6 commit dd5eaf7
Showing 1 changed file with 1 addition and 1 deletion.
Expand Up @@ -293,7 +293,7 @@ private void startJnlpProc() throws Exception {
Handler handler = new ConsoleHandler();
handler.setLevel(Level.ALL);
LOGGER.addHandler(handler);
// Cannot use regular JenkinsRule.createSlave, since its slave dir is thrown out after a restart.
// Cannot use regular JenkinsRule.createSlave due to JENKINS-26398.
// Nor can we can use JenkinsRule.createComputerLauncher, since spawned commands are killed by CommandLauncher somehow (it is not clear how; apparently before its onClosed kills them off).
DumbSlave s = new DumbSlave("dumbo", "dummy", tmp.getRoot().getAbsolutePath(), "1", Node.Mode.NORMAL, "", new JNLPLauncher(), RetentionStrategy.NOOP, Collections.<NodeProperty<?>>emptyList());
story.j.jenkins.addNode(s);
Expand Down

0 comments on commit dd5eaf7

Please sign in to comment.