Skip to content

Commit

Permalink
Start BEEFY client by default for Polkadot nodes (#1913)
Browse files Browse the repository at this point in the history
Fellowship companion:
polkadot-fellows/runtimes#65

This starts the BEEFY client by default for Polkadot nodes.

Governance/sudo call is later required to enable/start consensus.

Part of paritytech/parity-bridges-common#2420
  • Loading branch information
serban300 authored Oct 18, 2023
1 parent 8b3905d commit 6c39bb4
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 8 deletions.
5 changes: 2 additions & 3 deletions polkadot/cli/src/command.rs
Original file line number Diff line number Diff line change
Expand Up @@ -195,9 +195,8 @@ where
.map_err(Error::from)?;
let chain_spec = &runner.config().chain_spec;

// By default, enable BEEFY on all networks except Polkadot (for now), unless
// explicitly disabled through CLI.
let mut enable_beefy = !chain_spec.is_polkadot() && !cli.run.no_beefy;
// By default, enable BEEFY on all networks, unless explicitly disabled through CLI.
let mut enable_beefy = !cli.run.no_beefy;
// BEEFY doesn't (yet) support warp sync:
// Until we implement https://github.com/paritytech/substrate/issues/14756
// - disallow warp sync for validators,
Expand Down
5 changes: 0 additions & 5 deletions polkadot/node/service/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -749,11 +749,6 @@ pub fn new_full<OverseerGenerator: OverseerGen>(
Some(backoff)
};

// Warn the user that BEEFY is still experimental for Polkadot.
if enable_beefy && config.chain_spec.is_polkadot() {
gum::warn!("BEEFY is still experimental, usage on Polkadot network is discouraged.");
}

let disable_grandpa = config.disable_grandpa;
let name = config.network.node_name.clone();

Expand Down

0 comments on commit 6c39bb4

Please sign in to comment.