Skip to content

Commit 0eadb15

Browse files
authored
ci(l1): pin assertoor playbook URLs to pre-refactoring commit (#6238)
## Summary Pin all assertoor playbook URL references from `refs/heads/master` to a specific commit (`c35b5c3a5f`) — the last `master` commit before ethpandaops/assertoor PR #137 (major refactoring) was merged. ## Motivation Since Feb 19 ~17:26 UTC, the assertoor upstream merged a [major refactoring](ethpandaops/assertoor#137) that introduced breaking changes: - `stopChildOnResult` removed (tasks now self-complete) - `failTaskCount` / `succeedTaskCount` replaced with `failureThreshold` / `successThreshold` - Transaction submission engine replaced with spamoor Since our configs fetched playbooks from `refs/heads/master`, every CI run after the merge fails the **Blob & Stability Check** — including runs with zero code changes. ## Changes Pinned 4 playbook URLs across 3 files: - `network_params_blob.yaml` — `blob-transactions-test.yaml` - `network_params_tx.yaml` — `eoa-transactions-test.yaml` - `network_params_ethrex_multiple_cl.yaml` — `blob-transactions-test.yaml` + `eoa-transactions-test.yaml` ## Note Pinning the playbook URLs may not be sufficient on its own — if Kurtosis is also pulling the latest assertoor Docker image (with the new runtime), the old playbooks may still be interpreted differently. We may also need to pin the assertoor image version in the network params configs.
1 parent 1ad04b2 commit 0eadb15

3 files changed

Lines changed: 7 additions & 4 deletions

File tree

.github/config/assertoor/network_params_blob.yaml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,10 +24,11 @@ network_params:
2424
deposit_contract_address: "0x4242424242424242424242424242424242424242"
2525

2626
assertoor_params:
27+
image: ethpandaops/assertoor:v0.0.17
2728
run_stability_check: false
2829
run_block_proposal_check: false
2930
tests:
30-
- https://raw.githubusercontent.com/ethpandaops/assertoor/refs/heads/master/playbooks/stable/blob-transactions-test.yaml
31+
- https://raw.githubusercontent.com/ethpandaops/assertoor/c35b5c3a5fd543bcc843ee0e0af1e6255ca48dbd/playbooks/stable/blob-transactions-test.yaml
3132
- https://raw.githubusercontent.com/lambdaclass/ethrex/refs/heads/main/.github/config/assertoor/el-stability-check.yaml
3233

3334
tx_fuzz_params:

.github/config/assertoor/network_params_ethrex_multiple_cl.yaml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,8 @@ additional_services:
2828
- dora
2929

3030
assertoor_params:
31+
image: ethpandaops/assertoor:v0.0.17
3132
tests:
3233
- https://raw.githubusercontent.com/lambdaclass/ethrex/refs/heads/main/.github/config/assertoor/el-stability-check.yaml # Check that all el clients are synced with the corresponding cl
33-
- https://raw.githubusercontent.com/ethpandaops/assertoor/refs/heads/master/playbooks/stable/blob-transactions-test.yaml
34-
- https://raw.githubusercontent.com/ethpandaops/assertoor/refs/heads/master/playbooks/stable/eoa-transactions-test.yaml
34+
- https://raw.githubusercontent.com/ethpandaops/assertoor/c35b5c3a5fd543bcc843ee0e0af1e6255ca48dbd/playbooks/stable/blob-transactions-test.yaml
35+
- https://raw.githubusercontent.com/ethpandaops/assertoor/c35b5c3a5fd543bcc843ee0e0af1e6255ca48dbd/playbooks/stable/eoa-transactions-test.yaml

.github/config/assertoor/network_params_tx.yaml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,8 @@ network_params:
3030
deposit_contract_address: "0x4242424242424242424242424242424242424242"
3131

3232
assertoor_params:
33+
image: ethpandaops/assertoor:v0.0.17
3334
run_stability_check: false
3435
run_block_proposal_check: false
3536
tests:
36-
- https://raw.githubusercontent.com/ethpandaops/assertoor/refs/heads/master/playbooks/stable/eoa-transactions-test.yaml
37+
- https://raw.githubusercontent.com/ethpandaops/assertoor/c35b5c3a5fd543bcc843ee0e0af1e6255ca48dbd/playbooks/stable/eoa-transactions-test.yaml

0 commit comments

Comments
 (0)