Skip to content

[windows] test rocm-systems integration with bump amdgpu-windows-interop to version 20250820#1323

Merged
amd-justchen merged 3 commits into
users/marbre/rocm-systemsfrom
users/justchen/pr1296-rocm-systems-test-updated-PAL
Aug 26, 2025
Merged

[windows] test rocm-systems integration with bump amdgpu-windows-interop to version 20250820#1323
amd-justchen merged 3 commits into
users/marbre/rocm-systemsfrom
users/justchen/pr1296-rocm-systems-test-updated-PAL

Conversation

@amd-justchen
Copy link
Copy Markdown
Contributor

Progress on #1321

Try to unblock #1296 with updated PAL

@amd-justchen
Copy link
Copy Markdown
Contributor Author

Per @ScottTodd suggestion, adding gfx115X-windows to try testing on other non default gpu archs.

@amd-justchen
Copy link
Copy Markdown
Contributor Author

Adding the label after 150f45e seems to have removed the link to the successful run here:
https://github.com/ROCm/TheRock/actions/runs/17220771301/job/48855549747?pr=1323

But, it did succeed, so merged in the amdgpu-windows-interop PR for 20250820a here ROCm/amdgpu-windows-interop@4a54868

and updated submodule commit to this.

@amd-justchen
Copy link
Copy Markdown
Contributor Author

WIndows Build was successful, though some test timeouts on gfx1151 for windows

@amd-justchen amd-justchen merged commit 222c4be into users/marbre/rocm-systems Aug 26, 2025
30 of 38 checks passed
@github-project-automation github-project-automation Bot moved this from TODO to Done in TheRock Triage Aug 26, 2025
@amd-justchen amd-justchen deleted the users/justchen/pr1296-rocm-systems-test-updated-PAL branch August 26, 2025 02:06
@ScottTodd
Copy link
Copy Markdown
Member

WIndows Build was successful, though some test timeouts on gfx1151 for windows

Nearly all of the tests failed. Those timeouts and failures should have blocked merging this.

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

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants