Closed natemccurdy closed 11 months ago
@natemccurdy this looks great. Suggested a simple whitespace change. I just pinged at Cloud Posse again on getting an approval on the README issues you're running into in your other PRs, so we'll get those merged first, you can rebase this PR, and then we'll get it merged. Thanks for the patience!
@natemccurdy thank you
can you please run the following commands and commit the changes
make init
make github/init
make readme
@natemccurdy thank you
can you please run the following commands and commit the changes
Hi @aknysh, thanks for looking. I've got 3 other PR's with the exact same issue of the boilerplate and readmes being out of sync. I fixed that in one of those PR's, https://github.com/cloudposse/terraform-aws-lambda-function/pull/45 , so I was hoping that rather than do the same thing 3 times (one for each PR), I can just rebase after merging https://github.com/cloudposse/terraform-aws-lambda-function/pull/45 and only have to do it once.
@natemccurdy thank you can you please run the following commands and commit the changes
Hi @aknysh, thanks for looking. I've got 3 other PR's with the exact same issue of the boilerplate and readmes being out of sync. I fixed that in one of those PR's, #45 , so I was hoping that rather than do the same thing 3 times (one for each PR), I can just rebase after merging #45 and only have to do it once.
https://github.com/cloudposse/terraform-aws-lambda-function/pull/45 is approved and merged, thank you. Please rebase the other PRs
Done. Thanks for your help @aknysh !
/terratest
Thanks @milldr 🍻
what
Replace the
toset()
in theaws_iam_role_policy_attachment
resource'sfor_each
attribute with a map of name:ARN pairs.why
Prior to this patch, specifying
custom_iam_policy_arns
for IAM Policies that do not exist yet and would be created in the same Terraform run that creates the Lambda Execution Role would cause the following error:This is due to the ARN's of those policies not being known at apply time and the usage of
toset()
in theaws_iam_role_policy_attachment
resource'sfor_each
parameter. As the set's values are unknown at apply time, Terraform can't create a dependency graph.references
Similar issues with similar fixes in other CloudPosse modules: