[operations] docs: document netlify-error-reporter.yml silent-integration-failure gap - #6779
Conversation
…tion-failure gap Cross-references #6778 from runbooks/deploy-rollback.md's 'Detecting a bad deploy' section: netlify-error-reporter.yml only runs when GitHub receives a status event from Netlify's GitHub integration, so a broken/misconfigured integration produces zero deploy-failure signal. Neither that workflow nor the existing/proposed healthz readiness check would catch a deploy stuck on stale content, since /api/healthz stays 200 for old-but-valid content. Signed-off-by: kubestellar-hive[bot] <223556219+Copilot@users.noreply.github.com>
|
Important Held for human sign-off on the direction, not on the code. This PR's only tracked rationale is #6778, which the hive filed itself — issue #6778 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 kubestellar-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
[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 |
|
Hi @kubestellar-hive[bot]. Thanks for your PR. I'm waiting for a kubestellar member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
PR needs rebase. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
Summary
Relates to #6778 (netlify-error-reporter.yml only fires on a GitHub
statusevent — a broken/revoked Netlify↔GitHub integration produceszero deploy-failure signal, and neither that workflow nor the readiness
check would catch a deploy stuck serving stale content).
runbooks/deploy-rollback.md's "Detectinga bad deploy" section, explaining the gap and giving on-call a concrete
next step (check the Netlify dashboard directly if the site looks stuck
on old content with no matching alert).
latest Netlify deploy's
commit_ref(via the same Netlify API callalready used in
netlify-error-reporter.yml) againstmain's HEAD SHA.Why this PR is docs-only
Per the established pattern in this repo (e.g. #6700, #6719), agent
tokens lack the
workflowspermission needed to add or modify.github/workflows/*, so the actual scheduled check is not implementedhere — only documented, for a maintainer (or a token with
workflowspermission) to add.
Scope note
No new metrics backend or exporter is proposed; the recommended fix
reuses the Netlify API call the existing
netlify-error-reporter.ymlworkflow already makes.
Testing
(unpatched)
netlify-error-reporter.ymland/api/healthzbehavior.Not merging this myself per policy.
— hive: agent=operations backend=copilot model=claude-sonnet-4-6