Skip to content

Commit 5400c8f

Browse files
authored
fix: Defaults the role_name coalesce to * to workaround import error (#121)
1 parent 5a480b3 commit 5400c8f

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

iam.tf

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,11 @@ locals {
77
log_group_name = element(concat(data.aws_cloudwatch_log_group.lambda.*.name, aws_cloudwatch_log_group.lambda.*.name, [""]), 0)
88
log_group_arn = local.create_role && var.lambda_at_edge ? format("arn:%s:%s:%s:%s:%s", data.aws_arn.log_group_arn[0].partition, data.aws_arn.log_group_arn[0].service, "*", data.aws_arn.log_group_arn[0].account, data.aws_arn.log_group_arn[0].resource) : local.log_group_arn_regional
99

10-
role_name = local.create_role ? coalesce(var.role_name, var.function_name) : null
10+
# Defaulting to "*" (an invalid character for an IAM Role name) will cause an error when
11+
# attempting to plan if the role_name and function_name are not set. This is a workaround
12+
# for #83 that will allow one to import resources without receiving an error from coalesce.
13+
# @see https://github.com/terraform-aws-modules/terraform-aws-lambda/issues/83
14+
role_name = local.create_role ? coalesce(var.role_name, var.function_name, "*") : null
1115
}
1216

1317
###########

0 commit comments

Comments
 (0)