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.

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

DutyBuilderRegistration Re Broadcast Every Epoch #1009

Closed
ciaranmcveigh5 opened this issue Aug 20, 2022 · 1 comment
Closed

DutyBuilderRegistration Re Broadcast Every Epoch #1009

ciaranmcveigh5 opened this issue Aug 20, 2022 · 1 comment

Comments

@ciaranmcveigh5
Copy link
Contributor

Problem to be solved

Currently, for builder registration, Charon submits only one registration after boot.

When registrations are re-submitted at the start of every epoch from the VC clients charon notes that the duty has already been completed and therefore does not forward the request onto the beacon node.

After speaking with Teku we found out that the beacons are implemented to keep a local store of registrations and prune them after 3 epoch if a re-registration doesn't occur. This differs from the relays who (last time I check with flashbots) persist the registration permanently.

When a validator isn't registered from teku's perspective it defaults to local block builds rather than builder network blocks

From Teku

"registered_validators will go back to empty on restart, that is why Teku registers all validators on startup
also there is expiry which is 3 epoch currently
so if you don't call register endpoint, then the validator registration will expire"

Proposed solution

Re cast component to re broadcast the message to beacon

recast component:

  • it sits next to bcast
  • so it subscribes to sigagg events, so get all aggregates signatures (same as bcast and aggsigdb)
  • it stores the high slot DutyRegistration it receives (ignores other duties)
  • it also listens to epoch_events (need to add this functionality to scheduler to emit slot and epoch events)
  • resend DutyRegistration to broadcast on receipt of epoch event.

Out of Scope

n/a

obol-bulldozer bot pushed a commit that referenced this issue Aug 22, 2022
)

Adds the recaster component to rebroadcast builder registrations every epoch.

category: feature
ticket: #1009
@corverroos
Copy link
Contributor

Closing since done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants