← Back to team overview

sts-sponsors team mailing list archive

[Merge] ~thorsten-merten/maas-site-manager:make-tox-rebuild-reqs-add-healthcheck into maas-site-manager:main

 

Thorsten Merten has proposed merging ~thorsten-merten/maas-site-manager:make-tox-rebuild-reqs-add-healthcheck into maas-site-manager:main.

Commit message:
chore: tox should rebuild dependencies on --build

also adds healthcheck



Requested reviews:
  MAAS Committers (maas-committers)

For more details, see:
https://code.launchpad.net/~thorsten-merten/maas-site-manager/+git/maas-site-manager/+merge/441106
-- 
Your team MAAS Committers is requested to review the proposed merge of ~thorsten-merten/maas-site-manager:make-tox-rebuild-reqs-add-healthcheck into maas-site-manager:main.
diff --git a/backend/Dockerfile b/backend/Dockerfile
index 85e6045..3968a15 100644
--- a/backend/Dockerfile
+++ b/backend/Dockerfile
@@ -19,4 +19,5 @@ COPY . /app/
 # Expose port
 EXPOSE 8000
 
-ENTRYPOINT ["python", "-m", "tox", "run", "-e", "run"]
+#ENTRYPOINT ["tail", "-f", "/dev/null"]  # to keep it running for debugging
+ENTRYPOINT ["python", "-m", "tox", "run", "-re", "run"]
diff --git a/docker-compose.yaml b/docker-compose.yaml
index b1dbec9..38e90ad 100644
--- a/docker-compose.yaml
+++ b/docker-compose.yaml
@@ -30,6 +30,11 @@ services:
       - "8000:8000"
     links:
       - "postgres:db"
+    healthcheck:
+      test: python3 -c "import urllib.request; repr(urllib.request.urlopen(urllib.request.Request('http://localhost:8000')).status)"
+      interval: 30s
+      timeout: 30s
+      retries: 5
 
 volumes:
   postgres-data:

Follow ups