Repository navigation
feat(rocshmem): add TheRock CI and Test Filter Standardization #5691
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
Changes from 4 commits
49cc26b
ba457d2
6df63da
f6cdf68
2deb912
812c28b
8554947
f643d64
5578d0d
e8211e4
4a01bcc
f623822
18a7530
db84137
897f6fd
d16b3ae
e45ece3
5467361
f739bf1
e2f9faa
309a77b
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -446,6 +446,20 @@ def _build_container_options(job_config: dict, platform: str) -> dict: | |
| # Architectures that we have multi GPU setup for testing | ||
| "multi_gpu": {"linux": ["gfx94X-dcgpu", "gfx950-dcgpu"]}, | ||
| }, | ||
| # rocSHMEM tests | ||
| "rocshmem": { | ||
| "job_name": "rocshmem", | ||
| "fetch_artifact_args": "--rocshmem --tests", | ||
| "timeout_minutes": 30, | ||
| "test_script": f"python {_get_script_path('test_runner.py')}", | ||
| "platform": ["linux"], | ||
| "total_shards_dict": { | ||
| "linux": 1, | ||
| }, | ||
| # rocSHMEM functional/unit tests launch via mpirun with RANKS 2..64, so | ||
| # they need a multi-GPU runner (same setup as rccl). | ||
| "multi_gpu": {"linux": ["gfx94X-dcgpu", "gfx950-dcgpu"]}, | ||
|
Comment on lines
+499
to
+501
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. fyi @amd-shiraz new 8 GPU runner like
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. this will cause huge queues with current capacity for gfx942-8gpu. do these actually need multi gpu? cc @suchitravemuri1 |
||
| }, | ||
| # rocprofiler-sdk tests | ||
| "rocprofiler-sdk": { | ||
| "job_name": "rocprofiler-sdk", | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -33,6 +33,7 @@ | |
| [--prim | --no-prim] | ||
| [--rand | --no-rand] | ||
| [--rccl | --no-rccl] | ||
| [--rocshmem | --no-rocshmem] | ||
| [--rocdecode | --no-rocdecode] | ||
| [--rocjpeg | --no-rocjpeg] | ||
| [--rocjitsu | --no-rocjitsu] | ||
|
|
@@ -358,6 +359,7 @@ def retrieve_artifacts_by_run_id(args): | |
| args.mpi, | ||
| args.rand, | ||
| args.rccl, | ||
| args.rocshmem, | ||
| args.rocdecode, | ||
| args.rocjpeg, | ||
| args.rocjitsu, | ||
|
|
@@ -447,6 +449,16 @@ def retrieve_artifacts_by_run_id(args): | |
| extra_artifacts.append("rand") | ||
| if args.rccl: | ||
| extra_artifacts.append("rccl") | ||
| if args.rocshmem: | ||
| extra_artifacts.append("rocshmem") | ||
| # The functional test binary (bin/rocshmem_functional_tests) and | ||
| # bin/rocshmem_info live in the _run component; the install-tree | ||
| # CTestTestfile.cmake references them via relative paths. | ||
| argv.append("rocshmem_run") | ||
| # rocSHMEM tests launch via mpirun and link against TheRock's | ||
| # vendored OpenMPI, so pull it (with its run component for mpiexec). | ||
| extra_artifacts.append("openmpi") | ||
| argv.append("openmpi_run") | ||
|
Comment on lines
+499
to
+502
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We don't build openmpi by default and we don't distribute a vendored version. You may be interested in how rocprofiler-sdk has recently started running mpi tests by using a test dockerfile with a system install: |
||
| if args.rocprofiler_sdk: | ||
| extra_artifacts.append("rocprofiler-sdk") | ||
| extra_artifacts.append("aqlprofile") | ||
|
|
@@ -788,6 +800,13 @@ def main(argv): | |
| action=argparse.BooleanOptionalAction, | ||
| ) | ||
|
|
||
| artifacts_group.add_argument( | ||
| "--rocshmem", | ||
| default=False, | ||
| help="Include 'rocshmem' artifacts", | ||
| action=argparse.BooleanOptionalAction, | ||
| ) | ||
|
|
||
| artifacts_group.add_argument( | ||
| "--mpi", | ||
| default=False, | ||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.
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.
What tests do you specifically expect to run with this script? Looking at the logs on https://github.com/ROCm/TheRock/actions/runs/30112055659/job/89560360151, I see
That seems pretty light on tests, even for the "quick" filter? The "No GPU suite found" logs are also concerning given that this claims it should be running multi-GPU tests?