feat: EigenDA M0 data availability client #3041
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Add blob id to batch * Update submodule * Add script for blob commitments in L1 * Clean up script * Address PR comments * Add blobs to file * Add missing change * Remove yq requirement * Address PR comments
feat: EigenDA Toolbox
* add more metrics * eigenda integration docs * changes to integration doc * remove blob retriever
* Fix get blobs l1 * Remove logs
* initial commit * use Notify for a more deterministic approach * replace atomic for mutex * move const to config
* initial commit * add more steps * add backup and restore ecosystem scripts * remove unnecessary step * improve docs * fix docs * fix the fix docs * add extra step * fix restore path * simplify restoration note * more docs * fix paths in backup restoration * fix whitespace * replacement fixes * moved holesky rpc url to env var
chore: Eigen DA Merge Main
* Remove unneeded formatting * Add script explanations * Remove observability changes
* Add initial implementation disperser client * Add holesky tests * Add error handling * Remove proxy from name * Add new configs * Update eigenda-integration.md * Address pr comments * initial commit * add conditional compilation attribute to test * remove unused imports * improve err * remove unwraps * implement `IntoResponse` for `RequestProcessorError` * use a single `MemStoreConfig` * add new step * change suggested api_node_url * memstore integration * remove comments & fix memstore test * fix memstore config * initial commit * remove unused imports * modularize code * non auth: wait for dispersal * auth: wait for dispersal * add config for auth dispersal * implement get blob data for remote disperser * remove unwraps, improve tests * remove eigenda_proxy layer * remove field from cfg & update readme * add padding before dispersal request * remove unwrap * remove proxy mention from integration doc * Fix pr comments --------- Co-authored-by: Gianbelinche <39842759+gianbelinche@users.noreply.github.com>
…evert-changes fix(da-eigen-implementation-m0): revert incorrect changes
juan518munoz
requested review from
yorik,
alexandrst88,
artmakh,
hatemosphere,
onyxet,
otani88,
iluwaa and
zksync-admin-bot2
|
Hello folks. What's the status of this PR? Any path to integration? cc: @dimazhornyk |
|
This PR must've been abandoned in favor of #3243. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What ❔
M0: Read and Write integration
Scope: Spin up a local EigenDA dev environment, spin up a local zksync-era dev environment. Instead of sending blobs to 4844, the zksync-era sends blobs to EigenDA. EigenDA provides a high level client called eigenda-proxy, and it should be used. On L1, mock the verification logics, such that blocks continue building. Increase the blob size from 4844 size to 2MiB blob.
Integration docs available on eigenda-integration.md
PR should be merged alongside contract changes
This PR also adds a concurrent optimization for the da dispatcher.
Why ❔
EigenDA M0 implementation based on Trusted Verification Strategy (M0)
Checklist
zk_supervisor fmtandzk_supervisor lint.