review(regall): batches 5-6 verified; A-regall-2 filed for plausible backup_restore=fail
All checks were successful
continuous-integration/drone/push Build is passing
All checks were successful
continuous-integration/drone/push Build is passing
Batch 5 results: - uptime-kuma (748): L5 all pass ✓ - lasuite-drive (749): L5 all pass ✓ - plausible (750): L2, backup_restore=FAIL — regression from baseline L5 - ci_marker not found after restore; no-op upgrade (3.0.1+v2.0.0→3.0.1+v2.0.0) - Builder re-running as Drone 754 Batch 6 results: - custom-html-tiny (752): L5, upgrade=pass, backup_restore=skip (expected) ✓ - bluesky-pds (753): L5, upgrade=skip (expected/EXPECTED_NA), backup_restore=pass ✓ A-regall-2: plausible backup_restore=fail — prevb regression or flake TBD. Run 750 shows no-op upgrade (prevb UPGRADE_BASE_VERSION path) vs baseline run 658 genuine upgrade (git ref). Same failure seen in m2r/m2rr-plausible during prevb development. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
@ -40,6 +40,34 @@
|
||||
|
||||
## Adversary findings
|
||||
|
||||
### A-regall-2 [adversary] OPEN @2026-06-17T03:25Z — plausible backup_restore=fail; classify prevb regression or flake
|
||||
|
||||
**Filed:** 2026-06-17T03:25Z
|
||||
**Severity:** MEDIUM — backup_restore failure drops plausible from baseline L5 to L2. Blocks M1 classification.
|
||||
|
||||
**Run:** 750 (Drone 750, PR#4). Result: level=2, backup_restore=fail.
|
||||
**Baseline:** run 658, level=5, backup_restore=pass.
|
||||
|
||||
**Failure:** `test_restore_returns_state` — `ERROR: relation "ci_marker" does not exist` after restore.
|
||||
- Backup test passed (only checks artifact file exists, 0.134s — does NOT verify ci_marker content)
|
||||
- Restore completes (test_restore_healthy passes), but ci_marker table absent from DB
|
||||
|
||||
**Prevb-specific difference:**
|
||||
- Run 750 upgrade: `version=3.0.1+v2.0.0→3.0.1+v2.0.0` (NO-OP: UPGRADE_BASE_VERSION='3.0.1+v2.0.0' matches recipe.yml version)
|
||||
- Run 658 upgrade: `version=d77adba4698b` (git ref — genuine upgrade from published base to tested commit)
|
||||
- Hypothesis: prevb's new base-resolution path resolves UPGRADE_BASE_VERSION to a static version; if recipe.yml also pins that same version, the upgrade is a no-op, which may change the DB state sequence enough to break backup/restore
|
||||
- Same failure pattern in m2r-plausible and m2rr-plausible (prevb development runs) — both level=2, backup_restore=fail
|
||||
|
||||
**Builder rerun:** Drone 754 (triggered after 750 failed) — verdict pending.
|
||||
|
||||
**Adversary stance:** Cannot classify as flake until rerun verdict AND upgrade-version diff is understood.
|
||||
If 754 fails: requires Builder to diagnose no-op upgrade path + fix or explicitly suppress plausible from UPGRADE_BASE_VERSION treatment.
|
||||
If 754 passes: likely flake; request 3-of-3 spot-check before classifying GREEN.
|
||||
|
||||
**Adversary closes after:** rerun verdict received, root cause established, and Builder records classification decision.
|
||||
|
||||
---
|
||||
|
||||
### A-regall-1 [adversary] CLOSED @2026-06-17T02:20Z — mailu baseline table corrected
|
||||
|
||||
**CLOSED:** Builder corrected STATUS-regall.md in commit 7c6134a: mailu upgrade rung now shows "pass" not "skip (no deployable base)".
|
||||
|
||||
@ -123,6 +123,34 @@ Cold-read Drone logs for custom-html (build 737):
|
||||
- `base = main-tip/ref 2b82ebabde74 → chaos deploy of the checked-out ref` ✓
|
||||
**PASS: prevb previous/ overlay correctly scoped to UPGRADE_BASE_VERSION recipes only.**
|
||||
|
||||
### Batch 5 partial-verified @2026-06-17T03:20Z — uptime-kuma/lasuite-drive L5; plausible FAIL (rerun pending)
|
||||
|
||||
From Drone builds API + cc-ci run results.json:
|
||||
- **uptime-kuma** (run 748, Drone 748, PR#?): level=5, all rungs pass (upgrade=pass, backup_restore=pass) ✓
|
||||
- **lasuite-drive** (run 749, Drone 749, PR#?): level=5, all rungs pass (upgrade=pass, backup_restore=pass) ✓
|
||||
- **plausible** (run 750, Drone 750, PR#4): level=2, backup_restore=**FAIL** — REGRESSION from baseline L5
|
||||
|
||||
**Plausible failure analysis:**
|
||||
- Error: `ERROR: relation "ci_marker" does not exist` in `test_restore_returns_state`
|
||||
- upgrade line: `version=3.0.1+v2.0.0→3.0.1+v2.0.0` — NO-OP upgrade (base = head version; same)
|
||||
- Baseline run 658 used `version=d77adba4698b` (genuine git ref → genuine upgrade)
|
||||
- Same failure pattern seen in `m2r-plausible` and `m2rr-plausible` during prevb development
|
||||
- Backup test passed (0.134s, checks artifact only — does NOT verify ci_marker content)
|
||||
- After restore, `SELECT v FROM ci_marker` fails: relation does not exist
|
||||
- Hypothesis A (prevb regression): UPGRADE_BASE_VERSION='3.0.1+v2.0.0' + recipe.yml version='3.0.1+v2.0.0' creates no-op upgrade path that affects backup state
|
||||
- Hypothesis B (flake): pre-existing intermittent failure in postgres backup/restore
|
||||
- **Builder triggered rerun as Drone 754 — verdict pending**
|
||||
|
||||
**See A-regall-2 in BACKLOG.**
|
||||
|
||||
### Batch 6 cold-verified @2026-06-17T03:25Z — custom-html-tiny/bluesky-pds L5
|
||||
|
||||
From Drone builds API + cc-ci run results.json:
|
||||
- **custom-html-tiny** (run 752, Drone 752, PR#?): level=5, upgrade=pass, backup_restore=skip (expected) ✓
|
||||
- **bluesky-pds** (run 753, Drone 753, PR#3): level=5, upgrade=skip (expected — no deployable upgrade base, moving tag), backup_restore=pass ✓
|
||||
|
||||
Bluesky-pds upgrade=skip reason confirms prevb is correctly handling the EXPECTED_NA path (no deployable base). ✓
|
||||
|
||||
### Batch 4 cold-verified @2026-06-17T03:00Z — all L5, no regressions
|
||||
|
||||
From Drone builds API + cc-ci run results.json:
|
||||
|
||||
Reference in New Issue
Block a user