-
Notifications
You must be signed in to change notification settings - Fork 12
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
Add Sigkill handling to node binary #480
Conversation
33ade29
to
f79c25a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, just double checking it that todo comment is a leftover.
nomos-services/system-sig/src/lib.rs
Outdated
async fn should_stop_service(msg: LifecycleMessage) -> bool { | ||
match msg { | ||
LifecycleMessage::Shutdown(sender) => { | ||
// TODO: Try to finish pending transactions if any and close connections properly |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe there won't be any pending txs in this service
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hopefully not 😅
f79c25a
to
3bda511
Compare
5dbb11b
to
82dee8f
Compare
CI codeconv was unrelated |
No description provided.