Skip to content

Commit 94ca428

Browse files
authored
Merge pull request #3 from worldcoin/feat/aws-load-balancer/add-variable-to-control-aws-load-balancer-iam-role-creation
INFRA-5029(aws-load-balancer): Add variable to control AWS Load Balancer IAM role creation
2 parents 4ebe612 + f646307 commit 94ca428

File tree

2 files changed

+19
-1
lines changed

2 files changed

+19
-1
lines changed

iam-aws-load-balancer.tf

Lines changed: 13 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -329,13 +329,25 @@ data "aws_iam_policy_document" "aws_load_balancer" {
329329
}
330330

331331
resource "aws_iam_role" "aws_load_balancer" {
332+
count = var.aws_load_balancer_iam_role_enabled ? 1 : 0
332333
name = "aws-load-balancer-controller-${var.cluster_name}"
333334
path = "/system/"
334335
assume_role_policy = data.aws_iam_policy_document.aws_load_balancer_assume_role_policy.json
335336
}
336337

337338
resource "aws_iam_role_policy" "aws_load_balancer" {
339+
count = var.aws_load_balancer_iam_role_enabled ? 1 : 0
338340
name = "aws-load-balancer-controller-${var.cluster_name}"
339-
role = aws_iam_role.aws_load_balancer.id
341+
role = aws_iam_role.aws_load_balancer[0].id
340342
policy = data.aws_iam_policy_document.aws_load_balancer.json
341343
}
344+
345+
moved {
346+
from = aws_iam_role.aws_load_balancer
347+
to = aws_iam_role.aws_load_balancer[0]
348+
}
349+
350+
moved {
351+
from = aws_iam_role_policy.aws_load_balancer
352+
to = aws_iam_role_policy.aws_load_balancer[0]
353+
}

variables.tf

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -656,3 +656,9 @@ variable "vpc_cni_external_snat" {
656656
type = string
657657
default = false
658658
}
659+
660+
variable "aws_load_balancer_iam_role_enabled" {
661+
description = "Whether to enable the IAM role for the AWS Load Balancer"
662+
type = bool
663+
default = true
664+
}

0 commit comments

Comments
 (0)