Skip to content

feat(ci): pin TheRock workflows to 7b19fce7 (2026-08-25) - #10693

Closed
geomin12 wants to merge 7 commits into
developfrom
users/geomin12/hash-update-7b19fce
Closed

feat(ci): pin TheRock workflows to 7b19fce7 (2026-08-25)#10693
geomin12 wants to merge 7 commits into
developfrom
users/geomin12/hash-update-7b19fce

Conversation

@geomin12

@geomin12 geomin12 commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

pin TheRock workflows to 7b19fce7 (2026-08-25)

geomin12 and others added 5 commits August 25, 2026 07:33
@geomin12 @claude
ci: pin TheRock workflows to 7b19fce75b97c4604609daaf9ad70975563108a0
6f09c4c 
Update all multi-arch CI workflows to use a specific TheRock commit
instead of @main for reproducible builds.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@geomin12 @claude
ci: pin TheRock checkout to 7b19fce75b97c4604609daaf9ad70975563108a0
e486166 
Update therock-ci.yml and therock-test-packages.yml to use the same
TheRock commit as the multi-arch workflows.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@geomin12 @claude
ci: pin remaining TheRock refs to 7b19fce75b97c4604609daaf9ad70975563…
4562336 
…108a0

Update all remaining workflow files with TheRock references:
- therock-build-linux.yml
- therock-ci-linux.yml
- therock-ci-windows.yml
- therock-rccl-ci-linux.yml
- therock-rccl-test-packages-single-node.yml
- therock-rccl-test-packages-multi-node.yml
- therock-rccl-test-pytorch-distributed.yml
- therock-rccl-test-jax-collective.yml
- therock-rccl-test-rocprof.yml
- rccl-ci.yml
- projects/rccl/.github/workflows/therock-ci-linux.yml

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@geomin12 @claude
ci: replace needs.setup.outputs.ref with pinned hash
4ae820b 
Replace dynamic ref resolution with the pinned TheRock commit hash
for consistent builds across all jobs.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@geomin12 @claude
ci: update therock-test-component.yml and remove stale comments
6c758fa 
- Pin therock-test-component.yml to 7b19fce75b97c4604609daaf9ad70975563108a0
- Remove outdated "Concrete SHA resolved once by setup" comments

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@geomin12
geomin12 requested review from a team August 25, 2026 14:43
@geomin12
geomin12 requested a review from a team as a code owner August 25, 2026 14:43
@geomin12 @claude
ci: revert projects/rccl therock-ci-linux.yml to original hash
3b6e021 
This file is managed separately and should not be updated.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions github-actions Bot added github actions Pull requests that update GitHub Actions code project: rccl labels Aug 25, 2026
@therock-pr-bot

therock-pr-bot Bot commented Aug 25, 2026

Copy link
Copy Markdown

❌ PR Check — Action Required

Check Status Details
📝 PR Description ❌ Fail Error: PR description must reference a JIRA ID, ISSUE ID, or a GitHub closing keyword.
Expected: include a JIRA ID / ISSUE ID line (separator : or -, or omitted; value may be a JIRA key, a number with/without #, or a link), OR a closing keyword + issue reference. Accepted examples:
• JIRA ID : TESTAUTO-6039
• JIRA ID - #330
• JIRA ID #330
• JIRA ID (on separate line)
ROCM-25757
• ISSUE ID : TESTUTO-3334
• ISSUE ID #3334
• ISSUE ID - TESTAUTO-3433
• ISSUE ID (on separate line)
AIRUNTIME-2352
• ISSUE ID : https://github.com/<org_name>/<repo_name>/issues/1234
• Closes #10
• Fixes octo-org/octo-repo#100
• Resolves: #123
• #123
• https://github.com/<org_name>/<repo_name>/issues/123
Current: no valid JIRA/ISSUE/closing-keyword reference found
⛔ Forbidden Files ✅ Pass —
🧪 Unit Test ✅ Pass PR does not contain code files — Unit Test auto-passed
🔎 pre-commit ⏳ Pending ⏳ Still running…
🚫 Draft PR 🔜 To Be Enabled —
🚩 Feature Flag 🔜 To Be Enabled —
📊 Code Coverage 🔜 To Be Enabled —

⚠️ 1 policy check(s) failed. Please address the issues above before this PR can be Reviewed.

🚫 Please fix the failed policies

  • ❌ PR Description

The Not ready to Review label was added to this PR. Once all policies pass, the label is removed automatically.

📖 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 25, 2026

Copy link
Copy Markdown

🚫 Please fix the failed policies before requesting reviews.

The following policy checks failed:

  • ❌ PR Description

The Not ready to Review label has been added to this PR.
Once all policies pass, the label will be removed automatically.

erman-gurses
erman-gurses approved these changes Aug 25, 2026
View reviewed changes

@erman-gurses erman-gurses left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@jayhawk-commits

jayhawk-commits commented Aug 26, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator

#10748 pins to a newer SHA

@geomin12 geomin12 closed this Aug 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@erman-gurses erman-gurses erman-gurses approved these changes

Assignees

No one assigned

Labels

github actions Pull requests that update GitHub Actions code Not ready to Review organization: ROCm project: rccl

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@geomin12 @jayhawk-commits @erman-gurses