Skip to content

[BBots] Use TheRock nightly in bot image#2252

Merged
jplehr merged 1 commit into
ROCm:aomp-devfrom
jplehr:feat/use-therock-nightly-in-buildbots
Jun 11, 2026
Merged

[BBots] Use TheRock nightly in bot image#2252
jplehr merged 1 commit into
ROCm:aomp-devfrom
jplehr:feat/use-therock-nightly-in-buildbots

Conversation

@jplehr

@jplehr jplehr commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Motivation

To live closer to the development version of ROCm, switch the way that we install ROCm from a long-living release to a nightly artifact from TheRock.

Long-term this also paves the way to unify the fleet image into a single one.

Technical Details

This will download the "latest" nightly when no specific date is given but accepts an override to pin to a specific date for reproducibility.

Test Plan

Manual testing

Test Result

Able to execute upstream bot recipes inside the container.

To live closer to the development version of ROCm, switch the way that
we install ROCm from a long-living release to a nightly artifact from
TheRock.
This will download the "latest" nightly when no specific date is given
but accepts an override to pin to a specific date for reproducibility.

@Kewen12 Kewen12 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!

The helper scripts are identical, we could combine the them in the future work.

@jplehr

jplehr commented Jun 11, 2026

Copy link
Copy Markdown
Contributor Author

The helper scripts are identical, we could combine the them in the future work.

With this change, large parts of the actual images are identical, so the next step is to only use a single one. But I wanted to split it for ease of review.

@jplehr jplehr merged commit 94e0abf into ROCm:aomp-dev Jun 11, 2026
1 check passed
@jplehr jplehr deleted the feat/use-therock-nightly-in-buildbots branch June 11, 2026 19:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants