Skip to content

Commit

Permalink
chore(hive): explicitly only run prague tests via --sim.limit (#25)
Browse files Browse the repository at this point in the history
This PR shouldn't change the current behavior as
[pectra-devnet-5@v1.2.0](https://github.com/ethereum/execution-spec-tests/releases/tag/pectra-devnet-5%40v1.2.0)
only contains tests filled for `Prague` and the `CancunToPragueAt15k`
transition.

It does, however, ensure the current behavior is maintained for this
hive instance for upcoming EEST releases that contain tests filled for
previously deployed forks, which will be the case when this PR gets
merged:
- ethereum/execution-spec-tests#1053

Feel free to disregard this PR, as running tests for all forks might be
desirable 😆 here's what the impact would currently be:
- Prague-only: 4062 tests.
- Until Prague: 6525 tests.
  • Loading branch information
danceratopz authored Jan 23, 2025
1 parent 4e0ce3f commit e18d2f8
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions ansible/inventories/devnet-5/group_vars/hive.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -50,6 +50,7 @@ hive_simulations_tests:
- --client.checktimelimit=60s
- --sim.parallelism=8
#- --sim.timelimit=2h
- --sim.limit "fork_CancunToPrague or fork_Prague"
- --sim.buildarg fixtures=https://github.com/ethereum/execution-spec-tests/releases/download/pectra-devnet-5%40v1.2.0/fixtures_pectra-devnet-5.tar.gz
- --sim.buildarg branch=pectra-devnet-5@v1.2.0
# Consume RLP
Expand All @@ -64,6 +65,7 @@ hive_simulations_tests:
- --client.checktimelimit=60s
- --sim.parallelism=8
#- --sim.timelimit=2h
- --sim.limit "fork_CancunToPrague or fork_Prague"
- --sim.buildarg fixtures=https://github.com/ethereum/execution-spec-tests/releases/download/pectra-devnet-5%40v1.2.0/fixtures_pectra-devnet-5.tar.gz
- --sim.buildarg branch=pectra-devnet-5@v1.2.0
# Consume RLP
Expand Down

0 comments on commit e18d2f8

Please sign in to comment.