-
-
Notifications
You must be signed in to change notification settings - Fork 109
feat: Ignore GSI changes #141
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
base: main
Are you sure you want to change the base?
Conversation
/terratest |
64f3805
to
20452fd
Compare
20452fd
to
50b50d9
Compare
Until the provider addresses this bug, this situation appears fair. We might need to add a README note to document that setting this option means Terraform will ignore any changes to GSIs and require manual or automated application. |
Sounds good 👍🏻 |
💥 This pull request now has conflicts. Could you fix it @pablotp? 🙏 |
aa6c434
to
0d22a4c
Compare
/terratest I can't think of any alternative solutions to this issue, but I'd love to get a second set of eyes from the team on this change. |
Right now, I can't either. |
what
why
global_secondary_index.read|write_capacity
by the autoscaler causes drift.references