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

Merge bug: besu producing zero transaction blocks #3890

Closed
garyschulte opened this issue May 24, 2022 · 6 comments · Fixed by #3944 or #4186
Closed

Merge bug: besu producing zero transaction blocks #3890

garyschulte opened this issue May 24, 2022 · 6 comments · Fixed by #3944 or #4186
Assignees
Labels
bug Something isn't working mainnet

Comments

@garyschulte
Copy link
Contributor

garyschulte commented May 24, 2022

Description

besu/consensus client combinations on mainnet-shadowfork-5 are producing blocks with zero transactions.

metrics indicate that besu has stopped seeing new transactions, possibly since the merge.

Steps to Reproduce (Bug)

  1. log into msnf5 node, observe transaction pool depth of 0
@jflo jflo added bug Something isn't working mainnet labels May 25, 2022
@garyschulte garyschulte self-assigned this May 25, 2022
@garyschulte
Copy link
Contributor Author

This appears to be a problem that occurs at merge transition that prevents remote transactions from getting added to besu's transaction pool. Short term, this has a very simple mitigation: just restart besu after the merge and remote transactions make it into the txpool just fine.

@jflo jflo self-assigned this Jun 4, 2022
@jflo
Copy link
Contributor

jflo commented Jun 4, 2022

We know why Besu has been proposing empty blocks after hitting TTD. It turns out, that incoming transactions cannot be added to the transaction pool unless besu is synced within 100 blocks of its best peer. When one of those peers is a vanilla mainnet peer, and besu is on a shadowfork, that can be a problem.

@jflo
Copy link
Contributor

jflo commented Jun 16, 2022

It seems this functionality has caused about a 50% reduction in the number of peers that Besu can maintain on Ropsten. Reopening to investigate and address.

@jflo jflo reopened this Jun 16, 2022
@jflo
Copy link
Contributor

jflo commented Jul 1, 2022

PR 3944 above which addressed this has been reverted.

@non-fungible-nelson
Copy link
Contributor

Will test the combination of fixes with the Goerli shadow forks. @garyschulte and @jflo to confirm or close after.

@jflo
Copy link
Contributor

jflo commented Jul 27, 2022

Current case can be seen on Goerli Shadowfork 5. The Prysm(bonsai) and Teku (bonsai) instances are the only ones that are submitting proposals with transactions, and has txs in their pools. They both had been restarted at some point more than 13 hours after TTD.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working mainnet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants