Skip to content

ci: stop pinning remote-daemon macOS tests to the macOS 26 pool - #14128

Merged
teamleaderleo merged 1 commit into
mainfrom
claude/remote-daemon-background-lane
Sep 24, 2026
Merged

teamleaderleo merged 1 commit into
mainfrom
claude/remote-daemon-background-lane

Conversation

@teamleaderleo

@teamleaderleo teamleaderleo commented Sep 24, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

remote-daemon-macos-tests runs go -C daemon/remote test ./... twice, once with -race. It uses no Xcode and no GUI, so any Mac can run it. It was pinned to MACOS_RUNNER_26 (Blacksmith 26), the pool that pull-request app-host work queues on. At 03:37Z it was one of the jobs waiting on that pool; on #14107 it had been queued since 02:29Z.

It now uses the same routing as the other pull-request macOS jobs, such as cli-pipe-regressions:

The background lane (MACOS_RUNNER_BACKGROUND) would fit this job better, but check_background_macos_lane does not allow it in a workflow_call workflow that pull requests reach.

Testing

  • tests/test_ci_self_hosted_guard.sh, tests/test_ci_fork_runner_routing.py, tests/test_ci_change_areas.py, tests/test_runner_label_policy.py and tests/test_ci_linux_guard_routing.py pass.
  • actionlint is clean on remote-daemon.yml.

Checklist

  • I added or updated tests for behavior changes (routing only; the existing guards cover it)
  • I updated docs/changelog if needed (none needed)

🤖 Generated with Claude Code

https://claude.ai/code/session_01EduXdN9PKnGsMQztJK7WeE


Generated by Claude Code


Summary by cubic

Stops routing remote-daemon-macos-tests to the contended macOS 26 pool, which was causing long queue times on pull requests.

The job runs plain go test with no Xcode or GUI, so it now follows the same routing as the other pull-request macOS jobs: MACOS_RUNNER_PR (fallback blacksmith-6vcpu-macos-15) on pull requests, blacksmith-6vcpu-macos-15 on other events, and hosted macos-15 on forks.

Written for commit 38f1868. Summary will update on new commits.

Review in cubic

remote-daemon-macos-tests runs plain go test with no Xcode or GUI, but
was pinned to MACOS_RUNNER_26 (Blacksmith 26), the pool pull-request
app-host work queues on. It now follows the pull-request lane
(MACOS_RUNNER_PR) on pull requests and Blacksmith 15 otherwise, like the
other pull-request macOS jobs. Forks keep hosted macos-15.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01EduXdN9PKnGsMQztJK7WeE
@cursor

cursor Bot commented Sep 24, 2026

Copy link
Copy Markdown

Bugbot is paused — on-demand spend limit reached

Bugbot uses usage-based billing for this team and has hit its on-demand spend limit.

A team admin can raise the spend limit in the Cursor dashboard, or wait for the next billing cycle to continue.

@github-actions

Copy link
Copy Markdown
Contributor

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@teamleaderleo
teamleaderleo merged commit 35a6bb1 into main Sep 24, 2026
20 of 21 checks passed
@coderabbitai

coderabbitai Bot commented Sep 24, 2026

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration

Configuration used: Repository: manaflow-ai/cmux/.coderabbit.yaml

Review profile: ASSERTIVE

Plan: Advanced

Run ID: d08a51f4-1efd-476a-b9f7-cab37d4e4630

📥 Commits

Reviewing files that changed from the base of the PR and between 1ba6d77 and 38f1868.

📒 Files selected for processing (1)
  • .github/workflows/remote-daemon.yml
 ______________________________
< Blink twice if you're a bug. >
 ------------------------------
  \
   \   \
        \ /\
        ( )
      .( o ).
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

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.

rustybret pushed a commit to rustybret/bmux that referenced this pull request Sep 24, 2026
5b646b7 ci: apply the queue janitor threshold per runner pool (manaflow-ai#14131)
d49a1b1 ci: reuse the headless cmux-tui build in SDK conformance (manaflow-ai#14108)
ba85a1b ci: key reload-build caches on the commit and fall back across branches (manaflow-ai#14099)
27fb3bf ci: hand focused test-macos-suite dispatches to run-e2e.sh (manaflow-ai#14075)
d18c1b9 ci: let a failed compile admission mark a run doomed for the queue janitor (manaflow-ai#14129)
dfdce2c ci: bind pull request product reuse to the merge it compiled (manaflow-ai#14080)
afacff3 ci: sparse-checkout the Claude wrapper regression job (manaflow-ai#14088)
35a6bb1 ci: stop pinning remote-daemon macOS tests to the macOS 26 pool (manaflow-ai#14128)
1ba6d77 ci: run macOS jobs on GitHub-hosted runners alongside Blacksmith (manaflow-ai#14097)
587de87 Import CmuxWorkspaces where CodexTurnRestoreIntentPolicy names its liveness type (manaflow-ai#14123)

# Conflicts:
#	.github/workflows/ci-guards.yml
#	.github/workflows/ci-macos.yml
#	.github/workflows/ci-queue-janitor.yml
#	.github/workflows/ci.yml
#	.github/workflows/cmux-tui-sdks.yml
#	.github/workflows/reload-build.yml
#	.github/workflows/remote-daemon.yml
#	.github/workflows/test-macos-suite.yml
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