Skip to content

test: use SIGKILL instead of SIGTERM#8612

Merged
jsha merged 1 commit intomainfrom
hardkill
Feb 2, 2026
Merged

test: use SIGKILL instead of SIGTERM#8612
jsha merged 1 commit intomainfrom
hardkill

Conversation

@jsha
Copy link
Copy Markdown
Contributor

@jsha jsha commented Jan 29, 2026

Now that we audit log clean shutdowns, it's very hard to find error messages among the logspam during shutdown. SIGKILL prevents the processes from logging anything on their way down.

Now that we audit log clean shutdowns, it's very hard to find error messages
among the logspam during shutdown. SIGKILL prevents the processes from logging
anything on their way down.
@jsha jsha marked this pull request as ready for review January 29, 2026 22:39
@jsha jsha requested a review from a team as a code owner January 29, 2026 22:39
@jsha jsha requested a review from aarongable January 29, 2026 22:39
@jsha jsha requested review from a team and beautifulentropy and removed request for a team February 2, 2026 18:27
@jsha
Copy link
Copy Markdown
Contributor Author

jsha commented Feb 2, 2026

Test only, merging on one review.

@jsha jsha merged commit 6a75950 into main Feb 2, 2026
17 checks passed
@jsha jsha deleted the hardkill branch February 2, 2026 18:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants