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

fix: NAT gw route table associations shouldn't be created when enable_nat_gateway=false #1090

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kkonstant-in
Copy link

@kkonstant-in kkonstant-in commented Jun 5, 2024

Description

Fixes creating NAT gateway route table associations even when enable_nat_gateway=false

Motivation and Context

More detailed description of the issue is here #1031

Breaking Changes

no breaking changes

How Has This Been Tested?

Tested in production TGW environment with no NAT gw route table associations

@kkonstant-in kkonstant-in changed the title Fix creating nat gw aws_route_table_association's when enable_nat_gateway=false fix: creating nat gw aws_route_table_association's when enable_nat_gateway=false Jun 5, 2024
@kkonstant-in kkonstant-in changed the title fix: creating nat gw aws_route_table_association's when enable_nat_gateway=false fix: NAT gw route table associations shouldn't be created when enable_nat_gateway=false Jun 5, 2024
@kkonstant-in
Copy link
Author

Hi there.

@bryantbiggs do you need a more detailed description or anything else for this issue?
Looks like a simple fix, isn't it?

@kkonstant-in
Copy link
Author

Hi, any updates?

Copy link

github-actions bot commented Aug 8, 2024

This PR has been automatically marked as stale because it has been open 30 days
with no activity. Remove stale label or comment or this PR will be closed in 10 days

@github-actions github-actions bot added the stale label Aug 8, 2024
@kkonstant-in
Copy link
Author

Hi, any updates?

@github-actions github-actions bot removed the stale label Aug 9, 2024
@kkonstant-in
Copy link
Author

Hi, any updates?

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.

1 participant