patch: Fix lambda role logic (#18)

jamengual · cloudpossebot · web-flow · commit 12fca4ed89de · 2022-06-07T20:34:34.000-04:00
* Fixing lambda function iam role logic

* Fixing lambda function iam role logic

* Auto Format

Co-authored-by: cloudpossebot &lt;11232728+cloudpossebot@users.noreply.github.com&gt;
diff --git a/iam-role.tf b/iam-role.tf
@@ -32,14 +32,14 @@ resource "aws_iam_role_policy_attachment" "cloudwatch_insights" {
 }
 
 resource "aws_iam_role_policy_attachment" "vpc_access" {
-  count = local.enabled && try(length(var.vpc_config), 0) > 0 ? 1 : 0
+  count = local.enabled && var.vpc_config != null ? 1 : 0
 
   policy_arn = "arn:${local.partition}:iam::aws:policy/service-role/AWSLambdaVPCAccessExecutionRole"
   role       = aws_iam_role.this[0].name
 }
 
 resource "aws_iam_role_policy_attachment" "xray" {
-  count = local.enabled && try(length(var.tracing_config_mode), 0) > 0 ? 1 : 0
+  count = local.enabled && var.tracing_config_mode != null ? 1 : 0
 
   policy_arn = "arn:${local.partition}:iam::aws:policy/AWSXRayDaemonWriteAccess"
   role       = aws_iam_role.this[0].name

Original file line number	Diff line number	Diff line change
`@@ -32,14 +32,14 @@ resource "aws_iam_role_policy_attachment" "cloudwatch_insights" {`
`32`	`32`	`}`
`33`	`33`
`34`	`34`	`resource "aws_iam_role_policy_attachment" "vpc_access" {`
`35`		`- count = local.enabled && try(length(var.vpc_config), 0) > 0 ? 1 : 0`
	`35`	`+ count = local.enabled && var.vpc_config != null ? 1 : 0`
`36`	`36`
`37`	`37`	`policy_arn = "arn:${local.partition}:iam::aws:policy/service-role/AWSLambdaVPCAccessExecutionRole"`
`38`	`38`	`role = aws_iam_role.this[0].name`
`39`	`39`	`}`
`40`	`40`
`41`	`41`	`resource "aws_iam_role_policy_attachment" "xray" {`
`42`		`- count = local.enabled && try(length(var.tracing_config_mode), 0) > 0 ? 1 : 0`
	`42`	`+ count = local.enabled && var.tracing_config_mode != null ? 1 : 0`
`43`	`43`
`44`	`44`	`policy_arn = "arn:${local.partition}:iam::aws:policy/AWSXRayDaemonWriteAccess"`
`45`	`45`	`role = aws_iam_role.this[0].name`