Skip to content

Commit cb79358

Browse files
authored
[ci] Ignoring smoke tests for gfx1151 windows (#3615)
To remediate windows gfx1151 queues (#3299 and #3050), we are only running Windows gfx1151 tests if the `test_type` is full. This means no smoke tests will be run for this architecture Tests ran via workflow_dispatch: - [full tests](https://github.com/ROCm/TheRock/actions/runs/22412198467/job/64888425151#step:4:77) (cancelled to save resources but you can see it ran both) - [smoke tests](https://github.com/ROCm/TheRock/actions/runs/22412250700/job/64888603214#step:4:75) (cancelled to save resources but you can see it skipped gfx1151) Adding `skip-ci` label as tests above prove it working
1 parent 8f17ad6 commit cb79358

2 files changed

Lines changed: 7 additions & 0 deletions

File tree

build_tools/github_actions/amdgpu_family_matrix.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -104,6 +104,8 @@
104104
"family": "gfx1151",
105105
"fetch-gfx-targets": ["gfx1151"],
106106
"build_variants": ["release"],
107+
# TODO(#3299): Re-enable smoke tests once capacity is available for Windows gfx1151
108+
"run-full-tests-only": True,
107109
},
108110
},
109111
"gfx120x": {

build_tools/github_actions/configure_ci.py

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -661,6 +661,11 @@ def main(base_args, linux_families, windows_families):
661661
test_type = "full"
662662
test_type_reason = f"test label(s) specified: {combined_test_labels}"
663663

664+
# If the "run-full-tests-only" flag is set for this family, we do not run tests if it is a smoke test type
665+
for matrix_row in linux_variants_output + windows_variants_output:
666+
if matrix_row.get("run-full-tests-only", False) and test_type == "smoke":
667+
matrix_row["test-runs-on"] = ""
668+
664669
print(f"test_type decision: '{test_type}' (reason: {test_type_reason})")
665670

666671
# Format variants for summary - handle both regular and multi-arch modes

0 commit comments

Comments
 (0)