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

Validator client builder support #10713

Closed
wants to merge 23 commits into from

Conversation

james-prysm
Copy link
Contributor

What type of PR is this?

Feature
related to PR #10669

What does this PR do? Why is it needed?

This PR builds on top of #10669

This PR provides capabilities for users to define fee recipients and gaslimit information on their corresponding validator public keys and send them to the beacon node, which will in turn call the builder APIs. These user-defined configurations are important for custom builders to distribute funds correctly and are signed by the validator key for authenticity. The information used here is similar to the prepare_beacon_proposer APIs which are sent to the execution engine, the register validator API proxies to the builder APIs.

Please refer to the corresponding PRs for additional information
ethereum/builder-specs#2
https://github.com/ethereum/beacon-APIs/pull/209/files

@james-prysm james-prysm added the Builder PR or issue that supports builder related work label May 18, 2022
@james-prysm james-prysm self-assigned this May 18, 2022
@james-prysm james-prysm changed the base branch from develop to builder-1 May 18, 2022 21:50
@james-prysm james-prysm marked this pull request as ready for review May 20, 2022 16:30
@james-prysm james-prysm requested a review from a team as a code owner May 20, 2022 16:30
@james-prysm james-prysm requested review from kasey, prestonvanloon and saolyn and removed request for a team May 20, 2022 16:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Builder PR or issue that supports builder related work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants