Skip to content
This repository has been archived by the owner on Nov 1, 2022. It is now read-only.

Move images to Docker Hub #1949

Merged
merged 1 commit into from
Apr 17, 2019
Merged

Move images to Docker Hub #1949

merged 1 commit into from
Apr 17, 2019

Conversation

hiddeco
Copy link
Member

@hiddeco hiddeco commented Apr 17, 2019

I choose to use the Fully Qualified Registry Name everywhere (docker.io/weaveworks/flux) instead of using the shorthand (weaveworks/flux) so it gets picked up when someone is for example excluding docker.io images using --registry-exclude-image=docker.io/*.

@hiddeco hiddeco added the build About the build or test scaffolding label Apr 17, 2019
@hiddeco hiddeco force-pushed the build/switch-docker-hub branch 3 times, most recently from 19acd1d to 9d3ba2b Compare April 17, 2019 10:04
chart/flux/README.md Outdated Show resolved Hide resolved
@hiddeco hiddeco force-pushed the build/switch-docker-hub branch 2 times, most recently from 12714ff to a1df1f8 Compare April 17, 2019 10:17
@hiddeco hiddeco requested a review from 2opremio April 17, 2019 10:19
I choose to use the Fully Qualified Registry Name everywhere
(`docker.io/weaveworks/flux`) instead of using the shorthand
(`weaveworks/flux`) so it gets picked up when someone is for example
excluding docker.io images using `--registry-exclude-image=docker.io/*`
@hiddeco hiddeco force-pushed the build/switch-docker-hub branch from a1df1f8 to 992bdd3 Compare April 17, 2019 10:38
Copy link
Contributor

@bboreham bboreham left a comment

Choose a reason for hiding this comment

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

Check that password flag before merging, otherwise lgtm.

.circleci/config.yml Show resolved Hide resolved
Copy link
Contributor

@bricef bricef left a comment

Choose a reason for hiding this comment

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

👍 I don't see any issues here

@hiddeco hiddeco merged commit f37b02d into master Apr 17, 2019
@hiddeco hiddeco deleted the build/switch-docker-hub branch April 17, 2019 11:14
@squaremo squaremo added this to the v1.12.1 milestone Apr 24, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
build About the build or test scaffolding
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants