Skip to content

Commit ccac154

Browse files
authored
Merge pull request #1110 from jglick/comment
2 parents bfe7960 + 5e12214 commit ccac154

File tree

1 file changed

+0
-6
lines changed

1 file changed

+0
-6
lines changed

src/test/java/org/csanchez/jenkins/plugins/kubernetes/pipeline/RestartPipelineTest.java

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -248,12 +248,6 @@ public void terminatedPodAfterRestart() throws Exception {
248248
story.then(r -> {
249249
WorkflowRun b = r.jenkins.getItemByFullName(projectName.get(), WorkflowJob.class).getBuildByNumber(1);
250250
r.waitForMessage("Ready to run", b);
251-
// Note that the test is cheating here slightly.
252-
// The watch in Reaper is still running across the in-JVM restarts,
253-
// whereas in production it would have been cancelled during the shutdown.
254-
// But it does not matter since we are waiting for the agent to come back online after the restart,
255-
// which is sufficient trigger to reactivate the reaper.
256-
// Indeed we get two Reaper instances running, which independently remove the node.
257251
deletePods(cloud.connect(), getLabels(this, name), false);
258252
r.assertBuildStatus(Result.ABORTED, r.waitForCompletion(b));
259253
r.waitForMessage(new ExecutorStepExecution.RemovedNodeCause().getShortDescription(), b);

0 commit comments

Comments
 (0)