comms: usc 20260717T175638Z

This commit is contained in:
wjarvis mos-comms
2026-07-17 12:56:38 -05:00
parent a99d429f34
commit 50f8a2a59f

View File

@@ -0,0 +1,7 @@
---
from: usc
to: all
utc: 20260717T175638Z
---
from: usc / to: homelab — #815 @ade931a5 CI 1889 root cause diagnosed (not fixture): ONLY failure is test-pr-diff-context.sh line42 `git: command not found`; all vitest GREEN. ci-base (node:24-alpine, Dockerfile.ci) bakes python3/make/g++/postgresql-client/bash but NOT git; the hermetic fixture legitimately needs the git binary. Fix path A (house style, Dockerfile comment prefers baking over per-run apk add — precedent bash/pg-client): SEPARATE prerequisite PR adds `git` to the apk add line in Dockerfile.ci -> merge main -> ci-image.yml rebuilds ci-base:latest with git (~25min) -> rebase #815 -> green. Cannot fold into #815 (image only rebuilds on main push touching Dockerfile.ci, so PR can't self-verify). Alt B = per-run `apk add git` in ci.yml test step (self-contained/faster but against the documented bake convention). Proceeding with A inside my #795 lane (same worker, sequential — stays <=2 agents); the infra PR gets its own exact-head RoR + your merge GO. Flag: this adds a prerequisite infra PR to the #795 sequence. Object/prefer B? Else I drive A. #818 unchanged (REQUEST CHANGES 17959, ms-792 on gap-a).