From 27e0f1c2564457a71843f9594144227ac4b3e678 Mon Sep 17 00:00:00 2001 From: Jackson Weber Date: Thu, 19 Jun 2025 01:08:09 -0400 Subject: [PATCH] Remove creation of the otlp metrics exporter in AKS scenario. --- src/agent/aksLoader.ts | 5 ----- test/unitTests/agent/aksLoader.tests.ts | 15 --------------- 2 files changed, 20 deletions(-) diff --git a/src/agent/aksLoader.ts b/src/agent/aksLoader.ts index c2021ce57..642328900 100644 --- a/src/agent/aksLoader.ts +++ b/src/agent/aksLoader.ts @@ -14,11 +14,6 @@ export class AKSLoader extends AgentLoader { constructor() { super(); if (this._canLoad) { - // AKS specific configuration - this._options.otlpMetricExporterConfig = { - // Add OTLP if env variable is present - enabled: process.env["OTEL_EXPORTER_OTLP_METRICS_ENDPOINT"] ? true : false - }; (this._options.instrumentationOptions as InstrumentationOptions) = { ...this._options.instrumentationOptions, console: { enabled: true }, diff --git a/test/unitTests/agent/aksLoader.tests.ts b/test/unitTests/agent/aksLoader.tests.ts index b0faa3706..fcca79727 100644 --- a/test/unitTests/agent/aksLoader.tests.ts +++ b/test/unitTests/agent/aksLoader.tests.ts @@ -2,7 +2,6 @@ import * as assert from "assert"; import * as sinon from "sinon"; import { ProxyTracerProvider, metrics, trace } from "@opentelemetry/api"; import { logs } from "@opentelemetry/api-logs"; - import { AKSLoader } from "../../../src/agent/aksLoader"; import { DiagnosticLogger } from "../../../src/agent/diagnostics/diagnosticLogger"; import { FileWriter } from "../../../src/agent/diagnostics/writers/fileWriter"; @@ -62,18 +61,4 @@ describe("agent/AKSLoader", () => { assert.equal(loggerProvider.constructor.name, "LoggerProvider"); assert.equal(loggerProvider["_sharedState"]["registeredLogRecordProcessors"][1]["_exporter"].constructor.name, "AzureMonitorLogExporter"); }); - - it("should add OTLP exporter if env variable is present", () => { - const env = { - ["APPLICATIONINSIGHTS_CONNECTION_STRING"]: "InstrumentationKey=1aa11111-bbbb-1ccc-8ddd-eeeeffff3333", - ["OTEL_EXPORTER_OTLP_METRICS_ENDPOINT"]: "something", - }; - process.env = env; - const agent = new AKSLoader(); - agent.initialize(); - let meterProvider = metrics.getMeterProvider() as any; - assert.equal(meterProvider.constructor.name, "MeterProvider"); - assert.equal(meterProvider["_sharedState"]["metricCollectors"].length, 2); - assert.equal(meterProvider["_sharedState"]["metricCollectors"][1]["_metricReader"]["_exporter"].constructor.name, "OTLPMetricExporter"); - }); });