Skip to content

Commit 99242d6

Browse files
authored
fix: fix failing persistence test (ethereum#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.
1 parent f69c4a7 commit 99242d6

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

.github/tests/mix-persistence.yaml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,4 @@
11
participants:
2-
- el_type: besu
3-
cl_type: lighthouse
42
- el_type: geth
53
cl_type: teku
64
use_separate_vc: true
@@ -11,7 +9,9 @@ participants:
119
use_separate_vc: true
1210
- el_type: reth
1311
cl_type: lodestar
12+
- el_type: besu
13+
cl_type: lighthouse
1414
- el_type: ethereumjs
1515
cl_type: nimbus
1616
additional_services: []
17-
persistent: true
17+
persistent: true

0 commit comments

Comments
 (0)