Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Enable Quorum/IBFT1 to Besu migration #8262

Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

Enable Quorum/IBFT1 to Besu migration #8262

wants to merge 9 commits into from

Conversation

pullurib
Copy link
Contributor

@pullurib pullurib commented Feb 6, 2025 •
edited
Loading

PR description

  • Restored and updated the removed IBFT1 code required for parsing and validating IBFT1 extra data.
  • Made required changes to enable transitioning to QBFT after IBFT1 block import.
  • Added test for importing IBFT1 blocks into a Besu network that transitions to QBFT after the imported IBFT1 blocks.
  • Added an end-to-end test case to simulate a 5-node Besu network and validate the block import and transition process.

@pullurib pullurib mentioned this pull request Feb 6, 2025
Closed
pullurib and others added 4 commits February 13, 2025 02:13
@pullurib @matthew1001
Update besu/src/main/java/org/hyperledger/besu/controller/IbftLegacyB…
c78c53d 
…esuControllerBuilder.java

Co-authored-by: Matt Whitehead <[email protected]>
Signed-off-by: Bhanu Pulluri <[email protected]>
Review changes
828b257 
Signed-off-by: Bhanu Pulluri <[email protected]>
Merge branch 'quorum-ibft1-migration' of github.com:pullurib/besu int…
9a10a01 
…o quorum-ibft1-migration
update creating additional JSON RPC methods for all controllerbuidler…
58d2563 
…s in consensus schedule

Signed-off-by: Bhanu Pulluri <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@matthew1001 matthew1001 matthew1001 left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@pullurib @matthew1001