-
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
(aws-eks): instanceType is deprecated, use instanceTypes instead #22547
Comments
Working on it |
What's the bug you're describing exactly? We deprecated this prop two years ago, so you should only be able to use |
I've updated the description now to make it more clear |
Thanks @bora-7! Let us know once you've submitted a PR |
…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*
|
…Capacity functions (aws#22590) fixes aws#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*
Describe the bug
There are some integ tests in aws-eks using the deprecated instanceType instead of the instanceTypes and yarn test throws a lot of warnings.
Expected Behavior
No more warnings for deprecated 'instanceType' properties from 'yarn test'
Current Behavior
There are warning signs since instanceType is now deprecated and instanceTypes should be used instead
Reproduction Steps
yarn test on aws-eks
Possible Solution
Change mentions of instanceType to 'instanceTypes'
CDK CLI Version
2.44.0
Node.js Version
v16.13.2
OS
MacOS
Language
Typescript
The text was updated successfully, but these errors were encountered: