Skip to content
This repository was archived by the owner on Jan 16, 2025. It is now read-only.

Commit e0819f6

Browse files
authored
fix: add missing IAM permissions for runners from encrypted AMI (#3049)
This should fix missing IAM permissions when running from encrypted AMI. See [this issue](https://github.com/philips-labs/terraform-aws-github-runner/issues/2927)
1 parent 1de73bf commit e0819f6

File tree

2 files changed

+24
-0
lines changed

2 files changed

+24
-0
lines changed

Diff for: modules/runners/policies/lambda-scale-up.json

+12
Original file line numberDiff line numberDiff line change
@@ -63,6 +63,18 @@
6363
"kms:Decrypt"
6464
],
6565
"Resource": "${ami_kms_key_arn}"
66+
},
67+
{
68+
"Effect": "Allow",
69+
"Action": [
70+
"kms:CreateGrant"
71+
],
72+
"Resource": "${ami_kms_key_arn}",
73+
"Condition": {
74+
"Bool": {
75+
"aws:ViaAWSService": "true"
76+
}
77+
}
6678
%{ endif ~}
6779
}
6880
]

Diff for: modules/runners/pool/policies/lambda-pool.json

+12
Original file line numberDiff line numberDiff line change
@@ -54,6 +54,18 @@
5454
"kms:Decrypt"
5555
],
5656
"Resource": "${ami_kms_key_arn}"
57+
},
58+
{
59+
"Effect": "Allow",
60+
"Action": [
61+
"kms:CreateGrant"
62+
],
63+
"Resource": "${ami_kms_key_arn}",
64+
"Condition": {
65+
"Bool": {
66+
"aws:ViaAWSService": "true"
67+
}
68+
}
5769
%{ endif ~}
5870
}
5971
]

0 commit comments

Comments
 (0)