-
-
Notifications
You must be signed in to change notification settings - Fork 65
Add signal/crash handling to the miner CLI #4826
Copy link
Copy link
Closed
Labels
gittensor:featureGittensor-scored feature linked to a feature issue — scores a 0.25x multiplier.Gittensor-scored feature linked to a feature issue — scores a 0.25x multiplier.help wantedExtra attention is neededExtra attention is needed
Description
Metadata
Metadata
Assignees
Labels
gittensor:featureGittensor-scored feature linked to a feature issue — scores a 0.25x multiplier.Gittensor-scored feature linked to a feature issue — scores a 0.25x multiplier.help wantedExtra attention is neededExtra attention is needed
Problem: No process-level signal handling exists anywhere in the miner package — no
SIGINT/SIGTERM/uncaughtException/unhandledRejectionhandlers. The CLI dispatches through a chain of bareprocess.exit()calls with no cleanup hook, so an interrupted run leaves whatever state it was mid-writing in an undefined condition.Area: AMS / CLI
Proposal: Add top-level signal and error handlers to the CLI entrypoint that flush/close any open ledger connections and exit cleanly, rather than letting the process die mid-write.
Deliverables:
Acceptance criteria:
SIGINT/SIGTERMmid-run leaves every ledger in a valid, non-corrupt state.Resources:
Boundaries: