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

Fix non-sequential forkid #946

Merged
merged 1 commit into from
Aug 12, 2024
Merged

Conversation

cffls
Copy link

@cffls cffls commented Aug 9, 2024

Without this change, sequencer won't be able to process initial injected batch with forkid12 correctly with the log similar to below:

[cdk-erigon-sequencer-001] [INFO] [08-09|16:54:14.397] [5/13 Execution] Starting sequencing stage
[cdk-erigon-sequencer-001] [INFO] [08-09|16:54:14.397] Upgrading fork id                        from=0 to=12 batch=1
[cdk-erigon-sequencer-001] [EROR] [08-09|16:54:14.397] [5/13 Execution] Error setting fork id 12 to block 1
[cdk-erigon-sequencer-001] [INFO] [08-09|16:54:14.397] [5/13 Execution] Finished sequencing stage
[cdk-erigon-sequencer-001] [EROR] [08-09|16:54:14.397] Error while executing stage              err="[5/13 Execution] unknown fork id number 11"
[cdk-erigon-sequencer-001] [EROR] [08-09|16:54:14.397] Staged Sync                              err="[5/13 Execution] unknown fork id number 11"
[cdk-erigon-sequencer-001] [INFO] [08-09|16:54:14.898] [5/13 Execution] Starting sequencing stage

@cla-bot cla-bot bot added the cla-signed label Aug 9, 2024
Copy link

sonarqubecloud bot commented Aug 9, 2024

@mandrigin mandrigin merged commit 1da0c44 into 0xPolygonHermez:zkevm Aug 12, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants