Skip to content

Commit f3d03c1

Browse files
committed
Switch to OIDC role for uploading builds
1 parent 5ea2535 commit f3d03c1

File tree

1 file changed

+18
-35
lines changed
  • terragrunt/modules/rustup

1 file changed

+18
-35
lines changed

terragrunt/modules/rustup/s3.tf

Lines changed: 18 additions & 35 deletions
Original file line numberDiff line numberDiff line change
@@ -6,46 +6,29 @@ resource "aws_s3_bucket" "builds" {
66
bucket = "rustup-builds"
77
}
88

9-
module "aws_iam_user" {
10-
source = "../gha-iam-user"
9+
module "ci_role" {
10+
source = "../gha-oidc-role"
1111
org = "rust-lang"
1212
repo = "rustup"
13+
branch = "master"
1314
}
1415

15-
data "aws_iam_policy_document" "upload_builds" {
16-
statement {
17-
sid = "WriteToRustupBuilds"
18-
effect = "Allow"
19-
20-
actions = [
21-
"s3:PutObject",
16+
resource "aws_iam_policy" "upload_builds" {
17+
name = "upload-rustup-builds"
18+
policy = jsonencode({
19+
Version = "2012-10-17"
20+
Statement = [
21+
{
22+
Sid = "WriteToRustupBuilds"
23+
Effect = "Allow"
24+
Action = ["s3:PutObject"]
25+
Resource = ["${aws_s3_bucket.builds.arn}/*"]
26+
}
2227
]
23-
24-
resources = ["${aws_s3_bucket.builds.arn}/*"]
25-
}
26-
}
27-
28-
resource "aws_iam_user_policy" "upload_builds" {
29-
name = "upload-rustup-builds"
30-
user = module.aws_iam_user.user_name
31-
policy = data.aws_iam_policy_document.upload_builds.json
32-
}
33-
34-
data "aws_iam_policy_document" "legacy_ci" {
35-
statement {
36-
sid = "WriteToDevStatic"
37-
effect = "Allow"
38-
39-
actions = [
40-
"s3:*",
41-
]
42-
43-
resources = ["arn:aws:s3:::dev-static-rust-lang-org/rustup/*"]
44-
}
28+
})
4529
}
4630

47-
resource "aws_iam_user_policy" "legacy_ci" {
48-
name = "legacy-ci"
49-
user = module.aws_iam_user.user_name
50-
policy = data.aws_iam_policy_document.legacy_ci.json
31+
resource "aws_iam_role_policy_attachment" "ci_upload_builds" {
32+
role = module.ci_role.role.id
33+
policy_arn = aws_iam_policy.upload_builds.arn
5134
}

0 commit comments

Comments
 (0)