Skip to content

fix(ci): Add build_variant to Test Artifacts run-name#6459

Open
geomin12 wants to merge 2 commits into
mainfrom
users/geomin12/issue-6456
Open

fix(ci): Add build_variant to Test Artifacts run-name#6459
geomin12 wants to merge 2 commits into
mainfrom
users/geomin12/issue-6456

Conversation

@geomin12

@geomin12 geomin12 commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Include the build_variant (release/asan) in the workflow run-name to easily distinguish between regular and ASAN test artifact runs.

Closes #6456

Working here: https://github.com/ROCm/TheRock/actions/runs/29054942544

Include the build_variant (release/asan) in the workflow run-name
to easily distinguish between regular and ASAN test artifact runs.

Fixes #6456

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@geomin12 geomin12 requested a review from marbre July 9, 2026 22:45
@geomin12 geomin12 added the ci:skip Skip all CI builds/tests for this PR label Jul 9, 2026
@therock-pr-bot

therock-pr-bot Bot commented Jul 9, 2026

Copy link
Copy Markdown

✅ All Checks Passed — Ready for Review

Check Status Details
🌿 Branch Name ✅ Pass
📝 PR Title/Description ✅ Pass
Forbidden Files ✅ Pass
🧪 Unit Test ✅ Pass PR does not contain code files — Unit Test auto-passed
🔎 pre-commit ✅ Pass
🚫 Draft PR 🔜 To Be Enabled
🚩 Feature Flag 🔜 To Be Enabled
📊 Code Coverage 🔜 To Be Enabled
🤖 therock-pr-bot ✅ Pass

🎉 All checks passed! This PR is ready for review.

📖 Need help? See the Policy FAQ for details on every check and how to fix failures.

@therock-pr-bot

therock-pr-bot Bot commented Jul 9, 2026

Copy link
Copy Markdown

🎉 All checks passed! This PR is ready for review.

@marbre marbre left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Checking the input in the test jobs linked in the issue both jobs show build_variant: release. Thus, this doesn't help to distinguish between non-ASAN and ASAN.

The test_artifacts run-name now includes build_variant, but the
multi_arch_release_linux workflow wasn't passing it through when
triggering test_artifacts. This caused ASAN builds to still show
"release" instead of "asan" in the job name.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@geomin12 geomin12 requested a review from marbre July 10, 2026 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci:skip Skip all CI builds/tests for this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Test Artifacts job doesn't directly reveal build type

2 participants