-
Notifications
You must be signed in to change notification settings - Fork 283
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
Implement validator aggregator selection changes for DVT #6851
Comments
It sounds like the latest DVT solution won't be requiring beacon-api changes, so these suggested changes are going to be fairly specific to this implementation - worth taking into consideration while prioritising... |
That spec is looking quite old right now, and doesn't deal with aggregators at all afaics (which might be a blind-spot). We should get some clarity on what the current state of play is. There are only two major implementers of DVT that I'm aware of at the moment, so anything we implement is inevitably going to be fairly application specific. Would definitely prefer to be adhering to whatever "standard" emerges, though. |
I was talking to @saltiniroberto , they're working on a fork... He's actively working on it though. |
Is there any update on this? |
As discussed with @paul Harris on Discord understanding 1) the spec is not complete, 2) some are testing on mainnet with Obol and 3) it's a small change, please could I request that this is included and put behind a feature toggle. |
Currently implementing an MVP of this feature. |
Link to detailed description
In short, when configured as part of a DVT (distributed validator technology) cluster, validators are unable to work out directly whether they are to perform attestation (or sync committee) aggregation duty for a committee. This is because they do not have access to the entire signing key needed to generate the signature used for aggregator selection.
To support this, some changes to the validator client are needed. At high-level (see the doc for details),
--distributed
flag on the VC) rather than automatic detection, but happy to discuss.These changes use new standard beacon APIs and should be generic across DVT solutions. (This is why I prefer manual configuration via a flag - automatic detection might be solution-specific or fragile.)
Tasks
The text was updated successfully, but these errors were encountered: