Skip to content

docs(evidence): close out the CI queue re-starvation — PR #206's unregistered runner label is the queue source - #214

Merged
nish3451 merged 1 commit into
mainfrom
ci/starvation-requeue-cleanup-lane1-20260815
Aug 14, 2026
Merged

docs(evidence): close out the CI queue re-starvation — PR #206's unregistered runner label is the queue source#214
nish3451 merged 1 commit into
mainfrom
ci/starvation-requeue-cleanup-lane1-20260815

Conversation

@nish3451

Copy link
Copy Markdown
Collaborator

Closes the lane-1 re-starvation item: "The CI queue has re-starved past the closed 1126 acceptance: 9 runs queued up to 2h03m".

Root cause (live-verified)

Main already routes both required jobs across two runner instances via the shared vps-verify label — runs execute on both netcup-rs2000-tinystudio-verify1 and -verify2 (verified on runs 31846412323 and 31846416272).

The re-starvation is PR #206's per-instance matrix: it routes to label tinystudio-io-verify1, which no runner registers (verify1's per-instance label is tinystudio-io; verify2's is tinystudio-io-verify2). Every push to that branch spawns 2 matrix legs that can never be scheduled — 7 of the 9 queued runs (18:00–21:01Z) are #206's branch. Its check rollup shows the verify1 legs QUEUED forever.

Change

  • No workflow change: main's shared vps-verify label already engages both runners (the correct capacity fix for item 1126).
  • Supersedes and closes PR ci: spread verify and Gitleaks across two runner instances #206 (its intended outcome is already live; its matrix is unmergeable by design).
  • Records the evidence in .lane/reports/ci-starvation-requeue-cleanup-lane1-20260815.md.

Verification

  • gh api .../actions/runners: verify1 [self-hosted, Linux, X64, vps-verify, tinystudio-io], verify2 [..., tinystudio-io-verify2] — no tinystudio-io-verify1 anywhere.
  • gh run list --status queued: 9 queued, 7 from the ci: spread verify and Gitleaks across two runner instances #206 branch.
  • main-tip checks on e0ee160: verify success, Gitleaks success.

…gistered runner label is the queue source

Main's shared vps-verify label already routes verify/Gitleaks across both
runner instances (netcup-rs2000-tinystudio-verify1/2, live-verified). The
re-starvation backing item 1126's closeout is PR #206's per-instance
matrix: its tinystudio-io-verify1 leg matches no registered runner and
queues forever on every push to that branch (9 queued runs, oldest 2h03m).

Co-authored-by: CommandCodeBot <noreply@commandcode.ai>
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Your trial has ended. Reactivate Greptile to resume code reviews.

@coderabbitai

coderabbitai Bot commented Aug 14, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@nish3451, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 46 minutes

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Free

Run ID: 8809626c-9836-46e7-ab38-62c2768789fa

📥 Commits

Reviewing files that changed from the base of the PR and between e0ee160 and 94d0228.

📒 Files selected for processing (1)
  • .lane/reports/ci-starvation-requeue-cleanup-lane1-20260815.md

Note

🎁 Summarized by CodeRabbit Free

Your organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login.

Comment @coderabbitai help to get the list of available commands.

@nish3451
nish3451 merged commit e7777f3 into main Aug 14, 2026
3 checks passed
nish3451 added a commit that referenced this pull request Aug 15, 2026
…ved by PR #214 close-out (#217)

Live queue check (2026-08-14 23:50Z): 11 queued runs all seconds old on
normal PR branches, zero runs on PR #206's branch, both vps-verify runners
online and engaged (main push on verify1, PR #112 on verify2). The item's
2h03m / 9-run observation was PR #206's unregistered tinystudio-io-verify1
matrix label, closed out by PR #214; the queue has not re-starved since.

Co-authored-by: nish3451 <nish3451@users.noreply.github.com>
Co-authored-by: CommandCodeBot <noreply@commandcode.ai>
nish3451 added a commit that referenced this pull request Aug 15, 2026
…ved (queue empty at 03:45Z) (#224)

* docs(evidence): re-verify CI queue re-starvation item — already resolved, queue empty at 03:45Z

Live check (2026-08-15 03:45Z): zero queued and zero in-progress runs, both
vps-verify runners online and engaged (recent main push drained in ~16s, the
03:01Z 8-PR batch in <=9m across both runners). The item's 9-run / 2h03m
observation was PR #206's unregistered tinystudio-io-verify1 matrix label,
closed out by PR #214 and re-verified by PR #217; PR #206 is CLOSED, no open
PR touches .github/workflows/*. No repo change required.

Co-authored-by: CommandCodeBot <noreply@commandcode.ai>

* docs(evidence): record 04:01Z wave observation — queue drained, PR #224 CI green

Observed the 04:01Z concurrent 8-run wave live: 4 queued at peak, oldest wait
~14m, fully drained by 04:15:28Z. PR #224's own verify+Gitleaks passed
(verify on netcup-rs2000-tinystudio-verify1, Gitleaks picked up at 04:10Z).
Max wait orders of magnitude below the item's 2h03m; no re-starvation.

Co-authored-by: CommandCodeBot <noreply@commandcode.ai>

---------

Co-authored-by: nish3451 <nish3451@users.noreply.github.com>
Co-authored-by: CommandCodeBot <noreply@commandcode.ai>
nish3451 added a commit that referenced this pull request Aug 21, 2026
…ved (both runners online, queue drains in minutes) (#283)

Re-confirms PRs #214/#217/#224: main's shared vps-verify label reaches
both self-hosted runners; the ~60 queued observation was the 19:01Z
wave of ~30 PRs × 2 jobs, drained in ~10 minutes with zero queue left.
No workflow change required.

Co-authored-by: minimax-vps <minimax-vps@local>
Co-authored-by: CommandCodeBot <noreply@commandcode.ai>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant