darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2026-01-15 23:21:41 -05:00
Code Issues Packages Projects Releases Wiki Activity

Add 5 second sleep before stopping old api instances

Browse Source 
This is to allow the new instances to start responding to the http
healthchecks and start serving traffic.
This commit is contained in:
Athanasios Voutsadakis
2018-04-11 15:01:45 +01:00
parent e2a05efae0
commit 23cfe73c48
1 changed files with 1 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
Makefile
View File

@@ -208,6 +208,7 @@ cf-deploy: ## Deploys the app to Cloud Foundry
cf rename ${CF_APP} ${CF_APP}-rollback
cf push ${CF_APP} -f <(make -s generate-manifest)
cf scale -i $$(cf curl /v2/apps/$$(cf app --guid ${CF_APP}-rollback) | jq -r ".entity.instances" 2>/dev/null || echo "1") ${CF_APP}
if [ "${CF_APP}" == "notify-api" ]; then sleep 5; fi
cf stop ${CF_APP}-rollback
# sleep for 10 seconds to try and make sure that all worker threads (either web api or celery) have finished before we delete
# when we delete the DB is unbound from the app, which can cause "permission denied for relation" psycopg2 errors.