github-mirrors/terraform-aws-eks
1
0
Fork 0
mirror of https://github.com/ysoftdevs/terraform-aws-eks.git synced 2026-03-31 06:23:04 +02:00
Code Issues Packages Projects Releases Wiki Activity

fix: Use customer managed policy instead of inline policy for cluster_elb_sl_role_creation (#1039)

Browse Source 
NOTE: The usage of customer managed policy, not an inline policy, for the `cluster_elb_sl_role_creation policy` is common for "enterprise" AWS users to disallow inline policies with an SCP rule for auditing-related reasons, and this accomplishes the same thing.
This commit is contained in:
Rich Lafferty
2020-10-06 03:24:22 -04:00
committed by GitHub
parent 93614b329d
commit c799fecd85
1 changed files with 8 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
cluster.tf
View File

@@ -158,9 +158,15 @@ data "aws_iam_policy_document" "cluster_elb_sl_role_creation" {
} }
} }
resource "aws_iam_role_policy" "cluster_elb_sl_role_creation" { resource "aws_iam_policy" "cluster_elb_sl_role_creation" {
count = var.manage_cluster_iam_resources && var.create_eks ? 1 : 0 count = var.manage_cluster_iam_resources && var.create_eks ? 1 : 0
name_prefix = "${var.cluster_name}-elb-sl-role-creation" name_prefix = "${var.cluster_name}-elb-sl-role-creation"
role = local.cluster_iam_role_name description = "Permissions for EKS to create AWSServiceRoleForElasticLoadBalancing service-linked role"
policy = data.aws_iam_policy_document.cluster_elb_sl_role_creation[0].json policy = data.aws_iam_policy_document.cluster_elb_sl_role_creation[0].json
} }
resource "aws_iam_role_policy_attachment" "cluster_elb_sl_role_creation" {
count = var.manage_cluster_iam_resources && var.create_eks ? 1 : 0
policy_arn = aws_iam_policy.cluster_elb_sl_role_creation[0].arn
role = local.cluster_iam_role_name
}