Update service URL and timeout in health_check script

Changed service URL to use container name and reduced timeout. Added job to dump Docker Compose service logs.
Cdaprod · Mar 4, 2024 · 7bfaab6 · 7bfaab6
1 parent 882e71c
commit 7bfaab6
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/.github/scripts/health_check.py b/.github/scripts/health_check.py
@@ -2,9 +2,9 @@
 import sys
 import time
 
-service_url = "http://localhost:8000"
+service_url = "http://python-app:8000"
 
-timeout = time.time() + 60*1  # 1 minute from now
+timeout = time.time() + 30*1
 while True:
     try:
         response = requests.get(service_url)

diff --git a/.github/workflows/docker-compose-log-build-test.yml b/.github/workflows/docker-compose-log-build-test.yml
@@ -12,6 +12,9 @@ jobs:
     - name: Start services
       run: docker-compose -f deploy/docker-compose.yaml up --build -d
 
+    - name: Dump Docker Compose Service Logs
+      run: docker-compose -f deploy/docker-compose.yaml logs
+
     - name: Check for python-app readiness
       run: python .github/scripts/health_check.py