Commit 99242d66 authored by pk910's avatar pk910 Committed by GitHub

fix: fix failing persistence test (#554)

fixed the failing persistence test by re-ordering the clients list.

The test fails because initialization of all EL clients takes longer
than the initial genesis delay.
Due to this, lighhouse returns an unhealty result for its health check
as it's post-genesis and not following a chain / has no peers.
The kurtosis package waits for a healthy result, which never occurs as
all other BNs & VCs are missing at that point.

The fix proposed in this PR is a workaround as it moves the lighthouse
node further down to ensure it can connect to other nodes initialized
before.

There is an alternative approach to fix this issue:
```
network_params:
  genesis_delay: 300
```
which ensures all clients are properly initialized before the network
starts.
parent f69c4a74
participants: participants:
- el_type: besu
cl_type: lighthouse
- el_type: geth - el_type: geth
cl_type: teku cl_type: teku
use_separate_vc: true use_separate_vc: true
...@@ -11,7 +9,9 @@ participants: ...@@ -11,7 +9,9 @@ participants:
use_separate_vc: true use_separate_vc: true
- el_type: reth - el_type: reth
cl_type: lodestar cl_type: lodestar
- el_type: besu
cl_type: lighthouse
- el_type: ethereumjs - el_type: ethereumjs
cl_type: nimbus cl_type: nimbus
additional_services: [] additional_services: []
persistent: true persistent: true
\ No newline at end of file
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment