#33362 has had some side-effects for anyone locally running the CI:
-
fanquake commented at 5:16 PM on May 29, 2026: member
- fanquake added the label Brainstorming on May 29, 2026
-
willcl-ark commented at 7:13 PM on May 29, 2026: member
Fixes I can think of:
- revert #33362 (and lose its test coverage)
- ~use (and cleanup!) per-container networks~
- use dynamic IPs and pass them in to tests (least favourite)
~I think I would tent towards using a network per-container. It will only be "messy" if we don't tidy them up properly. We don't currently clean up networks ever (and I ∴ always have two lurking in
docker network ls), and perhaps we should be, anyway.~actually I don't think 2) will work even, as the networks will still overlap :(