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

README.md: Update docs on building for AWS ECR #2020

Merged
merged 1 commit into from
Mar 28, 2022

Conversation

Jasper-Ben
Copy link
Contributor

  • Kaniko v.1.8.0 does not require additional cred helper logic for ECR,
    as it discovers ECR repositories automatically and acts accordingly.
    Thus removed from the documentation.
  • Add details on using IAM role based authentication for pushing to ECR.

Fixes #780
Fixes #1455

Signed-off-by: Jasper Orschulko Jasper.Orschulko@iris-sensing.com

- Kaniko v.1.8.0 does not require additional cred helper logic for ECR,
as it discovers ECR repositories automatically and acts accordingly.
Thus removed from the documentation.
- Add details on using IAM role based authentication for pushing to ECR.

Fixes GoogleContainerTools#780
Fixes GoogleContainerTools#1455

Signed-off-by: Jasper Orschulko <Jasper.Orschulko@iris-sensing.com>
@google-cla
Copy link

google-cla bot commented Mar 28, 2022

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

For more information, open the CLA check for this pull request.

@Jasper-Ben
Copy link
Contributor Author

CLA signed.

@imjasonh
Copy link
Collaborator

Just to confirm, you've tested this in your own EC2 instance pushing to ECR correct? This looks great I just want to make double sure it's all true 😄

@Jasper-Ben
Copy link
Contributor Author

Jasper-Ben commented Mar 28, 2022

Just to confirm, you've tested this in your own EC2 instance pushing to ECR correct? This looks great I just want to make double sure it's all true smile

I've tested this on EKS with worker nodes running on EC2 instances and pushing to ECR 👍

@Jasper-Ben
Copy link
Contributor Author

What I haven't tested is credential based access but I see no reason why this should not work 😅

@imjasonh
Copy link
Collaborator

Looks good to me then! Looks like it's still complaining about the CLA though 🤔

@Jasper-Ben
Copy link
Contributor Author

Looks good to me then! Looks like it's still complaining about the CLA though thinking

had to request a rescan :)

@imjasonh imjasonh merged commit b6d18cc into GoogleContainerTools:main Mar 28, 2022
@Jasper-Ben Jasper-Ben deleted the update_docs branch March 28, 2022 13:54
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.

Kaniko failing to push to ECR Support native AWS IAM integration with updated docker-credential-ecr-login
2 participants