From 9888ba3d4e7f941c06abca90a8e94f6336972f40 Mon Sep 17 00:00:00 2001 From: Athanasios Voutsadakis Date: Wed, 28 Mar 2018 17:19:02 +0100 Subject: [PATCH] Add http health-check for the api app Also rename the 'elb' argument to 'simple' to be more descriptive. The healthcheck is going to issue requests to `/_status?simple=true` just to check that gunicorn is up and running - we don't need to go all the way to the database, as this would have adverse impact when api is under load. --- app/status/healthcheck.py | 2 +- manifest-api-base.yml | 2 ++ 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/app/status/healthcheck.py b/app/status/healthcheck.py index e704e6b73..da305c9e2 100644 --- a/app/status/healthcheck.py +++ b/app/status/healthcheck.py @@ -11,7 +11,7 @@ status = Blueprint('status', __name__) @status.route('/_status', methods=['GET', 'POST']) def show_status(): - if request.args.get('elb', None): + if request.args.get('simple', None): return jsonify(status="ok"), 200 else: return jsonify( diff --git a/manifest-api-base.yml b/manifest-api-base.yml index e8197575f..bc043e351 100644 --- a/manifest-api-base.yml +++ b/manifest-api-base.yml @@ -51,6 +51,8 @@ memory: 1G applications: - name: notify-api + health-check-type: http + health-check-http-endpoint: /_status?simple=true - name: notify-api-db-migration command: sleep infinity