Skip to content

ci: give an owned Mac's full app rebuild 35 minutes to compile - #14594

Merged
teamleaderleo merged 1 commit into
mainfrom
ci/owned-compile-timeout-full-rebuild
Sep 25, 2026
Merged

teamleaderleo merged 1 commit into
mainfrom
ci/owned-compile-timeout-full-rebuild

Conversation

@teamleaderleo

@teamleaderleo teamleaderleo commented Sep 25, 2026 •

Copy link
Copy Markdown
Collaborator

Why

The owned-Mac compile step is capped at 20 minutes. A PR that touches a package or CLI/cmux.swift rebuilds the whole app even from warm DerivedData (kept_rebuilds_app: true), and on the slower minis that takes 19 to 21 minutes:

So a PR like #13877 cannot go green on a slow mini, no matter how many times it retries.

Change

Raise the glaeda cap from 20 to 35 minutes. The cap stays for the hung-compile case (job 107921038609 sat 63 minutes). The job limit is 75, so there's room.

🤖 Generated with Claude Code


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.


Summary by cubic

Raises the glaeda Mac compile step timeout from 20 to 35 minutes so full app rebuilds (triggered by package or CLI/cmux.swift changes) on slower minis—which take 19 to 21 minutes even from warm DerivedData—stop timing out. Keeps a cap for hung-compile cases, which have sat at 63 minutes.

Written for commit a31be7f. Summary will update on new commits.

Review in cubic

A package or CLI/cmux.swift change rebuilds the whole app even from warm
DerivedData. That takes 19 to 21 minutes on the slower minis, so the
20-minute step cap failed those PRs (#13877 twice) while 13287's passed
at 19.1. Keep a cap for the hung-compile case, at 35.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Sep 25, 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: a160c9a2-a8a0-4329-842e-5393721aa2e4

📥 Commits

Reviewing files that changed from the base of the PR and between 6b6fdc3 and a31be7f.

📒 Files selected for processing (1)
  • .github/workflows/ci-macos.yml
 ___________________________________________________________________________________________________________________________________
< Don't use manual procedures. A shell script or batch file will execute the same instructions, in the same order, time after time. >
 -----------------------------------------------------------------------------------------------------------------------------------
  \
   \   \
        \ /\
        ( )
      .( 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.

@teamleaderleo
teamleaderleo merged commit e3acb25 into main Sep 25, 2026
45 of 52 checks passed
@teamleaderleo
teamleaderleo deleted the ci/owned-compile-timeout-full-rebuild branch September 25, 2026 12:14
@github-actions

Copy link
Copy Markdown
Contributor

Merge receipt for a31be7fa05, merged 2026-09-25 12:14:20 UTC

  • Not verified at merge: ci-status (not reported), macOS compile admission (in progress), CLA Assistant (failure), CLA policy guard (failure)
  • Verified: Web complexity, web-validation, Fast static checks, GhosttyKit release check, guards (16), linux-preflight, macOS admission gate, receipt-contract, Testbox broker trust boundary, transport
  • Skipped by policy: browser, Claude wrapper regressions, remote-daemon, suite-coverage, swift-package-tests, web, web-build, web-database-tests, web-tests
  • Full suite: runs on main after merge.

Labeled merged-unverified: if main breaks near this merge, look here first.

@github-actions github-actions Bot added the merged-unverified A judging check was not green at merge; see the merge receipt comment label Sep 25, 2026
rustybret pushed a commit to rustybret/bmux that referenced this pull request Sep 25, 2026
c055747 ci: parse runner expressions in the fork guard, and gate LINUX_RUNNER on fork PRs (manaflow-ai#14192)
fc112a8 docs(testing): describe how run-e2e.sh actually picks the runner (manaflow-ai#14620)
0f6edfa ci(owned): keep seeds until the disk is actually short, not at a fixed 6 (manaflow-ai#14621)
1b78097 CI: one owned-pool rescue sweeper instead of a rescue run per CI run (manaflow-ai#14602)
c7e79f6 Hold Files tree reloads while its context menu is open (manaflow-ai#14451)
dbb24cb ci: read the owned-pool rescue's Actions API through the route App (manaflow-ai#14499)
36b8063 ci: store each app-host product file once in the product archive (manaflow-ai#14601)
0dba677 CI: run CmuxWorkspaces package tests (manaflow-ai#14592)
2244e98 Fix color detection in native tmux mirrors (manaflow-ai#14175)
3d2478e build: keep every built file in a project group so Xcode reuses its build description (manaflow-ai#14486)
a1a5ae9 ci(cmux-tui): cache cargo builds in the Rust jobs (manaflow-ai#14613)
560e640 ci(owned): keep 8 seeds per mini and pick seeds by cost, not a 2-commit cap (manaflow-ai#14607)
561d317 cmux-debug-cli: find a publish-hq build in the HQ tag app cache (manaflow-ai#14579)
c0aaac7 fix(events): survive receive-timeout reconfiguration churn during replay (manaflow-ai#13888)
3bd994a fix: keep split zoom when the zoomed pane outlives a tab close (manaflow-ai#12853)
ca984c7 Session snapshots can send binding actions to ghostty on a surface that is no longer live (manaflow-ai#12623)
a47d65b settings: expose local tmux session persistence (manaflow-ai#13210)
e3acb25 ci: give an owned Mac's full app rebuild 35 minutes to compile (manaflow-ai#14594)

# Conflicts:
#	.github/workflows/ci-artifact-transport.yml
#	.github/workflows/ci-cache-receipts.yml
#	.github/workflows/ci-guards.yml
#	.github/workflows/ci-macos.yml
#	.github/workflows/ci-owned-pool-rescue.yml
#	.github/workflows/ci-web.yml
#	.github/workflows/ci.yml
#	.github/workflows/cloud-machine-tests.yml
#	.github/workflows/cloud-vm-guest-install.yml
#	.github/workflows/cloud-vm-image-contract.yml
#	.github/workflows/cloud-vm-image-reachability.yml
#	.github/workflows/cloudflare-relay.yml
#	.github/workflows/cmux-skill-contract.yml
#	.github/workflows/cmux-tui-sdks.yml
#	.github/workflows/cmux-tui-spec.yml
#	.github/workflows/cmux-tui.yml
#	.github/workflows/indexnow-tests.yml
#	.github/workflows/iroh-v2.yml
#	.github/workflows/localization-catalog.yml
#	.github/workflows/r2-upload-tests.yml
#	.github/workflows/remote-daemon.yml
#	.github/workflows/repair-nightly-appcast-content-types.yml
#	.github/workflows/required-checks-drift.yml
#	.github/workflows/resolve-dispatch-ref.yml
#	.github/workflows/seed-derived-data.yml
#	.github/workflows/terminal-hang-diagnostics.yml
#	.github/workflows/test-e2e.yml
#	.github/workflows/test-ios.yml
#	.github/workflows/testbox-broker-guard.yml
#	.github/workflows/web-validation.yml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merged-unverified A judging check was not green at merge; see the merge receipt comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant