Skip to content

Conversation

raxhvl
Copy link
Member

@raxhvl raxhvl commented Sep 30, 2025

Adds amsterdam fork. This is required for testing out EIP-7928 Block Access List.

I'm not too sure about this:

setTime("HIVE_OSAKA_TIMESTAMP", cfg.OsakaTime)
setTime("HIVE_AMSTERDAM_TIMESTAMP", cfg.AmsterdamTime)

@raxhvl
Copy link
Member Author

raxhvl commented Oct 8, 2025

Hey @marioevz I have also added a flag to disable parallel txs in besu as suggested

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think that adding the dummy file on line 104 in this file:

blocks="$blocks dummy"

is not necessary anymore.

Copy link

@mirgee mirgee Oct 10, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh - it seems that it actually still is necessary - without it, negative scenarios where block should fail to import cause the client to terminate, but the tests rely on the client keep serving RPC requests after the import, resulting in a failing test which might pass otherwise.

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.

3 participants