-
Notifications
You must be signed in to change notification settings - Fork 245
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
OpenSearch: EBSOptions.Iops and EBSOptions.VolumeSize type should be int
#4504
Comments
Hi @JCKortlang , thanks for reaching out. I see the cloudformation implementation and CDK typescript implementation specifies both props as Int. This is code level implementation which also declares these as int -https:/aws/aws-cdk/blob/491434e19ea4566ef90ff137efe62e694bb03cca/packages/aws-cdk-lib/aws-opensearchservice/lib/domain.ts#L155 but this is incorrectly interpreted to .NET docs as |
@khushail it is not just the documentation. The cdk library in .NET models it as a double. Which allows the invalid values to be set. |
Describe the bug
EBSOptions models Iops and VolumeSize as
double
but the cloud formation template defines them asint
. Deployments fail when these values are set to none integer values.Expected Behavior
EBSOptions.Iops and EBSOptions.VolumeSize type should be
int
https://docs.aws.amazon.com/opensearch-service/latest/APIReference/API_EBSOptions.html
Current Behavior
EBSOptions.Iops and EBSOptions.VolumeSize are
double
Reproduction Steps
Possible Solution
Change EBSOptions.Iops and EBSOptions.VolumeSize to
int
Additional Information/Context
No response
CDK CLI Version
2.136.1 (build 6f21c1d)
Framework Version
2.139.0
Node.js Version
v21.7.2
OS
MacOS 13.6.6 (22G630)
Language
.NET
Language Version
8.0.101
Other information
No response
The text was updated successfully, but these errors were encountered: