Skip to content

Commit c379277

Browse files
authored
chore: use less privileged roles for deploying bigquery module (#376)
1 parent ab9ed8c commit c379277

File tree

6 files changed

+78
-12
lines changed

6 files changed

+78
-12
lines changed

metadata.yaml

+13-2
Original file line numberDiff line numberDiff line change
@@ -382,8 +382,19 @@ spec:
382382
- level: Project
383383
roles:
384384
- roles/bigquery.admin
385-
- roles/cloudkms.cryptoKeyEncrypterDecrypter
386-
- roles/owner
385+
- roles/aiplatform.admin
386+
- roles/cloudfunctions.admin
387+
- roles/dataform.admin
388+
- roles/datalineage.viewer
389+
- roles/iam.serviceAccountAdmin
390+
- roles/iam.serviceAccountTokenCreator
391+
- roles/iam.serviceAccountUser
392+
- roles/logging.configWriter
393+
- roles/resourcemanager.projectIamAdmin
394+
- roles/run.invoker
395+
- roles/serviceusage.serviceUsageAdmin
396+
- roles/storage.admin
397+
- roles/workflows.admin
387398
services:
388399
- cloudkms.googleapis.com
389400
- cloudresourcemanager.googleapis.com

modules/authorization/metadata.yaml

+13-2
Original file line numberDiff line numberDiff line change
@@ -94,8 +94,19 @@ spec:
9494
- level: Project
9595
roles:
9696
- roles/bigquery.admin
97-
- roles/cloudkms.cryptoKeyEncrypterDecrypter
98-
- roles/owner
97+
- roles/aiplatform.admin
98+
- roles/cloudfunctions.admin
99+
- roles/dataform.admin
100+
- roles/datalineage.viewer
101+
- roles/iam.serviceAccountAdmin
102+
- roles/iam.serviceAccountTokenCreator
103+
- roles/iam.serviceAccountUser
104+
- roles/logging.configWriter
105+
- roles/resourcemanager.projectIamAdmin
106+
- roles/run.invoker
107+
- roles/serviceusage.serviceUsageAdmin
108+
- roles/storage.admin
109+
- roles/workflows.admin
99110
services:
100111
- cloudkms.googleapis.com
101112
- cloudresourcemanager.googleapis.com

modules/data_warehouse/metadata.yaml

+13-2
Original file line numberDiff line numberDiff line change
@@ -136,8 +136,19 @@ spec:
136136
- level: Project
137137
roles:
138138
- roles/bigquery.admin
139-
- roles/cloudkms.cryptoKeyEncrypterDecrypter
140-
- roles/owner
139+
- roles/aiplatform.admin
140+
- roles/cloudfunctions.admin
141+
- roles/dataform.admin
142+
- roles/datalineage.viewer
143+
- roles/iam.serviceAccountAdmin
144+
- roles/iam.serviceAccountTokenCreator
145+
- roles/iam.serviceAccountUser
146+
- roles/logging.configWriter
147+
- roles/resourcemanager.projectIamAdmin
148+
- roles/run.invoker
149+
- roles/serviceusage.serviceUsageAdmin
150+
- roles/storage.admin
151+
- roles/workflows.admin
141152
services:
142153
- cloudkms.googleapis.com
143154
- cloudresourcemanager.googleapis.com

modules/scheduled_queries/metadata.yaml

+13-2
Original file line numberDiff line numberDiff line change
@@ -60,8 +60,19 @@ spec:
6060
- level: Project
6161
roles:
6262
- roles/bigquery.admin
63-
- roles/cloudkms.cryptoKeyEncrypterDecrypter
64-
- roles/owner
63+
- roles/aiplatform.admin
64+
- roles/cloudfunctions.admin
65+
- roles/dataform.admin
66+
- roles/datalineage.viewer
67+
- roles/iam.serviceAccountAdmin
68+
- roles/iam.serviceAccountTokenCreator
69+
- roles/iam.serviceAccountUser
70+
- roles/logging.configWriter
71+
- roles/resourcemanager.projectIamAdmin
72+
- roles/run.invoker
73+
- roles/serviceusage.serviceUsageAdmin
74+
- roles/storage.admin
75+
- roles/workflows.admin
6576
services:
6677
- cloudkms.googleapis.com
6778
- cloudresourcemanager.googleapis.com

modules/udf/metadata.yaml

+13-2
Original file line numberDiff line numberDiff line change
@@ -64,8 +64,19 @@ spec:
6464
- level: Project
6565
roles:
6666
- roles/bigquery.admin
67-
- roles/cloudkms.cryptoKeyEncrypterDecrypter
68-
- roles/owner
67+
- roles/aiplatform.admin
68+
- roles/cloudfunctions.admin
69+
- roles/dataform.admin
70+
- roles/datalineage.viewer
71+
- roles/iam.serviceAccountAdmin
72+
- roles/iam.serviceAccountTokenCreator
73+
- roles/iam.serviceAccountUser
74+
- roles/logging.configWriter
75+
- roles/resourcemanager.projectIamAdmin
76+
- roles/run.invoker
77+
- roles/serviceusage.serviceUsageAdmin
78+
- roles/storage.admin
79+
- roles/workflows.admin
6980
services:
7081
- cloudkms.googleapis.com
7182
- cloudresourcemanager.googleapis.com

test/setup/iam.tf

+13-2
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,19 @@
1717
locals {
1818
int_required_roles = [
1919
"roles/bigquery.admin",
20-
"roles/cloudkms.cryptoKeyEncrypterDecrypter",
21-
"roles/owner" // TODO: Descope
20+
"roles/aiplatform.admin",
21+
"roles/cloudfunctions.admin",
22+
"roles/dataform.admin",
23+
"roles/datalineage.viewer",
24+
"roles/iam.serviceAccountAdmin",
25+
"roles/iam.serviceAccountTokenCreator",
26+
"roles/iam.serviceAccountUser",
27+
"roles/logging.configWriter",
28+
"roles/resourcemanager.projectIamAdmin",
29+
"roles/run.invoker",
30+
"roles/serviceusage.serviceUsageAdmin",
31+
"roles/storage.admin",
32+
"roles/workflows.admin"
2233
]
2334
}
2435

0 commit comments

Comments
 (0)