Skip to content

Commit c05e02f

Browse files
committed
update IAM policies on production resources
1 parent 359db16 commit c05e02f

File tree

1 file changed

+10
-8
lines changed

1 file changed

+10
-8
lines changed

broker/setup_broker/lvk/setup_broker.sh

Lines changed: 10 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -58,8 +58,8 @@ ps_topic_alerts=$(define_GCP_resources "${survey}-alerts")
5858
ps_subscription_reservoir=$(define_GCP_resources "${survey}-alerts-reservoir")
5959
# topics and subscriptions involved in writing alert data to BigQuery
6060
ps_bigquery_subscription=$(define_GCP_resources "${survey}-bigquery-import-${versiontag}")
61-
ps_deadletter_topic=$(define_GCP_resources "${survey}-deadletter")
62-
ps_deadletter_subscription="${ps_deadletter_topic}"
61+
ps_deadletter_subscription=$(define_GCP_resources "${survey}-deadletter")
62+
ps_deadletter_topic="${ps_deadletter_subscription}"
6363

6464
# function used to create (or delete) GCP resources
6565
manage_resources() {
@@ -126,15 +126,17 @@ manage_resources() {
126126
--message-transforms-file=templates/ps_lvk_add_top_level_fields_smt.yaml
127127

128128
# set IAM policies on resources
129-
user="allUsers"
130-
roleid="roles/pubsub.subscriber"
131-
gcloud pubsub topics add-iam-policy-binding "${ps_topic_alerts}" --member="${user}" --role="${roleid}"
132-
gcloud pubsub topics add-iam-policy-binding "${ps_deadletter_topic}" \
129+
if [ "$testid" = "False" ]; then
130+
user="allUsers"
131+
roleid="roles/pubsub.subscriber"
132+
gcloud pubsub topics add-iam-policy-binding "${ps_topic_alerts}" --member="${user}" --role="${roleid}"
133+
gcloud pubsub topics add-iam-policy-binding "${ps_deadletter_topic}" \
133134
--member="serviceAccount:${service_account}" \
134135
--role="roles/pubsub.publisher"
135-
gcloud pubsub subscriptions add-iam-policy-binding "${ps_bigquery_subscription}" \
136+
gcloud pubsub subscriptions add-iam-policy-binding "${ps_bigquery_subscription}" \
136137
--member="serviceAccount:${service_account}" \
137-
--role="${roleid}"
138+
--role="roles/pubsub.subscriber"
139+
fi
138140

139141
#--- Create Artifact Registry Repository
140142
echo

0 commit comments

Comments
 (0)