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

Builder flow for Deneb / Blobs #4345

Closed

Conversation

jimmygchen
Copy link
Member

@jimmygchen jimmygchen commented May 29, 2023

Issue Addressed

#3689

Builder spec: ethereum/builder-specs#61
4844 Builder flow: https://hackmd.io/@jimmygchen/B1dLR74Io

Proposed Changes

Beacon Node

  • produceBlindedBlock endpoint updates
    • Handle new Deneb BuilderBid response from builder endpoint (new BlindedBlobsBundle type)
    • Build BlockContents response (containing kzg_commitments, proof and blinded_blob_sidecars)
  • submitBlindedBlock endpoint updates
    • Handle SignedBlockContents request and forward to builder
    • Handle unblinded ExecutionPayloadAndBlobsBundle response from builder endpoint, and publish block

Validator Client

  • Handle produceBlindedBlock response from Beacon API
  • Individual signing of blinded_blob_sidecars
  • Build SignedBlockContents and submit to Beacon API (submitBlindedBlock)

Task List

  • Cleanup: move types under deneb_types to their respective files.

@jimmygchen jimmygchen self-assigned this May 29, 2023
@jimmygchen jimmygchen added work-in-progress PR is a work-in-progress deneb labels May 29, 2023
@jimmygchen
Copy link
Member Author

jimmygchen commented Jun 21, 2023

There are some issues with my first attempt - code duplication & enum wrappers for blobs / blinded blobs were making things a bit messy and difficult to progress further - so I've started over again using generics instead, similar to how we implemented it for Payload in BeaconBlockBody and it seems a bit cleaner. New branch here: https://github.com/jimmygchen/lighthouse/tree/deneb-builder-take-2

I'll switch the PR to the new branch once I've moved everything I need over to that branch.

@jimmygchen
Copy link
Member Author

Closing in favour of #4428

@jimmygchen jimmygchen closed this Jun 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deneb work-in-progress PR is a work-in-progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant