-
Notifications
You must be signed in to change notification settings - Fork 4
Fulu support #584
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.
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
Fulu support #584
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR adds support for the Fulu fork to the SSV protocol implementation. Fulu is the next Ethereum fork scheduled after Electra, providing continuity for the beacon chain evolution. The changes integrate Fulu block types, update version handling, and ensure consistent testing coverage across all supported fork versions.
Key changes include:
- Added Fulu fork epoch and testing utilities with appropriate slot calculations
- Extended block processing to handle both regular and blinded Fulu blocks using Electra structures
- Updated version detection logic to properly sequence Electra → Fulu transitions
- Comprehensive test coverage for Fulu block validation scenarios
Reviewed Changes
Copilot reviewed 95 out of 152 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| types/testingutils/beacon_node_versions.go | Adds Fulu fork epoch constant and integrates it into version detection functions |
| types/testingutils/beacon_node_proposer.go | Implements Fulu block content generation and proposal handling |
| types/testingutils/beacon_node_attestation.go | Updates attestation utilities to handle Fulu version alongside Electra |
| types/testingutils/beacon_node_aggregator.go | Extends aggregator support for Fulu using Electra structures |
| types/testingutils/beacon_node.go | Adds Fulu block submission and processing logic |
| types/consensus_data.go | Implements unified block data handling for both blinded and regular blocks across all versions |
| Various test files | Adds comprehensive test coverage for Fulu block validation scenarios |
Comments suppressed due to low confidence (3)
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
|
Hey @olegshmuelov |
MatheusFranco99
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good work! Some small fixes
Adds support for the Fulu consensus layer upgrade (part of the upcoming Fusaka fork).
No major changes were required as Fulu reuses Electra's block and attestation structures. This PR mainly aligns with go-eth2-client's Fulu support.
Changes:
Specs: https://github.com/ethereum/consensus-specs/tree/dev/specs/fulu