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

registry-creds: Add support for Google Artifact Registry #9259

Closed
jonjohnsonjr opened this issue Sep 15, 2020 · 8 comments
Closed

registry-creds: Add support for Google Artifact Registry #9259

jonjohnsonjr opened this issue Sep 15, 2020 · 8 comments
Labels
addon/registry-creds registry-creds issues kind/feature Categorizes issue or PR as related to a new feature. lifecycle/rotten Denotes an issue or PR that has aged beyond stale and will be auto-closed. priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete.

Comments

@jonjohnsonjr
Copy link

I'd love to have Artifact Registry included in the registry-creds addon.

Authentication generally works the same way as GCR: https://cloud.google.com/artifact-registry/docs/docker/quickstart#auth

I'm not sure how registry-creds works or how one might go about adding this. If it's not too difficult, I may be able to contribute given some pointers/guidance.

@medyagh medyagh added kind/feature Categorizes issue or PR as related to a new feature. priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete. labels Sep 15, 2020
@medyagh
Copy link
Member

medyagh commented Sep 15, 2020

@jonjohnsonjr would be a great feature, I would happy accept a PR that would implements this. creating an addon for minikube is documented here, and could also look at one of the addon PRs.

https://minikube.sigs.k8s.io/docs/contrib/addons/

@jonjohnsonjr
Copy link
Author

I have some questions :)

It looks like this is what prompts the user to configure GCR credentials with the registry-creds addon. Does this only allow one URL for GCR? It appears to be impossible to configure both e.g. gcr.io and asia.gcr.io using this addon. Is that true?

If the above is true, would it be reasonable to just update some strings to indicate that you can supply an Artifact Registry URL instead of a GCR URL?

@tstromberg
Copy link
Contributor

@jonjohnsonjr - This code was added 4 years ago, so your guess is as good as anyone else's on the project at the moment. It seems bizarre to me that we wouldn't allow credentials to be provided for all GCR & AR endpoints.

@tstromberg tstromberg changed the title Add Artifact Registry to registry-creds addon registry-creds: Add support for Google Artifact Registry Dec 14, 2020
@tstromberg tstromberg added the addon/registry-creds registry-creds issues label Dec 14, 2020
@fejta-bot
Copy link

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Send feedback to sig-contributor-experience at kubernetes/community.
/lifecycle stale

@k8s-ci-robot k8s-ci-robot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Mar 14, 2021
@medyagh
Copy link
Member

medyagh commented Apr 5, 2021

@jonjohnsonjr we added GCR support for GCP-Auth addon recently, (on HEAD) #10853

@medyagh
Copy link
Member

medyagh commented May 3, 2021

@jonjohnsonjr we added GCR support for GCP-Auth addon #10853

do u mind trying with minikube 1.20.0beta ? and see if that fixes ? we believe we added this feature with integration tests

@fejta-bot
Copy link

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.

If this issue is safe to close now please do so with /close.

Send feedback to sig-contributor-experience at kubernetes/community.
/lifecycle rotten

@k8s-ci-robot k8s-ci-robot added lifecycle/rotten Denotes an issue or PR that has aged beyond stale and will be auto-closed. and removed lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. labels Jun 2, 2021
@sharifelgamal
Copy link
Collaborator

the gcp-auth addon supports artifact registry now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
addon/registry-creds registry-creds issues kind/feature Categorizes issue or PR as related to a new feature. lifecycle/rotten Denotes an issue or PR that has aged beyond stale and will be auto-closed. priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete.
Projects
None yet
Development

No branches or pull requests

6 participants