Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[JENKINS-49707] Pick up proposed final changes #630

Merged
merged 1 commit into from
Oct 11, 2022

Conversation

jglick
Copy link
Contributor

@jglick jglick commented Oct 11, 2022

Following up #512 and countermanding #532 if you are willing to run these prior to release @dduportal. Picks up jenkinsci/workflow-api-plugin#256 + jenkinsci/workflow-durable-task-step-plugin#180. Briefly, this would make it possible for retry blocks to work even if the agent outage was detected during a controller restart, as might occur for example during cluster updates (draining node pools, that sort of thing).

Copy link
Contributor

@dduportal dduportal left a comment

Choose a reason for hiding this comment

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

LGMT 👍 We are ok to run these elements!

Copy link
Contributor

@smerle33 smerle33 left a comment

Choose a reason for hiding this comment

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

🚀

@smerle33 smerle33 enabled auto-merge (squash) October 11, 2022 15:53
@smerle33 smerle33 merged commit b250359 into jenkins-infra:main Oct 11, 2022
@jglick jglick deleted the patch-1 branch October 11, 2022 16:23
@jglick
Copy link
Contributor Author

jglick commented Oct 14, 2022

Is this live? Any issues noticed so far?

@jglick
Copy link
Contributor Author

jglick commented Oct 17, 2022

No one screaming bloody regression so far, so I guess it is OK.

@dduportal
Copy link
Contributor

Hey sorry @jglick for the delayed answer. So yes it is live in infra.ci.jenkins.io since ~7 days and no issues.

We can proceed to release, unless you want also a test on ci.jenkins.io itself?

@jglick
Copy link
Contributor Author

jglick commented Oct 17, 2022

Thanks. I am proceeding to release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants