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

da: use celestia as da #3

Merged
merged 1 commit into from Apr 27, 2023
Merged

da: use celestia as da #3

merged 1 commit into from Apr 27, 2023

Conversation

tuxcanfly
Copy link
Collaborator

@tuxcanfly tuxcanfly commented Jan 16, 2023

Overview

This PR refactors the batcher to submit only data pointers to the batch inbox, the data
being made available on a off chain data layer, in this case celestia.

The serialization format of the data pointer is:

--------------------
| 8 bytes | 4 bytes |
--------------------
| height  | index   |
--------------------

where height is the block height at which the actual data was included and
index is the index at which the transaction was included in the block.

(TODO: clarify if we need tx index or share index or both here)

Since the batch calldata is now just a data pointer instead of the actual data, corresponding
refactor has been made to the op-node to read the data pointer off the batch inbox address first
and then retreive the data from the data layer.

This makes the rollup more modular as it makes the data available on it's own layer while using
ethereum for settlement, thereby inheriting it's security. Celestia is the data availability
layer of choice as it provides a high guarantee of the data availability.

Checklist

  • New and updated code has appropriate documentation
  • New and updated code has new and/or updated testing
  • Required CI checks are passing
  • Visual proof for any user facing features like CLI or documentation updates
  • Linked issues closed with keywords

ops-bedrock/docker-compose.yml Outdated Show resolved Hide resolved
op-node/rollup/derive/calldata_source.go Show resolved Hide resolved
@jcstein
Copy link
Member

jcstein commented Apr 26, 2023

Hey @tuxcanfly can we use latest, so v0.9.1?

@jcstein jcstein mentioned this pull request Apr 26, 2023
@tuxcanfly tuxcanfly marked this pull request as ready for review April 27, 2023 02:06
@tuxcanfly tuxcanfly changed the base branch from develop to celestia-develop April 27, 2023 02:08
@tuxcanfly
Copy link
Collaborator Author

Going to merge this into celestia-develop branch which will track all celestia changes from now on. New PRs should be created on top of and merged into this branch for now.

@tuxcanfly tuxcanfly merged commit 05296db into celestia-develop Apr 27, 2023
1 of 2 checks passed
@jcstein jcstein deleted the celestia branch November 14, 2023 15:50
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.

None yet

2 participants