Repository navigation
ci: route status reports through live light capacity - #34849
Conversation
Bugbot needs on-demand usage enabledBugbot uses usage-based billing for this team and requires on-demand usage to be enabled. A team admin can enable on-demand usage in the Cursor dashboard. |
|
Review (codex gpt-6-astra high): SHIP at abed44b Visual verdict: unchanged by this CI-only diff; no product UI is affected. Checked workflow wiring, runner declarations, YAML parsing, portable-placement tests, current-attempt guards, fork precedence, permissions, and kill-switch behavior. No correctness or regression findings. Follow-ups: none |
56a898a to
07f94f3
Compare
abed44b to
8b04512
Compare
Co-authored-by: Codex <199175422+chatgpt-codex-connector[bot]@users.noreply.github.com> Agent-Session: https://cmux.com/deeplink/agent?id=aid-11af0abc73e245862c06
Co-authored-by: Codex <199175422+chatgpt-codex-connector[bot]@users.noreply.github.com> Agent-Session: https://cmux.com/deeplink/agent?id=aid-11af0abc73e245862c06
8b04512 to
4ccf327
Compare
|
cmux status (one comment, edited in place; each line is updated by the job that owns it) Review App of this head (app.zip, kept 1 day)
Shipped in:
|
|
Found 6 test failures on Blacksmith runners: Failures
|
|
Review (codex gpt-6-astra high): SHIP at 4ccf327 Visual verdict: unchanged by this CI-only diff. Focused placement and routing tests pass. No correctness or regression findings. Follow-ups: none |
![Fix with [code]smith](https://pr-comments-assets.blacksmith.sh/codesmith/fix-with-codesmith-light-3x.png)
Problem
Status-report workflows were still choosing hosted or paid fallback routes even when the live fleet had capacity. The
pr-red-checks-commentandci-guardsreport consumers now use the same portable placement snapshot as light job execution, so an idle fleet Linux runner or owned mini is tried first for the current attempt.This PR is stacked on #34615 (
codex/light-routing) and should be retargeted tofeat-cmux-nextafter that PR lands.What changed
pr-red-checks-comment#commentandci-guards#guard-status.CI_RUNNER_STATEwithcontents: read; no App token, secret, or workflow permission was added.CI_PORTABLE_LIGHT_JOBS_OFF=1as an emergency kill switch; default behavior is on.The placement snapshot is intentionally fail-open for long-running workflows: a stale snapshot falls through to the existing fallback instead of blocking a report.
Performance
Wiring.test_status_reports_consume_live_capacity_without_a_routing_appLeo asked, “how’s CI? we still have bigger wins we can get? what’s the state of it all? more things we can skip/do better”. This slice removes avoidable hosted/paid placement for status reports while preserving a one-line rollback switch.
Validation
router-review(gpt-6-astra high): SHIP atabed44b8af99e8bb24ff14b5546de6b8a1b18d6a.🤖 Generated with Codex
Agent session: agent session
Open in [code]smith · Need help on this PR? Tag
@codesmith-botwith what you need.Summary by cubic
Moves
pr-red-checks-commentandci-guardsstatus reports to use live light capacity so an idle fleet Linux runner or owned mini is placed before hosted or paid fallbacks.CI_RUNNER_STATEwithcontents: read; no App token, secret, or new workflow permission, plus a wiring test asserting both reports consume an idle mini without secrets.CI_PORTABLE_LIGHT_JOBS_OFF=1is set.guard-statuswaits for the five guard jobs before placement, and the comment bootstrap runs only onfeat-cmux-nextbase PRs.Written for commit 4ccf327. Summary will update on new commits.