Skip to content

ci(signing): the owned signing pick accepts the nightly slot label - #27113

Merged
teamleaderleo merged 1 commit into
feat-cmux-nextfrom
leo/signing-nightly-slot
Oct 28, 2026
Merged

teamleaderleo merged 1 commit into
feat-cmux-nextfrom
leo/signing-nightly-slot

Conversation

@teamleaderleo

Copy link
Copy Markdown
Collaborator

Follow-up to #26944. An independent review at #26944's head (posted just before it landed) found that scripts/ci/signing_runner.py only accepted persistent pool labels. vars.CI_NIGHTLY_NEXT_BUILD_RUNNER may name glaeda's nightly slot label (glaeda-nightly-<class>-xcode-<ver>). When it does, the owned pick came back empty and signing fell back to Blacksmith.

Leo, 2026-10-28: "ok. idk what the "risk" was before but like.. we can just read the code and stuff."

What changed

  • The picker now accepts the same owned labels as nightly_build_runner.owned_label: a persistent pool or the nightly slot. A signing job on that runner is still admitted only as glaeda's exclusive class, and the claim gate still requires it.
  • tests/test_ci_signing_owned_mac.py covers a drained nightly-slot runner.

Before / after

  • Before: when the nightly build label is the nightly slot, the owned pick was empty, so signing went to Blacksmith.
  • After: an idle nightly-slot mini is offered like any drained owned mini.

Owned signing stays off until CI_SIGN_OWNED=1. Risk: low.

Agent session: agent session

🤖 Generated with Claude Code

…el, as the nightly build picker does

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Agent-Session: https://cmux.com/deeplink/agent?id=aid-85ca59f74f02d81734c3
@teamleaderleo

Copy link
Copy Markdown
Collaborator Author

Owned by an agent session · session

@github-actions

Copy link
Copy Markdown
Contributor

cmux status (one comment, edited in place; each line is updated by the job that owns it)

Review 8cae0db: HQ feed card · gallery before/after runs · checks

App of this head (app.zip, kept 1 day)

gh workflow run reload-build.yml -R manaflow-ai/cmux --ref feat-cmux-next -f tag=pr27113 -f ref=leo/signing-nightly-slot -f platform=macos -f backend_mode=local

@teamleaderleo

Copy link
Copy Markdown
Collaborator Author

Bead cx-nnymlp created for this PR by the cmux coordinator.

@teamleaderleo
teamleaderleo merged commit 53ab8e2 into feat-cmux-next Oct 28, 2026
129 of 130 checks passed
@teamleaderleo
teamleaderleo deleted the leo/signing-nightly-slot branch October 28, 2026 23:44
@github-actions

Copy link
Copy Markdown
Contributor

Shipped in:

This branch was previously deployed

1 inactive deployment
ci-routing-agent — 8cae0db6 Deployed Oct 28, 2026 by teamleaderleo via wait for same-key tree publisher #73883
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk:high Risk tier (docs/ci/risk-tiers.md)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant