Skip to content

Add fixes to properly catch shutdown signals#4731

Open
backspace wants to merge 1 commit intomainfrom
dev-all-cancel-worker-manager-cs-11084
Open

Add fixes to properly catch shutdown signals#4731
backspace wants to merge 1 commit intomainfrom
dev-all-cancel-worker-manager-cs-11084

Conversation

@backspace
Copy link
Copy Markdown
Contributor

This is a fix for the underlying problem that #4704 and #4727 were about: lingering worker and worker manager processes after a Ctrl-C on mise dev-all or mise dev.

I ran this locally with mise dev, which still had the dangling process because #4727 hasn’t been merged, you can see when I Ctrl-C, the associated Node process eventually ends.

CleanShot 2026-05-08 at 14 33 31

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 8, 2026

Host Test Results

    1 files      1 suites   1h 46m 24s ⏱️
2 643 tests 2 628 ✅ 15 💤 0 ❌
2 662 runs  2 647 ✅ 15 💤 0 ❌

Results for commit 6aa48f8.

Realm Server Test Results

    1 files      1 suites   16m 50s ⏱️
1 297 tests 1 297 ✅ 0 💤 0 ❌
1 376 runs  1 376 ✅ 0 💤 0 ❌

Results for commit 6aa48f8.

@backspace backspace marked this pull request as ready for review May 8, 2026 20:39
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.

1 participant