fix: modify end blocker order to avoid panic when reducing MaxValidators#3786
fix: modify end blocker order to avoid panic when reducing MaxValidators#3786
Conversation
|
Important Review skippedAuto incremental reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the 📝 WalkthroughWalkthroughThe changes update the initialization process by reordering the governance module within the application's end blockers list. In the code, the governance module is now appended in a different sequence, which affects its execution at the end of each block. The changelog has been amended to reflect this reordering. Additionally, a new JSON proposal file has been introduced to test changes to the Cosmos staking parameters, specifically reducing the maximum validator count to 1. Changes
Sequence Diagram(s)sequenceDiagram
participant App
participant EndBlocker
participant GovModule
participant OtherModules
App->>EndBlocker: Trigger end block operations
EndBlocker->>OtherModules: Process preliminary module tasks
EndBlocker->>GovModule: Process governance operations (reordered)
sequenceDiagram
participant Proposer
participant Governance
participant StakingModule
Proposer->>Governance: Submit proposal (MsgUpdateParams)
Governance->>StakingModule: Forward update parameters
StakingModule-->>Governance: Acknowledge update
Governance-->>Proposer: Return proposal status
Suggested labels
Suggested reviewers
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
Documentation and Community
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #3786 +/- ##
========================================
Coverage 64.18% 64.18%
========================================
Files 463 463
Lines 33156 33156
========================================
Hits 21282 21282
Misses 10897 10897
Partials 977 977 🚀 New features to boost your workflow:
|
…e-validator-set # Conflicts: # app/modules.go
lumtis
left a comment
There was a problem hiding this comment.
Can we add a link for the reference for the order
Added the link the code and the PR description |
Description
Closes : #3782
Order used for this refactoring has been derived from
https://github.com/cosmos/gaia/blob/main/app/modules.go
How Has This Been Tested?
Summary by CodeRabbit