Enable multi node attestation consensus for the VC#8445
Draft
eserilev wants to merge 64 commits intosigp:unstablefrom
Draft
Enable multi node attestation consensus for the VC#8445eserilev wants to merge 64 commits intosigp:unstablefrom
eserilev wants to merge 64 commits intosigp:unstablefrom
Conversation
…house into eager-send-attestation
…house into eager-send-attestation
…testation-data-service
…ger-send-attestation
|
Yes, I find it very interesting! Thank you! (This is the corpo account. I'm forwarding this to the Galaxy team as a potential option) |
Contributor
|
Just curious are you gonna just pick one BN to get the ghost head (beacon_block_root)? |
Member
Author
|
@0xmrree im actually doing most of my development on this branch at the moment i've isolated the changes there to make things a bit easier to review I'm not using the beacon block root at all. I'm just comparing the source and target checkpoints across all connected BN's and if a threshold agrees, I attest with one of the nodes that are in agreement. If a consensus threshold isnt reach, attestation is disabled for that epoch. |
f15f01e to
1665bbf
Compare
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
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.
Allow for the VC to query all BN's at the beginning of each epoch and only enable attestation for that epoch if source and target checkpoint matches for a threshold of BN's
TODO