Skip to content

Commit

Permalink
remove shell redirection for cumulus based collators
Browse files Browse the repository at this point in the history
  • Loading branch information
pepoviola committed Nov 17, 2023
1 parent 6518119 commit bff7ed6
Showing 1 changed file with 16 additions and 4 deletions.
20 changes: 16 additions & 4 deletions javascript/packages/orchestrator/src/configGenerator.ts
Original file line number Diff line number Diff line change
Expand Up @@ -353,7 +353,13 @@ export async function generateNetworkSpec(
? parachain.genesis_state_generator
: `${collatorBinary} ${DEFAULT_GENESIS_GENERATE_SUBCOMMAND}`;

computedStateCommand += ` > {{CLIENT_REMOTE_DIR}}/${GENESIS_STATE_FILENAME}`;
// TODO: we should remove this conditional ones
// https://github.com/paritytech/polkadot-sdk/pull/2375 and
// https://github.com/paritytech/polkadot-sdk/pull/2370
// and the jobs tha use a previous version of the collators reach those fixes.
computedStateCommand += `${
isCumulusBased ? " >" : ""
} {{CLIENT_REMOTE_DIR}}/${GENESIS_STATE_FILENAME}`;
}

if (parachain.genesis_wasm_path) {
Expand All @@ -376,7 +382,13 @@ export async function generateNetworkSpec(
? parachain.genesis_wasm_generator
: `${collatorBinary} ${DEFAULT_WASM_GENERATE_SUBCOMMAND}`;

computedWasmCommand += ` > {{CLIENT_REMOTE_DIR}}/${GENESIS_WASM_FILENAME}`;
// TODO: we should remove this conditional ones
// https://github.com/paritytech/polkadot-sdk/pull/2375 and
// https://github.com/paritytech/polkadot-sdk/pull/2370
// and the jobs tha use a previous version of the collators reach those fixes.
computedWasmCommand += `${
isCumulusBased ? " >" : ""
} {{CLIENT_REMOTE_DIR}}/${GENESIS_WASM_FILENAME}`;
}

let parachainSetup: Parachain = {
Expand Down Expand Up @@ -549,8 +561,8 @@ async function getCollatorNodeFromConfig(
collatorConfig.validator !== undefined
? collatorConfig.validator
: cumulusBased
? true
: false;
? true
: false;

if (collatorConfig.args)
args = args.concat(
Expand Down

0 comments on commit bff7ed6

Please sign in to comment.