Skip to content

feat: New CloudWAN and Transit Gateway subnets #1181

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

wiseelf
Copy link
Contributor

@wiseelf wiseelf commented Apr 22, 2025

Description

Added two additional subnets for Transit Gateway and CloudWAN.

Motivation and Context

In order to follow AWS best practices for CloudWAN and Transit Gateway there is a need in two additional subnets for each. Example of documentation: https://docs.aws.amazon.com/vpc/latest/tgw/tgw-best-design-practices.html

Use a separate subnet for each transit gateway VPC attachment. For each subnet, use a small CIDR, for example /28, so that you have more addresses for EC2 resources. When you use a separate subnet, you can configure the following:

  1. Keep the inbound and outbound network ACLs associated with the transit gateway subnets open.
  2. Depending on your traffic flow, you can apply network ACLs to your workload subnets.

Breaking Changes

No breaking changes

How Has This Been Tested?

  • I have updated at least one of the examples/* to demonstrate and validate my change(s)
  • I have tested and validated these changes using one or more of the provided examples/* projects
  • I have executed pre-commit run -a on my pull request

@wiseelf
Copy link
Contributor Author

wiseelf commented Apr 22, 2025

@antonbabenko @bryantbiggs sorry for the mention, but please review, not sure how to assign reviewers here.

@sknmi
Copy link

sknmi commented Apr 24, 2025

We would like to have this as well. Also this fixes: #888

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants