From d002266d51592de09fb91ebcc190a70558679144 Mon Sep 17 00:00:00 2001 From: Alan Cruz Date: Fri, 24 Jul 2026 10:01:28 -0400 Subject: [PATCH] ci(subscriptions): point channels workflow testcontainers at remote Docker host The self-hosted Linux runners no longer expose /var/run/docker.sock, so every testcontainer-based test in the channels workflow's unit-tests matrix panicked with SocketNotFoundError, and the email integration tests silently skipped via their docker_available() guard (#389). Add the DOCKER_HOST/DOCKER_HOST_IP env block that subscriptions-smoke.yml already uses, and run the unit-test step with --no-fail-fast so one failing test binary no longer hides the rest. Tests: validated by dispatching the workflow from feat/cluster-capable-state with this exact change (run 30099280985): all 112 jobs green, and the Email jobs' mailpit tests genuinely ran (5 passed in 1.94s vs the prior 0.01s silent skip). --- .github/workflows/subscriptions-channels.yml | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/.github/workflows/subscriptions-channels.yml b/.github/workflows/subscriptions-channels.yml index 198e80aa4..a26165e87 100644 --- a/.github/workflows/subscriptions-channels.yml +++ b/.github/workflows/subscriptions-channels.yml @@ -14,6 +14,10 @@ env: CARGO_TERM_COLOR: always CARGO_BUILD_JOBS: 2 CARGO_PROFILE_DEV_DEBUG: 0 + # Self-hosted Linux runners have no local /var/run/docker.sock; testcontainers + # reach Docker on a remote host, same as subscriptions-smoke.yml (issue #389). + DOCKER_HOST: ${{ secrets.DOCKER_HOST }} + DOCKER_HOST_IP: ${{ secrets.DOCKER_HOST_IP }} jobs: matrix-setup: @@ -89,7 +93,7 @@ jobs: # so dropping defaults breaks builds; this matrix exercises adding # additional versions to the R4 baseline. run: | - cargo test \ + cargo test --no-fail-fast \ --features "${{ matrix.features }}" \ -p helios-subscriptions -p helios-rest -p helios-persistence -p helios-fhirpath