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

experimental engine of opbnb panics for the static file mismatching #10448

Closed
1 task done
yutianwu opened this issue Aug 22, 2024 · 3 comments
Closed
1 task done

experimental engine of opbnb panics for the static file mismatching #10448

yutianwu opened this issue Aug 22, 2024 · 3 comments
Labels
C-bug An unexpected or incorrect behavior S-needs-triage This issue needs to be labelled

Comments

@yutianwu
Copy link
Contributor

yutianwu commented Aug 22, 2024

Describe the bug

When we switch to the experimental engine of opBNB after the syncing of the old version binary, it will complain that the static file mismatch as the following log:

2024-08-22T08:31:44.287182Z ERROR Persistence service failed err=ProviderError(UnexpectedStaticFileBlockNumber(Headers, 37436756, 37436640))
2024-08-22T08:31:44.288659Z ERROR Advancing persistence failed err=channel closed
2024-08-22T08:31:44.288708Z  INFO Block added to canonical chain number=37436760 hash=0xf93062a96aabf8255493af4c5d110f6519a619a79be08012063e3b12b7a99806 peers=0 txs=1 gas=49.44 Kgas gas_throughput=18.80 Mgas/second full=0.0% base_fee=0.00gwei blobs=0 excess_blobs=0 elapsed=2.629476ms
2024-08-22T08:31:44.288816Z ERROR Fatal error
2024-08-22T08:31:44.288849Z ERROR Fatal error in consensus engine
2024-08-22T08:31:44.288914Z ERROR shutting down due to error

Actually, block 37436755 has already been committed but has not been synced to static files. It will be synced to static files by the static producer hook in the old version.

The workaround solution is to unwind the blockchain to block 37436640 and then the experimental engine will work.

I am unsure if it is a bug or our issue when merging the code, you can refer to bnb-chain/reth#107.

Steps to reproduce

  1. update the binary
  2. check the log
2024-08-22T08:31:44.287182Z ERROR Persistence service failed err=ProviderError(UnexpectedStaticFileBlockNumber(Headers, 37436756, 37436640))
2024-08-22T08:31:44.288659Z ERROR Advancing persistence failed err=channel closed
2024-08-22T08:31:44.288708Z  INFO Block added to canonical chain number=37436760 hash=0xf93062a96aabf8255493af4c5d110f6519a619a79be08012063e3b12b7a99806 peers=0 txs=1 gas=49.44 Kgas gas_throughput=18.80 Mgas/second full=0.0% base_fee=0.00gwei blobs=0 excess_blobs=0 elapsed=2.629476ms
2024-08-22T08:31:44.288816Z ERROR Fatal error
2024-08-22T08:31:44.288849Z ERROR Fatal error in consensus engine
2024-08-22T08:31:44.288914Z ERROR shutting down due to error

Node logs

2024-08-22T08:31:44.287182Z ERROR Persistence service failed err=ProviderError(UnexpectedStaticFileBlockNumber(Headers, 37436756, 37436640))
2024-08-22T08:31:44.288659Z ERROR Advancing persistence failed err=channel closed
2024-08-22T08:31:44.288708Z  INFO Block added to canonical chain number=37436760 hash=0xf93062a96aabf8255493af4c5d110f6519a619a79be08012063e3b12b7a99806 peers=0 txs=1 gas=49.44 Kgas gas_throughput=18.80 Mgas/second full=0.0% base_fee=0.00gwei blobs=0 excess_blobs=0 elapsed=2.629476ms
2024-08-22T08:31:44.288816Z ERROR Fatal error
2024-08-22T08:31:44.288849Z ERROR Fatal error in consensus engine
2024-08-22T08:31:44.288914Z ERROR shutting down due to error

Platform(s)

Linux (x86)

What version/commit are you on?

v1.0.5

https://github.com/bnb-chain/reth/tree/merge-v1.0.5

What database version are you on?

Current database version: 2
Local database version: 2

Which chain / network are you on?

opBNB

What type of node are you running?

Archive (default)

What prune config do you use, if any?

No response

If you've built Reth from source, provide the full command you used

No response

Code of Conduct

  • I agree to follow the Code of Conduct
@yutianwu yutianwu added C-bug An unexpected or incorrect behavior S-needs-triage This issue needs to be labelled labels Aug 22, 2024
@mattsse
Copy link
Collaborator

mattsse commented Aug 22, 2024

this has since been fixed right @rkrasiuk @Rjected ?

@joshieDo
Copy link
Collaborator

should be solved by #10299

@yutianwu
Copy link
Contributor Author

thanks for u help, will close this issue

@github-project-automation github-project-automation bot moved this from Todo to Done in Reth Tracker Aug 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-bug An unexpected or incorrect behavior S-needs-triage This issue needs to be labelled
Projects
Archived in project
Development

No branches or pull requests

3 participants