nimbus-eth2 icon indicating copy to clipboard operation
nimbus-eth2 copied to clipboard

[FR] Per validator fee recipient

Open kanewallmann opened this issue 3 years ago • 3 comments

It is highly desirable for Rocket Pool (and likely other staking providers) that the feeRecipient on execution payloads be configurable on a per validator account basis.

In the case of Rocket Pool, users may be running both solo validators where they are entitled to the full reward and Rocket Pool validators where only a portion of the rewards belongs to them. Not having this feature would exclude Nimbus as a possible client for users running this setup.

Depending on our chosen implementation of priority fee processing, we may even require this feature for all Rocket Pool node operators as we may need to account for priority fees on an individual validator basis. In this situation, this feature would be critical for our interoperability with Nimbus.

A simple mapping of validator indices to fee recipients would be sufficient for our use case.

I can imagine that this flexibility would be desirable in other advanced use cases as well.

kanewallmann avatar Jan 17 '22 02:01 kanewallmann

For reference, how Teku does this: https://github.com/ConsenSys/teku/pull/4894

tersec avatar Feb 15 '22 13:02 tersec

The support for specifying per-validator fee recipient was implemented in https://github.com/status-im/nimbus-eth2/pull/3652

The issue will be closed after we document the configuration steps in the Nimbus guide (https://github.com/status-im/nimbus-eth2/issues/3723). In the first iteration, the user will have to manually create configuration files, while in the future we will also support a command-line interface for this (https://github.com/status-im/nimbus-eth2/issues/3723 and https://github.com/status-im/nimbus-eth2/issues/3725)

zah avatar Jun 09 '22 10:06 zah

~~We should also support the more simple and stateless --suggested-fee-recipient option for this - ie it's messy to have to create a file or use the keymanager api for simple use cases~~

Edit: Oops, we already do :)

arnetheduck avatar Jun 09 '22 15:06 arnetheduck