terraform-aws-iam icon indicating copy to clipboard operation
terraform-aws-iam copied to clipboard

terraform function update

Open robertdeheer opened this issue 4 years ago • 3 comments

Hi'

I'm getting this error when running the module. Do you have any plans to fix it?

Thanks, Rob iam_module_error.txt

robertdeheer avatar Sep 11 '21 00:09 robertdeheer

@robertdeheer thanks for reporting. I will ensure to get this module ready for TF >= 1.0

cytopia avatar Sep 11 '21 13:09 cytopia

Thanks, I really appreciate it.

Rob

On Sat, Sep 11, 2021 at 6:13 AM cytopia @.***> wrote:

@robertdeheer https://github.com/robertdeheer thanks for reporting. I will ensure to get this module ready for TF >= 1.0

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/cytopia/terraform-aws-iam/issues/16#issuecomment-917405796, or unsubscribe https://github.com/notifications/unsubscribe-auth/AD22AX6GMVSF5OOP2XU4NVLUBNIZNANCNFSM5D2KCY3A . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

robertdeheer avatar Sep 11 '21 16:09 robertdeheer

It looks like a simple fix.

diff --git a/main.tf b/main.tf

index b455056..7d0f3a7 100644

--- a/main.tf

+++ b/main.tf

@@ -143,7 +143,7 @@ resource "aws_iam_user" "users" {

permissions_boundary = each.value.permissions_boundary

tags = merge(

  • map("Name", lookup(each.value, "name")),
  • {"Name" = lookup(each.value, "name")},

    var.tags

)

}

@@ -243,7 +243,7 @@ resource "aws_iam_role" "roles" {

force_detach_policies = var.role_force_detach_policies

tags = merge(

  • map("Name", lookup(each.value, "name")),
  • {"Name" = lookup(each.value, "name")},

    var.tags

)

}

On Sat, Sep 11, 2021 at 9:58 AM Robert de Heer @.***> wrote:

Thanks, I really appreciate it.

Rob

On Sat, Sep 11, 2021 at 6:13 AM cytopia @.***> wrote:

@robertdeheer https://github.com/robertdeheer thanks for reporting. I will ensure to get this module ready for TF >= 1.0

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/cytopia/terraform-aws-iam/issues/16#issuecomment-917405796, or unsubscribe https://github.com/notifications/unsubscribe-auth/AD22AX6GMVSF5OOP2XU4NVLUBNIZNANCNFSM5D2KCY3A . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

robertdeheer avatar Sep 15 '21 00:09 robertdeheer