Skip to content

Commit

Permalink
Enable approval-voting-parallel by default on kusama (#6218)
Browse files Browse the repository at this point in the history
The approval-voting-parallel introduced with
#4849 has been tested on
`versi` and approximately 3 weeks on parity's existing kusama nodes
paritytech/devops#3583, things worked as
expected, so enable it by default on all kusama nodes in the next
release.

The next step will be enabling by default on polkadot if no issue
arrises while running on kusama.

---------

Signed-off-by: Alexandru Gheorghe <[email protected]>
  • Loading branch information
alexggh authored Oct 24, 2024
1 parent 5682f9a commit c0df223
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 5 deletions.
9 changes: 4 additions & 5 deletions polkadot/node/service/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -759,13 +759,12 @@ pub fn new_full<
Some(backoff)
};

// Running approval voting in parallel is enabled by default on all networks except Polkadot and
// Kusama, unless explicitly enabled by the commandline option.
// Running approval voting in parallel is enabled by default on all networks except Polkadot
// unless explicitly enabled by the commandline option.
// This is meant to be temporary until we have enough confidence in the new system to enable it
// by default on all networks.
let enable_approval_voting_parallel = (!config.chain_spec.is_kusama() &&
!config.chain_spec.is_polkadot()) ||
enable_approval_voting_parallel;
let enable_approval_voting_parallel =
!config.chain_spec.is_polkadot() || enable_approval_voting_parallel;

let disable_grandpa = config.disable_grandpa;
let name = config.network.node_name.clone();
Expand Down
9 changes: 9 additions & 0 deletions prdoc/pr_6218.prdoc
Original file line number Diff line number Diff line change
@@ -0,0 +1,9 @@
title: Enable approval-voting-parallel by default on kusama

doc:
- audience: Node Dev
description: |
Enable approval-voting-parallel by default on kusama
crates:
- name: polkadot-service
bump: patch

0 comments on commit c0df223

Please sign in to comment.