Backport release/v6.7: Log a pinned node's skipped migration kick-off once per batch size

[seidroid]

Backport of #4408 to release/v6.7.

[cursor]

PR Summary

^{Cursor Bugbot is generating a summary for commit 1ed6686. Configure here.}

[cursor]

PR Summary

Low Risk
Observability-only deduplication around existing skip paths; migration kick-off logic is unchanged aside from when logs emit.

Overview
Reduces log spam when a node skips the EVM migration kick-off because SetMigrationBatchSize is applied every block (e.g. from BeginBlock) while the store stays in memiavl_only.

rootmulti.Store now tracks migrationSkipLoggedBatchSize and gates the existing skip Info / Error lines through shouldLogMigrationSkip, so each positive batch size is logged at most once until governance changes it. Setting batch size to 0 clears that state so re-enabling migration logs again.

Adds TestRootMultiAutoKickoff_FixedMemiavlOnlySkipLoggedOncePerBatchSize with a counting slog handler to lock in once-per-batch-size behavior.

^{Reviewed by Cursor Bugbot for commit 2a0d094. Bugbot is set up for automated code reviews on this repo. Configure here.}

[github-actions]

The latest Buf updates on your PR. Results from workflow Buf / buf (pull_request).

Build	Format	Lint	Breaking	Updated (UTC)
✅ passed	✅ passed	✅ passed	✅ passed	Oct 1, 2026, 4:01 PM

[seidroid]

This is a clean backport of #4408. It adds a per-batch-size log dedup to SetMigrationBatchSize, so a pinned or unknown-mode node logs the skipped migration kick-off once per batch size instead of on every block, and a batch size of 0 resets the dedup. The field is only touched from BeginBlock, which runs on one goroutine, so no race is introduced. The new test swaps the package logger without t.Parallel, and it checks repeat, change and reset-after-0 cases.

Findings: 0 blocking | 0 non-blocking | 0 posted inline

Blockers

• None at the file/PR level.

Non-blocking

• None at the file/PR level.

[codecov]

Codecov Report

❌ Patch coverage is 80.00000% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 60.36%. Comparing base (ba590ac) to head (2a0d094).

Files with missing lines	Patch %	Lines
sei-cosmos/storev2/rootmulti/store.go	80.00%	2 Missing ⚠️

Additional details and impacted files

@@               Coverage Diff                @@
##           release/v6.7    #4411      +/-   ##
================================================
- Coverage         61.40%   60.36%   -1.04%     
================================================
  Files              2163     2064      -99     
  Lines            189115   177381   -11734     
================================================
- Hits             116121   107082    -9039     
+ Misses            62261    60513    -1748     
+ Partials          10733     9786     -947     

Flag	Coverage Δ
sei-chain-pr	68.37% <80.00%> (?)
sei-db	69.80% <ø> (ø)
sei-db-state-db	?

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
sei-cosmos/storev2/rootmulti/store.go	69.67% <80.00%> (+0.14%)	⬆️

... and 99 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[seidroid]

The skipped-migration log now fires once per batch size instead of on every block, and the change touches only logging. Approved with no findings. Its only caller is BeginBlock, on the consensus goroutine, so the new unguarded field has no concurrent access. No consensus or state behaviour changes, the counter resets when the batch size drops to 0, and the test covers both paths. The one other reading (codex) found nothing, which matches this review, so it adds nothing here.

_{seidroid review · decision approve · session 8a056eada1ae441cbf40675f0868395b · turn resp_claude_63e5cf605a443f4cefd4264577a31b58 · item 3f44da5f3d285291a9e3e7734444f558}

Findings: 0 blocking | 0 non-blocking | 0 posted inline