Skip to content

Bump container image#438

Merged
evaneschneider merged 7 commits intodevfrom
bump-container-image
Aug 15, 2025
Merged

Bump container image#438
evaneschneider merged 7 commits intodevfrom
bump-container-image

Conversation

@mabruzzo
Copy link
Collaborator

@mabruzzo mabruzzo commented Aug 4, 2025

This is a reissue of #437 (I literally just moved the branch to the Cholla Repository). I think this was the source of all my problems!


I've done a lot of reading in the past week (before I ultimately just started trying things out).

Initially, I tried to start with the regular ROCm image and then manually use sudo apt-get in the workflow to actually install the stuff we need... I've concluded that won't work... We need to install all build dependencies inside the container.

So then I moved onto actually building the container with the github action. Ideally we want to be able to make some tweaks to the Dockerfile and let GitHub make all of the updates for us. I spent a bunch of time tinkering around within #386, before I moved onto experimenting elsewhere...

I've done a lot of reading in the past week (before I ultimately just
started trying things out).

Initially, I tried to start with the regular ROCm image and then
manually use sudo apt-get in the workflow to actually install the stuff
we need... I've concluded that won't work... We need to install all
build dependencies inside the container.

So then I moved onto actually building the container with the github
action. Ideally we want to be able to make some tweaks to the Dockerfile
and let GitHub make all of the updates for us. I spent a bunch of time
tinkering around within #386, before I moved onto experimenting
elsewhere...

I have verified that this GHA **does** work. If it fails in this PR,
it's because of a permissions issue
@mabruzzo mabruzzo force-pushed the bump-container-image branch from 00900c5 to ee7531d Compare August 5, 2025 14:07
@mabruzzo mabruzzo force-pushed the bump-container-image branch from ee7531d to 5fc3258 Compare August 5, 2025 14:16
@mabruzzo
Copy link
Collaborator Author

mabruzzo commented Aug 5, 2025

@evaneschneider, this is ready for review.

Please take a look at the .github/workflows/README.md file that I added and let me know if you find any part of it to be confusing (or if you think any part of it needs clarification). The goal was to provide detailed enough instructions to make it possible for anyone to bump the docker image (and provide basic context in case somebody eventually needs to make a more significant change)

Aside: There is a small chance that I'll need to open a 2nd PR that adds the build_image.yml to the main branch, if the button for manually running the workflow doesn't show up after we merge this PR.1

Footnotes

  1. I'm a little hesitant to directly merge this PR into main because the README talks about workflows that I previously modified (and the modifications are not present in the main branch)

Copy link
Collaborator

@evaneschneider evaneschneider left a comment

Choose a reason for hiding this comment

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

All looks good to me!

@evaneschneider evaneschneider merged commit f638c55 into dev Aug 15, 2025
15 checks passed
@mabruzzo mabruzzo deleted the bump-container-image branch August 18, 2025 02:25
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