Skip to content

Commit

Permalink
fix: load prover node config from env (#8525)
Browse files Browse the repository at this point in the history
Fixes spartan deployments.

`BigInt(config.archiverL1StartBlock)` was failing because
`config.archiverL1StartBlock` was not defined in the environment, nor on
the command line, and the existing code did not actually use the
`parseEnv` command, which is how this default value seems to be loaded.
  • Loading branch information
just-mitch authored Sep 12, 2024
1 parent d9e3f4d commit 7065962
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion yarn-project/aztec/src/cli/cmds/start_prover_node.ts
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ import {
type ProverNodeConfig,
createProverNode,
createProverNodeRpcServer,
getProverNodeConfigFromEnv,
proverNodeConfigMappings,
} from '@aztec/prover-node';
import { createAndStartTelemetryClient, telemetryClientConfigMappings } from '@aztec/telemetry-client/start';
Expand All @@ -29,7 +30,8 @@ export const startProverNode = async (
}

const proverConfig = {
...extractRelevantOptions<ProverNodeConfig>(options, proverNodeConfigMappings, 'proverNode'),
...getProverNodeConfigFromEnv(), // get default config from env
...extractRelevantOptions<ProverNodeConfig>(options, proverNodeConfigMappings, 'proverNode'), // override with command line options
l1Contracts: extractL1ContractAddresses(options),
};

Expand Down

0 comments on commit 7065962

Please sign in to comment.