Skip to content

Commit 0a81171

Browse files
fix roles and propagation
1 parent a04b4c9 commit 0a81171

File tree

2 files changed

+20
-9
lines changed

2 files changed

+20
-9
lines changed

4-projects/business_unit_1/shared/example_infra_pipeline.tf

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -65,19 +65,18 @@ resource "google_storage_bucket_iam_member" "cloudbuild_storage_read" {
6565
member = "serviceAccount:${module.app_infra_cloudbuild_project[0].sa}"
6666
}
6767

68-
resource "google_storage_bucket_iam_member" "bucket_object_admin" {
69-
bucket = "${local.cloudbuild_project_id}_cloudbuild"
70-
role = "roles/storage.objectAdmin"
71-
member = "serviceAccount:${module.app_infra_cloudbuild_project[0].sa}"
68+
resource "google_storage_bucket_iam_member" "cloudbuild_sa_storage_admin" {
69+
bucket = module.infra_pipelines[0].log_buckets["bu1-example-app"]
70+
role = "roles/storage.admin"
71+
member = "serviceAccount:tf-cb-builder-sa@${local.cloudbuild_project_id}.iam.gserviceaccount.com"
7272
}
7373

74-
resource "google_storage_bucket_iam_member" "bucket_object_viewer" {
74+
resource "google_storage_bucket_iam_member" "cloudbuild_bucket_admin" {
7575
bucket = "${local.cloudbuild_project_id}_cloudbuild"
76-
role = "roles/storage.objectViewer"
76+
role = "roles/storage.admin"
7777
member = "serviceAccount:${module.app_infra_cloudbuild_project[0].sa}"
7878
}
7979

80-
8180
module "app_infra_cloudbuild_project" {
8281
source = "../../modules/single_project"
8382
count = local.enable_cloudbuild_deploy ? 1 : 0
@@ -133,8 +132,8 @@ resource "time_sleep" "wait_iam_propagation" {
133132
google_storage_bucket_iam_member.cloudbuild_storage_read,
134133
google_artifact_registry_repository_iam_member.builder_on_artifact_registry,
135134
google_project_iam_member.cloudbuild_logging,
136-
google_storage_bucket_iam_member.bucket_object_viewer,
137-
google_storage_bucket_iam_member.bucket_object_admin,
135+
google_storage_bucket_iam_member.cloudbuild_sa_storage_admin,
136+
google_storage_bucket_iam_member.cloudbuild_bucket_admin,
138137
]
139138
}
140139

5-app-infra/modules/confidential_space/main.tf

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -134,9 +134,21 @@ module "confidential_compute_instance" {
134134
resource_manager_tags = local.resource_manager_tags
135135
}
136136

137+
resource "time_sleep" "wait_workload_pool_propagation" {
138+
create_duration = "60s"
139+
140+
depends_on = [
141+
google_iam_workload_identity_pool.confidential_space_pool
142+
]
143+
}
144+
137145
resource "google_service_account_iam_member" "workload_identity_binding" {
138146
service_account_id = "projects/${local.env_project_id}/serviceAccounts/${local.confidential_space_workload_sa}"
139147
role = "roles/iam.workloadIdentityUser"
140148
member = "principalSet://iam.googleapis.com/projects/${local.confidential_space_project_number}/locations/global/workloadIdentityPools/confidential-space-pool/*"
149+
150+
depends_on = [
151+
time_sleep.wait_workload_pool_propagation
152+
]
141153
}
142154

0 commit comments

Comments
 (0)