Skip to content

Debug docker push for RPM-based container images#22

Merged
bwarden merged 1 commit intomainfrom
feat/add-fedora-support
Apr 15, 2026
Merged

Debug docker push for RPM-based container images#22
bwarden merged 1 commit intomainfrom
feat/add-fedora-support

Conversation

@bwarden
Copy link
Copy Markdown
Contributor

@bwarden bwarden commented Apr 15, 2026

RPM-based containers weren't being pushed to docker successfully, probably due to an issue with the embedded bash.

  • Add an explicit format to docker image ls for dependable scripting.
  • Use a while read loop to capture image names
  • Avoid using ${var} form for bash variables in case it conflicts with workflow substitutions

Containers aren't being pushed successfully; I suspect issues with how
the embedded bash script is executed, or the default formatting of
`docker image ls` on the runners. Use an explicit format string to name
the images in a dependable way. If this fails, we can try JSON output
instead.
Add another step to list the images so we can tell they're there.

Signed-off-by: Brett T. Warden <bwarden@qti.qualcomm.com>
@bwarden bwarden marked this pull request as ready for review April 15, 2026 20:30
Copy link
Copy Markdown
Contributor

@simonbeaudoin0935 simonbeaudoin0935 left a comment

Choose a reason for hiding this comment

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

LGTM

@bwarden bwarden merged commit 27312d6 into main Apr 15, 2026
14 checks passed
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.

4 participants