Skip to content

test: sweep the two remaining cr-category count baselines to 71#35

Merged
sqlrush merged 1 commit into
mainfrom
fix/73-cr-count-sweep
Jul 10, 2026
Merged

test: sweep the two remaining cr-category count baselines to 71#35
sqlrush merged 1 commit into
mainfrom
fix/73-cr-count-sweep

Conversation

@sqlrush

@sqlrush sqlrush commented Jul 10, 2026

Copy link
Copy Markdown
Owner

nightly-73b (run 29127607103) verified the PR#34 fixes (foundation-core green, t/288 boots) and exposed the last L462 sweep miss: t/306 + t/311 assert the cr dump-category count, auto-merged clean at main's 70 while the merged truth is 71 (+ cr_server_fence_refused_count). Locally green (29/29, 27/27).

With this, stage5-ges-locking's only red files are cleared; remaining nightly reds = pre-existing main baseline set (stage4-wal / stage6-crossnode-a/b / stage6-storage / stage7-reconfig-liveness / stage5-integrated t/328+334).

Test plan:

  • t/306 + t/311 locally green
  • fast-gate
  • re-tag nightly after merge

Spec: spec-7.3-lms-worker-pool.md

The spec-7.3 merge recomputed the cr dump-category count to 71 (main's
70 + the lane's cr_server_fence_refused_count row) and updated the
three files that CONFLICTED (t/215/216/300) -- but t/306 and t/311
assert the same fact, auto-merged clean carrying main's 70, and only
run in nightly (caught by run 29127607103; the L462 lesson verbatim:
sweep every file asserting the same fact, not just the conflicted set).

Verified locally: t/306 29/29, t/311 27/27.

Spec: spec-7.3-lms-worker-pool.md
@sqlrush sqlrush merged commit 2c428bf into main Jul 10, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant