Skip to content

[CI] Fix TheRock runner-matrix import after upstream rename - #324

Merged
maarquitos14 merged 1 commit into
ROCm:amd-stagingfrom
maarquitos14:maronas/fix-ci-import
Oct 1, 2026
Merged

maarquitos14 merged 1 commit into
ROCm:amd-stagingfrom
maarquitos14:maronas/fix-ci-import

Conversation

@maarquitos14

Copy link
Copy Markdown
Contributor

The Select gfx942 test runner job fails on every PR with:

ImportError: cannot import name 'amdgpu_family_info_matrix_presubmit' from 'amdgpu_family_matrix'

We check out ROCm/TheRock at main. TheRock 0383100a27 ("consolidate GPU family matrix with trigger-based config", ROCm/TheRock#8322) merged the per-trigger matrices into a single amdgpu_family_info_matrix, where each family lists the triggers it applies to.

@maarquitos14
maarquitos14 requested a review from idubinov October 1, 2026 09:42

@idubinov idubinov left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@maarquitos14
maarquitos14 merged commit e531cd7 into ROCm:amd-staging Oct 1, 2026
10 of 11 checks passed
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