Skip to content

ci(subscriptions): point channels workflow testcontainers at remote Docker host - #392

Merged
smunini merged 1 commit into
mainfrom
ci/389-channels-docker-host
Jul 27, 2026
Merged

ci(subscriptions): point channels workflow testcontainers at remote Docker host#392
smunini merged 1 commit into
mainfrom
ci/389-channels-docker-host

Conversation

@aacruzgon

Copy link
Copy Markdown
Contributor

Summary

Fixes #389. The subscriptions-channels.yml workflow fails on the current [self-hosted, Linux] runner pool: the runners no longer expose a local /var/run/docker.sock, so every testcontainer-based test in the unit-tests matrix panics with SocketNotFoundError — and the email integration tests silently skip via their docker_available() guard, reporting green without testing anything. subscriptions-smoke.yml already adapted by pointing testcontainers at a remote Docker host via DOCKER_HOST; the channels workflow never got that env block.

Changes

Testing

  • Validated by dispatching the workflow with this exact change from feat/cluster-capable-state (run 30099280985): all 112 jobs green.
  • The Email matrix jobs' mailpit tests genuinely ran — 5 passed in 1.94s, versus the prior 0.01s silent skip — because docker_available() also honors DOCKER_HOST, so this change simultaneously un-breaks the unit tests and reactivates real email delivery coverage.

Notes

…ocker 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).
@codecov

codecov Bot commented Jul 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@smunini
smunini merged commit 4d732dd into main Jul 27, 2026
20 checks passed
@smunini
smunini deleted the ci/389-channels-docker-host branch July 27, 2026 19:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

subscriptions-channels.yml fails on self-hosted Linux runners: no docker.sock (needs DOCKER_HOST env like smoke workflow)

2 participants