Skip to content
This repository has been archived by the owner on Jun 11, 2024. It is now read-only.

Update genesis state handling for DPoS #7730

Merged
merged 5 commits into from
Nov 7, 2022

Conversation

shuse2
Copy link
Collaborator

@shuse2 shuse2 commented Nov 3, 2022

What was the problem?

This PR resolves #7669

How was it solved?

  • Add validation for the reward sharing in initGenesisState
  • Update BFT property calculation in finalizeGenesisState
  • Update Remove snapshot usage in finalizeGenesisState
  • Add new properties to init command genesis creation
  • Update genesis block in example

How was it tested?

  • Run DPoS example app
  • Add additional test for validations

@shuse2 shuse2 requested review from bobanm and mosmartin November 3, 2022 07:44
@shuse2 shuse2 self-assigned this Nov 3, 2022
Copy link
Contributor

@mosmartin mosmartin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👏

@shuse2 shuse2 force-pushed the 7670-update-after_transactions-execute branch from 4da2f67 to 9bf1b0b Compare November 3, 2022 16:41
@shuse2 shuse2 force-pushed the 7669-update_genesis_state branch from fd33628 to cb9d379 Compare November 3, 2022 16:49
@shuse2 shuse2 requested a review from bobanm November 7, 2022 10:21
Base automatically changed from 7670-update-after_transactions-execute to feature/7622-update-dpos-module November 7, 2022 11:22
@shuse2 shuse2 force-pushed the 7669-update_genesis_state branch from 89a43fe to 93e29c7 Compare November 7, 2022 11:44
@shuse2 shuse2 merged commit 8614063 into feature/7622-update-dpos-module Nov 7, 2022
@shuse2 shuse2 deleted the 7669-update_genesis_state branch November 7, 2022 14:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants