cmd: Log goroutines (stack traces) on SIGUSR1 #471
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Task
Produce core dump (goroutines stack trace) on demand
Perhaps a custom handler for SIGUSR1 that dumps the core into a file and/or logs it.
(For services (= "the API"), there's a fairly standard way in Go to add an endpoint, but I'm not sure what the best way would be to expose the endpoint only internally. Might not be worth the additional hassle; we've never needed a core dump there.)
Motivation:
Currently, the only way to obtain the stacks of goroutines is to send a SIGQUIT, which also terminates the program. And in a way that does not safely close the KVStore 💀, at that. When analyzers get stuck, the list of goroutines can be very helpful for diagnosing the live system.
This PR
This PR maxes the
nexus
binary respond to theSIGUSR1
os signal by logging the stack traces of all goroutines.A good way to read the stack traces is to pass the (last minute of) logs through on of:
Tested locally.