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

Refactor PBS modules to go via the mux #242

Open
ltitanb opened this issue Jan 16, 2025 · 1 comment
Open

Refactor PBS modules to go via the mux #242

ltitanb opened this issue Jan 16, 2025 · 1 comment
Labels
discuss Needs to be discussed and/or researched pbs Pbs module / Builder API

Comments

@ltitanb
Copy link
Collaborator

ltitanb commented Jan 16, 2025

Commit boost has a PBS module that can be extended/replaced (example). Given that there can be at most one PBS module per validator, it requires significant effort for module developers to get buy-in from node operators, and it's likely that many would want to experiment with only a subset of keys initially. Now that we have the PBS mux (and considering that relays implement the Builder API just like commit boost does), we should consider allowing external PBS modules to go via the mux only, allowing NOs to experiment more easily. Effectively this means the PBS module would mux to both relays (standard MEV boost behaviour), and to other PBS modules. The new flow becomes: CL -> commit boost PBS module -> extended PBS module -> mux relays, while without an additional sidecar (so eg muxing on just relays): CL -> commit boost PBS module -> mux relays.

With this we would be able to remove the builder API indirection (routes vs mev_boost). We should also modify the init command to make it easier to spin up a mux with a given sidecar (ie adding a docker image field to the mux config)

@ltitanb ltitanb added discuss Needs to be discussed and/or researched pbs Pbs module / Builder API labels Jan 16, 2025
@ltitanb
Copy link
Collaborator Author

ltitanb commented Jan 16, 2025

This would also solve issues like #105

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discuss Needs to be discussed and/or researched pbs Pbs module / Builder API
Projects
None yet
Development

No branches or pull requests

1 participant