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

Polygon POS Problem #13493

Open
sidohin-felix opened this issue Jan 18, 2025 · 4 comments · May be fixed by #13520
Open

Polygon POS Problem #13493

sidohin-felix opened this issue Jan 18, 2025 · 4 comments · May be fixed by #13520
Labels

Comments

@sidohin-felix
Copy link

sidohin-felix commented Jan 18, 2025

When launching the latest build of erigon built from main the following happens when syncing with the polygon pos flags enabled after a certain time.

pos sync store failed: nonsequential block in bridge processing: 66842880 != 66842848"

This leads to erigon to crash and restarting it works for a few hours then the error comes back again.

Any suggestions on this?

All I found is the following in the source:

expectedNextBlockNum := lastProcessedBlockInfo.BlockNum + s.borConfig.CalculateSprintLength(blockNum)
		if blockNum != expectedNextBlockNum {
			return fmt.Errorf("nonsequential block in bridge processing: %d != %d", blockNum, expectedNextBlockNum)
		}

I am not exactly sure how this works for Polygon, but my guess would be it might that we should have <= rather than not equals since we were faster.

@taratorio
Copy link
Member

taratorio commented Jan 18, 2025

@sidohin-felix we released a new flow for polygon pos for preparation for beta which requires re-sync from scratch (most preferable). Alternatively you can try stopping erigon, then rm -rf datadir/chaindata && rm -rf datadir/heimdall && rm -rf datadir/polygon-bridge and starting again - if that doesn't help please try re-syncing.

@sidohin-felix
Copy link
Author

sidohin-felix commented Jan 18, 2025

@taratorio so I tried re-synching a few times before but the error always comes back, I also thought this was related but to no avail. I applied my strange suggestion but I used a strict less than and for now everything seems stable (even though I think the solution is wrong). I can always revert to the original code and try the the rm -rf solution...

@taratorio
Copy link
Member

taratorio commented Jan 18, 2025

@sidohin-felix please upload your full logs file with --log.console.verbosity=debug without your change with latest main once you get the error again. We haven't seen this error in our tests. The check is correct as it is and if you are seeing an error it means that something else is going wrong (for which I need your entire log file with debug verbosity to troubleshoot).

@sidohin-felix
Copy link
Author

Ok I will revert to the original code and set to the log level - and yes it crashed again but with a different error. I will also provide all the build details.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants