-
Notifications
You must be signed in to change notification settings - Fork 6
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
feat: sovereign integration #88
Conversation
96596f7
to
a139b63
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you mean to update solidity-ibc-eureka
submodule? Does the existing rollkit demo work (up to step 2)?
Do we need any FLUP issues to track testing the Sovereign image and adapting the demo to use it? |
Co-authored-by: Rootul P <[email protected]>
I feel like this issue should uncover more #97 |
@ninabarbakadze Did you mean to update solidity-ibc-eureka submodule? I'm bumping it in a separate PR but it involves bumping Cosmos SDK so we prob want to revert the bump in this PR. |
yeah I'll revert the bump. It updated on its own. I'm going to convert this to draft again because i'm adding a few things. |
update: the integration seems to be correct but i'm hitting an internal panic when trying to deploy contracts. likely an issue not related to the setup. sovereign is helping debug. |
|
The commit in your screenshot differs from the commit on main. We're currently on https://github.com/cosmos/solidity-ibc-eureka/tree/febaabb6915eccfd3e1922793bc0936cd0b4fdfb To resolve:
|
after updating my main it now shows |
merging main fixed. |
Yes. Most recently bumped in #67 |
@@ -0,0 +1,58 @@ | |||
[da] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[nit] can we rename this file to sovereign_rollup_config.toml
or put it in a directory that makes it obvious it's for sovereign?
@@ -0,0 +1,16 @@ | |||
{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[question] why the directory /celestia/
in this file's path? Seems like these accounts are on the Sovereign roll-up based on the addresses.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I followed their structure. They use other DA solutions too so this is probably to indicate Sov SDK rollup + Celestia DA
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we can just make it sov genesis
Co-authored-by: Rootul P <[email protected]>
Overview
Sov SDK rollup is up and running with an EVM module. I'll add more docs once I start testing it with our existing scripts. The demo still primarily uses RollKit.
Fixes #76
Running Sov SDK rollup
Requires building the image locally. Once the image is battle tested i'll push it to ghcr.