-
Notifications
You must be signed in to change notification settings - Fork 316
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
Request: lifecycle hooks for before kill signal #4923
Comments
Valid as of v0.57 |
@gfodor Could you provide some concrete usecase examples of your proposed hooks, along the lines of "Application W could use hook X because that would allow it to do Y, which is impossible/impractical/etc. currently because Z"? Just wanting to make sure that whatever we end up implementing serves the intended purpose. Thanks! |
Sure.
|
I think |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. We value your input and contribution. Please leave a comment if this issue still affects you. |
We would like to be able to de-register our service from a load balancer when it is about to be restarted by Habitat, by running a hook before the termination signal is sent. The feature proposed in #2447 is somewhat overlapping since it could be used to solve the same problem, but that solution requires the application handle it, and often it's not possible to change the application or it is the wrong separation of concerns.
Specifically, some lifecycle events which would be useful are:
pre-config-restart
- Run before the service is sent the kill signal when it is being restarted due to a config change or file upload (ie there is noreload
hook)pre-upgrade-restart
- Run before the service is sent the kill signal when it is being restarted due to a package upgradepre-stop
- Run before the service is sent the kill signal, when it is being shut down.pre-kill
- Run before the service is sent the kill signal, regardless of the reason.The text was updated successfully, but these errors were encountered: