docs(miner): add unattended scheduling & failure-alerting guidance - #5584
docs(miner): add unattended scheduling & failure-alerting guidance#5584nickmopen wants to merge 1 commit into
Conversation
Operational guidance (JSONbored#4840) for running the miner's two scheduled commands (`manage poll`, `discover`) unattended: the shared exit-code contract (0 success / 2 failure) that alerting keys on, the `--no-update-check` / `--json` flags for scheduled runs, worked cron and systemd service+timer examples, and three failure-alerting patterns (cron MAILTO/logger, systemd OnFailure=, and a wrapper script). Closes JSONbored#4840
|
Superagent didn't find any vulnerabilities or security issues in this PR. |
|
Caution 🟥🟥🟥🟥🟥🟥🟥🟥🟥🟥🟥🟥 🛑 Gittensory review result - reject/close recommendedReview updated: 2026-07-13 07:18:29 UTC
🛑 Suggested Action - Reject/Close Review summary Blockers
Nits — 6 non-blocking
Why this is blocked
📋 Copy for AI agents — paste into your coding agentCI checks failing
Linked issue satisfactionAddressed Review context
Contributor next steps
Signal definitions
[BETA] Chat with GittensoryAsk Gittensory a question about this PR directly in a comment — grounded only in the same cached, public-safe facts shown above, never a new claim.
Full command reference: https://gittensory.aethereal.dev/docs/gittensory-commands 🟩 Safe / merged · 🟦 Advisory · 🟨 Held for review · 🟥 Blocked / closed 💰 Earn for open-source contributions like this. Gittensor lets GitHub contributors earn for the work they already do — register to start earning →. Checked by Gittensory, a quiet PR intelligence layer for OSS maintainers.
|
|
Gittensory is closing this pull request on the maintainer's behalf (CI is failing (Build UI preview artifact); AI reviewers agree on a likely critical defect: packages/gittensory-miner/docs/unattended-scheduling.md:30 claims cron mails `MAILTO` on any non-zero exit, but cron mails command output rather than exit status and the shown `|| logger` branch returns logger's status, so a silent command failure can be hidden from cron; change the comment/text to say MAILTO only sends emitted output and either rely on `logger` as the alert path or preserve failure with `|| { status=$?; logger -t gittensory-miner "manage poll failed (exit $status)"; exit "$status"; }`.). This is an automated maintenance action — to pursue this change, please open a new pull request with the issues resolved. Closed PRs may be analyzed later to improve review accuracy, but they are not automatically reopened or re-reviewed. |
Summary
Adds operational guidance (#4840) for running the miner's two scheduled commands —
manage pollanddiscover— unattended, and for alerting on failures.Doc:
packages/gittensory-miner/docs/unattended-scheduling.md. Covers:0success /2failure) that every scheduler keys on for alerting — verified againstmanage-poll.jsanddiscover-cli.js.--no-update-check/GITTENSORY_MINER_NO_UPDATE_CHECK=1(skip the npm nudge) and--json(machine-parseable output).MAILTO+ a|| loggerfallback that captures the failing exit code).oneshotservice + timer, usingOnFailure=as the alerting hook.MAILTO/logger, systemdOnFailure=templated unit, and a scheduler-agnostic wrapper script.Scope
type(scope): short summaryConventional Commit format.CONTRIBUTING.mdand does not reintroduce GitHub Pages, VitePress,site/, orCNAME.Closes #4840.Validation
git diff --checknpm run docs:drift-check(passes — no mechanically-checkable claim goes stale)manage-poll.js,discover-cli.jsexit0/2;update-check.js--no-update-check/GITTENSORY_MINER_NO_UPDATE_CHECK)If any required check was skipped, explain why:
.md, no code). Nosrc/**/apps/**/package changes, so Codecov and the backend/UI/MCP CI jobs are path-skipped; there is no code to test.Safety
Notes
operations-runbook.mdand../DEPLOYMENT.mdrather than duplicating their scope.Closes #4840