Skip to content

Celestia integration #68

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.

Sign up for GitHub

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

Jump to bottom
Draft
Sneh1999 wants to merge 9 commits into
base: integration
Choose a base branch
from
Draft

Celestia integration #68

Sneh1999 wants to merge 9 commits into from

Conversation

@Sneh1999
Copy link

@Sneh1999 Sneh1999 commented Nov 24, 2024 •
edited
Loading

This PR outlines the difference in our celestia-integration vs the integration branch.

Sneh1999
Sneh1999 commented Nov 24, 2024
View reviewed changes
.github/workflows/smoke-test.yml
[
./smoke-test.bash,
./smoke-test-l3.bash,
./smoke-test-nitro-simple.bash,
Copy link
Author

@Sneh1999 Sneh1999 Nov 24, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we dont need all these tests for celestia integration

@ImJeremyHe
Copy link
Member

ImJeremyHe commented Nov 25, 2024

Thanks! @Sneh1999
I was planning to do this today -_-|||

@Sneh1999
Copy link
Author

Sneh1999 commented Nov 25, 2024

Thanks! @Sneh1999 I was planning to do this today -_-|||

Sorry @ImJeremyHe, we had to get a working demo ready for caldera and I had to test our existing code. There are still a lot of improvements we can do to this lol

@Sneh1999 Sneh1999 mentioned this pull request Nov 25, 2024
Closed
2 tasks
ImJeremyHe and others added 7 commits December 2, 2024 15:22
@ImJeremyHe
Fix the batch poster
cff153e 
In this commit, a dummy TEE Verifier address is being used, we will
use a correct one later
@ImJeremyHe
Merge pull request #71 from EspressoSystems/jh/fix
da587f3 
Fix the batch poster
@zacshowa
Update migration test for TEE integration (#73)
947fb44 
* Update migration test for TEE integration

This commit updates the migration-test.bash script to reflect the new
migration flow for the TEE integration. It also adds some utility
scripts and updates some exsisting example files to assist in the
migration test.

* Remove changes to docker-compose.yml

* Respond to review feedback
@ImJeremyHe @Sneh1999
Sync with celestia integration (#88)
dbf800a 
* Update espresso image to point to integration (#83)

* Update espresso image to point to integration

* update espresso dev node

* Update light client address

---------

Co-authored-by: Jeremy <297323986@qq.com>

* add blobSchedule for new genesis creator requirement (#84)

* add blobSchedule for new genesis creator requirement

* Rename

(cherry picked from commit 01b9f3a)

* Fix caff node test (#85)

* WIP

* WIP

* small fixes

* rename to caff

* fix

* use bc for big numbers

* Send transaction through caff node

---------

Co-authored-by: Sneh Koul <snehkoul1999@gmail.com>
(cherry picked from commit bfaaf1d)

* Add EspressoTEEVerifier address (#86)

* Add EspressoTEEVerifier address

* Fix the TEE address

---------

Co-authored-by: Jeremy <297323986@qq.com>
(cherry picked from commit b1ce7e1)

* Fix caff node test

(cherry picked from commit 589b44b)

* Remove simple test

---------

Co-authored-by: Sneh Koul <35871990+Sneh1999@users.noreply.github.com>
@zacshowa
Celestia 2.1.3 migration (#94)
eed7e48 
* create celestia-migration-test script

* work on migration test for celestia-2.1.3

* Update to celestia-migration-test to start on espresso contracts

* update orbit-actions submodule

* add celestia-migration-test.bash and associated .env file

* fix orbit-actions subodule commit

* Rename celestia-2.1.3-migration-test.bash script
@jjeangal @Sneh1999 @ImJeremyHe
Update Testnode for Celestia (#101)
acb4ee1 
* Fix caff node test (#100)

* Fix caff node test

* comment out l3

* fix launch bash

* fix caff node test

* cleanup

* fix smoke test

* fix migration test

* fix l3 test

* comment out l3 test

* Restart sequencer when it hangs

---------

Co-authored-by: Jeremy <yiliang.he@qq.com>

* update celestia contracts version

* add migration ci check

* fix docker file

* Fix l3 test

* Fix caff node

* Correct the configuration

---------

Co-authored-by: Sneh Koul <35871990+Sneh1999@users.noreply.github.com>
Co-authored-by: Jeremy <yiliang.he@qq.com>
@jjeangal @Ferret-san
@gvladika @tsahee @Tristan-Wilson @eljobe @gzeoneth @yahgwai @PlasmaPower @hkalodner @joshuacolvin0 @Sneh1999 @ImJeremyHe
Update testnode to support Celestia v3.5.6 (#104)
57760b8 
* Update config.ts

* Change defaults and add Celestia DA Server

* Do not transfer all supply to L2, leave some for L1-L3 bridging test

* Bump nitro contracts

* Disable rebuild of dev docker imgs w --build false

* Make UpgradeExecutor L3 arbowner

* Clarify logic around dev builds and rebuilding

The concepts of using the dev builds for nitro and blockscout and
rebuilding are now split into different variables internally to the
test-node.bash script and can be controlled independently.

The --dev flag by itself means to use the dev builds and to rebuild
them. --no-build-dev-nitro and -no-build-dev-blockscout disable
rebuilding the respective dev images. --no-build disables rebuilding the
dev images and also rebuilding contracts related images, and the node
images. This finer grained control saves time in the development cycle
by allowing the user to rebuild only what is needed.

An example command line to reinitialize the blockchain, use the dev
images, but not rebuild anything except blockscout would be:
./test-node.bash --init --dev --no-build --build-dev-blockscout --blockscout

Flags are read from left to right; flags to the right cancel
the effects of flags to the left.

* No skip_build

* Add comment

* Contracts and node images should rebuild on --init

* Declare variable before use for -u flag

* Change contracts to utils, add --force-build-utils

* Upgrade to the consensus-v31 release

* Update versions of contracts token-bridge and node

These updated versions are needed to work with the consensus-v31
release.

* fix: prysm config

* fix: remove irrelevant changes

* fix: pos

* fix: working config

* chore: reduce slot time to speed up deployment

* fix: enable electra since genesis

* fix: use newer geth

* fix: remove --config-name=interop

* fix: disable electra

* fix: use latest geth

* fix: add fee recipient

* fix: enable deneb

* fix: wait for tx sync

* fix: format

* feat: shorter slot for faster testing

* fix: readd dev flags

* fix: readd condition

* fix: reduce diff

* fix: docker compose

* ci: test pos

* ci: fix

* ci: debug log

* fix: also use 1337 for pos setup

* Test commit

* Test passing all the args through

* Test tokenbridge and l3node startups

* Changed the runner names

* Cleanup in ci bash

* Dont test for l3 tx if no l3node

* Added l3 node comment

* Add --l2-anytrust option to run in AnyTrust mode

test-node.bash can now be run with --l2-anytrust which runs the l2
nodes in AnyTrust mode. It creates a committee of 2 daserver with
assumed-honest set to 1, requiring successful stores of the batch data
to both of them. Nitro nodes sync from a mirror daserver. All dasevers
use local file storage for the batch data on their own volumes.

BLS keys for the committee are automatically generated and the keyset is
activated automatically on the SequencerInbox contract.

* Use latest Nitro release v3.1.2

* Added some testing for l3node with custom fee token

* Rename token6

* Set lower transfer amounts

* Update to nitro v3.2.1

* Update blockscout pin to arbitrum branch

* Add clabot config

* Fix PR comments

* Update ci.yml

* Simple local script on Arabica testnet

Simple local script on Arabica testnet. Run `./celestia-start.sh`, can pass `--validate` for WASM validation and `--anytrust` to spin up a local anytrust setup in order to test fallbacks and other things

* Update .gitignore

* fix cancun and light up

* containerize and simplify

* Update docker-compose.yaml

* Update docker-compose.yaml

* Update docker-compose.yaml

* Fix caff node test (#100)

* Fix caff node test

* comment out l3

* fix launch bash

* fix caff node test

* cleanup

* fix smoke test

* fix migration test

* fix l3 test

* comment out l3 test

* Restart sequencer when it hangs

---------

Co-authored-by: Jeremy <yiliang.he@qq.com>

* update celestia contracts version

* add migration ci check

* fix docker file

* Fix l3 test

* Fix caff node

* Correct the configuration

* add flooding script

* Update Dockerfile

* update docker image

* fix image

* Remove redundancy

* update docker compose indenting

* Remove unused fallback urls

* update latest devnode image

* remove platform specification

---------

Co-authored-by: Diego <diego@celestia.org>
Co-authored-by: Goran Vladika <goran.vladika@gmail.com>
Co-authored-by: Tsahi Zidenberg <65945052+tsahee@users.noreply.github.com>
Co-authored-by: Tristan Wilson <twilson@offchainlabs.com>
Co-authored-by: Pepper Lebeck-Jobe <pepper@offchainlabs.com>
Co-authored-by: Pepper Lebeck-Jobe <eljobe@gmail.com>
Co-authored-by: gzeon <hng@offchainlabs.com>
Co-authored-by: gzeon <im@gzeon.dev>
Co-authored-by: Chris Buckland <cpbuckland88@gmail.com>
Co-authored-by: Tristan-Wilson <87238672+Tristan-Wilson@users.noreply.github.com>
Co-authored-by: Lee Bousfield <ljbousfield@gmail.com>
Co-authored-by: Tsahi Zidenberg <tsahi@offchainlabs.com>
Co-authored-by: Harry Kalodner <harry@offchainlabs.com>
Co-authored-by: Joshua Colvin <jcolvin@offchainlabs.com>
Co-authored-by: Diego <31937514+Ferret-san@users.noreply.github.com>
Co-authored-by: Sneh Koul <35871990+Sneh1999@users.noreply.github.com>
Co-authored-by: Jeremy <yiliang.he@qq.com>
ImJeremyHe pushed a commit that referenced this pull request Aug 11, 2025
@gvladika
Merge pull request #68 from OffchainLabs/bump-version
aab133a 
Bump nitro-contracts and token-bridge
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ImJeremyHe ImJeremyHe Awaiting requested review from ImJeremyHe ImJeremyHe will be requested when the pull request is marked ready for review ImJeremyHe is a code owner

@alysiahuggins alysiahuggins Awaiting requested review from alysiahuggins alysiahuggins will be requested when the pull request is marked ready for review alysiahuggins is a code owner

@zacshowa zacshowa Awaiting requested review from zacshowa zacshowa will be requested when the pull request is marked ready for review zacshowa is a code owner

@sveitser sveitser Awaiting requested review from sveitser sveitser will be requested when the pull request is marked ready for review sveitser is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@Sneh1999 @ImJeremyHe @zacshowa @jjeangal