Skip to content

Commit b14172d

Browse files
authored
fix(eks): missing permissions to add and remove tags when creating EKS cluster resource (#7302)
Added missing permissions for `eks:TagResource` and `eks:UntagResource`. Updated unit and integ test expectations to include the added permissions. Closes #7163
1 parent 1475d5a commit b14172d

File tree

3 files changed

+19
-4
lines changed

3 files changed

+19
-4
lines changed

packages/@aws-cdk/aws-eks/lib/cluster-resource.ts

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,16 @@ export class ClusterResource extends Construct {
7878
}));
7979

8080
this.creationRole.addToPolicy(new iam.PolicyStatement({
81-
actions: [ 'eks:CreateCluster', 'eks:DescribeCluster', 'eks:DeleteCluster', 'eks:UpdateClusterVersion', 'eks:UpdateClusterConfig', 'eks:CreateFargateProfile' ],
81+
actions: [
82+
'eks:CreateCluster',
83+
'eks:DescribeCluster',
84+
'eks:DeleteCluster',
85+
'eks:UpdateClusterVersion',
86+
'eks:UpdateClusterConfig',
87+
'eks:CreateFargateProfile',
88+
'eks:TagResource',
89+
'eks:UntagResource'
90+
],
8291
resources: resourceArns
8392
}));
8493

packages/@aws-cdk/aws-eks/test/integ.eks-cluster.expected.json

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -784,7 +784,9 @@
784784
"eks:DeleteCluster",
785785
"eks:UpdateClusterVersion",
786786
"eks:UpdateClusterConfig",
787-
"eks:CreateFargateProfile"
787+
"eks:CreateFargateProfile",
788+
"eks:TagResource",
789+
"eks:UntagResource"
788790
],
789791
"Effect": "Allow",
790792
"Resource": [

packages/@aws-cdk/aws-eks/test/test.cluster.ts

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -718,7 +718,9 @@ export = {
718718
'eks:DeleteCluster',
719719
'eks:UpdateClusterVersion',
720720
'eks:UpdateClusterConfig',
721-
'eks:CreateFargateProfile'
721+
'eks:CreateFargateProfile',
722+
'eks:TagResource',
723+
'eks:UntagResource'
722724
],
723725
Effect: 'Allow',
724726
Resource: [ {
@@ -826,7 +828,9 @@ export = {
826828
'eks:DeleteCluster',
827829
'eks:UpdateClusterVersion',
828830
'eks:UpdateClusterConfig',
829-
'eks:CreateFargateProfile'
831+
'eks:CreateFargateProfile',
832+
'eks:TagResource',
833+
'eks:UntagResource'
830834
],
831835
Effect: 'Allow',
832836
Resource: [ '*' ]

0 commit comments

Comments
 (0)