darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2026-09-10 01:56:03 -04:00
Code Issues Packages Projects Releases Wiki Activity

BAU: Use port health checks for API

Browse Source 
- We've been seeing an issue when traffic spikes of the http health
  checks taking over 1s and PaaS killing the app
- Port health checks won't care about being stuck in a queue so should
  continue to work even at high loads
- We have functional tests to catch if a deployment brings up the app
  (and so passes port health check) but then doesn't work
This commit is contained in:
Andy Paine
2019-05-30 11:56:19 +01:00
parent 82e27c4064
commit adf81ef689
1 changed files with 2 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
manifest.yml.j2
View File

@@ -4,7 +4,7 @@
'staging': ['api.staging-notify.works'], 'staging': ['api.staging-notify.works'],
'production': ['api.notifications.service.gov.uk'], 'production': ['api.notifications.service.gov.uk'],
}, },
'healthcheck-endpoint': '/_status?simple=true', 'health-check-type': 'port',
}, },
'notify-api-db-migration': {'NOTIFY_APP_NAME': 'api', 'instances': 0}, 'notify-api-db-migration': {'NOTIFY_APP_NAME': 'api', 'instances': 0},
@@ -39,12 +39,7 @@ applications:
- route: {{ route }} - route: {{ route }}
{%- endfor%} {%- endfor%}
- route: {{ CF_APP }}-{{ environment }}.cloudapps.digital - route: {{ CF_APP }}-{{ environment }}.cloudapps.digital
{% if 'healthcheck-endpoint' in app %} health-check-type: {{ app.get('health-check-type', 'process') }}
health-check-type: http
health-check-http-endpoint: {{ app['healthcheck-endpoint'] }}
{% else %}
health-check-type: process
{% endif %}
services: services:
- notify-db - notify-db