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

GCR Nested Paths #34

Merged
merged 1 commit into from
Sep 14, 2020
Merged

GCR Nested Paths #34

merged 1 commit into from
Sep 14, 2020

Conversation

bnutt
Copy link
Contributor

@bnutt bnutt commented Sep 14, 2020

Unlike quay.io, docker hub, GCR registries support nested paths nested paths in image names. This change will let sinker generate nested paths for images in GCR repositories.

Unlike quay.io, docker hub, GCR registries support nested paths nested paths in image names. This change will let sinker generate nested paths for images in GCR repositories.
@jpreese
Copy link
Collaborator

jpreese commented Sep 14, 2020

Thanks for the PR @bnutt! Admittedly I did take an educated guess with this. I read through https://cloud.google.com/container-registry/docs/pushing-and-pulling and looking at the naming restrictions of PROJECT-ID it didn't look possible.

I'm not seeing any naming restrictions for the name of the image, so slashes must be allowed in that piece of the URL then?

@bnutt
Copy link
Contributor Author

bnutt commented Sep 14, 2020

Thanks for the PR @bnutt! Admittedly I did take an educated guess with this. I read through https://cloud.google.com/container-registry/docs/pushing-and-pulling and looking at the naming restrictions of PROJECT-ID it didn't look possible.

I'm not seeing any naming restrictions for the name of the image, so slashes must be allowed in that piece of the URL then?

That's correct @jpreese 😄 , as the docs suggest, the image is in the format of

docker tag [SOURCE_IMAGE] [HOSTNAME]/[PROJECT-ID]/[IMAGE]

IMAGE can contain slashes in it.

@jpreese
Copy link
Collaborator

jpreese commented Sep 14, 2020

Excellent, makes sense to me! Thanks again! I'll cut a new release.

@jpreese jpreese merged commit 8cb1840 into plexsystems:main Sep 14, 2020
@jpreese
Copy link
Collaborator

jpreese commented Sep 14, 2020

@luomoxu
Copy link

luomoxu commented Sep 29, 2020

could add an option to set registries use nested paths or not,use a list to mark is not a complete solution

@jpreese
Copy link
Collaborator

jpreese commented Sep 29, 2020

could add an option to set registries use nested paths or not,use a list to mark is not a complete solution

I dont have any objections to adding some mechanism for users to choose nested or not, if the target supports it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants