Skip to content

Little cleanup #79

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Little cleanup #79

wants to merge 6 commits into from

Conversation

gw0
Copy link

@gw0 gw0 commented Feb 18, 2021

This PR builds upon #78 and cleans up a few things.

It contains one non-backward compatible change. It affects only those who override REGISTRIES env var, use DockerHub, but do not explicitly set REGISTRIES="auth.docker.io registry-1.docker.io". As part of the organization all default configurations were put in Dockerfile (including previously internally hard-coded values, such as DockerHub servers). This change is necessary to cover the use cases where one does not want to cache DockerHub, but wants to cache another registry. If one does not set the REGISTRIES env var (and the default in Dockerfile is used), everything works as it used to (it works even for GitLab.com).

@gw0 gw0 force-pushed the gw0/little-cleanup branch from d06ce90 to b366362 Compare February 18, 2021 17:37
@rpardini rpardini added the outdated Outdated, needs large rework against the master branch label Feb 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
outdated Outdated, needs large rework against the master branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants