MEV block proposal #3883

tersec · 2022-07-17T18:48:32Z

This doesn't affect node operations unless --payload-builder-enable is specified.

github-actions · 2022-07-17T21:42:37Z

Unit Test Results

      12 files ±0     860 suites ±0 1h 20m 6s ⏱️ + 3m 57s
  1 911 tests ±0   1 764 ✔️ ±0 147 💤 ±0 0 ❌ ±0
10 341 runs ±0 10 151 ✔️ ±0 190 💤 ±0 0 ❌ ±0

Results for commit ae32a6f. ± Comparison against base commit dfd4afc.

♻️ This comment has been updated with latest results.

Refers: status-im/nimbus-eth2#3883 Signed-off-by: Artur Marud <artur@status.im>

…02` (#118) Refers: status-im/nimbus-eth2#3883 Signed-off-by: Artur Marud <artur@status.im>

zah · 2022-07-21T20:09:30Z

beacon_chain/conf.nim

+      payloadBuilder* {.
+        hidden
+        desc: "Payload builder URL"
+        name: "payload-builder" .}: Option[string]


Technically, it's possible to use multiple builders, right?

One is supposed to do this with an external builder/relay setup, but yes, it's technically possible to incorporate that multiplexing into a CL. This was discussed at DevConnect and the recommendation was not to do this within the CL, however.

In particular, what you're describing is the role of a "relay", e.g., https://github.com/flashbots/mev-boost/

This project is part of the Flashbots research towards proposer/builder separation for Maximal Extractable Value (MEV) mitigation. mev-boost can connect to relays that aggregate multiple builders. The builders prepare full blocks, optimizing for MEV extraction and fair distribution of the rewards. The Consensus Layer client of the validator proposes the most profitable block received from mev-boost.

i.e. the question of whether a CL should be its own relay, or meta-relay. The protocol allows it -- there used to be a distinction between the "builder" and "relay" protocols, which is gone now, to allow composing these layers more freely -- but they're still generally better to outsource to more specialized software.

For example, the publicly available URLs are not of builders but relays:

Sepolia testnet

Run mev-boost pointed at our Sepolia Relay:

./mev-boost -sepolia -relays https://0x845bd072b7cd566f02faeb0a4033ce9399e42839ced64e8b2adcfc859ed1e8e1a5a293336a49feac6d9a5edb779be53a@builder-relay-sepolia.flashbots.net

That's a reasonable point, but I think users would still be concerned about losing connectivity to a particular relay (either due to a particular internet link going down or due to the relay going offline), so having the option to specify more than one may be desirable.

https://github.com/flashbots/mev-boost should accomplish this, and it's already maintained and will be improved by Flashbots.

beacon_chain/nimbus_beacon_node.nim

beacon_chain/spec/signatures.nim

beacon_chain/spec/state_transition.nim

beacon_chain/validators/validator_duties.nim

zah · 2022-07-21T21:10:30Z

beacon_chain/validators/validator_duties.nim

+
+  return ok validatorRegistration
+
+proc registerValidators(node: BeaconNode) {.async.} =


Maybe the registrations can be evenly spread out through the period instead of producing a spike of CPU usage from time to time.

7780e21 adds a TODO here -- I agree with this as a goal, but not sure it works best as part of this PR.

beacon_chain/validators/validator_pool.nim

The `enable` flag is hidden in the if since this is not merged yet: status-im/nimbus-eth2#3883 Signed-off-by: Jakub Sokołowski <jakub@status.im>

zah · 2022-08-01T06:30:21Z

beacon_chain/conf.nim

+        hidden
+        desc: "Enable external payload builder"
+        defaultValue: false
+        name: "payload-builder-enable" .}: bool


We don't use this -enable suffix on other similar flags (e.g. --rest:on, --keymanager:on, --metrics:on, etc). --light-client-enable is the only existing precedent, but we haven't released this functionality officially yet, so we can fix it as well.

#3928
status-im/infra-role-beacon-node-linux@df10599

Note that macOS and Windows use separate infra-role-beacon-node repos and need separate updating.

Marudny pushed a commit to status-im/infra-role-beacon-node-linux that referenced this pull request Jul 18, 2022

add support for payload_builder option

9ec8b15

Refers: status-im/nimbus-eth2#3883 Signed-off-by: Artur Marud <artur@status.im>

Marudny mentioned this pull request Jul 18, 2022

add support for payload_builder option status-im/infra-role-beacon-node-linux#11

Merged

Marudny pushed a commit to status-im/infra-nimbus that referenced this pull request Jul 18, 2022

sepolia: implement and configure "payload_builder" feature for node-02

8d2c5db

Refers: status-im/nimbus-eth2#3883 Signed-off-by: Artur Marud <artur@status.im>

Marudny mentioned this pull request Jul 18, 2022

sepolia: implement and configure "payload_builder" feature for node-02 status-im/infra-nimbus#118

Merged

Marudny pushed a commit to status-im/infra-role-beacon-node-linux that referenced this pull request Jul 18, 2022

add support for payload_builder option

abf08b4

Refers: status-im/nimbus-eth2#3883 Signed-off-by: Artur Marud <artur@status.im>

Marudny added a commit to status-im/infra-role-beacon-node-linux that referenced this pull request Jul 18, 2022

add support for payload_builder option (#11)

3841d94

Refers: status-im/nimbus-eth2#3883 Signed-off-by: Artur Marud <artur@status.im>

Marudny pushed a commit to status-im/infra-nimbus that referenced this pull request Jul 18, 2022

sepolia: implement and configure "payload_builder" feature for node-02

74d3f0a

Refers: status-im/nimbus-eth2#3883 Signed-off-by: Artur Marud <artur@status.im>

Marudny added a commit to status-im/infra-nimbus that referenced this pull request Jul 18, 2022

sepolia: implement and configure "payload_builder" feature for `node-…

65d8f36

…02` (#118) Refers: status-im/nimbus-eth2#3883 Signed-off-by: Artur Marud <artur@status.im>

tersec force-pushed the mev-register-validators branch from 20db6ba to f9d05a8 Compare July 19, 2022 07:19

tersec changed the title ~~MEV validator registration~~ MEV validator registration and blinded payload retrieval Jul 19, 2022

tersec changed the title ~~MEV validator registration and blinded payload retrieval~~ MEV block proposal Jul 21, 2022

tersec force-pushed the mev-register-validators branch from 038bd4e to 6381cb8 Compare July 21, 2022 16:54

zah reviewed Jul 21, 2022

View reviewed changes

tersec force-pushed the mev-register-validators branch 7 times, most recently from be06b87 to 4a19a9c Compare July 29, 2022 08:10

tersec force-pushed the mev-register-validators branch 2 times, most recently from a0940ba to b1e963e Compare July 29, 2022 10:16

tersec added 6 commits July 29, 2022 11:37

MEV validator registration

0d7bbcd

add nearby canary to detect new beacon chain forks

0953e4f

remove special MEV graffiti

68438ae

web3signer support

44277bf

fix trace logging

77acbd8

Nim 1.2 needs raises Defect

13e5b7b

tersec added 4 commits July 29, 2022 11:38

use template rather than proc in REST JSON parsing

7a243f8

use --payload-builder-enable and --payload-builder-url

dc667f7

explicitly default MEV to disabled

6c98c75

explicitly empty default value for payload builder URL

3793103

tersec force-pushed the mev-register-validators branch from 2a45e1d to 3793103 Compare July 29, 2022 11:38

revert attestation pool to unstable version

ae32a6f

zah reviewed Aug 1, 2022

View reviewed changes

zah merged commit d62d13a into unstable Aug 1, 2022

zah deleted the mev-register-validators branch August 1, 2022 06:41

tersec mentioned this pull request Aug 1, 2022

Implement the required changes for interoperability with mev-boost #3200

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

MEV block proposal #3883

MEV block proposal #3883

tersec commented Jul 17, 2022 •

edited

Loading

github-actions bot commented Jul 17, 2022 •

edited

Loading

zah Jul 21, 2022

tersec Jul 22, 2022

tersec Jul 22, 2022

zah Jul 25, 2022

tersec Jul 27, 2022

zah Jul 21, 2022

tersec Jul 28, 2022

zah Aug 1, 2022 •

edited

Loading

zah Aug 1, 2022

etan-status Aug 2, 2022


		return ok validatorRegistration

		proc registerValidators(node: BeaconNode) {.async.} =

MEV block proposal #3883

MEV block proposal #3883

Conversation

tersec commented Jul 17, 2022 • edited Loading

github-actions bot commented Jul 17, 2022 • edited Loading

Unit Test Results

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Sepolia testnet

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

zah Aug 1, 2022 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

tersec commented Jul 17, 2022 •

edited

Loading

github-actions bot commented Jul 17, 2022 •

edited

Loading

zah Aug 1, 2022 •

edited

Loading