Skip to content
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

Update docker repo #3

Merged
merged 1 commit into from
Oct 24, 2018
Merged

Update docker repo #3

merged 1 commit into from
Oct 24, 2018

Conversation

brancz
Copy link
Member

@brancz brancz commented Oct 24, 2018

I created the docker repo on quay and pushed the current state of this repo to it, so we can move forward with testing. Once this is merged I would tag the v0.1.0 tag on this repo and branch a release-0.1 branch.

@metalmatze @s-urbaniak

@openshift-ci-robot openshift-ci-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Oct 24, 2018
@@ -7,7 +7,7 @@ OUT_DIR=_output
BIN?=prom-label-proxy
VERSION?=$(shell cat VERSION)
PKGS=$(shell go list ./... | grep -v /vendor/)
DOCKER_REPO?=quay.io/openshift/prom-label-proxy
DOCKER_REPO?=quay.io/coreos/prom-label-proxy
Copy link
Contributor

Choose a reason for hiding this comment

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

silly question: shouldn't these all be openshift?

Copy link
Member Author

Choose a reason for hiding this comment

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

silly answer: I don't have permission to create repos under that, this seemed like the path of least resistance.

Copy link
Contributor

@s-urbaniak s-urbaniak left a comment

Choose a reason for hiding this comment

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

resistance is futile \o/

/lgtm

@brancz
Copy link
Member Author

brancz commented Oct 24, 2018

you will be assimilated

@brancz brancz merged commit 2c239b0 into master Oct 24, 2018
@brancz brancz deleted the update-docker-repo branch October 24, 2018 12:15
nima-rx added a commit to snapp-incubator/prom-label-proxy that referenced this pull request Aug 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants