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

bug: If genesis block have an extra event the sanity check fails #154

Open
joanestebanr opened this issue Aug 24, 2024 · 0 comments
Open
Assignees

Comments

@joanestebanr
Copy link

Version: v0.6.4+cdk.7

The node check that genesis block provider have the expected event: a ForkID but It assume that must be the first event. In case that an L1InfoTreeUpdate is the first one the check fails!

trace:

{"level":"error","ts":1724428936.925428,"caller":"synchronizer/synchronizer.go:594","msg":"error getting rollupInfoByBlockRange after set the genesis. Expected ForkIDsOrder, got L1InfoTreeOrder","pid":1,"version":"0.6.4+cdk.7","stacktrace":"github.com/0xPolygonHermez/zkevm-node/synchronizer.sanityCheckForGenesisBlockRollupInfo\n\t/home/runner/work/cdk-validium-node/cdk-validium-node/synchronizer/synchronizer.go:594\ngithub.com/0xPolygonHermez/zkevm-node/synchronizer.(*ClientSynchronizer).RequestAndProcessRollupGenesisBlock\n\t/home/runner/work/cdk-validium-node/cdk-validium-node/synchronizer/synchronizer.go:566\ngithub.com/0xPolygonHermez/zkevm-node/synchronizer.(*ClientSynchronizer).processGenesis\n\t/home/runner/work/cdk-validium-node/cdk-validium-node/synchronizer/synchronizer.go:374\ngithub.com/0xPolygonHermez/zkevm-node/synchronizer.(*ClientSynchronizer).Sync\n\t/home/runner/work/cdk-validium-node/cdk-validium-node/synchronizer/synchronizer.go:412\nmain.runSynchronizer\n\t/home/runner/work/cdk-validium-node/cdk-validium-node/cmd/run.go:402"}
@joanestebanr joanestebanr self-assigned this Aug 24, 2024
joanestebanr added a commit that referenced this issue Sep 4, 2024
Cherry-pick fixes on release 0.6.4-cdk.8:
- #154: fix sanity check for genesis block
- #153: Synchronization of L1InfoTree events before genesis allows to resume
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

No branches or pull requests

1 participant