Bump rocm-systems from 093b66c to 96e30b4#3984
Conversation
Bumps [rocm-systems](https://github.com/ROCm/rocm-systems) from `093b66c` to `96e30b4`. - [Release notes](https://github.com/ROCm/rocm-systems/releases) - [Commits](ROCm/rocm-systems@093b66c...96e30b4) --- updated-dependencies: - dependency-name: rocm-systems dependency-version: 96e30b429c074f8a3fcf66be9c014df4d7cbc681 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com>
|
It looks like the builds are still failing to apply the patch: |
|
Docs for how to work with patch files: https://github.com/ROCm/TheRock/tree/main/patches#resolving-conflicts-with-patches |
|
The fix is to delete the patch. See ROCm/rocm-systems@b7167d0 |
geomin12
left a comment
There was a problem hiding this comment.
patches need to be fixed!
rocm-systems at 96e30b4 includes commit b7167d0 which sets kComgrVersioned = amd::IS_HIP (versioned for HIP, unversioned for OpenCL). Per patches/README.md: when the conflicting commit is equivalent to the patch, delete the patch. No longer apply this patch in TheRock CI. Made-with: Cursor
Recommend: failures are pre-existing test/infra issues, not introduced by rocm-systems bump 093b66c→96e30b4 or patch delete. hipsparse pattern may overlap with rocm-libraries #5502. rocm-sdk test should skip directories in testSharedLibrariesLoad. Re-run only for infra (gfx110X) if needed; merge acceptable with known test failures tracked elsewhere. cc @geomin12 |
|
Re-based as we now have gfx94X runners to test on |
|
A newer version of rocm-systems exists, but since this PR has been edited by someone other than Dependabot I haven't updated it. You'll get a PR for the updated version as normal once this PR is merged. |
|
For rccl test failures, existing issue: #3774 |
|
Reran tests as there appears to be some flaky tests. Namely rocfft, hipsparselt and rocgdb. |
amd-aakash
left a comment
There was a problem hiding this comment.
Awaiting summary of known issues - lets merge this
|
#3999 should really be fixed before we merge this. I don't see anyone actually working on it yet though. It should be a 1 line change to a python test file... |
The fix for this issue seems to have landed in 4 hours ago. We are starting another Bump PR cycle right after we merge this one. |
Summary for known issuesIssues with workarounds (while a full fix is pending)
Issues with no update
|
No? As pointed out on that PR, it does not fix the issue. It is related, but there is still a bug in the python tests. |
Sorry read through the PR but also saw "should also fix issue - #3999" in the PR description. Will hold off on Merging |
amd-aakash
left a comment
There was a problem hiding this comment.
Lets address the changes mentioned by @ScottTodd - create a fresh Bump and merge that in after watching if we fix these issues.
|
I suspect #4094 will fix #3999. We're testing now. I'd say we don't necessarily need to sync/rebase/restart this submodule update PR after that, we can just merge the PRs in sequence. It's a test at the leaf of the packages (but would affect users who pip install then run |
ScottTodd
left a comment
There was a problem hiding this comment.
Approving the patch deletion for CODEOWNERS
The test failures in rocm-sdk test should be fixed now that #3999 is fixed (I tested with artifacts from this PR itself, see
- https://github.com/ROCm/TheRock/actions/runs/23366263964
- https://github.com/ROCm/TheRock/actions/runs/23366544519/job/67981590817
I haven't personally been tracking the other issues spotted on this submodule update, so trusting others there.
Bumps [rocm-systems](https://github.com/ROCm/rocm-systems) from `093b66c` to `96e30b4`. <details> <summary>Commits</summary> <ul> <li><a href="https://github.com/ROCm/rocm-systems/commit/96e30b429c074f8a3fcf66be9c014df4d7cbc681"><code>96e30b4</code></a> call HIP_SKIP_TEST properly if device has no image support (<a href="https://redirect.github.com/ROCm/rocm-systems/issues/4040">#4040</a>)</li> <li><a href="https://github.com/ROCm/rocm-systems/commit/e1c229e313e137b8b17d074d9b13bfa1697886b0"><code>e1c229e</code></a> SWDEV-581075 - Skip if managed memory not supported (<a href="https://redirect.github.com/ROCm/rocm-systems/issues/3598">#3598</a>)</li> <li><a href="https://github.com/ROCm/rocm-systems/commit/ce28e5fcce37b4e77f4e7e12d79b245905147e8e"><code>ce28e5f</code></a> copy amd_icd_file to stage/etc/OpenCL/vendors (<a href="https://redirect.github.com/ROCm/rocm-systems/issues/3818">#3818</a>)</li> <li><a href="https://github.com/ROCm/rocm-systems/commit/55b57eec37a7d128a5a760b93d28a0df084e11e7"><code>55b57ee</code></a> [ci] disabling mi325 test runners (<a href="https://redirect.github.com/ROCm/rocm-systems/issues/4062">#4062</a>)</li> <li><a href="https://github.com/ROCm/rocm-systems/commit/c873a9b2aabf64696f54dd1f47461f97aa23b09b"><code>c873a9b</code></a> [rocprofiler-compute][tui] Align TUI's analysis logic to GUI (<a href="https://redirect.github.com/ROCm/rocm-systems/issues/3641">#3641</a>)</li> <li><a href="https://github.com/ROCm/rocm-systems/commit/485f87f6bc0e2bcc822321d446b9298cc97dc845"><code>485f87f</code></a> [rocprofiler-systems] Enable the roctx-api-runtime-instrument tests (<a href="https://redirect.github.com/ROCm/rocm-systems/issues/3990">#3990</a>)</li> <li><a href="https://github.com/ROCm/rocm-systems/commit/a8b23edbe91e58655a19a0ab1e10ecd7be983b16"><code>a8b23ed</code></a> [TheRock CI] Fix workflow_dispatch project selection being overridden by CI f...</li> <li><a href="https://github.com/ROCm/rocm-systems/commit/ceda96122210ad48835ef6e5594497b0cdff5547"><code>ceda961</code></a> [HSA][CLR][rocprofiler-sdk] hsa_amd_memory_async_batch_copy cherry-pick <a href="https://redirect.github.com/ROCm/rocm-systems/issues/3824">#3824</a>...</li> <li><a href="https://github.com/ROCm/rocm-systems/commit/d7d61f29a44c3af7f5973a888d4da81b62242520"><code>d7d61f2</code></a> [rocdecode]users/rrawther/media libs test fix (<a href="https://redirect.github.com/ROCm/rocm-systems/issues/4056">#4056</a>)</li> <li><a href="https://github.com/ROCm/rocm-systems/commit/45365fb0fea22e64b83194e41c4e8a268e338e26"><code>45365fb</code></a> [rocprofiler-systems] Fix the ubuntu-jammy workflow (<a href="https://redirect.github.com/ROCm/rocm-systems/issues/4055">#4055</a>)</li> <li>Additional commits viewable in <a href="https://github.com/ROCm/rocm-systems/compare/093b66caa35ff225bb2846c8174c09f4ce62830c...96e30b429c074f8a3fcf66be9c014df4d7cbc681">compare view</a></li> </ul> </details> <br /> Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`. [//]: # (dependabot-automerge-start) [//]: # (dependabot-automerge-end) --- <details> <summary>Dependabot commands and options</summary> <br /> You can trigger Dependabot actions by commenting on this PR: - `@dependabot rebase` will rebase this PR - `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it - `@dependabot show <dependency name> ignore conditions` will show all of the ignore conditions of the specified dependency - `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself) </details> --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Chi <chi.liu@amd.com> Co-authored-by: Geo Min <geomin12@amd.com> Co-authored-by: Aakash Sudhanwa <9801456+amd-aakash@users.noreply.github.com>
Bumps rocm-systems from
093b66cto96e30b4.Commits
96e30b4call HIP_SKIP_TEST properly if device has no image support (#4040)e1c229eSWDEV-581075 - Skip if managed memory not supported (#3598)ce28e5fcopy amd_icd_file to stage/etc/OpenCL/vendors (#3818)55b57ee[ci] disabling mi325 test runners (#4062)c873a9b[rocprofiler-compute][tui] Align TUI's analysis logic to GUI (#3641)485f87f[rocprofiler-systems] Enable the roctx-api-runtime-instrument tests (#3990)a8b23ed[TheRock CI] Fix workflow_dispatch project selection being overridden by CI f...ceda961[HSA][CLR][rocprofiler-sdk] hsa_amd_memory_async_batch_copy cherry-pick #3824...d7d61f2[rocdecode]users/rrawther/media libs test fix (#4056)45365fb[rocprofiler-systems] Fix the ubuntu-jammy workflow (#4055)Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)