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

Deploying L2 contracts is super slow and makes for bad devex #68

Closed
samlaf opened this issue Sep 4, 2024 · 2 comments · Fixed by #74
Closed

Deploying L2 contracts is super slow and makes for bad devex #68

samlaf opened this issue Sep 4, 2024 · 2 comments · Fixed by #74

Comments

@samlaf
Copy link

samlaf commented Sep 4, 2024

I've been playing around with the devnet trying to add features to it, but the L2 deployment step is super slow and makes for a pretty bad dev experience (https://xkcd.com/303/)

image

Not super sure what the culprit is here. I noticed that the docker-compose devnet uses op-node genesis l1 to prepare an L1 genesis file, but the kurtosis devnet does not. But that probably under the hood just deploys the contracts using forge and then builds the genesis file...?

Otherwise would it just be the fact that the deployment is done inside a docker container as opposed to using forge subcommand execution? I have noticed in the past that forge deploys from within a docker container were much much slower than using native forge, but I'm unsure why.

Any other reasons that could explain the discrepancy?

@mslipper
Copy link
Collaborator

mslipper commented Sep 4, 2024

Hey Sam!

I'm working on some improvements to the deployment setup which should make things faster. The Kurtosis package wraps some poorly maintained OP Labs Docker images, and I'll be replacing them with a more modular and performant setup.

@samlaf
Copy link
Author

samlaf commented Sep 4, 2024

Awesome looking forward!

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 a pull request may close this issue.

2 participants