Skip to content

fix(ci): route trusted Strix metadata to control runners - #2436

Merged
seonghobae merged 2 commits into
mainfrom
fix/fmls2114-strix-control-routing
Sep 27, 2026
Merged

seonghobae merged 2 commits into
mainfrom
fix/fmls2114-strix-control-routing

Conversation

@seonghobae

Copy link
Copy Markdown
Contributor

Problem

fast-mlsirm #2220 current head 4eaeb799a6647ea29f3f4902d9ca79a1377e795c queues Strix admission job 108617323217 on ubuntu-24.04. The additional self-hosted control runners cannot accept that hosted-only selector.

Change

Reuse the deployed control allocation for four metadata-only jobs. Require exact central strix.yml@refs/heads/main identity and caller central or fast-mlsirm. Preserve hosted fallback for other sources/callers and the actual model scan image. No PR checkout, provider, deadline, verdict, credential or evidence rule changes.

Group 6 now permits only the additional central strix.yml@refs/heads/main source; every previous entry and restriction is preserved. Old queued jobs retain their workflow revision.

Verification

  • Routing regression fails on original source.
  • 21 affected runner, scope and dependency-hash tests pass.
  • actionlint and git diff --check pass.

Local verification is not a completed consumer gate. Delivery may use maintainer-authorized bypass for queued hosted checks; any bypass must be recorded as such.

@coderabbitai

coderabbitai Bot commented Sep 27, 2026 •

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

Next included review available in 39 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 8ec4211f-cee1-4236-aa15-ae5550f537a8

📥 Commits

Reviewing files that changed from the base of the PR and between fd2a03e and 5db8ff9.

📒 Files selected for processing (4)
  • .github/workflows/strix.yml
  • docs/doctoring/central-dedicated-runner-routing-20260927.md
  • tests/test_docs_only_pr_runner_admission.py
  • tests/test_required_review_runner_image_contract.py

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.

❤️ Share

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

@seonghobae
seonghobae merged commit 3b36b89 into main Sep 27, 2026
4 of 19 checks passed
@seonghobae
seonghobae deleted the fix/fmls2114-strix-control-routing branch September 27, 2026 12:35
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