CI: Add basic per-PR build + smoke test workflow - #9
Merged
Conversation
Adds a GitHub Actions workflow that runs on every pull request: CPU-only release build via autogen/configure-release, then a ucx_info smoke test. Starter scope only; gtest is intentionally excluded to keep runtime short. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
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
1 participant
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.
Goal
Add a basic per-PR CI check for this fork. Every pull request (any base branch) now runs a fast CPU-only build and a smoke test on
ubuntu-latest; the workflow can also be dispatched manually.What it does
Single job
build-smoke(timeout-minutes: 30, concurrency group cancels superseded runs on the same PR):autoconf,automake,libtool,pkg-config,libnuma-dev)../autogen.sh+./contrib/configure-release --without-verbs(the canonical release configure used by upstream'sbuildlib/tools/builds.sh;--without-verbssince GitHub runners have no RDMA stack).make -j$(nproc)../src/tools/info/ucx_info -vand-dto prove the build produced a working binary and enumerates transports.This is intentionally a starter stub: the gtest suite is too slow to build and run within a reasonable PR budget, so it is not included. The existing
blossom-ci.yml(NVIDIA's comment-triggered CI inherited from upstream) is untouched to keep upstream merges clean.Admin follow-up (cannot be done from a workflow file)
CPU build + ucx_info smoke testjob as a required status check via branch protection / ruleset onmaster.Test Plan
actionlinton the workflow file — clean.yaml.safe_load).ucx_inforeports library version 1.22.0 and enumerates transports.🤖 Generated with Claude Code