Skip to content

Commit 7b82541

Browse files
Rohooliojameslinnell
authored andcommitted
feature/PI-560-fix_proxygen_urls Reversed the check
1 parent 73de748 commit 7b82541

File tree

4 files changed

+12
-16
lines changed

4 files changed

+12
-16
lines changed

.github/workflows/_deploy.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -131,7 +131,7 @@ jobs:
131131
- uses: ./.github/actions/make/
132132
if: ${{ env.SCOPE == 'per_workspace'}}
133133
with:
134-
command: apigee--deploy
134+
command: apigee--deploy PERSISTENT_ENVIRONMENT_BUILD="true"
135135
requires-aws: true
136136
restore-from-cache: "true"
137137
cache-suffix: ${{ env.CACHE_NAME }}

scripts/infrastructure/apigee.mk

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ apigee--delete: aws--login
2121
AWS_ACCESS_KEY_ID=$(AWS_ACCESS_KEY_ID) \
2222
AWS_SECRET_ACCESS_KEY=$(AWS_SECRET_ACCESS_KEY) \
2323
AWS_SESSION_TOKEN=$(AWS_SESSION_TOKEN) \
24-
bash $(PATH_TO_INFRASTRUCTURE)/apigee/proxygen.sh delete_proxy $(DEV_BUILD)
24+
bash $(PATH_TO_INFRASTRUCTURE)/apigee/proxygen.sh delete_proxy $(PERSISTENT_ENVIRONMENT_BUILD)
2525

2626
apigee--clean:
2727
[[ -f $(PROXYGEN_TIMESTAMP) ]] && rm $(PROXYGEN_TIMESTAMP) || :
@@ -34,7 +34,7 @@ apigee--attach-product: aws--login
3434
AWS_ACCESS_KEY_ID=$(AWS_ACCESS_KEY_ID) \
3535
AWS_SECRET_ACCESS_KEY=$(AWS_SECRET_ACCESS_KEY) \
3636
AWS_SESSION_TOKEN=$(AWS_SESSION_TOKEN) \
37-
bash $(PATH_TO_INFRASTRUCTURE)/apigee/apigee.sh attach_product $(DEV_BUILD)
37+
bash $(PATH_TO_INFRASTRUCTURE)/apigee/apigee.sh attach_product $(PERSISTENT_ENVIRONMENT_BUILD)
3838

3939
apigee--detach-product: aws--login
4040
WORKSPACE_OUTPUT_JSON=$(WORKSPACE_OUTPUT_JSON) \
@@ -44,7 +44,7 @@ apigee--detach-product: aws--login
4444
AWS_ACCESS_KEY_ID=$(AWS_ACCESS_KEY_ID) \
4545
AWS_SECRET_ACCESS_KEY=$(AWS_SECRET_ACCESS_KEY) \
4646
AWS_SESSION_TOKEN=$(AWS_SESSION_TOKEN) \
47-
bash $(PATH_TO_INFRASTRUCTURE)/apigee/apigee.sh detach_product $(DEV_BUILD)
47+
bash $(PATH_TO_INFRASTRUCTURE)/apigee/apigee.sh detach_product $(PERSISTENT_ENVIRONMENT_BUILD)
4848

4949
$(PROXYGEN_TIMESTAMP): aws--login $(SWAGGER_APIGEE) $(WORKSPACE_OUTPUT_JSON)
5050
[[ -f $(PROXYGEN_TIMESTAMP) ]] && rm $(PROXYGEN_TIMESTAMP) || :
@@ -57,6 +57,6 @@ $(PROXYGEN_TIMESTAMP): aws--login $(SWAGGER_APIGEE) $(WORKSPACE_OUTPUT_JSON)
5757
AWS_ACCESS_KEY_ID=$(AWS_ACCESS_KEY_ID) \
5858
AWS_SECRET_ACCESS_KEY=$(AWS_SECRET_ACCESS_KEY) \
5959
AWS_SESSION_TOKEN=$(AWS_SESSION_TOKEN) \
60-
bash $(PATH_TO_INFRASTRUCTURE)/apigee/proxygen.sh generate_proxy $(DEV_BUILD)
60+
bash $(PATH_TO_INFRASTRUCTURE)/apigee/proxygen.sh generate_proxy $(PERSISTENT_ENVIRONMENT_BUILD)
6161

6262
touch $(PROXYGEN_TIMESTAMP)

scripts/infrastructure/apigee/apigee.sh

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ source ./scripts/infrastructure/terraform/terraform-constants.sh
55

66
PATH_TO_HERE="scripts/infrastructure/apigee"
77
APIGEE_DEPLOYMENT_ROLE="NHSDeploymentRole"
8-
DEV_BUILD="${2:-true}"
8+
PERSISTENT_ENVIRONMENT_BUILD="${2:-false}"
99
API_NAME="connecting-party-manager"
1010

1111

@@ -93,7 +93,7 @@ function attach_product(){
9393
# Currently hardcoded to CPM PTL id for PR running purposes, could be passed in for adjusting other apps in the future
9494
_app_id="9d28b416-311b-4523-bda9-686baa2fc437"
9595

96-
if [ "$DEV_BUILD" = "true" ]; then
96+
if [ "$PERSISTENT_ENVIRONMENT_BUILD" = "false" ]; then
9797
API_NAME=$_workspace_name
9898
fi
9999

@@ -179,7 +179,7 @@ function detach_product(){
179179
# Currently hardcoded to CPM PTL id for PR running purposes, could be passed in for adjusting other apps in the future
180180
_app_id="9d28b416-311b-4523-bda9-686baa2fc437"
181181

182-
if [ "$DEV_BUILD" = "true" ]; then
182+
if [ "$PERSISTENT_ENVIRONMENT_BUILD" = "false" ]; then
183183
API_NAME=$_workspace_name
184184
fi
185185

scripts/infrastructure/apigee/proxygen.sh

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,8 @@ set -e
33
PATH_TO_HERE="scripts/infrastructure/apigee"
44
APIGEE_DEPLOYMENT_ROLE="NHSDeploymentRole"
55
API_NAME="connecting-party-manager"
6-
DEV_BUILD="${2:-true}"
7-
echo "DEV_BUILD is: $DEV_BUILD"
6+
PERSISTENT_ENVIRONMENT_BUILD="${2:-false}"
7+
echo "PERSISTENT_ENVIRONMENT_BUILD is: $PERSISTENT_ENVIRONMENT_BUILD"
88

99

1010
if [[ -z ${WORKSPACE_OUTPUT_JSON} ]]; then
@@ -111,14 +111,10 @@ function generate_proxy(){
111111
poetry run python ${PATH_TO_HERE}/download_pem.py ${_apigee_stage} ${APIGEE_DEPLOYMENT_ROLE}
112112
fi
113113

114-
echo "API_NAME before is: $API_NAME"
115-
116-
if [ "$DEV_BUILD" = "true" ]; then
114+
if [ "$PERSISTENT_ENVIRONMENT_BUILD" = "false" ]; then
117115
API_NAME=$_workspace_name
118116
fi
119117

120-
echo "API_NAME is: $API_NAME"
121-
122118
DOT_PROXYGEN=${APIGEE_CONFIG_PATH}/${_apigee_stage} \
123119
poetry run \
124120
python ${PATH_TO_HERE}/proxygen.py instance deploy \
@@ -165,7 +161,7 @@ function delete_proxy(){
165161
_workspace_name="sandbox"
166162
fi
167163

168-
if [ "$DEV_BUILD" = "true" ]; then
164+
if [ "$PERSISTENT_ENVIRONMENT_BUILD" = "false" ]; then
169165
API_NAME=$_workspace_name
170166
fi
171167

0 commit comments

Comments
 (0)