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

simulators/eth2/dencun: Fixes to builder tests #1028

Merged
merged 4 commits into from
Mar 5, 2024

Conversation

marioevz
Copy link
Member

@marioevz marioevz commented Mar 5, 2024

Changes included

  • Builder tests now wait until all clients' builders have proposed at least one block with blobs, because some tests were erroneously failing during the verification phase.
  • Bumped the Dencun fork epoch on the builder tests for clients that do not require finalization, since nimbus was now taking one epoch before starting to request payloads from the builder, which resulted in a test fail.
  • Use fix in simulators/ethereum/engine: TransactionCreator - Keep track of blobs #1030 to keep track of the blobs in the blob spammer used by all eth2/dencun tests, which resulted in the blob KZG cache not being used due to the blob IDs being too high

@marioevz marioevz merged commit 80304c6 into ethereum:master Mar 5, 2024
6 checks passed
@marioevz marioevz deleted the dencun-fixes branch March 5, 2024 23:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant