Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(eks): support EKS service link role #7637

Merged
merged 5 commits into from
May 1, 2020
+0 −25
Merged

chore(eks): support EKS service link role #7637

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
673ef87
remove AmazonEKSServicePolicy
pahud Apr 28, 2020
01607b6
Merge branch 'master' of https://github.com/aws/aws-cdk into eks-serv…
pahud Apr 30, 2020
329299e
update integ test
pahud Apr 30, 2020
6ff144f
Merge branch 'master' into eks-serviceLinkRole
pahud May 1, 2020
d58fa1d
Merge branch 'master' into eks-serviceLinkRole
mergify[bot] May 1, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
remove AmazonEKSServicePolicy
@pahud
pahud committed Apr 28, 2020
commit 673ef87a37d64fc088973814f208cfe470581805
1 change: 0 additions & 1 deletion packages/@aws-cdk/aws-eks/lib/cluster.ts
View file
Original file line number Diff line number Diff line change
@@ -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'),
],
});

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