Skip to content

Commit 2bb9d82

Browse files
NRL-1595 Workflow didn't have permission to pull CA certs for dev-sandbox env. Retrying with just server certs instead.
1 parent 3c0c778 commit 2bb9d82

File tree

3 files changed

+5
-7
lines changed

3 files changed

+5
-7
lines changed

.github/workflows/deploy-account-wide-infra.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ jobs:
6262
env:
6363
ACCOUNT_NAME: ${{ vars.ACCOUNT_NAME }}
6464
run: |
65-
make truststore-pull-all-for-account ACCOUNT=${ACCOUNT_NAME}
65+
make truststore-pull-server-for-account ACCOUNT=${ACCOUNT_NAME}
6666
6767
- name: Terraform Init
6868
env:
@@ -124,7 +124,7 @@ jobs:
124124
env:
125125
ACCOUNT_NAME: ${{ vars.ACCOUNT_NAME }}
126126
run: |
127-
make truststore-pull-all-for-account ACCOUNT=${ACCOUNT_NAME}
127+
make truststore-pull-server-for-account ACCOUNT=${ACCOUNT_NAME}
128128
129129
- name: Terraform Init
130130
env:

Makefile

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -202,8 +202,8 @@ truststore-build-ca: check-warn ## Build a CA (Certificate Authority)
202202
truststore-build-cert: check-warn ## Build a certificate
203203
@./scripts/truststore.sh build-cert "$(CA_NAME)" "$(CERT_NAME)" "$(CERT_SUBJECT)"
204204

205-
truststore-pull-all-for-account: check-warn ## Pull all certificates for each environment in a given account
206-
@./scripts/truststore.sh pull-all-for-account "$(ACCOUNT)"
205+
truststore-pull-server-for-account: check-warn ## Pull all certificates for each environment in a given account
206+
@./scripts/truststore.sh pull-server-for-account "$(ACCOUNT)"
207207

208208
truststore-pull-all: check-warn ## Pull all certificates
209209
@./scripts/truststore.sh pull-all "$(ENV)"

scripts/truststore.sh

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -312,7 +312,7 @@ function _truststore_pull_all() {
312312
return 0
313313
}
314314

315-
function _truststore_pull_all_for_account() {
315+
function _truststore_pull_server_for_account() {
316316
account=$1
317317

318318
# sets envs_array
@@ -322,8 +322,6 @@ function _truststore_pull_all_for_account() {
322322

323323
for env in ${envs_array[@]}; do
324324
echo "⏳ Pulling ${env} truststore certs"
325-
_truststore_pull_ca $env
326-
_truststore_pull_client $env
327325
_truststore_pull_server $env
328326
done
329327

0 commit comments

Comments
 (0)