-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
incompatible changes when defining a policy #13
Comments
Marius, My bad that this was not introduced in the first release of this module. Unfortunately, this was not possible to do as you suggest because of a limitation in Terraform which says Before this is 100% fixed in Terraform, we can't do anything better than requiring users to set Closing this issue. Thank you! |
still an incompatible change that crashes when relying on semantic versioning.. which can be fixed by increasing version number as this reflects a breaking change. |
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. |
just a small heads up:
When releasing newer minor versions the last days, you added incompatible changes:
in 1.0.0 we had defined
policy = ....
which created a bucket policy ... when using the latest the bucket policy is not created anymore even if set and you have to define the policy and add anattach_policy = true
attribute.maybe a good idea to assume you want to attach a policy if you define a policy (not
null
or""
)?thanks a lot.
The text was updated successfully, but these errors were encountered: