search

cloudposse / terraform-aws-tfstate-backend

Terraform module that provision an S3 bucket to store the `terraform.tfstate` file and a DynamoDB table to lock the state file to prevent concurrent modifications and state corruption.
https://cloudposse.com/accelerate
Apache License 2.0
408 stars 177 forks source link

Decoupling various resources from context module #108

Closed thiagoalmeidasa closed 1 year ago

thiagoalmeidasa commented 3 years ago

Removing dependency on some resources (such as IAM role, policy, and bucket replication rule) from the label module. Also added replication to the complete example.

nitrocode commented 3 years ago

Why is this change being made ? What problem is this solving ?

thiagoalmeidasa commented 3 years ago

Why is this change being made? What problem is this solving?

Created a new issue to describe it better: https://github.com/cloudposse/terraform-aws-tfstate-backend/issues/109

nitrocode commented 2 years ago

Thank you for the contribution but it's unclear what this is solving for. Updating the example makes sense but the rest of the changes do not. Could you explain more? This is one reason why we request filling out the PR template.