ci: right-size runner registration caps#97119
Conversation
|
Codex review: needs maintainer review before merge. Reviewed June 26, 2026, 6:58 PM ET / 22:58 UTC. Summary PR surface: Tests 0, Docs +1, Config 0. Total +1 across 4 files. Reproducibility: not applicable. this is CI capacity tuning rather than a product bug. The review path is source inspection plus live operations checks for the workflow caps, docs, guard tests, CI status, and rate-limit evidence. Review metrics: 2 noteworthy metrics.
Merge readiness Overall follows the weaker of proof and patch quality, so missing proof can cap an otherwise strong patch. Rank-up moves:
Risk before merge
Maintainer options:
Next step before merge
Security Review detailsBest possible solution: Land this synchronized workflow/docs/test/skill update only after maintainers confirm the production runner-registration bucket and exact-head CI gates are healthy. Do we have a high-confidence way to reproduce the issue? Not applicable: this is CI capacity tuning rather than a product bug. The review path is source inspection plus live operations checks for the workflow caps, docs, guard tests, CI status, and rate-limit evidence. Is this the best way to solve the issue? Yes, this is the right code shape because the workflow, docs, guard test, and CI-limits skill move together. The unresolved question is operational confirmation of the production runner-registration bucket, not a narrower code fix. AGENTS.md: found and applied where relevant. Codex review notes: model internal, reasoning high; reviewed against aca905cce50a. Label changesLabel justifications:
Evidence reviewedPR surface: Tests 0, Docs +1, Config 0. Total +1 across 4 files. View PR surface stats
What I checked:
Likely related people:
What the crustacean ranks mean
Shiny media proof means a screenshot, video, or linked artifact directly shows the changed behavior. Runtime, network, CSP, and security claims still need visible diagnostics. How this review workflow works
|
|
Merged via squash.
|
What Problem This Solves
GitHub increased the OpenClaw runner-registration bucket to 3,000 self-hosted registrations per 5 minutes. Our CI limits and maintainer guidance were still sized around the older 1,500 bucket, leaving safe throughput on the table while main CI remained slower than needed.
Why This Change Was Made
This keeps the existing burst protections, main-push debounce, compact PR plan, and platform caps, but right-sizes the Linux CI max-parallel values for the new bucket. The operating target moves from 1,000 to 2,000 registrations per 5 minutes so we still reserve roughly one-third of the org bucket for ClawSweeper, ClawHub, retries, and overlapping repo activity.
User Impact
Main and PR CI should admit more Linux work once a run passes the debounce, without adding jobs or moving CodeQL back to Blacksmith. Windows and Android stay capped at two.
Evidence
node scripts/run-vitest.mjs test/scripts/ci-workflow-guards.test.tsnode scripts/check-workflows.mjsnode scripts/docs-list.js./node_modules/.bin/oxfmt --check .github/workflows/ci.yml docs/ci.md test/scripts/ci-workflow-guards.test.ts .agents/skills/openclaw-ci-limits/SKILL.mdgit diff --check.agents/skills/autoreview/scripts/autoreview --mode branch --base origin/maincleanOperational note: attempted to cancel 77 stale queued OpenClaw Actions runs older than 24h. GitHub returned backend 500s for normal cancel and force-cancel; sample force-cancel request id:
10DD:262C01:388DA56:396FE3F:6A3F0152.