Skip to content

Commit

Permalink
chore(aws-eks): changed instanceType to instanceTypes in addNodegroup…
Browse files Browse the repository at this point in the history
…Capacity functions (#22590)

fixes #22547 

---
 
Since `instanceType` is deprecated, I changed it to `instanceTypes` in all the `addNodegroupCapacity` functions

----

### 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

* [ ] Have you added the new feature to an [integration test](https://github.com/aws/aws-cdk/blob/main/INTEGRATION_TESTS.md)?
	* [ ] 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
MrSyn88 committed Oct 20, 2022
1 parent 562bbfe commit 03210be
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions packages/@aws-cdk/aws-eks/test/integ.eks-cluster.ts
Original file line number Diff line number Diff line change
Expand Up @@ -192,7 +192,7 @@ class EksClusterStack extends Stack {
private assertNodeGroupX86() {
// add a extra nodegroup
this.cluster.addNodegroupCapacity('extra-ng', {
instanceType: new ec2.InstanceType('t3.small'),
instanceTypes: [new ec2.InstanceType('t3.small')],
minSize: 1,
// reusing the default capacity nodegroup instance role when available
nodeRole: this.cluster.defaultCapacity ? this.cluster.defaultCapacity.role : undefined,
Expand Down Expand Up @@ -241,7 +241,7 @@ class EksClusterStack extends Stack {
private assertNodeGroupArm() {
// add a extra nodegroup
this.cluster.addNodegroupCapacity('extra-ng-arm', {
instanceType: new ec2.InstanceType('m6g.medium'),
instanceTypes: [new ec2.InstanceType('m6g.medium')],
minSize: 1,
// reusing the default capacity nodegroup instance role when available
nodeRole: this.cluster.defaultCapacity ? this.cluster.defaultCapacity.role : undefined,
Expand All @@ -250,7 +250,7 @@ class EksClusterStack extends Stack {
private assertNodeGroupGraviton3() {
// add a Graviton3 nodegroup
this.cluster.addNodegroupCapacity('extra-ng-arm3', {
instanceType: new ec2.InstanceType('c7g.large'),
instanceTypes: [new ec2.InstanceType('c7g.large')],
minSize: 1,
// reusing the default capacity nodegroup instance role when available
nodeRole: this.cluster.defaultCapacity ? this.cluster.defaultCapacity.role : undefined,
Expand Down

0 comments on commit 03210be

Please sign in to comment.