Skip to content
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-cdk-lib/aws-ec2: CfnSubnet AvailabilityZoneId parameter removed #19563

Closed
fwieffering-blockfi opened this issue Mar 25, 2022 · 2 comments
Closed
Assignees
Labels
@aws-cdk/aws-cloudformation Related to AWS CloudFormation @aws-cdk/aws-ec2 Related to Amazon Elastic Compute Cloud bug This issue is a bug. effort/small Small work item – less than a day of effort p1

Comments

@fwieffering-blockfi
Copy link

What is the problem?

The AvailabilityZoneId attribute for the CfnSubnet struct was removed in this pr https://github.com/aws/aws-cdk/pull/19347/files#diff-16a9c9617c24ccee4f2d2bfb75440efc6ce83f64529d724840df0bd3adde9f0dL6603-L6608

This parameter is still available on the cloudformation object https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-resource-ec2-subnet.html#aws-resource-ec2-subnet-properties

I believe this is a bug. I was relying on this attribute.

Reproduction Steps

n/a

What did you expect to happen?

I expected fields not to be removed from the cloudformation constructs

What actually happened?

fields were removed from the cloudformation constructs

CDK CLI Version

2.17.0 (build f9cd009)

Framework Version

2.17.0

Node.js Version

v17.7.2

OS

macos

Language

Python

Language Version

3.10.2

Other information

No response

@fwieffering-blockfi fwieffering-blockfi added bug This issue is a bug. needs-triage This issue or PR still needs to be triaged. labels Mar 25, 2022
@github-actions github-actions bot added the @aws-cdk/aws-cloudformation Related to AWS CloudFormation label Mar 25, 2022
@ryparker ryparker added p1 effort/small Small work item – less than a day of effort @aws-cdk/aws-ec2 Related to Amazon Elastic Compute Cloud labels Mar 25, 2022
@peterwoodworth
Copy link
Contributor

Sorry for the inconvenience here @fwieffering-blockfi,

The properties have since been merged back #19457, however we haven't pushed a new v2 release since then (it should be coming soon!)

@peterwoodworth peterwoodworth removed the needs-triage This issue or PR still needs to be triaged. label Mar 28, 2022
@github-actions
Copy link

⚠️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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
@aws-cdk/aws-cloudformation Related to AWS CloudFormation @aws-cdk/aws-ec2 Related to Amazon Elastic Compute Cloud bug This issue is a bug. effort/small Small work item – less than a day of effort p1
Projects
None yet
Development

No branches or pull requests

4 participants