ci: integration test fixes

Author: evansims

.github/workflows/ci.yml

@@ -252,16 +252,18 @@ jobs:
         run: |
           docker compose up -d ledger
           echo "Waiting for Ledger to be ready..."
+          # Check from host side (port 50052 is mapped to container's 50051)
+          # Distroless container doesn't have nc/shell utilities
           for i in {1..30}; do
-            if docker compose exec -T ledger nc -z localhost 50051 2>/dev/null; then
+            if nc -z localhost 50052 2>/dev/null; then
               echo "Ledger is ready!"
               break
             fi
             echo "Attempt $i/30: Ledger not ready yet..."
             sleep 2
           done
           # Final check
-          docker compose exec -T ledger nc -z localhost 50051 || (docker compose logs ledger && exit 1)
+          nc -z localhost 50052 || (docker compose logs ledger && exit 1)
 
       - name: Run Ledger integration tests
         env:

docker/ledger-integration-tests/docker-compose.yml

@@ -29,12 +29,9 @@ services:
       RUST_LOG: "info,inferadb_ledger_server=debug"
     tmpfs:
       - /var/lib/ledger:uid=65532,gid=65532
-    healthcheck:
-      test: ["CMD-SHELL", "nc -z localhost 50051 || exit 1"]
-      interval: 2s
-      timeout: 5s
-      retries: 30
-      start_period: 10s
+    # Note: Healthcheck removed because distroless image has no shell/utilities.
+    # CI uses host-side port checking instead. For local development with
+    # docker-compose, the test-runner will retry connections automatically.
     deploy:
       resources:
         limits:
@@ -53,8 +50,7 @@ services:
     networks:
       - control-ledger-test-network
     depends_on:
-      ledger:
-        condition: service_healthy
+      - ledger
     environment:
       LEDGER_ENDPOINT: "http://ledger:50051"
       LEDGER_NAMESPACE_ID: "1"