Skip to content

Commit ca0ce74

Browse files
authored
Merge pull request #42 from Homebrew/cleanup
roles: cleanup and import
2 parents 8c50f42 + baba2b2 commit ca0ce74

File tree

2 files changed

+11
-8
lines changed

2 files changed

+11
-8
lines changed

aws/roles.tf

+6-8
Original file line numberDiff line numberDiff line change
@@ -11,13 +11,6 @@ data "aws_iam_policy_document" "codebuild_policy_document" {
1111
"s3:Put*",
1212
"s3:DeleteObject",
1313
"s3:DeleteObjectVersion",
14-
"ecs:*",
15-
"ecr:*",
16-
"apigateway:*",
17-
"elasticloadbalancing:*",
18-
"iam:DeleteGroupMembership",
19-
"iam:DetachRolePolicy",
20-
"iam:DeletePolicy"
2114
]
2215
resources = [
2316
"arn:aws:s3:::homebrew-terraform-state/*",
@@ -29,7 +22,12 @@ data "aws_iam_policy_document" "codebuild_policy_document" {
2922
effect = "Allow"
3023
actions = [
3124
"iam:*",
32-
"sso:TagResource"
25+
"sso:TagResource",
26+
"ecs:*",
27+
"ecr:*",
28+
"apigateway:*",
29+
"elasticloadbalancing:*",
30+
"identitystore:*"
3331
]
3432
resources = ["*"]
3533
}

import.tf

+5
Original file line numberDiff line numberDiff line change
@@ -45,3 +45,8 @@ import {
4545
to = module.aws.aws_iam_openid_connect_provider.github_actions
4646
id = "arn:aws:iam::765021812025:oidc-provider/token.actions.githubusercontent.com"
4747
}
48+
49+
import {
50+
to = module.aws.aws_iam_role.github_tf
51+
id = "GitHubActionsRole"
52+
}

0 commit comments

Comments
 (0)