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

WIP feat(Dockerfile): Add reth to container image so we can run both in same container for kurtosis. #200

Draft
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

ryanschneider
Copy link
Contributor

📝 Summary

This is a very quick work-around for #199 mainly to unblock @parithosh so he can work on the entrypoint script to start and monitor both binaries. I used a recent ethpandaops/reth container

Definitely at this point still a draft PR.

At a minimum this still needs:

  • figure out how better to parameterize which reth image to use as the builder container source.
  • add an ENTRYPOINT

💡 Motivation and Context

The EF devops and reth teams are trying to get rbuilder working in kurtosis for pectra devnet-4


✅ I have completed the following steps:

  • Run make lint
  • Run make test
  • Added tests (if applicable)

Copy link

Benchmark results for 69f2c25

Report: https://flashbots-rbuilder-ci-stats.s3.us-east-2.amazonaws.com/benchmark/69f2c25-c886cf1/report/index.html

Date (UTC) 2024-09-30T23:43:04+00:00
Commit 69f2c25800ed6c33864364fd10673f7f568ae7a0
Base SHA c886cf1278e8f12e459adaeaef5f58b12515e8ee

Significant changes

None

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.

1 participant