darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2026-03-21 18:50:19 -04:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #1432 from alphagov/cf-deploy-sleep

Browse Source 
Add sleep after stopping the old app instances on deploy
This commit is contained in:
Leo Hemsted
2017-11-27 14:15:09 +00:00
committed by GitHub
parent ecde8b6575 bcee95214e
commit 2ddf05a645
1 changed files with 4 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
Makefile
View File

@@ -277,6 +277,10 @@ cf-deploy: ## Deploys the app to Cloud Foundry
cf rename ${CF_APP} ${CF_APP}-rollback
cf push ${CF_APP} -f ${CF_MANIFEST_FILE}
cf scale -i $$(cf curl /v2/apps/$$(cf app --guid ${CF_APP}-rollback) | jq -r ".entity.instances" 2>/dev/null || echo "1") ${CF_APP}
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.
sleep 10
cf delete -f ${CF_APP}-rollback
.PHONY: cf-deploy-api-db-migration