-
-
Notifications
You must be signed in to change notification settings - Fork 21
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
✨ Stop worker gracefully on sigterm #407
Conversation
e9afc3e
to
2e28b89
Compare
Overmind decides to kill the worker in the dev setup after a couple of seconds but I think this is okay |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it still possible to end the worker with a keyboard combination after this?
(e.g. ctrl-c ctrl-c)
Yes this is intentional
ctrl-c should work as expected and stop the worker immediately since it sends SIGINT |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Other than the comments LGTM
869736d
to
06b7ef9
Compare
No description provided.