Merge branch 'spiigot/add-pallet-drand' into commit-reveal-v3

Author: orriin

node/src/benchmarking.rs

@@ -2,7 +2,7 @@
 //!
 //! Should only be used for benchmarking as it may break in other contexts.
 
-use crate::service::Client;
+use crate::client::FullClient;
 
 use node_subtensor_runtime as runtime;
 use node_subtensor_runtime::check_nonce;
@@ -21,12 +21,12 @@ use std::{sync::Arc, time::Duration};
 //
 // Note: Should only be used for benchmarking.
 pub struct RemarkBuilder {
-    client: Arc<Client>,
+    client: Arc<FullClient>,
 }
 
 impl RemarkBuilder {
     // Creates a new [`Self`] from the given client.
-    pub fn new(client: Arc<Client>) -> Self {
+    pub fn new(client: Arc<FullClient>) -> Self {
         Self { client }
     }
 }
@@ -58,14 +58,14 @@ impl frame_benchmarking_cli::ExtrinsicBuilder for RemarkBuilder {
 //
 // Note: Should only be used for benchmarking.
 pub struct TransferKeepAliveBuilder {
-    client: Arc<Client>,
+    client: Arc<FullClient>,
     dest: AccountId,
     value: Balance,
 }
 
 impl TransferKeepAliveBuilder {
     // Creates a new [`Self`] from the given client.
-    pub fn new(client: Arc<Client>, dest: AccountId, value: Balance) -> Self {
+    pub fn new(client: Arc<FullClient>, dest: AccountId, value: Balance) -> Self {
         Self {
             client,
             dest,
@@ -105,7 +105,7 @@ impl frame_benchmarking_cli::ExtrinsicBuilder for TransferKeepAliveBuilder {
 //
 // Note: Should only be used for benchmarking.
 pub fn create_benchmark_extrinsic(
-    client: &Client,
+    client: &FullClient,
     sender: sp_core::sr25519::Pair,
     call: runtime::RuntimeCall,
     nonce: u32,

node/src/chain_spec/localnet.rs

@@ -3,7 +3,7 @@
 
 use super::*;
 
-pub fn localnet_config() -> Result<ChainSpec, String> {
+pub fn localnet_config(single_authority: bool) -> Result<ChainSpec, String> {
     let wasm_binary = WASM_BINARY.ok_or_else(|| "Development wasm not available".to_string())?;
 
     // Give front-ends necessary data to present to users
@@ -32,11 +32,15 @@ pub fn localnet_config() -> Result<ChainSpec, String> {
     .with_genesis_config_patch(localnet_genesis(
         // Initial PoA authorities (Validators)
         // aura | grandpa
-        vec![
-            // Keys for debug
-            authority_keys_from_seed("Alice"),
-            authority_keys_from_seed("Bob"),
-        ],
+        if single_authority {
+            // single authority allows you to run the network using a single node
+            vec![authority_keys_from_seed("Alice")]
+        } else {
+            vec![
+                authority_keys_from_seed("Alice"),
+                authority_keys_from_seed("Bob"),
+            ]
+        },
         // Pre-funded accounts
         true,
     ))

node/src/client.rs

@@ -1,76 +1,18 @@
-use scale_codec::Codec;
-// Substrate
-use sp_runtime::traits::{Block as BlockT, MaybeDisplay};
-
-use crate::ethereum::EthCompatRuntimeApiCollection;
-use crate::service::RuntimeExecutor;
+use node_subtensor_runtime::{opaque::Block, RuntimeApi};
+use sc_executor::WasmExecutor;
 
 /// Full backend.
-pub type FullBackend<B> = sc_service::TFullBackend<B>;
+pub type FullBackend = sc_service::TFullBackend<Block>;
 /// Full client.
-pub type FullClient<B, RA> = sc_service::TFullClient<B, RA, RuntimeExecutor>;
-
-/// A set of APIs that every runtime must implement.
-pub trait BaseRuntimeApiCollection<Block: BlockT>:
-    sp_api::ApiExt<Block>
-    + sp_api::Metadata<Block>
-    + sp_block_builder::BlockBuilder<Block>
-    + sp_offchain::OffchainWorkerApi<Block>
-    + sp_session::SessionKeys<Block>
-    + sp_transaction_pool::runtime_api::TaggedTransactionQueue<Block>
-{
-}
-
-impl<Block, Api> BaseRuntimeApiCollection<Block> for Api
-where
-    Block: BlockT,
-    Api: sp_api::ApiExt<Block>
-        + sp_api::Metadata<Block>
-        + sp_block_builder::BlockBuilder<Block>
-        + sp_offchain::OffchainWorkerApi<Block>
-        + sp_session::SessionKeys<Block>
-        + sp_transaction_pool::runtime_api::TaggedTransactionQueue<Block>,
-{
-}
-
-/// A set of APIs that Subtensor runtime must implement.
-pub trait RuntimeApiCollection<
-    Block: BlockT,
-    AuraId: Codec,
-    AccountId: Codec,
-    Nonce: Codec,
-    Balance: Codec + MaybeDisplay,
->:
-    BaseRuntimeApiCollection<Block>
-    + EthCompatRuntimeApiCollection<Block>
-    + sp_consensus_aura::AuraApi<Block, AuraId>
-    + sp_consensus_grandpa::GrandpaApi<Block>
-    + frame_system_rpc_runtime_api::AccountNonceApi<Block, AccountId, Nonce>
-    + pallet_transaction_payment_rpc_runtime_api::TransactionPaymentApi<Block, Balance>
-    + subtensor_custom_rpc_runtime_api::DelegateInfoRuntimeApi<Block>
-    + subtensor_custom_rpc_runtime_api::NeuronInfoRuntimeApi<Block>
-    + subtensor_custom_rpc_runtime_api::SubnetInfoRuntimeApi<Block>
-    + subtensor_custom_rpc_runtime_api::SubnetRegistrationRuntimeApi<Block>
-{
-}
-
-impl<Block, AuraId, AccountId, Nonce, Balance, Api>
-    RuntimeApiCollection<Block, AuraId, AccountId, Nonce, Balance> for Api
-where
-    Block: BlockT,
-    AuraId: Codec,
-    AccountId: Codec,
-    Nonce: Codec,
-    Balance: Codec + MaybeDisplay,
-    Api: BaseRuntimeApiCollection<Block>
-        + EthCompatRuntimeApiCollection<Block>
-        + sp_consensus_aura::AuraApi<Block, AuraId>
-        + sp_consensus_grandpa::GrandpaApi<Block>
-        + frame_system_rpc_runtime_api::AccountNonceApi<Block, AccountId, Nonce>
-        + pallet_transaction_payment_rpc_runtime_api::TransactionPaymentApi<Block, Balance>
-        + subtensor_custom_rpc_runtime_api::DelegateInfoRuntimeApi<Block>
-        + subtensor_custom_rpc_runtime_api::NeuronInfoRuntimeApi<Block>
-        + subtensor_custom_rpc_runtime_api::SubnetInfoRuntimeApi<Block>
-        + subtensor_custom_rpc_runtime_api::SubnetRegistrationRuntimeApi<Block>,
-{
-}
+pub type FullClient = sc_service::TFullClient<Block, RuntimeApi, RuntimeExecutor>;
+/// Always enable runtime benchmark host functions, the genesis state
+/// was built with them so we're stuck with them forever.
+///
+/// They're just a noop, never actually get used if the runtime was not compiled with
+/// `runtime-benchmarks`.
+pub type HostFunctions = (
+    sp_io::SubstrateHostFunctions,
+    sp_crypto_ec_utils::bls12_381::host_calls::HostFunctions,
+    frame_benchmarking::benchmarking::HostFunctions,
+);
+pub type RuntimeExecutor = WasmExecutor<HostFunctions>;

node/src/command.rs

@@ -41,7 +41,8 @@ impl SubstrateCli for Cli {
 
     fn load_spec(&self, id: &str) -> Result<Box<dyn sc_service::ChainSpec>, String> {
         Ok(match id {
-            "local" => Box::new(chain_spec::localnet::localnet_config()?),
+            "dev" => Box::new(chain_spec::localnet::localnet_config(true)?),
+            "local" => Box::new(chain_spec::localnet::localnet_config(false)?),
             "finney" => Box::new(chain_spec::finney::finney_mainnet_config()?),
             "devnet" => Box::new(chain_spec::devnet::devnet_config()?),
             "" | "test_finney" => Box::new(chain_spec::testnet::finney_testnet_config()?),