Skip to content

[JENKINS-49707] Allow node blocks from deleted pods to be retried#1190

Merged
Vlatombe merged 11 commits intojenkinsci:masterfrom
jglick:retry-JENKINS-49707-base
Jul 13, 2022
Merged

[JENKINS-49707] Allow node blocks from deleted pods to be retried#1190
Vlatombe merged 11 commits intojenkinsci:masterfrom
jglick:retry-JENKINS-49707-base

Conversation

@jglick
Copy link
Member

@jglick jglick commented Jun 3, 2022

Downstream of jenkinsci/workflow-api-plugin#217, jenkinsci/workflow-durable-task-step-plugin#231, & jenkinsci/workflow-basic-steps-plugin#203. Main functionality (without support for Declarative syntax) extracted from #1083, and without cross-restart tests.

@jglick jglick requested review from Vlatombe and dduportal June 6, 2022 23:27
@jglick jglick marked this pull request as ready for review July 7, 2022 20:54
Comment on lines +93 to +96
if (!(n instanceof KubernetesSlave)) {
LOGGER.fine(() -> node + " was not a K8s agent");
return false;
}
Copy link
Member Author

Choose a reason for hiding this comment

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

@jglick jglick marked this pull request as draft July 8, 2022 21:29
@jglick
Copy link
Member Author

jglick commented Jul 8, 2022

Still a couple tweaks I want to make here.

@jglick jglick marked this pull request as ready for review July 12, 2022 20:52
@jglick
Copy link
Member Author

jglick commented Jul 12, 2022

@Vlatombe ready for release from my perspective. ci.jenkins.io is already running an earlier version of this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants