change p2p default port#27
Merged
Merged
Conversation
a-ok123
approved these changes
May 14, 2025
akobrin1
added a commit
that referenced
this pull request
Jul 4, 2026
A validator that fell out of the active set purely on stake weight (Unbonded, not jailed) was permanently deadlocked: MsgMigrateValidator rejected both Unbonding and Unbonded, while MsgClaimLegacyAccount rejects any validator operator. Such an operator could recover neither legacy keys, funds, nor rewards, and had no economic path back across the active-set threshold. Narrow the guard to reject only Unbonding, whose live entry in the SDK unbonding-validator queue (keyed by completion time -> operator address) would be orphaned by Step V8's DeleteValidatorRecordNoHooks and halt the chain at maturity, since the module's StakingKeeper interface has no validator-queue methods. Unbonded is safe and is the recovery path: that queue entry was already dequeued (the transition that makes it Unbonded), so nothing is orphaned. MigrationEstimate now reports would_succeed=true for Unbonded non-jailed. Jailed stays blocked (checked first; recovery is unjail). Also in this change: - Remove MigrateClaim: claim records are no longer re-keyed during migration. The claim DB is frozen (claiming ended 2025-01-01) and kept for reference only; re-keying DestAddress was cosmetic (funds already moved) and scanning the ~18K-record store per migration was unbounded gas with no benefit. Drops the ClaimKeeper interface, keeper field, and depinject wiring. - Docs: record Bug #27 (delegation stake stored as tokens, not shares) and the 2026-07-03 testnet gas calibration; update the migration guides and the evmigration unit/integration test-index docs. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.