Skip to content

Commit

Permalink
Make policyReader binding additive in bootstrap (#2470)
Browse files Browse the repository at this point in the history
  • Loading branch information
juliocc authored Aug 6, 2024
1 parent b3efa95 commit 89333a5
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 5 deletions.
2 changes: 1 addition & 1 deletion fast/stages/0-bootstrap/organization-iam.tf
Original file line number Diff line number Diff line change
Expand Up @@ -156,7 +156,6 @@ locals {
}
(module.automation-tf-resman-r-sa.iam_email) = {
authoritative = [
"roles/accesscontextmanager.policyReader",
"roles/essentialcontacts.viewer",
"roles/logging.viewer",
"roles/resourcemanager.folderViewer",
Expand All @@ -165,6 +164,7 @@ locals {
]
additive = concat(
[
"roles/accesscontextmanager.policyReader",
# the organizationAdminViewer custom role is granted via the SA module
"roles/orgpolicy.policyViewer"
],
Expand Down
4 changes: 2 additions & 2 deletions tests/fast/stages/s0_bootstrap/checklist.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -379,9 +379,9 @@ counts:
google_logging_organization_sink: 4
google_logging_project_bucket_config: 4
google_org_policy_policy: 22
google_organization_iam_binding: 29
google_organization_iam_binding: 28
google_organization_iam_custom_role: 9
google_organization_iam_member: 41
google_organization_iam_member: 42
google_project: 3
google_project_iam_audit_config: 1
google_project_iam_binding: 19
Expand Down
4 changes: 2 additions & 2 deletions tests/fast/stages/s0_bootstrap/simple.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -20,9 +20,9 @@ counts:
google_logging_organization_sink: 4
google_logging_project_bucket_config: 4
google_org_policy_policy: 22
google_organization_iam_binding: 29
google_organization_iam_binding: 28
google_organization_iam_custom_role: 9
google_organization_iam_member: 28
google_organization_iam_member: 29
google_project: 3
google_project_iam_audit_config: 1
google_project_iam_binding: 19
Expand Down

0 comments on commit 89333a5

Please sign in to comment.