-
Notifications
You must be signed in to change notification settings - Fork 4k
Commit
…dd error checks (#31510) ### Issue # (if applicable) Closes #31148 ### Reason for this change When adding an arm-based instance, optimal instance is set by default which consists of an x86-based instance. This causes errors since arm and x86 instances can't be mixed together. ### Description of changes - useOptimalInstanceClasses is set to false for all arm-based instances - Throws error when trying to mix x86 and arm instances - Case 1: Instantiating the class - Case 2: addInstanceClass and addInstanceType functions - Warning useOptimalInstanceClasses is being set to false for arm-based instances ### Description of how you validated changes - Unit test where optimal doesn't get added for arm-based instances - Unit tests to display errors when instantiating the class - Unit tests to display errors when addInstanceClass and addInstanceType is adding mixed instances - Unit tests to display warning for arm based instances ### Checklist - [x] My code adheres to the [CONTRIBUTING GUIDE](https://github.com/aws/aws-cdk/blob/main/CONTRIBUTING.md) and [DESIGN GUIDELINES](https://github.com/aws/aws-cdk/blob/main/docs/DESIGN_GUIDELINES.md) ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
- Loading branch information
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.