Skip to content

Use absolute Docker Image URLs#396

Merged
mhutter merged 1 commit into
masterfrom
fix/docker-image-url
Dec 28, 2021
Merged

Use absolute Docker Image URLs#396
mhutter merged 1 commit into
masterfrom
fix/docker-image-url

Conversation

@mhutter

@mhutter mhutter commented Dec 28, 2021

Copy link
Copy Markdown
Contributor

Especially on Podman systems, docker.io may not be the default, thus breaking image pulls.

Be explicit where to pull images from by configuring a registry.

Checklist

  • Keep pull requests small so they can be easily reviewed.
  • Categorize the PR by setting a good title and adding one of the labels:
    bug, enhancement, documentation, change, breaking, dependency
    as they show up in the changelog

Especially on Podman systems, docker.io may not be the default.

Be explicit where to pull images from by configuring a registry.

Signed-off-by: Manuel Hutter <manuel@hutter.io>
@mhutter mhutter added the bug Something isn't working label Dec 28, 2021
@mhutter mhutter requested a review from simu December 28, 2021 10:27

@simu simu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The modulesync-control template (https://github.com/projectsyn/modulesync-control/blob/master/moduleroot/Makefile.vars.mk.erb) should be updated as well, if the component template in Commodore is changed, otherwise modulesync will revert the absolute image URLs in the component template when components are onboarded to modulesync.

@mhutter

mhutter commented Dec 28, 2021

Copy link
Copy Markdown
Contributor Author

Good point, projectsyn/modulesync-control#53

@mhutter mhutter merged commit afd924a into master Dec 28, 2021
@mhutter mhutter deleted the fix/docker-image-url branch December 28, 2021 14:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants