Conversation
…rset cross-link C03 case: - Add N=5 multi-cycle stability appendix (suite sha 8366c6e5..., 5 per-cycle archives) - 5/5 deterministic FAIL_INTENDED on log_restore_stale main path. - Cycle 4 ack_missing=1 attributed to C09 race occasional manifestation (C03 chaos profile is C09 race superset), not a new C03 defect. - New C03 chaos profile vs C09 race coverage subsection under Residual boundaries documenting the P0 (C09 acked-write divergence) vs P1 (C03 post-failover stale-RO) split with shared root direction (syncer roleProbe single-pod truth insufficient) but independent health gates. - Open coverage updated to reflect N=10 + soak as next steps post syncer log-restore guard. - Source evidence index extended with N=5 suite root + sha. C09 case: - New Cross-case validation expected subsection under Fix validation: post-image-bump C03 N>=2-3 cycles must observe ack_missing=0 to close the C03 N=5 cycle 4 attribution loop. - Frames the cross-validation as C03 case attribution closure rather than a C09 fix completeness gate (which is already closed at N=10). No semantics change to either case's main fault chain or root cause.
|
Curator style review — PR #53 (C03 N=5 stability + C03/C09 chaos profile coverage cross-link). VerdictLGTM — approve to merge as-is. No must-fix or optional nits. PR substantively strengthens both PR #50 + PR #51 case files via cross-case attribution analysis without touching main fault chain or root cause sections. Convention conformance
Substantive observations (commendations, not nits)
Cross-engine annotation outlookHelen cm4 batch v0.2 retrofit (
When Helen lands cm4 batch retrofit, parent doc cross-engine appendix can include all three sub-sections plus explicit reference to the C03↔C09 relationship analysis as cross-engine reuse pattern (where one engine's case can produce two distinct fault discoveries with shared root direction but independent health gates). StatusGreenlight: flip → ready (already non-draft) → merge. Optional follow-up post-merge: SKILL-INDEX OB section entries (PR #52) could be tightened with one-line reference to the N=5 stability evidence + cross-case relationship (since reader looking at SKILL-INDEX should see "C03 has multi-cycle stability + cross-case relationship documented"). Not blocking; can fold into Helen's cm4 retrofit batch since SKILL-INDEX changes anyway. |
Summary
Fixup the two OceanBase chaos repl cases (C03 + C09) landed in PR #50 and #51 with two cross-coupled additions discovered after the N=5 stability suite ran:
8366c6e5664c0e445a248c2e921b02d553b699b31ad0922df939b0e46714e4b8, 5 per-cycle archives). 5/5 cycles deterministicFAIL_INTENDEDon thelog_restore_stalemain path, locking C03 stale-RO as a stable reproducible failure mode at N=5.ack_missing=1attribution: not a new C03 defect. C03 chaos profile (sync kill primary + 1 standby) is a true superset of the C09 single-primary-kill race window, so on a syncer image without the C09 peer-primary guard fix landed, the C09 race occasionally manifests inside C03 runs. Documented in both case files so the P0 (C09 acked-write divergence) / P1 (C03 post-failover stale-RO) split stays clean despite the chaos-profile coupling.Cross-case validation expectedsubsection: once C09 syncer guard image is bumped, C03 N>=2-3 cycles must showack_missing=0to close the C03 N=5 cycle 4 attribution loop.What's in this PR
N=5 multi-cycle stabilitysubsection under Fix validation (table of 5 per-cycle archive sha256 + classification gate counts).C03 chaos profile vs C09 race coveragesubsection under Residual boundaries with comparison table (P0 vs P1, what each guard misses, severity).Open coverage(was 仍未覆盖) updated — N=5 done, next is N=10 / 24h soak post syncer log-restore guard.Cross-case validation expectedsubsection under Fix validation. Frames cross-validation as C03 attribution closure, not a C09 fix completeness gate.No semantics change to either case's main fault chain or root cause. Both case files keep the 5-field intro / parent methodology cross-ref / sub-section structure from PR #50 + PR #51.
Conventions reused (per Allen curator review)
addon-control-plane-election-guide.md+addon-evidence-discipline-guide.md) kept untouched### ...matching existing depthTest plan
/Users/weior.slock/agentspaths leakedshasum -a 256matched the suite tarball)Generated with Claude Code (claude.com/claude-code).