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

Retain task states for post stop tasks at the time of node GC #18005

Merged

Conversation

stswidwinski
Copy link
Contributor

Persist the task states for post stop tasks at the time of killing to resolve #17971.

Please note that this is the safer of the two options discussed within the original issue description.

@stswidwinski
Copy link
Contributor Author

I believe that test-ui / finalize (pull_request) has failed for unrelated reasons. I would appreciate it if the reviewer could double check

@schmichael schmichael added this to the 1.6.x milestone Jul 21, 2023
@schmichael schmichael added backport/1.4.x backport to 1.4.x release line backport/1.5.x backport to 1.5.x release line backport/1.6.x backport to 1.6.x release line labels Jul 21, 2023
@schmichael
Copy link
Member

Tested and looks good to me! Thanks for the great work debugging and fixing this. I pushed up a changelog entry, added backport labels, and will merge as soon as tests are happy.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/1.4.x backport to 1.4.x release line backport/1.5.x backport to 1.5.x release line backport/1.6.x backport to 1.6.x release line
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Post Stop tasks are never killed on the client node (also at GC time) resulting in task events loss.
2 participants