-
-
Notifications
You must be signed in to change notification settings - Fork 9.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Deploy Sequoia runner, 🔪 Monterey #18314
Conversation
04baa50
to
9141bb0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @carlocab! Unclear from the code but: we should only do this for Apple Silicon on Sequoia, not Intel. Intel bottles should only be manually dispatched and not added into the normal rotation.
Yes, already handled in brew/Library/Homebrew/github_runner_matrix.rb Line 184 in c5d09b4
|
@carlocab Great, thanks, ship away! |
Actually, I think we're in good shape now to deploy the ARM Sequoia runner on all homebrew-core PRs. We usually do it at around 60% of formulas bottled, and now we're at 57%. Moreover, we have relatively few breakages (and they're often easy to fix with clang flags) so it should be fine at this point. I'd suggest making this unconditional, and stoping macOS 12 CI altogether. |
Cool, will adjust. Note: may take me until tomorrow to refactor out test failures nicely. (Or I can hardcode more assumptions into tests if we're in a hurry.) |
8dab515
to
4ee4abe
Compare
We will now build on Ventura (macOS 13) to Sequoia (macOS 15). Runners for macOS 15-x86_64 will not be allocated by default due to a lack of appropriate hardware.
4ee4abe
to
51c4421
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. I'll let you judge when best to merge this, bearing in mind 4.3.21 which dropped the "no bottle available" error for macOS 12 can take up to 24 hours to reach everyone (on default settings).
Cool, let's wait till tomorrow then |
is this merge the cause of this job failure? https://github.com/Homebrew/brew/actions/runs/10854929440/job/30126441752?pr=18305 |
I think that'll have been #18296 and the latest tag. See changes to |
### Rationale for this change Homebrew dropped support for macOS 12 (Monterey): Homebrew/brew#18314 ### What changes are included in this PR? Use macOS 13 instead of macOS 12 as much as possible. ### Are these changes tested? Yes. ### Are there any user-facing changes? No. * GitHub Issue: #44206 Authored-by: Sutou Kouhei <kou@clear-code.com> Signed-off-by: Antoine Pitrou <antoine@python.org>
Homebrew has stopped building Monterey bottles (Homebrew/brew#18314)
Homebrew has stopped building Monterey bottles (Homebrew/brew#18314)
Homebrew has stopped building Monterey bottles (Homebrew/brew#18314)
brew style
with your changes locally?brew typecheck
with your changes locally?brew tests
with your changes locally?This will allow us to avoid the loss of bottles to Homebrew/core PRs.