-
Notifications
You must be signed in to change notification settings - Fork 788
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
Collecting fee recipients from pre-merge validators #2994
Comments
Here is Teku's work along these lines: Consensys/teku#4894 |
Here is Prysm's implementation that follows something similar prysmaticlabs/prysm#10292 |
Closing this as we now support the fee recipient HTTP API and fee recipients in the validator definitions YAML |
for the SFR would that be an ETHv1 or ETHv2 address? Any reason why it should not be the same as the validator public address? |
@djbakerman We just added an FAQ on this exact point! You can give it a test drive here: https://lighthouse-book.sigmaprime.io/suggested-fee-recipient.html#why-do-i-have-to-nominate-an-ethereum-address-as-the-fee-recipient Let us know if that helps or if you think there's anything you think we should add |
Thank you that was helpful. Perhaps obvious, but it's worth mentioning something like: "If the execution node is testnet, ensure the ETHv1 address is a testnet address" |
Description
After Bellatrix, it will eventually be necessary for Lighthouse to know one more detail for each validator: the
suggested_fee_recipient
(SFR) for producing an execution payload ("eth1 block").For new validators, it makes sense to add a SFR to the
lighthouse account validator
commands that create a validator.In #2924, the
suggested_fee_recipient
can be loaded from CLI, from file and from thevalidator_definitions.yml
file. To me, it seems like the clearest strategy would be to get users to edit thevalidator_definitions.yml
. Perhaps it would be nice if we wrote alighthouse account
tool that can be pointed at avalidator_definitions.yml
file and it will request a SFR for each validator and update the file.The text was updated successfully, but these errors were encountered: