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

TEST ISSUE in some missing ancestor reorg tests #1135

Closed
Rjected opened this issue Jul 3, 2024 · 3 comments · Fixed by #1148
Closed

TEST ISSUE in some missing ancestor reorg tests #1135

Rjected opened this issue Jul 3, 2024 · 3 comments · Fixed by #1148
Assignees

Comments

@Rjected
Copy link
Contributor

Rjected commented Jul 3, 2024

When running:

Invalid Missing Ancestor Syncing ReOrg, GasLimit, EmptyTxs=False, CanonicalReOrg=True, Invalid P8

I get:

FAIL (Invalid Missing Ancestor Syncing ReOrg, GasLimit, EmptyTxs=False, CanonicalReOrg=True, Invalid P8): TEST ISSUE - Failed to set invalid block: triedb layer [0x5bb856b70f63d46b95dff7d2b07700194311a87c8a7494ad0de0e72de23d624b] is disk layer

We caught these when testing recent updates to our CI which should improve catching individual test regressions. I'm not really sure how to fix this, but it says TEST ISSUE and fails on reth / geth / erigon, so I assume this is a hive issue.

@Rjected
Copy link
Contributor Author

Rjected commented Aug 6, 2024

just bumping this issue as we are currently rewriting a lot of engine logic, and having coverage from these tests would be great

@marioevz
Copy link
Member

marioevz commented Aug 6, 2024

Sorry about the delay, taking a look today!

@marioevz
Copy link
Member

marioevz commented Aug 7, 2024

The problem seems to be in the geth node module we use to simulate a client delivering bad blocks through the p2p network: https://github.com/ethereum/hive/blob/master/simulators/ethereum/engine/client/node/node.go

Since the error suggests that tree leaf could not be modified since it made its way to the disk, I tried modifying the parameters for the database cache to no success:

DatabaseCache: 256,
DatabaseHandles: 256,

Will keep looking into it.

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 a pull request may close this issue.

3 participants
@Rjected @marioevz and others