Repository navigation
feat(ci): Update TheRock hash to 7c4f91f (2026-08-18) - #10317
vedithal-amd wants to merge 1 commit into
Conversation
Picks up the rocprofiler-compute changes merged in TheRock since 48ec94d: ROCm/TheRock#7425, ROCm/TheRock#7421, ROCm/TheRock#7427. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
❌ PR Check — Action Required
📖 Need help? See the Policy FAQ for details on every check and how to fix failures. |
|
🚫 Please fix the failed policies before requesting reviews. The following policy checks failed:
The |
There was a problem hiding this comment.
Pull request overview
Updates the pinned ROCm/TheRock commit SHA used by several GitHub Actions workflows in rocm-systems, bringing CI runs in sync with TheRock commit 7c4f91f (2026-08-18) and the upstream fixes listed in the PR description.
Changes:
- Bump
ROCm/TheRockcheckoutreffrom48ec94d…to7c4f91f…across affected CI workflows. - Align the in-file annotated date comments on the pinned SHA to
2026-08-18.
Reviewed changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| .github/workflows/therock-test-packages.yml | Update pinned TheRock checkout SHA used by the packages test workflow. |
| .github/workflows/therock-test-component.yml | Update pinned TheRock checkout SHA used by the component test workflow. |
| .github/workflows/therock-rccl-test-pytorch-distributed.yml | Update pinned TheRock checkout SHA used by the RCCL PyTorch distributed test workflow. |
| .github/workflows/therock-rccl-test-packages-single-node.yml | Update pinned TheRock checkout SHA used by the single-node RCCL package tests. |
| .github/workflows/therock-rccl-test-packages-multi-node.yml | Update pinned TheRock checkout SHA used by the multi-node RCCL package tests. |
| .github/workflows/therock-rccl-ci-linux.yml | Update pinned TheRock checkout SHA used by RCCL CI on Linux. |
| .github/workflows/therock-ci.yml | Update pinned TheRock checkout SHA used by the main TheRock CI workflow. |
| .github/workflows/therock-ci-windows.yml | Update pinned TheRock checkout SHA used by TheRock CI on Windows. |
| .github/workflows/therock-ci-linux.yml | Update pinned TheRock checkout SHA used by TheRock CI on Linux. |
| .github/workflows/therock-build-linux.yml | Update pinned TheRock checkout SHA used by the Linux build workflow. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
jbonnell-amd
left a comment
There was a problem hiding this comment.
LGTM, assuming all CI checks pass and no regressions present
|
This got superseded - closing |
Update TheRock hash to
7c4f91f(2026-08-18)Picks up the rocprofiler-compute changes merged in TheRock since
48ec94d:rocm_sysdepszlib.pc reliably relocatable