From 790d79490059eb736d17963944273ecf1a674ede Mon Sep 17 00:00:00 2001 From: Kit OC5 Date: Wed, 23 Sep 2026 19:51:26 -0400 Subject: [PATCH] bridge: drop eternitas ci/build from BRIDGE_NO_DAEMON (converted to ci/smoke, #179) Co-Authored-By: Claude Opus 5.5 --- api/tests/test_pr_status_bridge.py | 5 +++-- docs/CUTOVER.md | 2 +- scripts/pr_status_bridge.py | 3 ++- 3 files changed, 6 insertions(+), 4 deletions(-) diff --git a/api/tests/test_pr_status_bridge.py b/api/tests/test_pr_status_bridge.py index 6b7dae4..41cdcd0 100644 --- a/api/tests/test_pr_status_bridge.py +++ b/api/tests/test_pr_status_bridge.py @@ -425,5 +425,6 @@ def test_named_no_daemon_job_is_not_posted_for_that_repo_only(fake, monkeypatch) assert sorted(p["context"] for p in f2.posted) == ["windy-git/ci/build", "windy-git/ci/test"] -def test_default_no_daemon_named_is_eternitas_build(): - assert bridge.NO_DAEMON_NAMED.get("eternitas") == {"ci/build"} +def test_default_no_daemon_named_is_empty(): + """eternitas converted its ci/build to a no-Docker ci/smoke (#179); nothing left.""" + assert bridge.NO_DAEMON_NAMED == {} diff --git a/docs/CUTOVER.md b/docs/CUTOVER.md index bb78cf1..8656fda 100644 --- a/docs/CUTOVER.md +++ b/docs/CUTOVER.md @@ -111,7 +111,7 @@ their CI permanently, not a stopgap: have no Docker daemon by design (I-5), so they are red on every commit. A rootless builder (BuildKit rootless / buildx in the capped dind) is the open decision that would bring them back. Jobs that need Docker but are named otherwise go in - `BRIDGE_NO_DAEMON` (default `eternitas:ci/build`). DECIDED 09-23 (orchestrator, + `BRIDGE_NO_DAEMON` (empty since eternitas converted to ci/smoke, #179). DECIDED 09-23 (orchestrator, option A): lanes convert these jobs to no-Docker smoke tests (job `services:` + start the app + curl /health); the real image build is the deploy step on the target host. ci-hygiene flags docker build/compose/run in workflows ("needs docker"). diff --git a/scripts/pr_status_bridge.py b/scripts/pr_status_bridge.py index e1f6851..b6c57b2 100755 --- a/scripts/pr_status_bridge.py +++ b/scripts/pr_status_bridge.py @@ -79,7 +79,8 @@ NO_DAEMON_JOB = re.compile(r"docker", re.IGNORECASE) # each lane converts the job to a no-Docker smoke test; until then it is not # posted). Format: "repo:workflow/job,...;repo2:...". NO_DAEMON_NAMED: dict[str, set[str]] = {} -for _entry in os.environ.get("BRIDGE_NO_DAEMON", "eternitas:ci/build").split(";"): +# eternitas:ci/build dropped 09-23: converted to a no-Docker ci/smoke (#179). +for _entry in os.environ.get("BRIDGE_NO_DAEMON", "").split(";"): if ":" in _entry: _repo, _jobs = _entry.split(":", 1) NO_DAEMON_NAMED[_repo.strip()] = {j.strip() for j in _jobs.split(",") if j.strip()}