note(2): pre-assess ghost base-grace overlay compose.ccci.yml (7feeadd) — static read policy-compliant (minimal/justified/grace-only); NOT a PASS, durable proof = green upgrade-to-latest run; VETO stands

This commit is contained in:
2026-05-30 17:56:05 +01:00
parent 7feeadd0ec
commit fe135d3d55

View File

@ -2306,3 +2306,21 @@ Orchestrator FYI: the ccci overlay convention is now a SINGLE uniform `compose.c
the filename `compose.ccci.yml`, single uniform per recipe.
- NB: ghost/discourse overlays are currently DELETED (literal-recipe-PR bump path). If the upgrade-to-latest
requirement brings the discourse re-pin overlay back, it should land as `compose.ccci.yml`. No verdict here.
## NOTE (pre-assessment, NOT a verdict, does NOT clear the VETO) @2026-05-30T16:56Z — ghost base-grace overlay `compose.ccci.yml` (Builder feat `7feeadd`)
Pre-examined the re-introduced ghost overlay against VETO-checklist item 1 (overlay minimality). Static read:
- **Minimal/single-purpose:** overrides ONLY `services.app.healthcheck.start_period: 15m`; deep-merges onto
the base healthcheck (test/interval/timeout/retries preserved — correct compose override semantics).
- **Justified header:** cites the exact abra limitation I independently reproduced (REVIEW-2 `4b862f6` — abra
FATA on env-interpolated start_period, pre-substitution duration validation) + upgrade-to-latest mandate +
base 1.1.1+6 ships 1m grace → swarm kill mid-migration → held migrations_lock deadlock.
- **Masks no defect / weakens no test:** start_period is grace-only (a healthy check marks healthy at once;
normal healthchecking resumes after the window). TIMEOUT=1200s bounds a genuine failure (~20min, not a
blackout). Idempotent on the PR head (head already ships literal 15m), widens base 1m→15m only.
- **Plumbing:** install_steps.sh copies the cc-ci overlay into the recipe checkout; CHAOS_BASE_DEPLOY=True
skips abra's clean-tree gate on the untracked overlay; COMPOSE_FILE=compose.yml:compose.ccci.yml.
PROVISIONAL CONCLUSION: appears `plan-ccci-compose-overlay-policy.md`-compliant on static read. **NOT a PASS**
— the durable proof is a green ghost full-suite run INCL upgrade-to-latest (deploy-count=1, P3≥2, P4 non-vacuous,
clean teardown), which the Builder has not yet claimed. When claimed I will (a) confirm the overlay on cc-ci is
byte-identical to git, (b) confirm upgrade-tier base actually deploys with it + converges, (c) confirm head
deploy is idempotent. VETO on DONE stands.