terraform-aws-modules / terraform-aws-transit-gateway

Terraform module to create AWS Transit Gateway resources 🇺🇦
https://registry.terraform.io/modules/terraform-aws-modules/transit-gateway/aws
Apache License 2.0
144 stars 226 forks source link

feat: Add support for security group referencing to transit-gateway m… #133

Open syedimranhassan opened 1 month ago

syedimranhassan commented 1 month ago

…odule

Added support for security_group_referencing_support in the transit-gateway module.

Description

Motivation and Context

AWS recently introduced [HERE] security group referencing for Transit Gateway VPC attachments. This feature was available in the standalone resource aws_ec2_transit_gateway_vpc_attachment but not in the transit-gateway module. Added support for it in the module.

Breaking Changes

no

How Has This Been Tested?

jhenry82 commented 1 month ago

Would love to see this too! One comment, this new setting exists at the Transit Gateway level too, not just attachments. So the module should also have a new variable for whether to enable it on the TGW. See https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/ec2_transit_gateway

syedimranhassan commented 1 month ago

@jhenry82 Thank you for pointing that out. I've updated the code to include SG referencing support at the TGW level as well.

syedimranhassan commented 1 month ago

@jhenry82 Can you please review this?

jhenry82 commented 1 month ago

I am not a maintainer of this module, my review wouldn't mean much :) I just use the module and came here to see if the feature had been added, and saw your MR. Hopefully it is able to be integrated soon.

syedimranhassan commented 1 month ago

@antonbabenko @bryantbiggs Can you please review this PR?

fintonic-plaform commented 3 weeks ago

We also need this functionality.