Skip to content

Conversation

@spencer-tb
Copy link
Contributor

🗒️ Description

Enables BPO configs for forks that occur after them such as Amsterdam. This aims to fix the blob related tests on the most recent BAL run on Hive CI: https://hive.ethpandaops.io/#/group/bal

As Amsterdam is based on top of Osaka within forks.py, the BPO hive variables are not included. This leads to chain spec issues for EL clients due to there ordering of forks, when all the timestamps are set to 0. This PR fixes the latter.

Additionally adds BPO5 and a transition fork from BPO5 to Amsterdam. This goes off of the assumption that we will have 5 BPOs on Osaka mainnet which is currently planned. Note BPO5 is added here for chain spec issues in EL clients.

🔗 Related Issues or PRs

N/A.

✅ Checklist

  • All: Ran fast tox checks to avoid unnecessary CI fails, see also Code Standards and Enabling Pre-commit Checks:
    uvx tox -e static
  • All: PR title adheres to the repo standard - it will be used as the squash commit message and should start type(scope):.
  • All: Considered adding an entry to CHANGELOG.md.
  • All: Considered updating the online docs in the ./docs/ directory.
  • All: Set appropriate labels for the changes (only maintainers can apply labels).

@spencer-tb spencer-tb added C-bug Category: this is a bug, deviation, or other problem A-test-cli-consume Area: Tests Consume CLI—pytest command (eg. `p/t/s/e/cli/pytest_commands/consume.py`) labels Nov 5, 2025
@spencer-tb spencer-tb marked this pull request as draft November 5, 2025 16:25
@spencer-tb
Copy link
Contributor Author

Set to draft for now as would like to wait for other EL to respond in STEEL discord.

@fselmo
Copy link
Contributor

fselmo commented Nov 6, 2025

These look like good changes to me. We can wait until there's consensus here but looks good on my end.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-test-cli-consume Area: Tests Consume CLI—pytest command (eg. `p/t/s/e/cli/pytest_commands/consume.py`) C-bug Category: this is a bug, deviation, or other problem

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants