Skip to content

Commit

Permalink
feat: configure prover as separate process
Browse files Browse the repository at this point in the history
  • Loading branch information
alexghr committed Apr 25, 2024
1 parent 545f216 commit 84d446b
Show file tree
Hide file tree
Showing 16 changed files with 186 additions and 30 deletions.
14 changes: 11 additions & 3 deletions yarn-project/aztec-node/src/aztec-node/config.ts
Original file line number Diff line number Diff line change
Expand Up @@ -17,22 +17,30 @@ export type AztecNodeConfig = ArchiverConfig &

/** A URL for an archiver service that the node will use. */
archiverUrl?: string;

proverAgents: number;
};

/**
* Returns the config of the aztec node from environment variables with reasonable defaults.
* @returns A valid aztec node config.
*/
export function getConfigEnvVars(): AztecNodeConfig {
const { SEQ_DISABLED, PROVER_DISABLED } = process.env;
const { SEQ_DISABLED, PROVER_DISABLED, ARCHIVER_URL, PROVER_AGENTS = '1' } = process.env;
let proverAgents = parseInt(PROVER_AGENTS, 10);
if (Number.isNaN(proverAgents)) {
proverAgents = 1;
}

const allEnvVars: AztecNodeConfig = {
...getSequencerVars(),
...getArchiverVars(),
...getP2PConfigEnvVars(),
...getWorldStateVars(),
disableSequencer: !!SEQ_DISABLED,
disableProver: !!PROVER_DISABLED,
archiverUrl: process.env.ARCHIVER_URL,
archiverUrl: ARCHIVER_URL,
disableProver: PROVER_DISABLED === '1',
proverAgents,
};

return allEnvVars;
Expand Down
7 changes: 6 additions & 1 deletion yarn-project/aztec-node/src/aztec-node/server.ts
Original file line number Diff line number Diff line change
Expand Up @@ -52,6 +52,7 @@ import { initStoreForRollup, openTmpStore } from '@aztec/kv-store/utils';
import { SHA256Trunc, StandardTree } from '@aztec/merkle-tree';
import { AztecKVTxPool, type P2P, createP2PClient } from '@aztec/p2p';
import { DummyProver, TxProver } from '@aztec/prover-client';
import { ProverPool } from '@aztec/prover-client/prover-pool';
import { type GlobalVariableBuilder, SequencerClient, getGlobalVariableBuilder } from '@aztec/sequencer-client';
import { PublicProcessorFactory, WASMSimulator } from '@aztec/simulator';
import {
Expand Down Expand Up @@ -151,7 +152,7 @@ export class AztecNodeService implements AztecNode {
const simulationProvider = await getSimulationProvider(config, log);
const prover = config.disableProver
? await DummyProver.new()
: await TxProver.new(config, worldStateSynchronizer, simulationProvider);
: await TxProver.new(worldStateSynchronizer, ProverPool.testPool(simulationProvider, config.proverAgents));

// now create the sequencer
const sequencer = config.disableSequencer
Expand Down Expand Up @@ -194,6 +195,10 @@ export class AztecNodeService implements AztecNode {
return this.sequencer;
}

public getProver(): ProverClient {
return this.prover;
}

/**
* Method to return the currently deployed L1 contract addresses.
* @returns - The currently deployed L1 contract addresses.
Expand Down
2 changes: 2 additions & 0 deletions yarn-project/aztec-node/src/index.ts
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
export { getSimulationProvider } from './aztec-node/simulator-factory.js';

export * from './aztec-node/config.js';
export * from './aztec-node/server.js';
export * from './aztec-node/http_rpc_server.js';
1 change: 1 addition & 0 deletions yarn-project/aztec/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,7 @@
"@aztec/noir-contracts.js": "workspace:^",
"@aztec/p2p": "workspace:^",
"@aztec/protocol-contracts": "workspace:^",
"@aztec/prover-client": "workspace:^",
"@aztec/pxe": "workspace:^",
"abitype": "^0.8.11",
"commander": "^11.1.0",
Expand Down
3 changes: 3 additions & 0 deletions yarn-project/aztec/src/cli/cli.ts
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,9 @@ export function getProgram(userLog: LogFn, debugLogger: DebugLogger): Command {
} else if (options.p2pBootstrap) {
const { startP2PBootstrap } = await import('./cmds/start_p2p_bootstrap.js');
await startP2PBootstrap(options, signalHandlers, userLog, debugLogger);
} else if (options.prover) {
const { startProver } = await import('./cmds/start_prover.js');
services = await startProver(options, signalHandlers, userLog);
}
if (services.length) {
const rpcServer = createNamespacedJsonRpcServer(services, debugLogger);
Expand Down
9 changes: 9 additions & 0 deletions yarn-project/aztec/src/cli/cmds/start_node.ts
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ import {
import { NULL_KEY } from '@aztec/ethereum';
import { type ServerList } from '@aztec/foundation/json-rpc/server';
import { type LogFn } from '@aztec/foundation/log';
import { createProvingJobSourceServer } from '@aztec/prover-client/prover-pool';
import { type PXEServiceConfig, createPXERpcServer, getPXEServiceConfig } from '@aztec/pxe';

import { mnemonicToAccount, privateKeyToAccount } from 'viem/accounts';
Expand Down Expand Up @@ -64,7 +65,10 @@ export const startNode = async (
}

if (!options.prover) {
userLog(`Prover is disabled, using mocked proofs`);
nodeConfig.disableProver = true;
} else {
nodeConfig = mergeEnvVarsAndCliOptions<AztecNodeConfig>(nodeConfig, parseModuleOptions(options.prover));
}

if (!nodeConfig.disableSequencer && nodeConfig.disableProver) {
Expand All @@ -78,6 +82,11 @@ export const startNode = async (
// Add node to services list
services.push({ node: nodeServer });

if (!nodeConfig.disableProver) {
const provingJobSource = createProvingJobSourceServer(node.getProver().getProvingJobSource());
services.push({ provingJobSource });
}

// Add node stop function to signal handlers
signalHandlers.push(node.stop);

Expand Down
34 changes: 34 additions & 0 deletions yarn-project/aztec/src/cli/cmds/start_prover.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,34 @@
import { getSimulationProvider } from '@aztec/aztec-node';
import { type ProvingJobSource } from '@aztec/circuit-types';
import { ProverPool, createProvingJobSourceClient } from '@aztec/prover-client/prover-pool';

import { type ServiceStarter, parseModuleOptions } from '../util.js';

type ProverOptions = Partial<{
jobsUrl: string;
agents: string;
}>;

export const startProver: ServiceStarter = async (options, signalHandlers, logger) => {
const proverOptions: ProverOptions = parseModuleOptions(options.prover);
let source: ProvingJobSource;

if (typeof proverOptions.jobsUrl === 'string') {
logger(`Connecting to prover at ${proverOptions.jobsUrl}`);
source = createProvingJobSourceClient(proverOptions.jobsUrl, 'provingJobSource');
} else {
throw new Error('Starting prover without an orchestrator is not supported');
}

const agentCount = proverOptions.agents ? parseInt(proverOptions.agents, 10) : 1;
if (agentCount === 0 || !Number.isSafeInteger(agentCount)) {
throw new Error('Cannot start prover without agents');
}

logger(`Starting ${agentCount} prover agents`);
const pool = ProverPool.testPool(await getSimulationProvider(options), agentCount);
await pool.start(source);
signalHandlers.push(() => pool.stop());

return Promise.resolve([]);
};
5 changes: 5 additions & 0 deletions yarn-project/aztec/src/cli/util.ts
Original file line number Diff line number Diff line change
Expand Up @@ -3,10 +3,15 @@ import { type AztecNodeConfig } from '@aztec/aztec-node';
import { type AccountManager, type Fr } from '@aztec/aztec.js';
import { type L1ContractAddresses, l1ContractsNames } from '@aztec/ethereum';
import { EthAddress } from '@aztec/foundation/eth-address';
import { type ServerList } from '@aztec/foundation/json-rpc/server';
import { type LogFn, createConsoleLogger } from '@aztec/foundation/log';
import { type P2PConfig } from '@aztec/p2p';
import { type PXEService, type PXEServiceConfig } from '@aztec/pxe';

export interface ServiceStarter<T = any> {
(options: T, signalHandlers: (() => Promise<void>)[], logger: LogFn): Promise<ServerList>;
}

/**
* Checks if the object has l1Contracts property
* @param obj - The object to check
Expand Down
3 changes: 3 additions & 0 deletions yarn-project/aztec/tsconfig.json
Original file line number Diff line number Diff line change
Expand Up @@ -51,6 +51,9 @@
{
"path": "../protocol-contracts"
},
{
"path": "../prover-client"
},
{
"path": "../pxe"
}
Expand Down
3 changes: 3 additions & 0 deletions yarn-project/circuit-types/src/interfaces/prover-client.ts
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
import { type BlockProver } from './block-prover.js';
import { type ProvingJobSource } from './proving-job.js';

/**
* The interface to the prover client.
Expand All @@ -8,4 +9,6 @@ export interface ProverClient extends BlockProver {
start(): Promise<void>;

stop(): Promise<void>;

getProvingJobSource(): ProvingJobSource;
}
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,7 @@ import { openTmpStore } from '@aztec/kv-store/utils';
import { AvailabilityOracleAbi, InboxAbi, OutboxAbi, RollupAbi } from '@aztec/l1-artifacts';
import { SHA256Trunc, StandardTree } from '@aztec/merkle-tree';
import { TxProver } from '@aztec/prover-client';
import { ProverPool } from '@aztec/prover-client/prover-pool';
import { type L1Publisher, getL1Publisher } from '@aztec/sequencer-client';
import { WASMSimulator } from '@aztec/simulator';
import { MerkleTrees, ServerWorldStateSynchronizer, type WorldStateConfig } from '@aztec/world-state';
Expand Down Expand Up @@ -143,7 +144,7 @@ describe('L1Publisher integration', () => {
};
const worldStateSynchronizer = new ServerWorldStateSynchronizer(tmpStore, builderDb, blockSource, worldStateConfig);
await worldStateSynchronizer.start();
builder = await TxProver.new({}, worldStateSynchronizer, new WASMSimulator());
builder = await TxProver.new(worldStateSynchronizer, ProverPool.testPool(new WASMSimulator(), 4));
l2Proof = Buffer.alloc(0);

publisher = getL1Publisher({
Expand Down
5 changes: 4 additions & 1 deletion yarn-project/prover-client/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,10 @@
"name": "@aztec/prover-client",
"version": "0.1.0",
"type": "module",
"exports": "./dest/index.js",
"exports": {
".": "./dest/index.js",
"./prover-pool": "./dest/prover-pool/index.js"
},
"bin": {
"bb-cli": "./dest/bb/index.js"
},
Expand Down
23 changes: 23 additions & 0 deletions yarn-project/prover-client/src/dummy-prover.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4,13 +4,18 @@ import {
PROVING_STATUS,
type ProcessedTx,
type ProverClient,
type ProvingJob,
type ProvingJobSource,
type ProvingRequest,
type ProvingSuccess,
type ProvingTicket,
} from '@aztec/circuit-types';
import { type GlobalVariables, makeEmptyProof } from '@aztec/circuits.js';
import { type Fr } from '@aztec/foundation/fields';

export class DummyProver implements ProverClient {
jobs = new DummyProvingJobSource();

public start(): Promise<void> {
return Promise.resolve();
}
Expand Down Expand Up @@ -54,4 +59,22 @@ export class DummyProver implements ProverClient {
setBlockCompleted(): Promise<void> {
return Promise.resolve();
}

getProvingJobSource(): ProvingJobSource {
return this.jobs;
}
}

class DummyProvingJobSource implements ProvingJobSource {
getProvingJob(): Promise<ProvingJob<ProvingRequest> | null> {
return Promise.resolve(null);
}

rejectProvingJob(): Promise<void> {
return Promise.resolve();
}

resolveProvingJob(): Promise<void> {
return Promise.resolve();
}
}
60 changes: 60 additions & 0 deletions yarn-project/prover-client/src/prover-pool/rpc.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,60 @@
import { type ProvingJobSource } from '@aztec/circuit-types';
import {
BaseOrMergeRollupPublicInputs,
BaseParityInputs,
BaseRollupInputs,
MergeRollupInputs,
ParityPublicInputs,
Proof,
PublicKernelCircuitPrivateInputs,
RootParityInputs,
RootRollupInputs,
RootRollupPublicInputs,
} from '@aztec/circuits.js';
import { createJsonRpcClient, makeFetch } from '@aztec/foundation/json-rpc/client';
import { JsonRpcServer } from '@aztec/foundation/json-rpc/server';

export function createProvingJobSourceServer(queue: ProvingJobSource): JsonRpcServer {
return new JsonRpcServer(
queue,
{
BaseParityInputs,
BaseOrMergeRollupPublicInputs,
BaseRollupInputs,
MergeRollupInputs,
ParityPublicInputs,
Proof,
RootParityInputs,
RootRollupInputs,
RootRollupPublicInputs,
PublicKernelCircuitPrivateInputs,
},
{},
);
}

export function createProvingJobSourceClient(
url: string,
namespace?: string,
fetch = makeFetch([1, 2, 3], false),
): ProvingJobSource {
return createJsonRpcClient(
url,
{
BaseParityInputs,
BaseOrMergeRollupPublicInputs,
BaseRollupInputs,
MergeRollupInputs,
ParityPublicInputs,
Proof,
RootParityInputs,
RootRollupInputs,
RootRollupPublicInputs,
PublicKernelCircuitPrivateInputs,
},
{},
false,
namespace,
fetch,
) as ProvingJobSource;
}
Loading

0 comments on commit 84d446b

Please sign in to comment.