ci: add ubuntu-latest test pipelines - #19
Merged
guru-desh merged 7 commits intoJul 1, 2026
Merged
Conversation
guru-desh
force-pushed
the
u/gurudesh/make-test-suite-pass-for-linux
branch
from
June 30, 2026 21:17
9a50a0a to
883ce88
Compare
guru-desh
changed the title
ubuntu-latest test pipelines
guru-desh
force-pushed
the
u/gurudesh/make-test-suite-pass-for-linux
branch
2 times, most recently
from
June 30, 2026 21:33
a7c6cb5 to
e3e9072
Compare
guru-desh
marked this pull request as ready for review
June 30, 2026 21:34
dengqiaoyu
previously approved these changes
Jun 30, 2026
|
Can we expand this to run on linux/arm64? I was trying to verify the linux tests on a linux container on my M4 laptop, and couldn't get past the platform check. |
Add 4 GitHub-hosted Linux pipelines that run on ubuntu-latest: - highest-torch slow, highest-torch fast - lowest-torch slow, lowest-torch fast Each runs the corresponding make target with --junit and uploads its test-results artifact. Ungated and independent of the macOS self-hosted jobs so they also run on forks.
Introduce a tiered CI structure: - Tier 1 (ubuntu-latest, ungated): make check, make test-smoke, make test-tutorials. - Tier 2 (needs all of Tier 1): the 4 linux test pipelines and the macOS test run. Move `check` off the macOS self-hosted runner to ubuntu-latest and drop its repo gate so the gate also runs on forks (otherwise Tier 2 would be skipped there). test-smoke/test-tutorials targets are added by a separate change; until then those jobs fail and Tier 2 is skipped.
Port the scripts/make changes from coreai-opt#497 so INSTALL_PRECOMMIT is a real knob: setup_env.sh defaults it (false in --ensure mode, true otherwise) and threads it to install_pre_commit_hooks.sh, which skips `pre-commit install -f` when INSTALL_PRECOMMIT=false.
Skip git-hook configuration during env setup for the 4 linux test jobs, the macOS test run, test-smoke, and test-tutorials. `check` is left as-is.
GitHub Actions doesn't support YAML anchors, so collapse the 4 linux-*
test jobs into a single `linux-tests` job with a strategy.matrix over
{highest,lowest} x {slow,fast}, declaring needs:/env:/steps once. Also
drop the redundant "Stage 2 runs only if..." comment.
guru-desh
force-pushed
the
u/gurudesh/make-test-suite-pass-for-linux
branch
from
June 30, 2026 22:55
e3e9072 to
3647512
Compare
aseemw
reviewed
Jun 30, 2026
guru-desh
force-pushed
the
u/gurudesh/make-test-suite-pass-for-linux
branch
from
June 30, 2026 23:31
3647512 to
442ff94
Compare
Contributor
Author
|
aseemw
approved these changes
Jun 30, 2026
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
4 participants
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.
Adds new Linux CI pipelines for the following:
make test-smokemake test-tutorialsThe following CI pipelines were edited:
make checknow runs on Linux