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

simulators/eth2/withdrawals: add builder tests #715

Merged
merged 13 commits into from
Feb 28, 2023

Conversation

marioevz
Copy link
Member

@marioevz marioevz commented Feb 16, 2023

Changes Included

  • Adds a mock builder to the eth2/common library that is able to relay payload requests to a local Execution client to simulate a working builder
    • Client receives and registers validators to produce payloads when requested
    • When requested, the builder queries the CL for its state to obtain the prevrandao value to be able to construct the correct payload attributes
    • The payload attributes are relayed to the EL and then it waits for the payload to be built
    • The builder constructs and signs a builder bid which is returned to the CL
    • When the signed blinded beacon block is received, the full payload (transactions + withdrawals) is sent as response
  • Changes all Beacon clients to include builder parameters

Clients passing tests:

  • Nimbus
  • Lighthouse
  • Prysm
  • Teku
  • Lodestar

Lodestar test is currently failing because it stops requesting headers from the builder at the Capella fork. Fixed by ChainSafe/lodestar#5179

@marioevz
Copy link
Member Author

@fjl let me know if it's ok to include client's changes in this same PR or I can split it up in a PR for each client (adds Builder API parameters to all consensus clients).

@marioevz marioevz force-pushed the add-builder-tests-to-eth2 branch from ddc6697 to 5782437 Compare February 27, 2023 19:23
@fjl fjl changed the title simulators/eth2/common,withdrawals: Add Builder Tests simulators/eth2: Add Builder Tests Feb 28, 2023
@fjl fjl changed the title simulators/eth2: Add Builder Tests simulators/eth2/withdrawals: add builder tests Feb 28, 2023
@fjl fjl merged commit 22afd9f into ethereum:master Feb 28, 2023
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

Successfully merging this pull request may close these issues.

2 participants