-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
GH-40536: [CI]: Migrate remaining jobs away from self-hosted mac runners. #40537
Conversation
@github-actions crossbow submit verify-rc-binaries-wheels-macos-14-arm64 r-binary-packages |
|
This comment was marked as outdated.
This comment was marked as outdated.
@github-actions crossbow submit verify-rc-binaries-wheels-macos-14-arm64 --param release=15.0.2 --param rc=3 |
ugh, brew doesn't support macos 11 anymore so it tries to build sccache from source...
Lets just bump to 12, 11 will be removed in Q2 anyway: actions/runner-images#9255 |
Revision: 346c036 Submitted crossbow builds: ursacomputing/crossbow @ actions-1797eb513d
|
@github-actions crossbow submit r-binary-packages |
Revision: 661fa28 Submitted crossbow builds: ursacomputing/crossbow @ actions-52e3f3edc1
|
verify-rc-binaries-wheels-macos-14-arm64 worked. I'll unify |
@github-actions crossbow submit verify-rc--macos- --param release=15.0.2 --param rc=3 |
This comment was marked as outdated.
This comment was marked as outdated.
Because preinstalled Homebrew's Python isn't latest.
@github-actions crossbow submit verify-rc--macos- --param release=15.0.2 --param rc=3 |
This comment was marked as outdated.
This comment was marked as outdated.
Revision: 9a5ecad Submitted crossbow builds: ursacomputing/crossbow @ actions-98b9c07bdc |
verify-rc-source-cpp-macos-arm64 and verify-rc-source-cpp-macos-conda-amd64 failures will be passed on main because main has 5127ef0#diff-aa8c7db0f10c202b555dd103c54828afa0e8cbd781e87694b5ac4705beba2cdeR1068 . |
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.
+1
Thanks both @kou and @assignUser for the effort! |
After merging your PR, Conbench analyzed the 5 benchmarking runs that have been run so far on merge-commit 9035c4e. There were no benchmark performance regressions. 🎉 The full Conbench report has more details. It also includes information about 18 possible false positives for unstable benchmarks that are known to sometimes produce them. |
Rationale for this change
CRAN no longer builds on 10.13 and gh provides M1 runners now.
What changes are included in this PR?
use m1 for verify-rc-binaries-wheels-macos-14-arm64:
and macos-11 for r-binary-packages
Are these changes tested?
crossbow