Repository navigation
ci: Skip macOS tests in forks - #27017
Merged
Merged
Conversation
github-actions
Bot
added
documentation
Member
Author
sdf-jkl
marked this pull request as ready for review
November 2, 2026 19:58
AdamGS
approved these changes
Nov 2, 2026
AdamGS
left a comment
AdamGS
left a comment
Contributor
There was a problem hiding this comment.
Also ran into this issue multiple times today, thanks!
Member
Author
alamb
approved these changes
Nov 2, 2026
| your fork. The merge queue runs the full workflow on the merged result before | ||
| a pull request merges, so a green fork run does not replace it. | ||
|
|
||
| macOS tests skip fork runs to avoid runner timeouts. They remain required in the |
Contributor
There was a problem hiding this comment.
it also reduces the CI time for forks
Contributor
|
Thank you @sdf-jkl |
2010YOUY01
approved these changes
Nov 3, 2026
2010YOUY01
left a comment
2010YOUY01
left a comment
Member
There was a problem hiding this comment.
Makes sense, thank you!
Question: Were the failures caused by flaky GitHub Actions runners?
Contributor
|
yes, we saw multiple jobs timeout getting a run runner. I think they’re just aren’t that many available and public forks that don’t pay probably get the lowest priority. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Which issue does this PR close?
Related to #25148.
Rationale for this change
Fork CI can fail waiting for a macOS runner before any tests start, blocking the
Rust (fork)check.What changes are included in this PR?
Skip macOS tests in fork repositories, register the condition with the existing workflow validator, and document the exclusion in the contributor guide. Upstream coverage is unchanged, including the required merge-queue check, following apache/arrow-rs#11991.
Windows remains unchanged; its job is already disabled.
What is the testing strategy for this PR?
Existing workflow-validator tests, all-features Clippy, and all checks in
./dev/rust_lint.shpass. No new tests: the existing validator checks exact job conditions against the policy table.Are there any user-facing changes?
Contributors no longer need a macOS runner for fork CI. No runtime or API changes.