ci: drop browser tests from CI pipeline #3

Merged

aureus merged 5 commits from feat/clean-ci into main

2026-05-25 14:43:50 +00:00

aureus commented

2026-05-25 14:31:21 +00:00

Owner

Workflow services cannot reach HOST-daemon Compose containers from
the job container — all bridging approaches require either a container
ID (unavailable on cgroup v2) or functional DNAT (not present in this
environment). Browser tests move to feat/browser-ci which redesigns
the approach using workflow services for Jellyfin and Seerr.

This branch makes CI green immediately: lint + unit + integration
tests still run; browser tests remain in-tree for local dev.

Workflow services cannot reach HOST-daemon Compose containers from the job container — all bridging approaches require either a container ID (unavailable on cgroup v2) or functional DNAT (not present in this environment). Browser tests move to feat/browser-ci which redesigns the approach using workflow services for Jellyfin and Seerr. This branch makes CI green immediately: lint + unit + integration tests still run; browser tests remain in-tree for local dev.

aureus added 1 commit

2026-05-25 14:31:21 +00:00

ci: drop browser tests from CI pipeline

PR Checks / test (pull_request) Failing after 1m19s

Details

396bd989ed

Workflow services cannot reach HOST-daemon Compose containers from
the job container — all bridging approaches require either a container
ID (unavailable on cgroup v2) or functional DNAT (not present in this
environment). Browser tests move to feat/browser-ci which redesigns
the approach using workflow services for Jellyfin and Seerr.

This branch makes CI green immediately: lint + unit + integration
tests still run; browser tests remain in-tree for local dev.

aureus added 1 commit

2026-05-25 14:35:30 +00:00

ci: install python3/make/g++ before npm ci for better-sqlite3

PR Checks / test (pull_request) Has been cancelled

Details

78345e4d6a

prebuild-install cannot reach GitHub releases in the CI environment, so
node-gyp rebuilds better-sqlite3 from source. node-gyp requires Python
and C++ build tools which aren't in the CI image; installing them before
npm ci on cache miss unblocks native module compilation.

aureus added 1 commit

2026-05-25 14:38:11 +00:00

ci: simplify CI image — strip Docker/Playwright, rely on python3 in image

PR Checks / test (pull_request) Has been cancelled

Details

a8e9a8bcc8

feat/clean-ci runs no browser tests and no Docker Compose, so the CI
image no longer needs Docker CLI, Compose v2, or Playwright system deps.
Strip those layers to reduce image size and build time.

python3/make/g++ are kept — they were already in Dockerfile.ci but the
image hadn't been rebuilt, causing node-gyp to fail compiling
better-sqlite3 on cache misses. Remove the workflow apt-get workaround
now that the image will have them baked in.

Rebuild and push the image from this branch before merging:
  docker build -f Dockerfile.ci -t <registry>/watchcraft-ci:latest .
  docker push <registry>/watchcraft-ci:latest

aureus added 1 commit

2026-05-25 14:38:44 +00:00

Revert "ci: simplify CI image — strip Docker/Playwright, rely on python3 in image"

PR Checks / test (pull_request) Has been cancelled

Details

f01ef26c50