Skip to content

feat(ec2-alpha): add CloudFormation Parameter support for VpcV2 CIDR#36884

Open
dgandhi62 wants to merge 1 commit intoaws:mainfrom
dgandhi62:fix-35245
Open

feat(ec2-alpha): add CloudFormation Parameter support for VpcV2 CIDR#36884
dgandhi62 wants to merge 1 commit intoaws:mainfrom
dgandhi62:fix-35245

Conversation

@dgandhi62
Copy link
Contributor

Enables VpcV2 to accept CloudFormation Parameters for CIDR blocks, allowing runtime CIDR specification for AWS Service Catalog deployments and multi-tenant scenarios.

  • Enhanced VpcV2 to accept Token-based CIDR values
  • Modified SubnetV2 validation to skip Token-based CIDRs
  • Added 5 new unit tests (175/175 passing)
  • Created integration test with Fn::Cidr examples
  • Updated README with CloudFormation Parameter usage

Closes #35245

Enables VpcV2 to accept CloudFormation Parameters for CIDR blocks,
allowing runtime CIDR specification for AWS Service Catalog deployments
and multi-tenant scenarios.

- Enhanced VpcV2 to accept Token-based CIDR values
- Modified SubnetV2 validation to skip Token-based CIDRs
- Added 5 new unit tests (175/175 passing)
- Created integration test with Fn::Cidr examples
- Updated README with CloudFormation Parameter usage

Closes aws#35245
@github-actions github-actions bot added beginning-contributor [Pilot] contributed between 0-2 PRs to the CDK effort/medium Medium work item – several days of effort labels Feb 4, 2026
@aws-cdk-automation aws-cdk-automation requested a review from a team February 4, 2026 22:35
@github-actions github-actions bot added feature-request A feature should be added or improved. p2 labels Feb 4, 2026
@mergify mergify bot added the contribution/core This is a PR that came from AWS. label Feb 4, 2026
Copy link
Collaborator

@aws-cdk-automation aws-cdk-automation left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The pull request linter fails with the following errors:

❌ Features must contain a change to an integration test file and the resulting snapshot.

If you believe this pull request should receive an exemption, please comment and provide a justification. A comment requesting an exemption should contain the text Exemption Request. Additionally, if clarification is needed, add Clarification Request to a comment.

@dgandhi62 dgandhi62 marked this pull request as ready for review February 4, 2026 22:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

beginning-contributor [Pilot] contributed between 0-2 PRs to the CDK contribution/core This is a PR that came from AWS. effort/medium Medium work item – several days of effort feature-request A feature should be added or improved. p2

Projects

None yet

Development

Successfully merging this pull request may close these issues.

aws_ec2: L2 Vpc Construct incompatible with CloudFormation Parameters for CIDR - breaks Service Catalog deployments

2 participants