(autoscaling): EbsDeviceVolumeType.IO2 is not a valid CloudFormation value
See original GitHub issueAWS::AutoScaling::LaunchConfiguration BlockDevice supports the following values for property VolumeType (see documentation):
Valid Values: standard | io1 | gp2 | st1 | sc1 | gp3
In CDK, values IO2 is also defined as a possible value in enum EbsDeviceVolumeType.
However, this value is not valid at the moment and should be removed.
Reproduction Steps
const vpc = ec2.Vpc.fromLookup(this, 'MyVPC', {
isDefault: true,
})
new autoscaling.AutoScalingGroup(this, 'MyAutoScalingGroup', {
blockDevices: [
{
deviceName: '/dev/sda1',
volume: {
ebsDevice: {
deleteOnTermination: true,
volumeType: autoscaling.EbsDeviceVolumeType.IO2,
}
},
}
],
vpc,
instanceType: ec2.InstanceType.of(ec2.InstanceClass.T3, ec2.InstanceSize.SMALL),
machineImage: new ec2.AmazonLinuxImage(),
});
What did you expect to happen?
Deployment of AutoScalingGroup should be successfully.
What actually happened?
Deployment failed with error:
io2 is invalid. Valid volume types are: standard, io1, gp2, st1, sc1, gp3 (Service: AmazonAutoScaling; Status Code: 400; Error Code: ValidationError; Request ID: e5dd0928-b959-4e36-beed-91b7b2ba1f6a; Proxy: null)
Environment
- **CDK CLI Version : 1.118.0
- **Framework Version: 1.118.0
- **Node.js Version: v14.16.0
- **OS : macOS
- **Language (Version): all
Other
Originally mentioned in issue #14685. I will submit a short PR and remove value IO2 as proposed for GP3 in the original issue.
This is 🐛 Bug Report
Issue Analytics
- State:
- Created 2 years ago
- Comments:7 (5 by maintainers)
Top GitHub Comments
There’s no mention in any of the original issues about Autoscaling, and Autoscaling has never supported io2. The PR seems to have just included the autoscaling enum as well even though autoscaling doesn’t support io2
I wonder if the Autoscaling volume type lags behind the ec2 volume type. In other words, I wonder if io2 will be supported soon for Autoscaling like gp3 was.
For now, I can confirm your PR is safe to merge. I’ll look into if io2 will be an option for autoscaling and if I can’t find anything about that I will go ahead and merge your change 😃
⚠️COMMENT VISIBILITY WARNING⚠️
Comments on closed issues are hard for our team to see. If you need more assistance, please either tag a team member or open a new issue that references this one. If you wish to keep having a conversation with other community members under this issue feel free to do so.