Skip to content

Commit b68eb21

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 b68eb21

File tree

3 files changed

+8
-9
lines changed

3 files changed

+8
-9
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: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@ function _truststore_help() {
2525
echo " pull-ca-key <ca> - Pull the certificate authority private key"
2626
echo " pull-client <env> - pull the files needed for a client connection"
2727
echo " pull-server <env> - pull the files needed for a server connection"
28+
echo " pull-server-for-account <acc> - pull the files needed for a server connection for all environments in a given account"
2829
echo " pull-all <env> - pull all the truststore files for an environment"
2930
echo " push-all <env> - push all the truststore files for an environment"
3031
echo " rotate-ca <env> - rotate the certificate authority, archiving the previous one"
@@ -312,18 +313,16 @@ function _truststore_pull_all() {
312313
return 0
313314
}
314315

315-
function _truststore_pull_all_for_account() {
316+
function _truststore_pull_server_for_account() {
316317
account=$1
317318

318319
# sets envs_array
319320
source ./scripts/get-envs-for-account.sh $account
320321

321-
echo "Pulling certs for environments ${envs_array[@]} in ${account} account"
322+
echo "Pulling server certs for environments ${envs_array[@]} in ${account} account"
322323

323324
for env in ${envs_array[@]}; do
324325
echo "⏳ Pulling ${env} truststore certs"
325-
_truststore_pull_ca $env
326-
_truststore_pull_client $env
327326
_truststore_pull_server $env
328327
done
329328

@@ -384,7 +383,7 @@ function _truststore() {
384383
"build-ca") _truststore_build_ca $args ;;
385384
"build-cert") _truststore_build_cert $args ;;
386385
"pull-all") _truststore_pull_all $args ;;
387-
"pull-all-for-account") _truststore_pull_all_for_account $args ;;
386+
"pull-server-for-account") _truststore_pull_server_for_account $args ;;
388387
"pull-server") _truststore_pull_server $args ;;
389388
"pull-client") _truststore_pull_client $args ;;
390389
"pull-ca") _truststore_pull_ca $args ;;

0 commit comments

Comments
 (0)