From 0ab6fefd2ca71623649722bd3ef4f6582513cb6c Mon Sep 17 00:00:00 2001 From: Danny Skubak <8206446+skubakdj@users.noreply.github.com> Date: Fri, 1 Sep 2023 09:54:33 -0400 Subject: [PATCH] Disable unused telemetry types (#10443) --- core/services/ocr2/delegate.go | 24 ++++++++++++++---------- 1 file changed, 14 insertions(+), 10 deletions(-) diff --git a/core/services/ocr2/delegate.go b/core/services/ocr2/delegate.go index 66da196dd77..9c1faca0c30 100644 --- a/core/services/ocr2/delegate.go +++ b/core/services/ocr2/delegate.go @@ -1214,6 +1214,8 @@ func (d *Delegate) newServicesOCR2Functions( ReportingPluginFactory: nil, // To be set by NewFunctionsServices } + noopMonitoringEndpoint := telemetry.NoopAgent{} + thresholdOracleArgs := libocr2.OCR2OracleArgs{ BinaryNetworkEndpointFactory: d.peerWrapper.Peer2, V2Bootstrappers: bootstrapPeers, @@ -1222,11 +1224,12 @@ func (d *Delegate) newServicesOCR2Functions( Database: thresholdOcrDB, LocalConfig: lc, Logger: ocrLogger, - MonitoringEndpoint: d.monitoringEndpointGen.GenMonitoringEndpoint(spec.ContractID, synchronization.OCR2Threshold), - OffchainConfigDigester: thresholdProvider.OffchainConfigDigester(), - OffchainKeyring: kb, - OnchainKeyring: kb, - ReportingPluginFactory: nil, // To be set by NewFunctionsServices + // Telemetry ingress for OCR2Threshold is currently not supported so a noop monitoring endpoint is being used + MonitoringEndpoint: &noopMonitoringEndpoint, + OffchainConfigDigester: thresholdProvider.OffchainConfigDigester(), + OffchainKeyring: kb, + OnchainKeyring: kb, + ReportingPluginFactory: nil, // To be set by NewFunctionsServices } s4OracleArgs := libocr2.OCR2OracleArgs{ @@ -1237,11 +1240,12 @@ func (d *Delegate) newServicesOCR2Functions( Database: s4OcrDB, LocalConfig: lc, Logger: ocrLogger, - MonitoringEndpoint: d.monitoringEndpointGen.GenMonitoringEndpoint(spec.ContractID, synchronization.OCR2S4), - OffchainConfigDigester: s4Provider.OffchainConfigDigester(), - OffchainKeyring: kb, - OnchainKeyring: kb, - ReportingPluginFactory: nil, // To be set by NewFunctionsServices + // Telemetry ingress for OCR2S4 is currently not supported so a noop monitoring endpoint is being used + MonitoringEndpoint: &noopMonitoringEndpoint, + OffchainConfigDigester: s4Provider.OffchainConfigDigester(), + OffchainKeyring: kb, + OnchainKeyring: kb, + ReportingPluginFactory: nil, // To be set by NewFunctionsServices } encryptedThresholdKeyShare := d.cfg.Threshold().ThresholdKeyShare()