fpco / terraform-aws-foundation

Establish a solid Foundation on AWS with these modules for Terraform
MIT License
203 stars 99 forks source link

Rework DLM policy modules. #305

Closed Magicloud closed 4 years ago

Magicloud commented 4 years ago

The original design has implict dependencies between dlm-lifecycle-iam-role and dlm-lifecycle-policy on IAM role. But TF currently does not support module dependency, and it does not get the dependency when a module is depends on another resource's name output.

So merged the two modules to avoid applying error that "the role does not exist".


name: Pull request template about: Make a PR to terraform-aws-foundation

Please include the following in your PR:

Please also note that these are not hard requirements, but merely serve to define what maintainers are looking for in PR's. Including these will more likely lead to your PR being reviewed and accepted.