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

Feature/rollupman ops #7

Merged
merged 3 commits into from
Aug 5, 2024

Conversation

arnaubennassar
Copy link
Collaborator

@arnaubennassar arnaubennassar commented Apr 17, 2024

add features:

  • deploy rollup manager
  • add rollup type
  • create rollup
  • deploy and setup DAC
  • mint tokens
  • approve tokens (not tested)

All of this seems to be working, I've followed the steps outlined on the example of the README and successfully run a bunch of E2E tests using the docker image generated from this repo on cdk-validium-node

There is a lot of cleaning to be done tho 😓

Important to notices

  • we still depend on zkevm-contracts to generate genesis files (it's easy to copy/paste them from that repo, and the adaptation to the node format is done automtically)
  • I beleive it's ok to use the rollupmanager from fork 7, while using consensus that are more recent (forks 8 and 9). The code is prepared to support fork 8 (although not tested). And adding fork 9 should be trivial

@arnaubennassar arnaubennassar merged commit 1fda710 into feature/deploy-rollup-manager Aug 5, 2024
@arnaubennassar arnaubennassar deleted the feature/rollupman-ops branch August 5, 2024 13:01
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