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

enhancement: UpdateTask when Task is waiting for ShutdownDelay #14775

Merged
merged 1 commit into from
Oct 6, 2022

Conversation

DarthBenro008
Copy link
Contributor

Signed-off-by: Hemanth Krishna [email protected]

@hashicorp-cla
Copy link

hashicorp-cla commented Oct 2, 2022

CLA assistant check
All committers have signed the CLA.

@cesarvspr
Copy link

this looks good @DarthBenro008 !

Would you please provide instructions on how can I test it?

Copy link

@cesarvspr cesarvspr left a comment

Choose a reason for hiding this comment

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

@DarthBenro008
Copy link
Contributor Author

@cesarvspr , you can create a Task with shutdown delay and then run nomad alloc status . It should show that the Task is waiting for the shutdown delay

@tgross tgross self-requested a review October 3, 2022 18:27
@tgross tgross self-assigned this Oct 3, 2022
@tgross tgross added the backport/1.4.x backport to 1.4.x release line label Oct 3, 2022
@tgross tgross added this to the 1.4.x milestone Oct 3, 2022
Copy link
Member

@tgross tgross left a comment

Choose a reason for hiding this comment

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

Hi @DarthBenro008, this is great!

I've left some suggestions this could be made even better. Also, can you add a changelog entry file? It'll be a file at .changelog/14775.txt that's in the format like this one. For the text, it can be something like:

client: Added a TaskEvent when task shutdown is waiting on shutdown_delay

This just missed the window for Nomad 1.4.0 GA, but once this is merged you can expect this to ship in the next regular release of Nomad. Thanks so much!

nomad/structs/structs.go Outdated Show resolved Hide resolved
client/allocrunner/taskrunner/task_runner.go Outdated Show resolved Hide resolved
@DarthBenro008
Copy link
Contributor Author

Hi @tgross, thanks a lot for the detailed review! 😃 . I have made the necessary amendments!

Copy link
Member

@tgross tgross left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks @DarthBenro008! This should ship in the next regular patch release of Nomad

@tgross tgross merged commit a138821 into hashicorp:main Oct 6, 2022
@github-actions
Copy link

github-actions bot commented Feb 4, 2023

I'm going to lock this pull request because it has been closed for 120 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 4, 2023
@tgross tgross modified the milestones: 1.4.x, 1.5.0 Mar 1, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backport/1.4.x backport to 1.4.x release line theme/task lifecycle
Projects
Development

Successfully merging this pull request may close these issues.

emit TaskEvent for task stops blocked by shutdown_delay
4 participants