Skip to content

Commit

Permalink
fix(eks): latest AlbController version isn't compatible with the ch…
Browse files Browse the repository at this point in the history
…art version (#20826)

Bump the chart version.

Fixes #20764

----

### All Submissions:

* [x] Have you followed the guidelines in our [Contributing guide?](https://github.com/aws/aws-cdk/blob/main/CONTRIBUTING.md)

### Adding new Unconventional Dependencies:

* [ ] This PR adds new unconventional dependencies following the process described [here](https://github.com/aws/aws-cdk/blob/main/CONTRIBUTING.md/#adding-new-unconventional-dependencies)

### New Features

* [x] Have you added the new feature to an [integration test](https://github.com/aws/aws-cdk/blob/main/INTEGRATION_TESTS.md)?
	* [x] Did you use `yarn integ` to deploy the infrastructure and generate the snapshot (i.e. `yarn integ` without `--dry-run`)?

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
  • Loading branch information
adriantaut authored and iliapolo committed Jul 6, 2022
1 parent 8e2ef42 commit f40662b
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 5 deletions.
2 changes: 1 addition & 1 deletion packages/@aws-cdk/aws-eks/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -535,7 +535,7 @@ From the docs:
> * It satisfies Kubernetes Service resources by provisioning Network Load Balancers.
To deploy the controller on your EKS cluster, configure the `albController` property:

```ts
new eks.Cluster(this, 'HelloEKS', {
version: eks.KubernetesVersion.V1_21,
Expand Down
2 changes: 1 addition & 1 deletion packages/@aws-cdk/aws-eks/lib/alb-controller.ts
Original file line number Diff line number Diff line change
Expand Up @@ -231,7 +231,7 @@ export class AlbController extends Construct {
// want to expose this since helm here is just an implementation detail
// for installing a specific version of the controller itself.
// https://github.com/aws/eks-charts/blob/v0.0.65/stable/aws-load-balancer-controller/Chart.yaml
version: '1.2.7',
version: '1.4.1',

wait: true,
timeout: Duration.minutes(15),
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1477,7 +1477,7 @@
},
"Release": "aws-load-balancer-controller",
"Chart": "aws-load-balancer-controller",
"Version": "1.2.7",
"Version": "1.4.1",
"Wait": true,
"Timeout": "900s",
"Values": {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1842,7 +1842,7 @@
},
"Release": "aws-load-balancer-controller",
"Chart": "aws-load-balancer-controller",
"Version": "1.2.7",
"Version": "1.4.1",
"Wait": true,
"Timeout": "900s",
"Values": {
Expand Down Expand Up @@ -2247,4 +2247,4 @@
"Default": "/aws/service/eks/optimized-ami/1.21/amazon-linux-2-gpu/recommended/image_id"
}
}
}
}

0 comments on commit f40662b

Please sign in to comment.