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

Optimisze coinbase address during recovery for the sequencer #1043

Merged
merged 4 commits into from
Aug 29, 2024

Conversation

zjg555543
Copy link

  • It's preferable to use batchState.getCoinbase to obtain the coinbase address.
  • The coinbase address has changed in X Layer testnet history. While recovering the sequencer data using the following steps, a mismatch issue may occur.
# Step 1
./build/bin/cdk-erigon --zkevm.sync-limit=12524591 --config="xlayerconfig-testnet.yaml"

# Step 2
CDK_ERIGON_SEQUENCER=1 ./build/bin/cdk-erigon --zkevm.l1-sync-start-block=4648290 --config="xlayerconfig-testnet.yaml"

@cla-bot cla-bot bot added the cla-signed label Aug 28, 2024
revitteth
revitteth previously approved these changes Aug 28, 2024
Copy link

sonarcloud bot commented Aug 29, 2024

@kstoykov kstoykov merged commit 19a54a2 into 0xPolygonHermez:zkevm Aug 29, 2024
6 checks passed
Stefan-Ethernal pushed a commit that referenced this pull request Sep 20, 2024
kstoykov pushed a commit that referenced this pull request Sep 25, 2024
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.

3 participants