Added dependency in role attachment

himsharma01 · himsharma01 · commit 7e94bc955587 · 2025-02-07T18:56:45.000+05:30
diff --git a/aws/cloudtrail/cloudtrail.tf b/aws/cloudtrail/cloudtrail.tf
@@ -87,6 +87,7 @@ resource "aws_iam_policy" "iam_policy" {
 }
 
 resource "aws_iam_role_policy_attachment" "source-role-policy-attach" {
+  for_each = toset(var.source_details.iam_details.create_iam_role ? ["source_iam_role"] : [])
   role       = aws_iam_role.source_iam_role["source_iam_role"].name
   policy_arn = aws_iam_policy.iam_policy["iam_policy"].arn
 }
diff --git a/aws/elasticloadbalancing/elb.tf b/aws/elasticloadbalancing/elb.tf
@@ -76,6 +76,7 @@ resource "aws_iam_policy" "iam_policy" {
 }
 
 resource "aws_iam_role_policy_attachment" "source-role-policy-attach" {
+  for_each = toset(var.source_details.iam_details.create_iam_role ? ["source_iam_role"] : [])
   role       = aws_iam_role.source_iam_role["source_iam_role"].name
   policy_arn = aws_iam_policy.iam_policy["iam_policy"].arn
 }
diff --git a/aws/elb/elb.tf b/aws/elb/elb.tf
@@ -76,6 +76,7 @@ resource "aws_iam_policy" "iam_policy" {
 }
 
 resource "aws_iam_role_policy_attachment" "source-role-policy-attach" {
+  for_each = toset(var.source_details.iam_details.create_iam_role ? ["source_iam_role"] : [])
   role       = aws_iam_role.source_iam_role["source_iam_role"].name
   policy_arn = aws_iam_policy.iam_policy["iam_policy"].arn
 }
diff --git a/aws/rootcause/rootcause.tf b/aws/rootcause/rootcause.tf
@@ -30,6 +30,7 @@ resource "aws_iam_policy" "iam_policy" {
 }
 
 resource "aws_iam_role_policy_attachment" "source-role-policy-attach" {
+  for_each = toset(var.iam_details.create_iam_role ? ["source_iam_role"] : [])
   role       = aws_iam_role.source_iam_role["source_iam_role"].name
   policy_arn = aws_iam_policy.iam_policy["iam_policy"].arn
 }

Original file line number	Diff line number	Diff line change
`@@ -87,6 +87,7 @@ resource "aws_iam_policy" "iam_policy" {`
`87`	`87`	`}`
`88`	`88`
`89`	`89`	`resource "aws_iam_role_policy_attachment" "source-role-policy-attach" {`
	`90`	`+ for_each = toset(var.source_details.iam_details.create_iam_role ? ["source_iam_role"] : [])`
`90`	`91`	`role = aws_iam_role.source_iam_role["source_iam_role"].name`
`91`	`92`	`policy_arn = aws_iam_policy.iam_policy["iam_policy"].arn`
`92`	`93`	`}`
Original file line number	Diff line number	Diff line change
`@@ -76,6 +76,7 @@ resource "aws_iam_policy" "iam_policy" {`
`76`	`76`	`}`
`77`	`77`
`78`	`78`	`resource "aws_iam_role_policy_attachment" "source-role-policy-attach" {`
	`79`	`+ for_each = toset(var.source_details.iam_details.create_iam_role ? ["source_iam_role"] : [])`
`79`	`80`	`role = aws_iam_role.source_iam_role["source_iam_role"].name`
`80`	`81`	`policy_arn = aws_iam_policy.iam_policy["iam_policy"].arn`
`81`	`82`	`}`
Original file line number	Diff line number	Diff line change
`@@ -30,6 +30,7 @@ resource "aws_iam_policy" "iam_policy" {`
`30`	`30`	`}`
`31`	`31`
`32`	`32`	`resource "aws_iam_role_policy_attachment" "source-role-policy-attach" {`
	`33`	`+ for_each = toset(var.iam_details.create_iam_role ? ["source_iam_role"] : [])`
`33`	`34`	`role = aws_iam_role.source_iam_role["source_iam_role"].name`
`34`	`35`	`policy_arn = aws_iam_policy.iam_policy["iam_policy"].arn`
`35`	`36`	`}`