Skip to content

Commit 81b9bf8

Browse files
committed
fix(e2e): only update protocol parameters on leader aggregator
Since now the follower read the network config from the leader, this means that the update of the protocol parameters is now a responsability of the leader only. This lead to flakiness because this step was restarting all aggregators, and sometimes the follower started before the leader and had a error when it executed its handle discrepencies because the leader http server was still down.
1 parent e348709 commit 81b9bf8

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

mithril-test-lab/mithril-end-to-end/src/end_to_end_spec.rs

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -136,7 +136,9 @@ impl Spec {
136136
"epoch after which the protocol parameters will change".to_string(),
137137
)
138138
.await?;
139-
assertions::update_protocol_parameters(aggregator).await?;
139+
if aggregator.is_first() {
140+
assertions::update_protocol_parameters(aggregator).await?;
141+
}
140142

141143
// Wait 6 epochs after protocol parameters update, so that we make sure that we use new protocol parameters as well as new stake distribution a few times
142144
target_epoch += 6;

0 commit comments

Comments
 (0)