Skip to content

fix(surfnet): terminates the cli program on sigterm#589

Open
0xzrf wants to merge 1 commit intosolana-foundation:mainfrom
0xzrf:sigterm_kill_fix
Open

fix(surfnet): terminates the cli program on sigterm#589
0xzrf wants to merge 1 commit intosolana-foundation:mainfrom
0xzrf:sigterm_kill_fix

Conversation

@0xzrf
Copy link
Contributor

@0xzrf 0xzrf commented Mar 23, 2026

Summery

Stops the surfpool surnet execution immediately after calling the SIGTERM

Changes

Added termination feature to the ctrlc crate, which signals the ctrlc:: set_handler to call the SimnetCommand::Terminate(None); on SIGTERM signal

Closes #514

@MicaiahReid
Copy link
Collaborator

@0xzrf were you able to successfully test this? I built and ran this branch, but SIGTERM didn't seem to do anything

@0xzrf
Copy link
Contributor Author

0xzrf commented Mar 23, 2026

@MicaiahReid umm, It does seem to run in my end.
I tested it by running the following:

kill -15 <PID>

@MicaiahReid MicaiahReid force-pushed the main branch 5 times, most recently from 5419677 to 2129ad9 Compare March 23, 2026 19:40
@MicaiahReid
Copy link
Collaborator

@MicaiahReid umm, It does seem to run in my end. I tested it by running the following:

kill -15 <PID>

Hmm, I'm on macos and that's not working for me 🤔

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.

Surfpool fails to stop after SIGTERM or exit RPC request

2 participants