Fix SirixDB healthcheck to use HTTP instead of HTTPS

- Changed healthcheck from https://localhost:9443 to http://localhost:9443
- Tests use HTTP, not HTTPS, so healthcheck must match
- Improved test.sh with better error handling and container status output
- Added exit with error and logs if SirixDB fails to start
- This should fix the infinite waiting loop in CI

Author: Johannes Lichtenberger

.github/workflows/test.yml

@@ -30,7 +30,7 @@ jobs:
           echo "Checking Keycloak..."
           curl -f http://localhost:8080/realms/master || echo "Keycloak not ready"
           echo "Checking SirixDB..."
-          curl -k -f https://localhost:9443 || echo "SirixDB not ready"
+          curl -f http://localhost:9443 || echo "SirixDB not ready"
       - name: install dependencies
         run: |
           python -m pip install --upgrade pip

test.sh

@@ -1,7 +1,35 @@
-echo "starting the docker environment"
+#!/bin/bash
+set -e
+
+echo "Starting the docker environment..."
 docker compose -f ./tests/resources/docker-compose.yml up -d keycloak
-echo "waiting for keycloak to be ready..."
+
+echo "Waiting for Keycloak to be ready..."
 bash ./tests/resources/wait.sh
+echo "Keycloak is ready!"
+
+echo "Starting SirixDB server..."
 docker compose -f ./tests/resources/docker-compose.yml up -d server
-echo "waiting for sirix server to be ready..."
-bash -c 'for i in {1..30}; do if curl -k -s -f https://localhost:9443 > /dev/null 2>&1; then echo "SirixDB is ready"; break; fi; echo "Waiting for SirixDB..."; sleep 2; done'
+
+echo "Waiting for SirixDB server to be ready..."
+for i in {1..30}; do 
+    if curl -s -f http://localhost:9443 > /dev/null 2>&1; then 
+        echo "SirixDB is ready!"
+        exit 0
+    fi
+    
+    # Show container status every 5 iterations
+    if [ $((i % 5)) -eq 0 ]; then
+        echo "Still waiting... (attempt $i/30)"
+        docker ps --filter "name=server" --format "table {{.Names}}\t{{.Status}}"
+    else
+        echo "Waiting for SirixDB... (attempt $i/30)"
+    fi
+    
+    sleep 2
+done
+
+echo "ERROR: SirixDB failed to become ready after 60 seconds"
+echo "Container logs:"
+docker compose -f ./tests/resources/docker-compose.yml logs server
+exit 1