Skip to content

feat(ci): enable automatic stage reuse for TheRock builds - #10533

Merged
geomin12 merged 3 commits into
developfrom
users/geomin12/dynamic-determination
Aug 7, 2026
Merged

geomin12 merged 3 commits into
developfrom
users/geomin12/dynamic-determination

Conversation

@geomin12

@geomin12 geomin12 commented Aug 7, 2026

Copy link
Copy Markdown
Collaborator
  • Remove hardcoded prebuilt_stages and baseline_run_id
  • Add stage_reuse_mode: reuse-stage to enable automatic baseline selection
  • TheRock now finds commit-compatible baselines and determines which stages to rebuild based on changed files in rocm-libraries

ISSUE ID: ROCm/TheRock#3343

- Remove hardcoded prebuilt_stages and baseline_run_id
- Add stage_reuse_mode: reuse-stage to enable automatic baseline selection
- TheRock now finds commit-compatible baselines and determines which
  stages to rebuild based on changed files in rocm-libraries

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions github-actions Bot added github actions project: none Does not target any component labels Aug 7, 2026
@geomin12
geomin12 marked this pull request as ready for review August 7, 2026 21:07
@geomin12
geomin12 requested a review from a team as a code owner August 7, 2026 21:07
@therock-pr-bot

therock-pr-bot Bot commented Aug 7, 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 7, 2026

Copy link
Copy Markdown

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

@geomin12 geomin12 added the skip-therockci If added, it will skip all builds and tests for TheRock CI label Aug 7, 2026
geomin12 and others added 2 commits August 7, 2026 14:40
The setup_multi_arch.yml workflow requires actions:read permission
for GitHub API calls to fetch baseline workflow runs.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

@endurthiabhilash endurthiabhilash left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@geomin12
geomin12 merged commit 7e90708 into develop Aug 7, 2026
24 checks passed
@geomin12
geomin12 deleted the users/geomin12/dynamic-determination branch August 7, 2026 21:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

github actions organization: ROCm project: none Does not target any component skip-therockci If added, it will skip all builds and tests for TheRock CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants