feat: integrate op-succinct #486
Merged
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
Signed-off-by: Ji Hwan <jkim@polygon.technology>
Signed-off-by: Ji Hwan <jkim@polygon.technology>
Signed-off-by: Ji Hwan <jkim@polygon.technology>
Signed-off-by: Ji Hwan <jkim@polygon.technology>
jhkimqd
changed the title
jhkimqd
changed the title
Signed-off-by: Ji Hwan <jkim@polygon.technology>
vcastellm
reviewed
Feb 18, 2025
…ntegration Signed-off-by: Ji Hwan <jkim@polygon.technology>
…ntegration Signed-off-by: Ji Hwan <jkim@polygon.technology>
jhkimqd
force-pushed
the
jihwan/op-succinct-integration
branch
from
1cd73c3 to
c6cf68b
Compare
Signed-off-by: Ji Hwan <jkim@polygon.technology>
jhkimqd
force-pushed
the
jihwan/op-succinct-integration
branch
from
c6cf68b to
711c97f
Compare
Signed-off-by: Ji Hwan <jkim@polygon.technology>
Signed-off-by: Ji Hwan <jkim@polygon.technology>
Signed-off-by: Ji Hwan <jkim@polygon.technology>
jhkimqd
force-pushed
the
jihwan/op-succinct-integration
branch
from
0ed32b1 to
23377ea
Compare
Signed-off-by: Ji Hwan <jkim@polygon.technology>
jhkimqd
force-pushed
the
jihwan/op-succinct-integration
branch
from
23377ea to
35665e8
Compare
jhkimqd
changed the title
jhkimqd
force-pushed
the
jihwan/op-succinct-integration
branch
2 times, most recently
from
d0e38b9 to
b2af755
Compare
Signed-off-by: Ji Hwan <jkim@polygon.technology>
jhkimqd
force-pushed
the
jihwan/op-succinct-integration
branch
from
b2af755 to
f90e59d
Compare
Signed-off-by: Ji Hwan <jkim@polygon.technology>
Signed-off-by: Ji Hwan <jkim@polygon.technology>
Signed-off-by: Ji Hwan <jkim@polygon.technology>
praetoriansentry
requested changes
Feb 24, 2025
Signed-off-by: Ji Hwan <jkim@polygon.technology>
jhkimqd
force-pushed
the
jihwan/op-succinct-integration
branch
from
28c5fe7 to
7572b46
Compare
Signed-off-by: Ji Hwan <jkim@polygon.technology>
Signed-off-by: Ji Hwan <jkim@polygon.technology>
jhkimqd
force-pushed
the
jihwan/op-succinct-integration
branch
from
5f3b607 to
0515def
Compare
Signed-off-by: Ji Hwan <jkim@polygon.technology>
praetoriansentry
previously approved these changes
Feb 26, 2025
praetoriansentry
previously approved these changes
Feb 26, 2025
jhkimqd
force-pushed
the
jihwan/op-succinct-integration
branch
from
6d659b5 to
bf98a0d
Compare
Signed-off-by: Ji Hwan <jkim@polygon.technology>
jhkimqd
force-pushed
the
jihwan/op-succinct-integration
branch
from
bf98a0d to
5dd3379
Compare
|
praetoriansentry
approved these changes
Feb 26, 2025
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.
Description
This PR integrates OP-Succinct into Kurtosis CDK.
It relies on custom built docker images to host the below images:
On v1.2.2 of OP-Succinct.
OP-Succinct has a mock-verifier option, but the OP-Succinct stack requires a valid
agglayer_prover_sp1_keyto work and submit proofs to L1.Note how there is no deployment suffix in the op-succinct components - this is because the current implementation of OP-Succinct components rely on having fixed names (
op-succinct-proposerandop-succinct-server) and is not easily configurable at the time of this PR.Testing
Refer to the the configuration in the docs and simply run
kurtosis run --enclave=cdk --args-file=./.github/tests/chains/op-succinct.yml .Monitor the L2 finalized block number incrementing using
kurtosis service logs cdk op-succinct-proposer -f