Skip to content

feat: add fast variant of CI build and smoke test - #109

Open
john00003 wants to merge 3 commits into
mainfrom
feat/dist-build-fast
Open

feat: add fast variant of CI build and smoke test#109
john00003 wants to merge 3 commits into
mainfrom
feat/dist-build-fast

Conversation

@john00003

Copy link
Copy Markdown
Collaborator

Motivation

We want a faster version of the dist-build and smoke-test Makefile targets.

Technical Details

The dist-build-fast target improves the build time by:

  1. compiling NIXL/UCX with optimizations and other debug tools disabled
  2. compiling only for gfx950 (the machines our CI targets)

Additionally, we disable building the tests/examples for both hipFile and NIXL to reduce the build time.

Currently the smoke-test-fast Makefile target doesn't run any different tests than the normal variant, it just uses the docker container built by the dist-build-fast variant.

Test Plan

Try the dist-build-fast and smoke-test-fast, ensure the build and tests pass. Measure the amount of time the reduced Makefile targets take compared to the regular targets.

Test Result

The reduced compile targets and disabling the tests and examples save ~30s. The reduced gfx compile targets save a couple minutes.

Submission Checklist

@john00003

Copy link
Copy Markdown
Collaborator Author

/run-ci

@john00003

Copy link
Copy Markdown
Collaborator Author

We'll need to update the workflows in /usr/ on the CI runners in order for the CI changes to reflect.

@github-actions

Copy link
Copy Markdown

✅ Hardware CI passed for dafc0cd (dist-build + smoke-test + tiny-test).

Comment thread .github/scripts/spur-dist-build.sh Outdated
Comment thread .github/scripts/spur-smoke-test.sh Outdated
Comment thread .github/workflows/aic-amd-dist-build-fast.yml
Comment thread docker/docker-compose.yml Outdated
Comment thread docker/Dockerfile Outdated
@john00003
john00003 force-pushed the feat/dist-build-fast branch from 2b207b0 to 3d69848 Compare July 29, 2026 21:02
@john00003

john00003 commented Jul 29, 2026

Copy link
Copy Markdown
Collaborator Author

@sbates130272 I addressed your comments! Sorry, I accidentally squashed the fixup commits, but I tried to address all of your comments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants