Skip to content

feat: spin up full mev - #97

Closed
h4ck3rk3y wants to merge 66 commits into
mainfrom
gyani/full-mev
Closed

feat: spin up full mev#97
h4ck3rk3y wants to merge 66 commits into
mainfrom
gyani/full-mev

Conversation

@h4ck3rk3y

@h4ck3rk3y h4ck3rk3y commented Jun 28, 2023

Copy link
Copy Markdown
Collaborator

To run this do

kurtosis run github.com/kurtosis-tech/eth2-package@gyani/full-mev '{"mev_type": "full"}'

@h4ck3rk3y
h4ck3rk3y marked this pull request as draft June 28, 2023 15:02
Comment thread .circleci/config.yml Outdated
@h4ck3rk3y
h4ck3rk3y requested a review from parithosh July 6, 2023 15:10
@h4ck3rk3y
h4ck3rk3y marked this pull request as ready for review July 6, 2023 15:10
Comment thread src/mev_flood/mev_flood_launcher.star Outdated
@h4ck3rk3y
h4ck3rk3y force-pushed the gyani/full-mev branch 2 times, most recently from 2dd5604 to 5d9b314 Compare July 7, 2023 14:46

mev_url = "http://{0}{1}:{2}".format(mev_prefix, num_participants, mev_port)

mev_participant = {

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this should be behind a massive if condition for mev type

@kamilchodola

Copy link
Copy Markdown
Contributor

Hi @h4ck3rk3y !

This is interesting PR and I'm checking it's capabilities and have a question:
Will tx-fuzzer also be included in transactions included for MEV? Or only MEV-flood will be responsible for creating txs included for MEV?

@kamilchodola

Copy link
Copy Markdown
Contributor

Also I just started entire kurtosis with command from first message:

kurtosis run github.com/kurtosis-tech/eth2-package@gyani/full-mev '{"mev_type": "full"}'
And in Flashbots boost relay I don't see any payloads delivered - shouldn't here I have all proposed blocks?

@h4ck3rk3y

Copy link
Copy Markdown
Collaborator Author

Hey @kamilchodola ! Thanks for your comments this is inspiring :) Wonder how you found Kurtosis.

mev-flood itself should be enough to create the required transactions but I am launching tx-fuzz just in case. Still hitting some edge cases with this hence this hasn't been merged yet.

flashbots/builder#89 <- have an ongoing issue

Also discussing this with people from Flashbots & Ethereum in a separate discord. We discovered that I wasn't setting the BUILDER_TX_SIGNING_KEY; got a bit further but the work is still ongoing

@kamilchodola

kamilchodola commented Jul 31, 2023

Copy link
Copy Markdown
Contributor

@h4ck3rk3y Thanks for those answers! Will be following those issues as well :)

I'm working in one of EF clients team and checking Kurtosis on how it works and exploring potential features and use-cases :)

In this case the call to relay is failing with signature checks!
Only see it on second node of mev boost
@h4ck3rk3y

Copy link
Copy Markdown
Collaborator Author

@kamilchodola We have a working full MEV setup. This PR works but this is dated. Use #115; which allows any size networks

@h4ck3rk3y h4ck3rk3y mentioned this pull request Aug 9, 2023
@h4ck3rk3y

Copy link
Copy Markdown
Collaborator Author

Closing in favor of #115

@h4ck3rk3y h4ck3rk3y closed this Aug 9, 2023
@barnabasbusa
barnabasbusa deleted the gyani/full-mev branch November 30, 2023 11:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants