Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Commit

Permalink
rebase issues
Browse files Browse the repository at this point in the history
  • Loading branch information
samelamin committed Dec 20, 2022
1 parent d32a010 commit 39a2878
Show file tree
Hide file tree
Showing 6 changed files with 12 additions and 22 deletions.
2 changes: 0 additions & 2 deletions client/consensus/common/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,6 @@

use polkadot_primitives::v2::{Hash as PHash, PersistedValidationData};


use sc_client_api::Backend;
use sc_consensus::{shared_data::SharedData, BlockImport, ImportResult};
use sp_runtime::traits::{Block as BlockT, Header as HeaderT};
Expand All @@ -26,7 +25,6 @@ use std::sync::Arc;
mod level_monitor;
mod parachain_consensus;

mod parachain_consensus;
#[cfg(test)]
mod tests;

Expand Down
6 changes: 1 addition & 5 deletions client/service/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -47,16 +47,14 @@ use sp_core::traits::SpawnNamed;
use sp_runtime::traits::Block as BlockT;

use futures::channel::mpsc;
use sp_runtime::traits::BlockIdTo;
use std::{sync::Arc, time::Duration};

// Given the sporadic nature of the explicit recovery operation and the
// possibility to retry infinite times this value is more than enough.
// In practice here we expect no more than one queued messages.
const RECOVERY_CHAN_SIZE: usize = 8;


use sp_runtime::traits::{Block as BlockT, BlockIdTo};
use std::{sync::Arc, time::Duration};
/// Parameters given to [`start_collator`].
pub struct StartCollatorParams<'a, Block: BlockT, BS, Client, RCInterface, Spawner> {
pub block_status: Arc<BS>,
Expand Down Expand Up @@ -276,8 +274,6 @@ pub async fn build_relay_chain_interface(
}
}



/// Parameters given to [`build_network`].
pub struct BuildNetworkParams<
'a,
Expand Down
9 changes: 3 additions & 6 deletions parachain-template/node/src/service.rs
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,8 @@ use cumulus_client_consensus_common::{
ParachainBlockImport as TParachainBlockImport, ParachainConsensus,
};
use cumulus_client_service::{
build_relay_chain_interface, prepare_node_config, start_collator, start_full_node,
StartCollatorParams, StartFullNodeParams,
build_network, build_relay_chain_interface, prepare_node_config, start_collator,
start_full_node, BuildNetworkParams, StartCollatorParams, StartFullNodeParams,
};
use cumulus_primitives_core::ParaId;
use cumulus_relay_chain_interface::{RelayChainError, RelayChainInterface};
Expand All @@ -29,8 +29,6 @@ use sc_telemetry::{Telemetry, TelemetryHandle, TelemetryWorker, TelemetryWorkerH
use sp_keystore::SyncCryptoStorePtr;
use substrate_prometheus_endpoint::Registry;

use polkadot_service::CollatorPair;

/// Native executor type.
pub struct ParachainNativeExecutor;

Expand Down Expand Up @@ -168,7 +166,6 @@ async fn start_node_impl(
s => s.to_string().into(),
})?;


let force_authoring = parachain_config.force_authoring;
let validator = parachain_config.role.is_authority();
let prometheus_registry = parachain_config.prometheus_registry().cloned();
Expand All @@ -180,7 +177,7 @@ async fn start_node_impl(
parachain_config: &parachain_config,
client: client.clone(),
transaction_pool: transaction_pool.clone(),
para_id: id,
para_id,
task_manager: &mut task_manager,
relay_chain_interface: relay_chain_interface.clone(),
import_queue: params.import_queue,
Expand Down
4 changes: 2 additions & 2 deletions polkadot-parachain/src/service.rs
Original file line number Diff line number Diff line change
Expand Up @@ -21,8 +21,8 @@ use cumulus_client_consensus_common::{
ParachainBlockImport as TParachainBlockImport, ParachainCandidate, ParachainConsensus,
};
use cumulus_client_service::{
build_relay_chain_interface, prepare_node_config, start_collator, start_full_node,
StartCollatorParams, StartFullNodeParams,
build_network, build_relay_chain_interface, prepare_node_config, start_collator,
start_full_node, BuildNetworkParams, StartCollatorParams, StartFullNodeParams,
};
use cumulus_primitives_core::{
relay_chain::v2::{Hash as PHash, PersistedValidationData},
Expand Down
8 changes: 4 additions & 4 deletions zombienet/tests/0007-full_node_warp_sync.feature
Original file line number Diff line number Diff line change
Expand Up @@ -8,10 +8,10 @@ charlie: is up
dave: is up
eve: is up
ferdie: is up
george: is up
george: pause
one: is up
one: pause
alice: parachain 2000 is registered within 225 seconds
dave: reports block height is at least 7 within 250 seconds
eve: reports block height is at least 7 within 250 seconds
george: resume
george: reports block height is at least 7 within 250 seconds
one: resume
one: reports block height is at least 7 within 250 seconds
5 changes: 2 additions & 3 deletions zombienet/tests/0007-full_node_warp_sync.toml
Original file line number Diff line number Diff line change
Expand Up @@ -49,10 +49,9 @@ cumulus_based = true
command = "test-parachain"
args = ["--reserved-only", "--reserved-nodes {{'charlie'|zombie('multiAddress')}}", "--relay-chain-rpc-url {{'alice'|zombie('wsUri')}}"]


# run cumulus george (a parachain full node) and wait for it to sync some blocks
# run cumulus one (a parachain full node) and wait for it to sync some blocks
[[parachains.collators]]
name = "george"
name = "one"
validator = false
image = "{{COL_IMAGE}}"
command = "test-parachain"
Expand Down

0 comments on commit 39a2878

Please sign in to comment.