-
Notifications
You must be signed in to change notification settings - Fork 997
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 remaining DockerHub-hosted repositories to use ECR #356
Comments
aws-node-termination-handler will be moving to ECR images soon. |
AWS ECR public gallery is out: https://gallery.ecr.aws/ We'll be moving the images to ECR gallery |
NTH Public ECR: https://gallery.ecr.aws/r6b0f9a1/aws-node-termination-handler Chart will be updated soon to use it by default. Images will still be pushed to Dockerhub in addition to ECR for now. |
The cloudwatch-agent's image tag that comes by default in the chart is not available in the public repository.
Should the default image tag be bumped? |
It would make sense that all EKS charts now use the ECR Public repository. [re aws#356]
It would make sense that all EKS charts now use the ECR Public repository. [re #356]
tl;dr: Please update
amazon/foo
registry values in these charts to use Amazon-hosted ECR versions of the same.Per the Coming Soon blogpost for the AWS public registry:
We went to update our
aws-node-termination-handler
to use this chart after getting DockerHub pull failures, but it doesn't look like it's going to make a difference:eks-charts/stable/aws-node-termination-handler/values.yaml
Line 6 in d721c97
There are several other places that would be subject to the same throttling:
And several public images that might need to be replicated as well, either as part of the deployment or as part of CI:
Please update the remaining
amazon/*
images (at least) to use an AWS ECR repository value instead of relying on the current DockerHub repo.The text was updated successfully, but these errors were encountered: