Repository navigation
ci: recover owned order during overflow failover - #15620
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository: manaflow-ai/cmux/.coderabbit.yaml Review profile: ASSERTIVE Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
24fd106
into
ci/pickers-account-capacity
What changed
Follow up cmux#15614's outage routing fix for an explicitly Blacksmith-only
CI_PR_POOL_ORDER. During an activeCI_CLOUD_OVERFLOW_SAVEDrecord, the picker now rebuilds the owned order for the pinned PR Xcode when the configured order has no owned labels. It still refuses to emit a dead Blacksmith candidate and keeps the retry route owned.Validation
git diff --checkpass.This is a dependent follow-up on #15609; it contains only the small recovery correction.
Summary by cubic
Fixes the failover path so a hand-set Blacksmith-only pool order can't turn an active cloud overflow record into a direct, unmeasured runner route. When the configured order drops to zero owned labels, the picker now rebuilds the owned order for the pinned PR Xcode and keeps the retry route owned.
Written for commit 4c06d91. Summary will update on new commits.