[operations] docs: add runbook for ACMM Level Monitor failure-alert gap - #23236
[operations] docs: add runbook for ACMM Level Monitor failure-alert gap#23236kubestellar-hive[bot] wants to merge 1 commit into
Conversation
acmm-level-monitor.yml has no if: failure() step and is not listed in workflow-failure-issue.yml's catch-all, so a job execution failure (distinct from the endpoint-unreachable false positive tracked in #23233) produces no notification. Adds a runbook documenting manual detection and the proposed catch-all fix, and links it from the docs/README.md operator index. Refs #23235. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Signed-off-by: operations-agent <operations-agent@users.noreply.github.com>
|
Important Held for human sign-off on the direction, not on the code. This PR's only tracked rationale is #23235, which the hive filed itself — issue #23235 was filed by kubestellar-hive[bot] and no human has acknowledged it. An agent-filed issue does not, on its own, establish that anyone agreed to the direction (hivecommons/hive#5117). The change may well be right; nothing here is a review of it. To release the hold, acknowledge the direction on that issue — comment on it, assign yourself, or add the |
✅ Deploy Preview for kubestellarconsole canceled.
|
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
👋 Hey @kubestellar-hive[bot] — thanks for opening this PR!
This is an automated message. |
Summary
Issue #23235 identified that
.github/workflows/acmm-level-monitor.yml("ACMM LevelMonitor") has no failure-alerting mechanism for the case where the job itself fails
to execute (as opposed to running successfully and reporting a genuine or
false-positive regression — that separate case is tracked in #23233 and is not
touched by this PR). It has no
if: failure()step and is not listed in.github/workflows/workflow-failure-issue.yml'sworkflow_runcatch-all.This PR adds
docs/runbooks/acmm-level-monitor-no-alert.md, a runbook documentingthe gap, a manual
gh run listdetection command, and the proposed one-line fix(add
"ACMM Level Monitor"to the catch-all'sworkflows:list) for a maintainerwith
workflowspermission to apply. It also links the new runbook fromdocs/README.md's operator index.Why no CI config file change
The
operationsagent's GitHub App token lacks theworkflowspermission requiredto create or update files under
.github/workflows/, confirmed in multiple priorsessions (see
docs/runbooks/upgrade-smoke-no-alert.md,docs/runbooks/console-live-promote-no-alert.md,docs/runbooks/stale-workflow-no-alert.md). This PR documents the interimmanual-detection workaround and the proposed fix for a maintainer with that
permission to apply.
Scope
Docs-only change: one new runbook file, one README index entry. No CI
configuration, code, or automation behavior is modified.
Refs #23235.
— hive: agent=operations backend=copilot model=claude-sonnet-5