Skip to content

Fully qualify seed container images#2202

Open
m-bull wants to merge 2 commits intostackhpc/2025.1from
fully-qualified-seed-image-names
Open

Fully qualify seed container images#2202
m-bull wants to merge 2 commits intostackhpc/2025.1from
fully-qualified-seed-image-names

Conversation

@m-bull
Copy link
Contributor

@m-bull m-bull commented Mar 9, 2026

Seed hosts configured to use the Podman container engine do not automatically resolve image short-names to docker.io like seed hosts using the Docker container engine.

Fully qualify seed container images so that Podman does not fail to resolve them out of the box.

Also update pre and post playbooks, which run before and after deploying seed containers, to call kayobe_container_* rather than docker_* modules, so that they respect the container_engine configuration variable.

Seed hosts configured to use the Podman container
engine do not automatically resolve image short-names
to docker.io like seed hosts using the Docker
container engine.

Fully qualify seed container images so that Podman
does not fail to resolve them out of the box.
@m-bull m-bull requested a review from a team as a code owner March 9, 2026 17:13
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates container image references in etc/kayobe/seed.yml to use fully qualified names. The pulp/pulp image is now specified as quay.io/pulp/pulp and stackhpc/squid as docker.io/stackhpc/squid. This change is to ensure compatibility with the Podman container engine, which does not automatically resolve short-names to a default registry. A release note is included to document this fix. The changes are consistent with the pull request's objective.

priteau
priteau previously approved these changes Mar 9, 2026
Use Kayobe action_plugins to direct calls to the correct
container_engine.
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