Skip to content

Commit

Permalink
fix: Create proving job queue when prover node started with no agents (
Browse files Browse the repository at this point in the history
…#7828)

If the prover node is started with no in-proc agents, then we should
create a proving job source so agents can attach to it.

Unrelated, also tweaks some logging statements on the agent.
  • Loading branch information
spalladino authored Aug 7, 2024
1 parent ef30c8f commit e2feaf8
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion yarn-project/aztec/src/cli/cmds/start_prover_node.ts
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,7 @@ export const startProverNode = async (

services.push({ node: createProverNodeRpcServer(proverNode) });

if (options.prover) {
if (!options.prover) {
const provingJobSource = createProvingJobSourceServer(proverNode.getProver().getProvingJobSource());
services.push({ provingJobSource });
}
Expand Down
4 changes: 2 additions & 2 deletions yarn-project/prover-client/src/prover-agent/prover-agent.ts
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,7 @@ export class ProverAgent {
);
}
} catch (err) {
// no-op
this.log.error(`Error fetching job`, err);
}
}
}, this.pollIntervalMs);
Expand All @@ -96,7 +96,7 @@ export class ProverAgent {
this.log.debug(`Picked up proving job id=${job.id} type=${ProvingRequestType[job.request.type]}`);
const [time, result] = await elapsed(this.getProof(job.request));
if (this.isRunning()) {
this.log.debug(
this.log.verbose(
`Processed proving job id=${job.id} type=${ProvingRequestType[job.request.type]} duration=${time}ms`,
);
await jobSource.resolveProvingJob(job.id, result);
Expand Down

0 comments on commit e2feaf8

Please sign in to comment.