Skip to content

hip-tests: Disable two hip-tests UTs blocked by Compiler ww28 SMP 2.5 - #7144

Open
lajagapp wants to merge 1 commit into
mainfrom
users/lajagapp/disable-hip-tests-comp
Open

hip-tests: Disable two hip-tests UTs blocked by Compiler ww28 SMP 2.5#7144
lajagapp wants to merge 1 commit into
mainfrom
users/lajagapp/disable-hip-tests-comp

Conversation

@lajagapp

@lajagapp lajagapp commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Summary

Temporarily exclude Unit_hip_linker_spirv_input and Unit_hipExtModuleLaunchKernel_CheckCodeObjAttr from TheRock hip-tests CI so TheRock#7052 (Compiler ww28 SMP 2.5) promotion is not blocked.

Motivation

Multi-Arch CI hip-tests fail with amd-llvm 7287e2062fa6 from #7052. These issues will be fixed in rocm-systems, which is exposed by the compiler promotion PR. CI exclusion is a short-term unblock until runtime/compiler owners deliver a fix.

Test plan

Related

Signed-off-by: Lenine Ajagappane <Lenine.Ajagappane@amd.com>
@therock-pr-bot

therock-pr-bot Bot commented Aug 6, 2026

Copy link
Copy Markdown

✅ All Checks Passed — Ready for Review

Check Status Details
📝 PR 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.

🙋 Wish to Override Policy?

@therock-pr-bot

therock-pr-bot Bot commented Aug 6, 2026

Copy link
Copy Markdown

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

@ronlieb
ronlieb self-requested a review August 6, 2026 11:52
@lajagapp
lajagapp marked this pull request as ready for review August 6, 2026 13:25
@ronlieb

ronlieb commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Is this landable now from DevOps perspective ?

@ScottTodd

Copy link
Copy Markdown
Member

Yes, though the list of skipped targets may not be exhaustive in that file. We run nightly tests on all devices we have test runners for in https://github.com/ROCm/rockrel/actions/workflows/test_artifacts.yml

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

Labels

None yet

Projects

Status: TODO

Development

Successfully merging this pull request may close these issues.

3 participants