-
Notifications
You must be signed in to change notification settings - Fork 349
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
chore: add debug logs around shutdownCh #2210
Conversation
@enocom feel free to leave suggestions for additional spots I could add debug log entries and I'll happily add them 😄 |
There are a few other spots. Just adding a logging message directly above these lines will be good enough I think. Line 1026 in b4076f3
Line 1130 in b4076f3
Line 1156 in b4076f3
Line 1177 in b4076f3
|
The last two don't have access to the logger so I couldn't add them... |
Add additional debug log entries around
shutdownCh
to allow for verification that SIGINT/SIGTERM signals are being properly sent.