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

[Feature Request] Add Support for Public ECR Gallery #1515

Closed
PatrickXYS opened this issue Dec 8, 2020 · 9 comments
Closed

[Feature Request] Add Support for Public ECR Gallery #1515

PatrickXYS opened this issue Dec 8, 2020 · 9 comments

Comments

@PatrickXYS
Copy link

PatrickXYS commented Dec 8, 2020

Given we have launched Public ECR registry last week, we should expect feature-requests fron ECR users for using Kaniko with Public ECR Authentication.

So the whole story will be:

Add support in amazon-ecr-credential-helper -> amazon-ecr-credential-helper cut new release -> Send PR to kaniko and make sure we build latest image of amazon-ecr-credential-helper into our executor image.

Related issue: awslabs/amazon-ecr-credential-helper#248

/cc @tejal29 @priyawadhwa

Any insights will be appreciated!

@andreyvelich
Copy link

We would highly appreciate if we push this forward once amazon-ecr-credential-helper changes are done.
I assume that Dockerfile is using the latest package for the docker-credential-ecr-login, so we don't need to modify Kaniko codebase after new release for the amazon-ecr-credential-helper.

Just to be aware, do we have any approximate timelines when Kaniko folks are planning to make the new release ?

/cc @gaocegege @johnugeorge

@PatrickXYS
Copy link
Author

We have PR for fixing awslabs/amazon-ecr-credential-helper#248

awslabs/amazon-ecr-credential-helper#253

So next step should be collaborating with Kaniko folks for embedding latest ECR Credential Helper and cut new release.

@andreyvelich
Copy link

Thank you @PatrickXYS!
Should we wait until this PR: awslabs/amazon-ecr-credential-helper#253 will be merged ?

@PatrickXYS
Copy link
Author

Thank you @PatrickXYS!
Should we wait until this PR: awslabs/amazon-ecr-credential-helper#253 will be merged ?

Right, as long as the PR merged, we issue another PR in kaniko repo, and get it shipped.

@theofpa
Copy link
Contributor

theofpa commented Feb 15, 2021

awslabs/amazon-ecr-credential-helper#253 has been merged and included in the latest 0.50 release. The next kaniko release will include it as it will be fetched during the docker build, I don't think we need a PR for that.

% go get -u github.com/awslabs/amazon-ecr-credential-helper/ecr-login/cli/docker-credential-ecr-login
go: downloading github.com/awslabs/amazon-ecr-credential-helper/ecr-login v0.0.0-20210211190242-b19192b6522b
go: downloading github.com/awslabs/amazon-ecr-credential-helper v0.5.0
...

@PatrickXYS
Copy link
Author

@theofpa Thanks for confirmation! How is it going with Kaniko folk's release cadence?

@snooyen
Copy link

snooyen commented Feb 22, 2021

Has this been included in kaniko v1.5.0-debug or is this slated for the next release?

@theofpa
Copy link
Contributor

theofpa commented Feb 22, 2021

Has this been included in kaniko v1.5.0-debug or is this slated for the next release?

yes!

% docker run -it --entrypoint=/kaniko/docker-credential-ecr-login gcr.io/kaniko-project/executor:debug version
0.6.3

@PatrickXYS this issue can be closed as @tejal29 silently pushed v1.5.0 last week!

@PatrickXYS
Copy link
Author

Yeah very nice, we can close the issue as of now

/close

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

No branches or pull requests

4 participants