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

Magi as a sequencer #2

Open
2 tasks
dndll opened this issue Sep 7, 2023 · 1 comment
Open
2 tasks

Magi as a sequencer #2

dndll opened this issue Sep 7, 2023 · 1 comment
Labels
long term wontfix This will not be worked on

Comments

@dndll
Copy link
Collaborator

dndll commented Sep 7, 2023

Description

Determine what needs to be done to use magi as a sequencer.

Additional info

The original op-node is incredibly unreliable, with panics and null pointers in most places, and is written in go. We have a partial implementation of DA on magi over at ./op-stack/optimism-rs. The maintainers of Magi are very receptive to working together and for us to support Magi's development, especially around the DA interface issues. It has been confirmed that we shouldn't use Magi in production yet because it is new and unaudited, but for us, it would probably be easier to work with Magi since it is in rust.

The main issue is magi is missing a bunch of endpoints that the sequencer/batcher actors need, whilst it is able to run as a non-sequencer, it needs a few endpoints to be able to work well. So far, the endpoints I found are:

  • optimism_rollupConfig
  • optimism_sync

Additionally, this work might mitigate any headaches we have with go -> rust as we build the client for the DA layer, since the current client is written in go(./op-stack/openrpc) and is mostly untested.

@dndll dndll added the wontfix This will not be worked on label Nov 15, 2023
@dndll
Copy link
Collaborator Author

dndll commented Nov 15, 2023

Wontfix right now, we should pick this up if we ship any op-stack DA l2s

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
long term wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

1 participant