-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
(eks): config aws-node-termination-handler image use public ECR #12134
Labels
@aws-cdk/aws-eks
Related to Amazon Elastic Kubernetes Service
effort/small
Small work item – less than a day of effort
feature-request
A feature should be added or improved.
p1
Comments
neilkuan
added
feature-request
A feature should be added or improved.
needs-triage
This issue or PR still needs to be triaged.
labels
Dec 17, 2020
github-actions
bot
added
the
@aws-cdk/aws-eks
Related to Amazon Elastic Kubernetes Service
label
Dec 17, 2020
neilkuan
changed the title
(eks): aws-node-termination-handler config default image pull from public ECR
(eks): config aws-node-termination-handler default image pull from public ECR
Dec 17, 2020
neilkuan
changed the title
(eks): config aws-node-termination-handler default image pull from public ECR
(eks): config aws-node-termination-handler image use public ECR
Dec 17, 2020
iliapolo
added
effort/small
Small work item – less than a day of effort
p1
and removed
needs-triage
This issue or PR still needs to be triaged.
labels
Dec 17, 2020
mergify bot
pushed a commit
that referenced
this issue
Dec 25, 2020
…the image from public ECR (#12141) close #12134 update `aws-node-termination-handler` version to `0.13.2` ```ts this._spotInterruptHandler = this.addHelmChart('spot-interrupt-handler', { chart: 'aws-node-termination-handler', version: '0.13.2', repository: 'https://aws.github.io/eks-charts', namespace: 'kube-system', values: { 'nodeSelector.lifecycle': LifecycleLabel.SPOT, }, }); ``` ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
|
flochaz
pushed a commit
to flochaz/aws-cdk
that referenced
this issue
Jan 5, 2021
…the image from public ECR (aws#12141) close aws#12134 update `aws-node-termination-handler` version to `0.13.2` ```ts this._spotInterruptHandler = this.addHelmChart('spot-interrupt-handler', { chart: 'aws-node-termination-handler', version: '0.13.2', repository: 'https://aws.github.io/eks-charts', namespace: 'kube-system', values: { 'nodeSelector.lifecycle': LifecycleLabel.SPOT, }, }); ``` ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
@aws-cdk/aws-eks
Related to Amazon Elastic Kubernetes Service
effort/small
Small work item – less than a day of effort
feature-request
A feature should be added or improved.
p1
If you want to use spot worker node,
aws-eks
construct will help you installaws-node-termination-handler
.Now helm chart will pull image from docker hub by default.
see this repo:
https://github.com/aws/aws-node-termination-handler/blob/7ec2ce5457fd7a8a3d4fe0fb3f95445cb1cde9fc/config/helm/aws-node-termination-handler/values.yaml#L5-L9
Since dockerhub has begun rate limiting pulls, it will affect many CDK users with
aws-eks
construct.aws-cdk/packages/@aws-cdk/aws-eks/lib/cluster.ts
Lines 1464 to 1474 in d5afb55
Proposed Solution
see this PR: aws/aws-node-termination-handler#325
aws-node-termination-handler
can pull from https://gallery.ecr.aws/r6b0f9a1/aws-node-termination-handler now.I will create the PR for this
Other
This is a 🚀 Feature Request
The text was updated successfully, but these errors were encountered: