fix(runbooks): correct Gitea runner fetch timing facts (post-#457) #478

Merged
core-lead merged 2 commits from sre/fix-gitea-runbook-network-quirks into main 2026-05-11 13:45:44 +00:00

2 Commits

Author SHA1 Message Date
3cd238c17d docs(runbooks): add Gitea combined-status null-false-positive quirk
All checks were successful
Block internal-flavored paths / Block forbidden paths (pull_request) Successful in 12s
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 14s
sop-tier-check / tier-check (pull_request) Successful in 15s
CI / Detect changes (pull_request) Successful in 41s
E2E API Smoke Test / detect-changes (pull_request) Successful in 43s
E2E Staging Canvas (Playwright) / detect-changes (pull_request) Successful in 44s
CI / Platform (Go) (pull_request) Successful in 7s
Handlers Postgres Integration / detect-changes (pull_request) Successful in 47s
Runtime PR-Built Compatibility / detect-changes (pull_request) Successful in 47s
CI / Canvas (Next.js) (pull_request) Successful in 8s
CI / Shellcheck (E2E scripts) (pull_request) Successful in 6s
CI / Python Lint & Test (pull_request) Successful in 7s
CI / Canvas Deploy Reminder (pull_request) Has been skipped
E2E API Smoke Test / E2E API Smoke Test (pull_request) Successful in 8s
E2E Staging Canvas (Playwright) / Canvas tabs E2E (pull_request) Successful in 7s
Runtime PR-Built Compatibility / PR-built wheel + import smoke (pull_request) Successful in 5s
Handlers Postgres Integration / Handlers Postgres Integration (pull_request) Successful in 5s
audit-force-merge / audit (pull_request) Successful in 13s
Document that Gitea reports combined state as `failure` when ALL
individual status contexts have `state: null`. This causes spurious
`[main-red]` issue filings from the watchdog — first seen on
issue #481 (2026-05-11).

Also removes the now-redundant "fetch-depth: 0" section (duplicate
of the "Large repo causes fetch timeout" section).

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
2026-05-11 13:42:32 +00:00
58be7b29ac fix(runbooks): correct Gitea runner network/fetch timing facts
SRE review of PR #457 flagged two factual errors that were not
addressed before merge. Applying corrections directly per SRE
mandate: no manual production changes without config-as-code.

Corrections:
1. Remove "git fetch --depth=1 times out" — shallow fetch succeeds
   in ~16s per PR #441 detect-changes evidence. Only fetch-depth:0
   and git clone time out due to ~75MB repo history size.
2. Rewrite "runner cannot reach git remote" to accurately state:
   runner CAN reach the remote; fetching full compressed history
   exceeds the ~15s network timeout window. Repo-size constraint,
   not network isolation.
3. Updated diagnosis snippet and verification section to match.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
2026-05-11 13:42:32 +00:00