Skip to content

Commit

Permalink
core(eks): support EKS service link role
Browse files Browse the repository at this point in the history
Prior to April 16, 2020, `AmazonEKSServicePolicy` was required for EKS cluster IAM role. With the new `AWSServiceRoleForAmazonEKS` service-linked role, that policy is no longer required.

This PR removes the `AmazonEKSServicePolicy` from the cluster role.

Closes #7634
  • Loading branch information
pahud authored May 1, 2020
1 parent 34af3d5 commit 1ecfca2
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 25 deletions.
1 change: 0 additions & 1 deletion packages/@aws-cdk/aws-eks/lib/cluster.ts
Original file line number Diff line number Diff line change
Expand Up @@ -368,7 +368,6 @@ export class Cluster extends Resource implements ICluster {
assumedBy: new iam.ServicePrincipal('eks.amazonaws.com'),
managedPolicies: [
iam.ManagedPolicy.fromAwsManagedPolicyName('AmazonEKSClusterPolicy'),
iam.ManagedPolicy.fromAwsManagedPolicyName('AmazonEKSServicePolicy'),
],
});

Expand Down
12 changes: 0 additions & 12 deletions packages/@aws-cdk/aws-eks/test/integ.eks-cluster.expected.json
Original file line number Diff line number Diff line change
Expand Up @@ -641,18 +641,6 @@
":iam::aws:policy/AmazonEKSClusterPolicy"
]
]
},
{
"Fn::Join": [
"",
[
"arn:",
{
"Ref": "AWS::Partition"
},
":iam::aws:policy/AmazonEKSServicePolicy"
]
]
}
]
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -613,18 +613,6 @@
":iam::aws:policy/AmazonEKSClusterPolicy"
]
]
},
{
"Fn::Join": [
"",
[
"arn:",
{
"Ref": "AWS::Partition"
},
":iam::aws:policy/AmazonEKSServicePolicy"
]
]
}
]
}
Expand Down

0 comments on commit 1ecfca2

Please sign in to comment.