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

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

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

Conversation

syedimranhassan
Copy link

@syedimranhassan syedimranhassan commented Sep 27, 2024

…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?

  • 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

@syedimranhassan syedimranhassan changed the title feat: add support for security group referencing to transit-gateway m… feat: Add support for security group referencing to transit-gateway m… Sep 27, 2024
…odule

AWS recently introduced 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.
@jhenry82
Copy link

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
Copy link
Author

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

@syedimranhassan
Copy link
Author

@jhenry82 Can you please review this?

@jhenry82
Copy link

jhenry82 commented Oct 2, 2024

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
Copy link
Author

@antonbabenko @bryantbiggs Can you please review this PR?

@fintonic-plaform
Copy link

We also need this functionality.

@tigpt
Copy link

tigpt commented Nov 19, 2024

BUMP we need this.

@savealive
Copy link

Can we get this feature released?

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.

5 participants