Merge c5e6aca into 97a86b5

Pipfile

@@ -17,6 +17,7 @@ whitenoise = "*"
 [dev-packages]
 "flake8" = "*"
 coverage = "*"
+ipython = "*"
 
 [requires]
 python_version = "3.6"

bothub/health/checks.py

@@ -1,23 +1,38 @@
+import logging
+
+from rest_framework import status
+
+
+logger = logging.getLogger('bothub.health.checks')
+
+CHECK_ACCESSIBLE_API_URL = '/api/repositories/'
+
+
 def check_database_connection(**kwargs):
     from django.db import connections
     from django.db.utils import OperationalError
-    db_conn = connections['default']
-    if not db_conn:
-        return False
-    try:
-        db_conn.cursor()
-        return True
-    except OperationalError as e:
+    if len(connections.all()) is 0:
         return False
+    logger.info('found {} database connection'.format(len(connections.all())))
+    for i, conn in enumerate(connections.all(), 1):
+        db_conn = connections['default']
+        try:
+            db_conn.cursor()
+            logger.info('#{} db connection OKAY'.format(i))
+        except OperationalError as e:
+            logger.warning('#{} db connection ERROR'.format(i))
+            return False
+    return True
 
 
 def check_accessible_api(request, **kwargs):
-    import requests
-    HTTP_HOST = request.META.get('HTTP_HOST')
-    repositories_url = 'http://{}/api/repositories/'.format(HTTP_HOST)
-    request = requests.get(repositories_url)
-    try:
-        request.raise_for_status()
+    from django.test import Client
+    logger.info('making request to {}'.format(CHECK_ACCESSIBLE_API_URL))
+    client = Client()
+    response = client.get(CHECK_ACCESSIBLE_API_URL)
+    logger.info('{} status code: {}'.format(
+        CHECK_ACCESSIBLE_API_URL,
+        response.status_code))
+    if response.status_code is status.HTTP_200_OK:
         return True
-    except requests.HTTPError as e:
-        return False
+    return False

bothub/settings.py

@@ -2,6 +2,7 @@
 import dj_database_url
 
 from decouple import config
+from django.utils.log import DEFAULT_LOGGING
 
 
 # Build paths inside the project like this: os.path.join(BASE_DIR, ...)
@@ -204,3 +205,21 @@
     'CSRF_COOKIE_SECURE',
     default=False,
     cast=bool)
+
+
+# Logging
+
+LOGGING = DEFAULT_LOGGING
+LOGGING['formatters']['bothub.health'] = {
+    'format': '[bothub.health] {message}',
+    'style': '{',
+}
+LOGGING['handlers']['bothub.health'] = {
+    'level': 'DEBUG',
+    'class': 'logging.StreamHandler',
+    'formatter': 'bothub.health',
+}
+LOGGING['loggers']['bothub.health.checks'] = {
+    'handlers': ['bothub.health'],
+    'level': 'DEBUG',
+}

docker/docker-compose.yml

@@ -9,13 +9,13 @@ services:
     ports:
       - 80:80
     environment:
-      - SECRET_KEY
+      - SECRET_KEY=SK
       - DEBUG
       - ALLOWED_HOSTS
       - DEFAULT_DATABASE
       - LANGUAGE_CODE
       - TIME_ZONE
-      - EMAIL_HOST
+      - EMAIL_HOST=
       - EMAIL_PORT
       - DEFAULT_FROM_EMAIL
       - SERVER_EMAIL
@@ -26,8 +26,8 @@ services:
       - ADMINS
       - CSRF_COOKIE_DOMAIN
       - CSRF_COOKIE_SECURE
-      - BOTHUB_WEBAPP_BASE_URL
-      - BOTHUB_NLP_BASE_URL
+      - BOTHUB_WEBAPP_BASE_URL=http://localhost/
+      - BOTHUB_NLP_BASE_URL=http://localhost:2657/
       - WEBAPP_REPO=https://github.com/Ilhasoft/bothub-webapp
       - WEBAPP_BRANCH=master
-      - API_BASE_URL
+      - API_BASE_URL=/api

docker/entrypoint.sh

@@ -6,7 +6,8 @@ cd $WORKDIR
 python manage.py migrate
 python manage.py collectstatic --noinput
 
-gunicorn -e DJANGO_SETTINGS_MODULE=bothub.settings -b unix:/tmp/bothub.sock -D bothub.wsgi
+gunicorn -e DJANGO_SETTINGS_MODULE=bothub.settings -b unix:/tmp/bothub.sock -D bothub.wsgi --log-file /var/log/gunicorn.log --log-level debug --capture-output
 
 mkdir -p /run/nginx/
-nginx -g "daemon off;"
+nginx
+tail -f /var/log/nginx/* /var/log/gunicorn.log