-
Notifications
You must be signed in to change notification settings - Fork 16
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
terraform function update #16
Comments
@robertdeheer thanks for reporting. I will ensure to get this module ready for TF >= 1.0 |
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
<#16 (comment)>,
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>.
|
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
> <#16 (comment)>,
> 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>.
>
>
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi'
I'm getting this error when running the module. Do you have any plans to fix it?
Thanks,
Rob
iam_module_error.txt
The text was updated successfully, but these errors were encountered: