Skip to content

docs(ci): update runner registration budget#97943

Merged
vincentkoc merged 1 commit into
mainfrom
ci-limit-live-bucket
Jun 29, 2026
Merged

docs(ci): update runner registration budget#97943
vincentkoc merged 1 commit into
mainfrom
ci-limit-live-bucket

Conversation

@vincentkoc

Copy link
Copy Markdown
Member

What Problem This Solves

Resolves a problem where OpenClaw CI operator guidance still used the older 3,000-runner-registration bucket and 2,000-registration target after live GitHub rate-limit evidence showed the current bucket is higher.

Why This Change Was Made

The CI runbook and maintainer skill now tell operators to verify actions_runner_registration live before tuning and to target about 60% of the live bucket. With the current 10,000-registration bucket, the documented operating target is 6,000 registrations per 5 minutes.

User Impact

Maintainers get current, reusable guidance for right-sizing Blacksmith-backed CI without overreacting to stale quota numbers. No workflow behavior changes in this PR.

Evidence

  • ghx api rate_limit --jq '{core:.resources.core,graphql:.resources.graphql,actions_runner_registration:.resources.actions_runner_registration}' reported actions_runner_registration.limit=10000 and remaining=10000 during this pass.
  • node scripts/run-vitest.mjs test/scripts/ci-workflow-guards.test.ts
  • node scripts/check-workflows.mjs
  • node scripts/docs-list.js
  • ./node_modules/.bin/oxfmt --check .github/workflows/ci.yml .github/workflows/codeql-critical-quality.yml docs/ci.md test/scripts/ci-workflow-guards.test.ts .agents/skills/openclaw-ci-limits/SKILL.md .agents/skills/openclaw-ci-limits/agents/openai.yaml
  • git diff --check
  • .agents/skills/autoreview/scripts/autoreview --mode local clean, no accepted/actionable findings

@openclaw-barnacle openclaw-barnacle Bot added docs Improvements or additions to documentation size: XS maintainer Maintainer-authored PR labels Jun 29, 2026
@clawsweeper

clawsweeper Bot commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Codex review: needs maintainer review before merge. Reviewed June 29, 2026, 7:25 PM ET / 23:25 UTC.

Summary
The branch updates the CI runbook and openclaw-ci-limits skill from fixed 3,000/2,000 runner-registration guidance to live actions_runner_registration checks with a 10,000/6,000 current target.

PR surface: Docs +7. Total +7 across 2 files.

Reproducibility: not applicable. for a docs/skill PR. The stale current-main text is source-verifiable in docs/ci.md and .agents/skills/openclaw-ci-limits/SKILL.md, and the live rate_limit check verifies the current 10,000 bucket claim.

Review metrics: 1 noteworthy metric.

  • CI runner budget guidance: 1 operating target changed across 2 guidance surfaces. The PR changes maintainer operating guidance from a fixed 2,000 registrations to 60% of the live bucket, which affects future CI tuning decisions even though this head does not change workflows.

Merge readiness
Overall: 🦞 diamond lobster
Proof: 🌊 off-meta tidepool
Patch quality: 🦞 diamond lobster
Result: ready for maintainer review.

Overall follows the weaker of proof and patch quality, so missing proof can cap an otherwise strong patch.

Risk before merge

  • [P1] The remaining non-code risk is operator guidance: if GitHub changes or scopes the runner-registration bucket differently, the 60%/6,000 target could be too aggressive for future CI tuning.

Maintainer options:

  1. Confirm and land guidance (recommended)
    A maintainer confirms the current live bucket and 60% target are the intended operating baseline, then lands this docs/skill-only PR while keeping workflow-cap changes separate.
  2. Pause for quota ambiguity
    If the live bucket scope is uncertain, hold this PR and keep the conservative 2,000 target until the runner-registration limit is rechecked.

Next step before merge

  • [P2] The PR has protected maintainer ownership and changes CI operating guidance; no narrow automated repair is identified beyond maintainer approval and normal checks.

Security
Cleared: The current diff is limited to Markdown docs and agent skill guidance, with no executable workflow, dependency, permission, secret-handling, or supply-chain change.

Review details

Best possible solution:

Land the docs/skill update only after a maintainer confirms the 10,000 live bucket and 60% target are the intended CI operating policy; keep workflow cap changes separate.

Do we have a high-confidence way to reproduce the issue?

Not applicable for a docs/skill PR. The stale current-main text is source-verifiable in docs/ci.md and .agents/skills/openclaw-ci-limits/SKILL.md, and the live rate_limit check verifies the current 10,000 bucket claim.

Is this the best way to solve the issue?

Yes. Updating the public CI runbook and matching maintainer skill is the narrowest maintainable fix for stale operator guidance; workflow cap changes should remain in a separate guarded PR.

AGENTS.md: found and applied where relevant.

Codex review notes: model internal, reasoning high; reviewed against 052dc59526a2.

Label changes

Label changes:

  • add P3: This is a low-risk maintainer docs/skill update for CI operator guidance with no runtime or workflow behavior change in the current head.
  • add merge-risk: 🚨 automation: The PR changes CI automation operating guidance that maintainers and agents may use for future runner-budget tuning, and green CI does not prove the live GitHub quota remains stable.

Label justifications:

  • P3: This is a low-risk maintainer docs/skill update for CI operator guidance with no runtime or workflow behavior change in the current head.
  • merge-risk: 🚨 automation: The PR changes CI automation operating guidance that maintainers and agents may use for future runner-budget tuning, and green CI does not prove the live GitHub quota remains stable.
  • rating: 🦞 diamond lobster: Overall readiness is 🦞 diamond lobster; proof is 🌊 off-meta tidepool and patch quality is 🦞 diamond lobster.
  • status: 👀 ready for maintainer look: ClawSweeper has no concrete contributor-facing blocker left for this PR. Not applicable: This is a maintainer/MEMBER docs and skill PR, so the external contributor real-behavior proof gate does not apply; the PR body includes live API output and validation commands.
Evidence reviewed

PR surface:

Docs +7. Total +7 across 2 files.

View PR surface stats
Area Files Added Removed Net
Source 0 0 0 0
Tests 0 0 0 0
Docs 2 20 13 +7
Config 0 0 0 0
Generated 0 0 0 0
Other 0 0 0 0
Total 2 20 13 +7

What I checked:

  • Live PR state: Live GitHub metadata shows the PR is open, mergeable, labeled maintainer, and currently at head 05c0f492969e68b19f8981a3693aa529ca16ebad with only .agents/skills/openclaw-ci-limits/SKILL.md and docs/ci.md modified. (05c0f492969e)
  • Current main stale public docs: Current main still says the runner-registration bucket allows 3,000 self-hosted registrations per 5 minutes and sets the target below 2,000 registrations. Public docs: docs/ci.md. (docs/ci.md:148, 052dc59526a2)
  • Current main stale skill guidance: Current main's CI limits skill repeats the 3,000 bucket and 2,000 operating target in its Core Facts and budget math. (.agents/skills/openclaw-ci-limits/SKILL.md:16, 052dc59526a2)
  • PR diff scope: The current diff updates only the two guidance surfaces to check actions_runner_registration live and use about 60% of the live bucket, with 10,000/6,000 as the current example. Public docs: docs/ci.md. (docs/ci.md:146, 05c0f492969e)
  • Live quota check: A read-only rate_limit API call returned actions_runner_registration.limit=10000 and remaining=10000, matching the PR body and proposed current target.
  • Workflow guard context: Current tests still pin the CI fan-out caps and hosted runner-admission debounce, which supports the PR body's claim that this head changes guidance rather than workflow behavior. (test/scripts/ci-workflow-guards.test.ts:178, 052dc59526a2)

Likely related people:

  • vincentkoc: Prior merged work introduced and adjusted the CI runner-registration cap guidance across the skill, public docs, workflow, and guard test; inclusion is based on that merged history, not just authorship of this PR. (role: recent CI-limits owner; confidence: high; commits: 137371c64f25, 6612f6b1bdea, a82902c725b2; files: .agents/skills/openclaw-ci-limits/SKILL.md, docs/ci.md, .github/workflows/ci.yml)
  • cxbAsDev: Current blame for the stale 3,000/2,000 guidance maps to commit 0ce10d779302bb12f3f287bb519299a11142fb5b; this is weaker routing evidence because the commit appears to be a broad docs/skill import rather than focused CI ownership. (role: line-level current-main provenance; confidence: low; commits: 0ce10d779302; files: docs/ci.md, .agents/skills/openclaw-ci-limits/SKILL.md)
What the crustacean ranks mean
  • 🦀 challenger crab: rare, exceptional readiness with strong proof, clean implementation, and convincing validation.
  • 🦞 diamond lobster: very strong readiness with only minor maintainer review expected.
  • 🐚 platinum hermit: good normal PR, likely mergeable with ordinary maintainer review.
  • 🦐 gold shrimp: useful signal, but proof or patch confidence is still limited.
  • 🦪 silver shellfish: thin signal; proof, validation, or implementation needs work.
  • 🧂 unranked krab: not merge-ready because proof is missing/unusable or there are serious correctness or safety concerns.
  • 🌊 off-meta tidepool: rating does not apply to this item.

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
  • ClawSweeper keeps one durable marker-backed review comment per issue or PR.
  • Re-runs edit this comment so the latest verdict, findings, and automation markers stay together instead of adding duplicate bot comments.
  • A fresh review can be triggered by eligible @clawsweeper re-review comments, exact-item GitHub events, scheduled/background review runs, or manual workflow dispatch.
  • PR/issue authors and users with repository write access can comment @clawsweeper re-review or @clawsweeper re-run on an open PR or issue to request a fresh review only.
  • Maintainers can also comment @clawsweeper review to request a fresh review only.
  • Fresh-review commands do not start repair, autofix, rebase, CI repair, or automerge.
  • Maintainer-only repair and merge flows require explicit commands such as @clawsweeper autofix, @clawsweeper automerge, @clawsweeper fix ci, or @clawsweeper address review.
  • Maintainers can comment @clawsweeper explain to ask for more context, or @clawsweeper stop to stop active automation.

@vincentkoc
vincentkoc marked this pull request as ready for review June 29, 2026 23:05
@vincentkoc vincentkoc self-assigned this Jun 29, 2026
@clawsweeper clawsweeper Bot added rating: 🦞 diamond lobster Very strong PR readiness with only minor maintainer review expected. status: 👀 ready for maintainer look ClawSweeper has no concrete contributor-facing blocker left for this PR. labels Jun 29, 2026
@vincentkoc
vincentkoc force-pushed the ci-limit-live-bucket branch from 34f1c7b to 8efcfbe Compare June 29, 2026 23:09
@vincentkoc
vincentkoc requested a review from a team as a code owner June 29, 2026 23:18
@github-actions github-actions Bot added the dependencies-changed PR changes dependency-related files label Jun 29, 2026
@github-actions

github-actions Bot commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Dependency graph guard cleared

This PR no longer has blocked dependency graph changes. A future dependency graph change requires a fresh /allow-dependencies-change comment after the guard blocks that new head SHA.

  • Current SHA: 05c0f492969e68b19f8981a3693aa529ca16ebad

@vincentkoc
vincentkoc force-pushed the ci-limit-live-bucket branch from 731dd92 to 30f68b5 Compare June 29, 2026 23:19
@github-actions github-actions Bot removed the dependencies-changed PR changes dependency-related files label Jun 29, 2026
@vincentkoc
vincentkoc merged commit 18b2ff6 into main Jun 29, 2026
46 of 48 checks passed
@vincentkoc
vincentkoc deleted the ci-limit-live-bucket branch June 29, 2026 23:27
@vincentkoc

Copy link
Copy Markdown
Member Author

Merged via squash.

github-actions Bot pushed a commit to Desicool/openclaw that referenced this pull request Jun 30, 2026
QiuYuang pushed a commit to QiuYuang/openclaw that referenced this pull request Jul 1, 2026
chenyangjun-xy pushed a commit to chenyangjun-xy/openclaw that referenced this pull request Jul 1, 2026
chenyangjun-xy pushed a commit to chenyangjun-xy/openclaw that referenced this pull request Jul 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Improvements or additions to documentation maintainer Maintainer-authored PR rating: 🦞 diamond lobster Very strong PR readiness with only minor maintainer review expected. size: XS status: 👀 ready for maintainer look ClawSweeper has no concrete contributor-facing blocker left for this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant