-
-
Notifications
You must be signed in to change notification settings - Fork 4.5k
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
change redshift_route_table_ids ? #738
Comments
Thanks for opening this issue and providing a specific line that broke it. I will make a fix now. |
This issue has been resolved in version 3.11.4 🎉 |
Thank you for your quick response and release!! |
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
Description
The private route table assigned to the redshift subnet is changed to the public route table.
Versions
Reproduction
I'm trying to use github actions to bring it up to date.
Today I noticed a diff.
Code Snippet to Reproduce
no set var.enable_public_redshift, so default is false right?
Expected behavior
no changes.
Actual behavior
Additional context
Caused by this change
https://github.com/terraform-aws-modules/terraform-aws-vpc/pull/729/files#diff-de6c47c2496bd028a84d55ab12d8a4f90174ebfb6544b8b5c7b07a7ee4f27ec7R253
Ignore the flag and use the public subnet.
The text was updated successfully, but these errors were encountered: