Fix: add electrs health check to prevent CI race conditions

- Add health check for electrs service in docker-compose.yml
- Add explicit wait for service health in CI workflow
- Prevent connection errors in tests due to services not being fully initialized

This addresses intermittent CI failures where Python tests would fail with
"Connection reset by peer" errors because they ran before electrs was fully
ready to accept requests.

Author: alexanderwiederin

.github/workflows/kotlin.yml

@@ -45,6 +45,12 @@ jobs:
       - name: Start bitcoind and electrs
         run: docker compose up -d
 
+      - name: Wait for services to be fully initialized
+        run: |
+          echo "Waiting for services to be ready..."
+          timeout 90 bash -c 'until docker compose ps | grep -q "bitcoin.*healthy" && docker compose ps | grep -q "electrs.*healthy"; do sleep 5; done'
+          echo "All services are healthy"
+
       - name: Run ldk-node-jvm tests
         run: |
           cd $LDK_NODE_JVM_DIR

.github/workflows/python.yml

@@ -24,6 +24,12 @@ jobs:
       - name: Start bitcoind and electrs
         run: docker compose up -d
 
+      - name: Wait for services to be fully initialized
+        run: |
+          echo "Waiting for services to be ready..."
+          timeout 90 bash -c 'until docker compose ps | grep -q "bitcoin.*healthy" && docker compose ps | grep -q "electrs.*healthy"; do sleep 5; done'
+          echo "All services are healthy"
+
       - name: Install testing prerequisites
         run: |
           pip3 install requests

docker-compose.yml

@@ -47,6 +47,11 @@ services:
       - "3002:3002"
     networks:
       - bitcoin-electrs
+    healthcheck:
+      test: [ "CMD", "curl", "-f", "http://localhost:3002/blocks/tip/hash" ]
+      interval: 5s
+      timeout: 10s
+      retries: 5
 
 networks:
   bitcoin-electrs: