Skip to content

ci: retry transient release workflow reads#102896

Merged
steipete merged 2 commits into
mainfrom
codex/release-read-retry-hardening
Jul 9, 2026
Merged

ci: retry transient release workflow reads#102896
steipete merged 2 commits into
mainfrom
codex/release-read-retry-hardening

Conversation

@steipete

@steipete steipete commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

What Problem This Solves

Full Release Validation now dispatches child workflows exactly once and can adopt an accepted child after an ambiguous POST failure. Its subsequent read-only GitHub API calls still failed immediately on transient HTTP 5xx and network errors, and deterministic dispatch failures could spend five minutes polling for a child that was never accepted.

Why This Change Was Made

This patch gives all six read-only retry helpers one shared transient server/network classifier. The five workflow dispatch POSTs remain raw one-shot calls: ambiguous HTTP 5xx/network failures enter exact-title adoption polling, while deterministic failures such as 400/401/403/404/422/429 and CLI usage errors fail immediately.

The classifier includes bare transport EOF, which gh can emit when the server accepted a POST but the response connection closed.

User Impact

Release validation is less likely to fail from temporary GitHub read errors, never blindly retries a mutating child dispatch, and fails faster for invalid dispatch requests. Product runtime behavior is unchanged.

Evidence

  • Blacksmith Testbox tbx_01kx3kpfkbsxac7akmdhw4zmgk: 92 focused workflow tests passed.
  • node scripts/check-workflows.mjs: actionlint, zizmor, and workflow guards passed.
  • pnpm check:changed: tooling lane passed with zero lint warnings/errors.
  • Targeted pnpm format:check and git diff --check: passed.
  • Fresh autoreview after the bare-EOF regression fix: clean at 0.94 confidence.
  • Live parent 29023866912 created exactly one correlated CI, Plugin, Release Checks, and Performance child at exact SHA 11dda0e5fd3ca670f4fdbd9eaf295162b8f81c59.

Refs #102858.

@openclaw-barnacle openclaw-barnacle Bot added size: S maintainer Maintainer-authored PR labels Jul 9, 2026
@steipete

steipete commented Jul 9, 2026

Copy link
Copy Markdown
Contributor Author

Land-ready proof for exact head 111c643cf2fe8e84d387273aee2acb63d36feb70:

  • CI 29025298027: 43 passed, 12 expected skips, 0 failures.
  • Workflow Sanity 29025294728: actionlint and no-tabs passed.
  • CodeQL 29025294282: 7/7 passed.
  • Blacksmith Testbox tbx_01kx3kpfkbsxac7akmdhw4zmgk: 92 focused workflow tests passed; changed-file tooling gate passed.
  • Targeted workflow checks, format, and git diff --check: passed.
  • Fresh autoreview after the bare-EOF fix: clean at 0.94 confidence.
  • Repo-native review artifacts validated with zero findings; OPENCLAW_TESTBOX=1 scripts/pr prepare-run 102896 passed.

All five dispatch POSTs remain one-shot. Only read-only GitHub calls retry transient failures, ambiguous transport/server failures use exact-child adoption, and deterministic dispatch errors fail immediately.

@steipete
steipete merged commit aca96d2 into main Jul 9, 2026
97 of 100 checks passed
@steipete
steipete deleted the codex/release-read-retry-hardening branch July 9, 2026 14:34
@steipete

steipete commented Jul 9, 2026

Copy link
Copy Markdown
Contributor Author

Merged via squash.

github-actions Bot pushed a commit to Desicool/openclaw that referenced this pull request Jul 10, 2026
steipete added a commit that referenced this pull request Jul 10, 2026
Backports the release pipeline speedups from main (ace2407 and
ancestors) so the 2026.7.1 train benefits immediately:
- metadata-only evidence reuse with input/harness/child-run matching
- Docker runtime-assets preflight runs in parallel with lanes
- release/* umbrella runs supersede in-progress duplicates
- beta profile treats live-provider suites as advisory
Syncing the umbrella workflow also picks up the main-only dispatch
idempotency and transient-read retries (#102858, #102896).
wheakerd pushed a commit to wheakerd/clawdbot that referenced this pull request Jul 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintainer Maintainer-authored PR size: S

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant