forked from ethereum/go-ethereum
-
Notifications
You must be signed in to change notification settings - Fork 281
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(L1 follower): adjust to recent CodecV7
and contract changes
#1120
Open
jonastheis
wants to merge
78
commits into
develop
Choose a base branch
from
jt/adjust-to-codecv7-changes
base: develop
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 77 commits
Commits
Show all changes
78 commits
Select commit
Hold shift + click to select a range
ea5a603
port changes from #1013
jonastheis b30006f
port changes from #1068
jonastheis de37d47
go.mod tidy
jonastheis e34fecf
fix compile error
jonastheis 1327771
fix goimports
jonastheis b05954d
fix log
jonastheis ce8f785
address review comments
jonastheis f10c383
upgrade golang.org/x/net to 0.23.0
jonastheis cb0a90e
Merge branch 'develop' into jt/l1-follower-mode
jonastheis b918a2b
port changes from #1018
jonastheis e51182d
fix tests and linter errors
jonastheis 4e6f759
address review comments
jonastheis fd6bff3
Merge branch 'develop' into jt/l1-follower-mode-l1-reader
jonastheis ab3e873
refactor rollup sync service / verifier to use CalldataBlobSource to …
jonastheis 4ced6f2
add configuration and initialize blob clients
jonastheis 6aafa74
fix unit tests
jonastheis da81a2e
remove unused code
jonastheis 8750045
address review comments
jonastheis 2499c69
address more review comments
jonastheis fb4fe7c
implement first version of new da-codec and to handle multiple batche…
jonastheis 9bf2f25
add CommitBatchDAV7 and handle multiple commit events submitted in a …
jonastheis d222f58
Merge remote-tracking branch 'origin/develop' into jt/rollup-verifier…
jonastheis c56be0d
Merge branch 'jt/rollup-verifier-use-code-from-l1-follower' into jt/l…
jonastheis 3950e58
fix bug due to previous batch being empty when processing the first b…
jonastheis a043d2f
Allow using MPT
omerfirmak 94c0ad5
Merge remote-tracking branch 'origin/omerfirmak/mpt' into jt/rollup-v…
jonastheis 67c1866
Merge branch 'jt/rollup-verifier-use-code-from-l1-follower' into jt/l…
jonastheis 43d54cb
update to latest da-codec
jonastheis 4290d16
add field to CommittedBatchMeta to store LastL1MessageQueueHash for C…
jonastheis 574dd53
adjust rollup verifier to support CodecV7 batches
jonastheis 53b6ebf
address review comments
jonastheis ab3bedf
Merge remote-tracking branch 'origin/develop' into jt/rollup-verifier…
jonastheis 3335654
fix issues after merge
jonastheis 3e18f7f
Merge remote-tracking branch 'origin/jt/rollup-verifier-use-code-from…
jonastheis 2342335
Merge remote-tracking branch 'origin/develop' into jt/rollup-verifier…
jonastheis 7c14639
Merge remote-tracking branch 'origin/jt/rollup-verifier-use-code-from…
jonastheis 634d1f1
go mod tidy
jonastheis 0fa3743
fix unit tests
jonastheis e048f53
Merge remote-tracking branch 'origin/develop' into jt/l1-follower-mod…
jonastheis a2a68e8
update da-codec
jonastheis ca6649e
add test TestValidateBatchCodecV7
jonastheis 80976ad
go mod tidy
jonastheis 4022989
do not log error on shutdown
jonastheis d4cc897
add sanity check for version to deserialization of committedBatchMetaV7
jonastheis a2cb3d1
port changes from #1073
jonastheis 6d5af23
chore: auto version bump [bot]
Thegaram 3c21f4e
address review comments
jonastheis 0bd6eb1
add more logs
jonastheis 9139e94
disable ENRUpdater if DA sync mode is enabled
jonastheis e9154ca
exit pipeline if context is cancelled
jonastheis 8820949
correctly handle override by setting the head of the chain to the par…
jonastheis 8ba140c
fix error with genesis event being nil
jonastheis e6af5b6
Merge branch 'jt/l1-follower-mode-update-da-codec' into jt/permission…
jonastheis b794dda
Merge remote-tracking branch 'origin/develop' into jt/permissionless-…
jonastheis 438ec09
chore: auto version bump [bot]
jonastheis 91aae90
Merge remote-tracking branch 'origin/develop' into jt/permissionless-…
jonastheis b715d4f
chore: auto version bump [bot]
jonastheis 4f74920
adjust to renaming in CodecV7
jonastheis ec6df85
implement carrying forward of L1 MessageQueue index
jonastheis 4d100c1
fix issue after upgrading from old storage to new format where batchI…
jonastheis ee53754
add new RevertBatch event
jonastheis 9fc77d8
add commitBatches to be able to read calldata of CodecV7/EuclidV2 com…
jonastheis cdc4b67
Merge remote-tracking branch 'origin/develop' into jt/adjust-to-codec…
jonastheis 8d2167b
implement finding of L1 message queue height for initial batch in rec…
jonastheis 5c6a8d5
add sanity checks for computed batches from events and batch hashes g…
jonastheis 5f08a5c
update ScrollChain ABI
jonastheis 87278ff
chore: auto version bump [bot]
jonastheis 4c3c16f
remove initial batch form DAQueue
jonastheis 973bd91
go mod tidy
jonastheis ffd2c13
fix underflow bug when l1DeploymentBlock==0
jonastheis 22215e8
fix bug with wrong parentBatchHash of first batch of batches submitte…
jonastheis 5b6a6ae
update to latest da-codec
jonastheis 5935e62
address review comments
jonastheis 46a5ef3
address review comments
jonastheis 86020e4
fix bug when l1MessageV2StartIndex==0 serialized to [] (empty slice) …
jonastheis bd6f89e
chore: auto version bump [bot]
jonastheis 6c2b57c
cache go dependencies in Dockerfile.mockccc
jonastheis 8078781
Merge remote-tracking branch 'origin/develop' into jt/adjust-to-codec…
jonastheis File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -3,3 +3,5 @@ | |
build/_workspace | ||
build/_bin | ||
tests/testdata | ||
|
||
tmp/ |
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
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
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
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
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
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
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
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
Oops, something went wrong.
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.
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.
What's the behavior if a node is not updated? Fail at the EuclidV2 transition, but then recover after the node is upgraded?