fix: telemetry stopping on shutdown (#9740)

TODO also stop in prover node

Author: ludamad

yarn-project/aztec-node/src/aztec-node/server.ts

@@ -374,6 +374,7 @@ export class AztecNodeService implements AztecNode {
     await this.p2pClient.stop();
     await this.worldStateSynchronizer.stop();
     await this.blockSource.stop();
+    await this.telemetry.stop();
     this.log.info(`Stopped`);
   }
 

yarn-project/aztec/src/cli/cmds/start_node.ts

@@ -100,7 +100,7 @@ export const startNode = async (
   services.push({ node: nodeServer });
 
   // Add node stop function to signal handlers
-  signalHandlers.push(node.stop);
+  signalHandlers.push(node.stop.bind(node));
 
   // Add a PXE client that connects to this node if requested
   let pxe: PXE | undefined;

yarn-project/aztec/src/cli/cmds/start_prover_node.ts

@@ -80,7 +80,7 @@ export const startProverNode = async (
     services.push({ provingJobSource });
   }
 
-  signalHandlers.push(proverNode.stop);
+  signalHandlers.push(proverNode.stop.bind(proverNode));
 
   // Automatically start proving unproven blocks
   await proverNode.start();

yarn-project/prover-node/src/prover-node.ts

@@ -171,6 +171,7 @@ export class ProverNode implements ClaimsMonitorHandler, EpochMonitorHandler, Pr
     await Promise.all(Array.from(this.jobs.values()).map(job => job.stop()));
     await this.worldState.stop();
     await this.coordination.stop();
+    await this.telemetryClient.stop();
     this.log.info('Stopped ProverNode');
   }