[Monitor OpenTelemetry] Fix OTLP Metric Exporter Creation (#35879)

### Packages impacted by this PR
@azure/monitor-opentelemtry

### Describe the problem that is addressed by this PR
This pull request introduces improvements to the Azure Monitor
OpenTelemetry SDK, focusing on enhanced support for metrics, dependency
updates, and better type safety for log processing. The most important
changes are grouped below:

**Metrics Support Improvements**
* Added support for multiple metric exporters by introducing a
`metricReaders` option to `AzureMonitorOpenTelemetryOptions`, allowing
users to supply additional `MetricReader` instances. The SDK now always
includes the Azure Monitor metric reader and any user-provided readers.
(`sdk/monitor/monitor-opentelemetry/src/index.ts`,
`sdk/monitor/monitor-opentelemetry/src/types.ts`,
`sdk/monitor/monitor-opentelemetry/review/monitor-opentelemetry-node.api.md`)
[[1]](diffhunk://#diff-e7e1dcf8f0ac5c3aea3811994ab032c757ad020b9ca8855dec69d18bb5343ef3R85-R94)
[[2]](diffhunk://#diff-48c64776365220d515750e313877e3ff6c92347670e2e5892c0df15179561028R7)
[[3]](diffhunk://#diff-053e8b2a667e27bc937e196833f522a1e6b137f1c1f4e64763144730dd4b149fR25)
* Updated the SDK initialization to use the new `metricReaders` array
instead of the deprecated single `metricReader` property.
(`sdk/monitor/monitor-opentelemetry/src/index.ts`)

**Dependency Updates**
* Upgraded multiple OpenTelemetry dependencies to their latest versions
in `package.json`, including exporters, instrumentations, and core
packages, ensuring compatibility and access to new features.
(`sdk/monitor/monitor-opentelemetry/package.json`)
[[1]](diffhunk://#diff-a55bcbd4eeed188b4872d7378e7bc7111e3a04059f36f5a6a4b52a74b29a971eL75-R95)
[[2]](diffhunk://#diff-a55bcbd4eeed188b4872d7378e7bc7111e3a04059f36f5a6a4b52a74b29a971eR59)
* Added the `@opentelemetry/exporter-metrics-otlp-http` dependency for
OTLP metric exporter support.
(`sdk/monitor/monitor-opentelemetry/package.json`)

**Log Processing Type Safety**
* Refactored all usages of the `LogRecord` type to the more accurate
`SdkLogRecord` type throughout the codebase, improving type safety and
compatibility with newer OpenTelemetry versions.
[[1]](diffhunk://#diff-40c7c732674ada132349217c585300c9623317b0d76d6af9354fcc05ace24c8bL5-R5)
[[2]](diffhunk://#diff-40c7c732674ada132349217c585300c9623317b0d76d6af9354fcc05ace24c8bL23-R23)
[[3]](diffhunk://#diff-6cb4c7dc3729862669d8e91910d3d2f4f7ea029135b0ed71a0c2bb9910a35c10L5-R5)
[[4]](diffhunk://#diff-6cb4c7dc3729862669d8e91910d3d2f4f7ea029135b0ed71a0c2bb9910a35c10L19-R19)
[[5]](diffhunk://#diff-01b317e2f20b9cfc18a7b7fbb904993e21fbe2240f20412387fc6453556f8663R7-L10)
[[6]](diffhunk://#diff-01b317e2f20b9cfc18a7b7fbb904993e21fbe2240f20412387fc6453556f8663L97-R97)
[[7]](diffhunk://#diff-89a783818735d150a51da07f52c0523bb4a68a241a174379fe0439d6489dc203R12)
[[8]](diffhunk://#diff-89a783818735d150a51da07f52c0523bb4a68a241a174379fe0439d6489dc203L23)
[[9]](diffhunk://#diff-89a783818735d150a51da07f52c0523bb4a68a241a174379fe0439d6489dc203L217-R217)
[[10]](diffhunk://#diff-e7fb1f5cf2a3336ec774595f29f7916a1d5df80a44162e4bd081a70422cd0a25R9-L14)
[[11]](diffhunk://#diff-e7fb1f5cf2a3336ec774595f29f7916a1d5df80a44162e4bd081a70422cd0a25L517-R517)
[[12]](diffhunk://#diff-eb7dc54dc1601d5790cfdb864a0976f71c58660fc5ad8258465d588fb268db01L7-R7)
[[13]](diffhunk://#diff-eb7dc54dc1601d5790cfdb864a0976f71c58660fc5ad8258465d588fb268db01L395-R395)
[[14]](diffhunk://#diff-46220be157c675f1ae424657fc5aaa60bebcf7c034e5790a926a1b5ead3d4a95R9-L14)
[[15]](diffhunk://#diff-46220be157c675f1ae424657fc5aaa60bebcf7c034e5790a926a1b5ead3d4a95L148-R148)
[[16]](diffhunk://#diff-0d388292857d67eba5aa26a7c3079cfeaa39ada1538619af33d94d486615a7eeR6)
[[17]](diffhunk://#diff-0d388292857d67eba5aa26a7c3079cfeaa39ada1538619af33d94d486615a7eeL37)
[[18]](diffhunk://#diff-0d388292857d67eba5aa26a7c3079cfeaa39ada1538619af33d94d486615a7eeL131-R131)
[[19]](diffhunk://#diff-0d388292857d67eba5aa26a7c3079cfeaa39ada1538619af33d94d486615a7eeL145-R145)
[[20]](diffhunk://#diff-0d388292857d67eba5aa26a7c3079cfeaa39ada1538619af33d94d486615a7eeL159-R159)

**Internal SDK Access for Testing**
* Added an internal `_getSdkInstance()` method to expose the SDK
instance for testing purposes.
(`sdk/monitor/monitor-opentelemetry/src/index.ts`,
`sdk/monitor/monitor-opentelemetry/review/monitor-opentelemetry-node.api.md`)
[[1]](diffhunk://#diff-e7e1dcf8f0ac5c3aea3811994ab032c757ad020b9ca8855dec69d18bb5343ef3R124-R131)
[[2]](diffhunk://#diff-053e8b2a667e27bc937e196833f522a1e6b137f1c1f4e64763144730dd4b149fR38-R40)

**Documentation**
* Updated the `CHANGELOG.md` to reflect bug fixes for metric exporter
support and dependency updates.
(`sdk/monitor/monitor-opentelemetry/CHANGELOG.md`)

### Are there test cases added in this PR? _(If not, why?)_
Yes

### Checklists
- [x] Added impacted package name to the issue description
- [ ] Does this PR needs any fixes in the SDK Generator?** _(If so,
create an Issue in the
[Autorest/typescript](https://github.com/Azure/autorest.typescript)
repository and link it here)_
- [x] Added a changelog (if necessary)

Author: JacksonWeber

pnpm-lock.yaml

@@ -1,5 +1,15 @@
 # Release History
 
+### 1.13.1 ()
+
+### Bugs Fixed
+
+- Fix support for multiple metric exporters and OTLP metric exporter creation.
+
+### Other Changes
+
+- Update OpenTelemetry dependencies.
+
 ### 1.13.0 (2025-09-05)
 
 ### Features Added

sdk/monitor/monitor-opentelemetry/CHANGELOG.md

@@ -254,6 +254,7 @@ Further information on usage of the browser SDK loader can be found [here](https
 You might set the Cloud Role Name and the Cloud Role Instance via [OpenTelemetry Resource](https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/resource/sdk.md#resource-sdk) attributes.
 
 ```ts snippet:ReadmeSampleSetRoleNameAndInstance
+import { emptyResource } from "@opentelemetry/resources";
 import {
   ATTR_SERVICE_NAME,
   SEMRESATTRS_SERVICE_NAMESPACE,
@@ -264,7 +265,7 @@ import { AzureMonitorOpenTelemetryOptions, useAzureMonitor } from "@azure/monito
 // ----------------------------------------
 // Setting role name and role instance
 // ----------------------------------------
-const customResource = Resource.EMPTY;
+const customResource = emptyResource();
 customResource.attributes[ATTR_SERVICE_NAME] = "my-helloworld-service";
 customResource.attributes[SEMRESATTRS_SERVICE_NAMESPACE] = "my-namespace";
 customResource.attributes[SEMRESATTRS_SERVICE_INSTANCE_ID] = "my-instance";
@@ -334,7 +335,7 @@ Use a custom span processor and log record processor in order to attach and corr
 import { SpanProcessor, ReadableSpan } from "@opentelemetry/sdk-trace-base";
 import { Span, Context, trace } from "@opentelemetry/api";
 import { AI_OPERATION_NAME } from "@azure/monitor-opentelemetry-exporter";
-import { LogRecordProcessor, LogRecord } from "@opentelemetry/sdk-logs";
+import { LogRecordProcessor, SdkLogRecord } from "@opentelemetry/sdk-logs";
 import { AzureMonitorOpenTelemetryOptions, useAzureMonitor } from "@azure/monitor-opentelemetry";
 
 class SpanEnrichingProcessor implements SpanProcessor {
@@ -361,7 +362,7 @@ class LogRecordEnrichingProcessor implements LogRecordProcessor {
   async shutdown(): Promise<void> {
     // shutdown code here
   }
-  onEmit(_logRecord: LogRecord, _context: Context): void {
+  onEmit(_logRecord: SdkLogRecord, _context: Context): void {
     const parentSpan = trace.getSpan(_context);
     if (parentSpan && "name" in parentSpan) {
       // If the parent span has a name we can assume it is a ReadableSpan and cast it.

sdk/monitor/monitor-opentelemetry/README.md

@@ -56,6 +56,7 @@
     "@azure/eslint-plugin-azure-sdk": "workspace:^",
     "@azure/functions": "^4.5.0",
     "@azure/functions-old": "npm:@azure/[email protected]",
+    "@opentelemetry/exporter-metrics-otlp-http": "^0.204.0",
     "@types/node": "catalog:",
     "@vitest/coverage-istanbul": "catalog:testing",
     "dotenv": "catalog:testing",
@@ -72,26 +73,26 @@
     "@azure/opentelemetry-instrumentation-azure-sdk": "^1.0.0-beta.7",
     "@microsoft/applicationinsights-web-snippet": "^1.2.1",
     "@opentelemetry/api": "^1.9.0",
-    "@opentelemetry/api-logs": "^0.200.0",
-    "@opentelemetry/core": "^2.0.0",
-    "@opentelemetry/instrumentation": "^0.200.0",
-    "@opentelemetry/instrumentation-bunyan": "^0.46.0",
-    "@opentelemetry/instrumentation-http": "^0.200.0",
-    "@opentelemetry/instrumentation-mongodb": "^0.53.0",
-    "@opentelemetry/instrumentation-mysql": "^0.46.0",
-    "@opentelemetry/instrumentation-pg": "^0.52.0",
-    "@opentelemetry/instrumentation-redis": "^0.47.0",
-    "@opentelemetry/instrumentation-redis-4": "^0.47.0",
-    "@opentelemetry/instrumentation-winston": "^0.45.0",
+    "@opentelemetry/api-logs": "^0.204.0",
+    "@opentelemetry/core": "^2.1.0",
+    "@opentelemetry/instrumentation": "^0.204.0",
+    "@opentelemetry/instrumentation-bunyan": "^0.50.0",
+    "@opentelemetry/instrumentation-http": "^0.204.0",
+    "@opentelemetry/instrumentation-mongodb": "^0.57.0",
+    "@opentelemetry/instrumentation-mysql": "^0.50.0",
+    "@opentelemetry/instrumentation-pg": "^0.57.0",
+    "@opentelemetry/instrumentation-redis": "^0.53.0",
+    "@opentelemetry/instrumentation-redis-4": "^0.49.0",
+    "@opentelemetry/instrumentation-winston": "^0.49.0",
     "@opentelemetry/resource-detector-azure": "^0.7.0",
-    "@opentelemetry/resources": "^2.0.0",
-    "@opentelemetry/sdk-logs": "^0.200.0",
-    "@opentelemetry/sdk-metrics": "^2.0.0",
-    "@opentelemetry/sdk-node": "^0.200.0",
-    "@opentelemetry/sdk-trace-base": "^2.0.0",
-    "@opentelemetry/sdk-trace-node": "^2.0.0",
+    "@opentelemetry/resources": "^2.1.0",
+    "@opentelemetry/sdk-logs": "^0.204.0",
+    "@opentelemetry/sdk-metrics": "^2.1.0",
+    "@opentelemetry/sdk-node": "^0.204.0",
+    "@opentelemetry/sdk-trace-base": "^2.1.0",
+    "@opentelemetry/sdk-trace-node": "^2.1.0",
     "@opentelemetry/semantic-conventions": "^1.32.0",
-    "@opentelemetry/winston-transport": "^0.11.0",
+    "@opentelemetry/winston-transport": "^0.15.0",
     "tslib": "^2.8.1"
   },
   "sideEffects": false,

sdk/monitor/monitor-opentelemetry/package.json

@@ -7,6 +7,8 @@
 import type { AzureMonitorExporterOptions } from '@azure/monitor-opentelemetry-exporter';
 import type { InstrumentationConfig } from '@opentelemetry/instrumentation';
 import type { LogRecordProcessor } from '@opentelemetry/sdk-logs';
+import type { MetricReader } from '@opentelemetry/sdk-metrics';
+import { NodeSDK } from '@opentelemetry/sdk-node';
 import type { Resource } from '@opentelemetry/resources';
 import type { SpanProcessor } from '@opentelemetry/sdk-trace-base';
 
@@ -20,6 +22,7 @@ export interface AzureMonitorOpenTelemetryOptions {
     enableTraceBasedSamplingForLogs?: boolean;
     instrumentationOptions?: InstrumentationOptions;
     logRecordProcessors?: LogRecordProcessor[];
+    metricReaders?: MetricReader[];
     resource?: Resource;
     samplingRatio?: number;
     spanProcessors?: SpanProcessor[];
@@ -32,6 +35,9 @@ export interface BrowserSdkLoaderOptions {
     enabled?: boolean;
 }
 
+// @internal
+export function _getSdkInstance(): NodeSDK | undefined;
+
 // @public
 export interface InstrumentationOptions {
     azureSdk?: InstrumentationConfig;

sdk/monitor/monitor-opentelemetry/review/monitor-opentelemetry-node.api.md

@@ -5,6 +5,7 @@ import { metrics, trace } from "@opentelemetry/api";
 import { logs } from "@opentelemetry/api-logs";
 import type { NodeSDKConfiguration } from "@opentelemetry/sdk-node";
 import { NodeSDK } from "@opentelemetry/sdk-node";
+import type { MetricReader } from "@opentelemetry/sdk-metrics";
 import { InternalConfig } from "./shared/config.js";
 import { MetricHandler } from "./metrics/index.js";
 import { TraceHandler } from "./traces/handler.js";
@@ -81,10 +82,16 @@ export function useAzureMonitor(options?: AzureMonitorOpenTelemetryOptions): voi
   const spanProcessors: SpanProcessor[] = options?.spanProcessors || [];
   const logRecordProcessors: LogRecordProcessor[] = options?.logRecordProcessors || [];
 
+  // Prepare metric readers - always include Azure Monitor
+  const metricReaders: MetricReader[] = [
+    metricHandler.getMetricReader(),
+    ...(options?.metricReaders || []),
+  ];
+
   // Initialize OpenTelemetry SDK
   const sdkConfig: Partial<NodeSDKConfiguration> = {
     autoDetectResources: true,
-    metricReader: metricHandler.getMetricReader(),
+    metricReaders: metricReaders,
     views: metricHandler.getViews(),
     instrumentations: instrumentations,
     logRecordProcessors: [
@@ -114,3 +121,12 @@ export function shutdownAzureMonitor(): Promise<void> {
   browserSdkLoader?.dispose();
   return sdk?.shutdown();
 }
+
+/**
+ * Get the internal SDK instance for testing purposes
+ * @internal
+ */
+// eslint-disable-next-line no-underscore-dangle
+export function _getSdkInstance(): NodeSDK | undefined {
+  return sdk;
+}

sdk/monitor/monitor-opentelemetry/src/index.ts

@@ -2,7 +2,7 @@
 // Licensed under the MIT License.
 
 import { TraceFlags } from "@opentelemetry/api";
-import type { LogRecord, LogRecordExporter } from "@opentelemetry/sdk-logs";
+import type { LogRecordExporter, SdkLogRecord } from "@opentelemetry/sdk-logs";
 import { BatchLogRecordProcessor } from "@opentelemetry/sdk-logs";
 
 /**
@@ -20,7 +20,7 @@ export class AzureBatchLogRecordProcessor extends BatchLogRecordProcessor {
     this._options = options;
   }
 
-  public onEmit(logRecord: LogRecord): void {
+  public onEmit(logRecord: SdkLogRecord): void {
     // Trace based sampling for logs
     if (this._options.enableTraceBasedSamplingForLogs) {
       if (logRecord.spanContext && logRecord.spanContext.spanId) {

sdk/monitor/monitor-opentelemetry/src/logs/batchLogRecordProcessor.ts

@@ -2,7 +2,7 @@
 // Licensed under the MIT License.
 
 import type { MetricHandler } from "../metrics/handler.js";
-import type { LogRecord, LogRecordProcessor } from "@opentelemetry/sdk-logs";
+import type { LogRecordProcessor, SdkLogRecord } from "@opentelemetry/sdk-logs";
 import { Logger } from "../shared/logging/index.js";
 
 /**
@@ -16,7 +16,7 @@ export class AzureLogRecordProcessor implements LogRecordProcessor {
     this._metricHandler = metricHandler;
   }
 
-  public onEmit(logRecord: LogRecord): void {
+  public onEmit(logRecord: SdkLogRecord): void {
     try {
       this._metricHandler.recordLog(logRecord);
     } catch (error) {

sdk/monitor/monitor-opentelemetry/src/logs/logRecordProcessor.ts

@@ -4,10 +4,10 @@
 import { AzureMonitorMetricExporter } from "@azure/monitor-opentelemetry-exporter";
 import type { PeriodicExportingMetricReaderOptions, ViewOptions } from "@opentelemetry/sdk-metrics";
 import { PeriodicExportingMetricReader } from "@opentelemetry/sdk-metrics";
+import type { SdkLogRecord } from "@opentelemetry/sdk-logs";
 import type { InternalConfig } from "../shared/config.js";
 import { StandardMetrics } from "./standardMetrics.js";
 import type { ReadableSpan, Span } from "@opentelemetry/sdk-trace-base";
-import type { LogRecord } from "@opentelemetry/sdk-logs";
 import { APPLICATION_INSIGHTS_NO_STANDARD_METRICS } from "./types.js";
 import { LiveMetrics } from "./quickpulse/liveMetrics.js";
 import { PerformanceCounterMetrics } from "./performanceCounters.js";
@@ -94,7 +94,7 @@ export class MetricHandler {
     this._performanceCounters?.recordSpan(span);
   }
 
-  public recordLog(logRecord: LogRecord): void {
+  public recordLog(logRecord: SdkLogRecord): void {
     this._standardMetrics?.recordLog(logRecord);
     this._liveMetrics?.recordLog(logRecord);
     this._performanceCounters?.recordLog(logRecord);

sdk/monitor/monitor-opentelemetry/src/metrics/handler.ts

@@ -9,6 +9,7 @@ import type {
   ObservableResult,
 } from "@opentelemetry/api";
 import { SpanKind, ValueType } from "@opentelemetry/api";
+import type { SdkLogRecord } from "@opentelemetry/sdk-logs";
 import { AzureMonitorMetricExporter } from "@azure/monitor-opentelemetry-exporter";
 import type {
   MeterProviderOptions,
@@ -20,7 +21,6 @@ import { PerformanceCounterMetricNames } from "./types.js";
 import type { AzureMonitorOpenTelemetryOptions } from "../types.js";
 import { getLogData, isExceptionData } from "./quickpulse/utils.js";
 import type { ExceptionData, TraceData } from "./quickpulse/types.js";
-import type { LogRecord } from "@opentelemetry/sdk-logs";
 import { Logger } from "../shared/logging/logger.js";
 import process from "node:process";
 
@@ -214,7 +214,7 @@ export class PerformanceCounterMetrics {
    * Record Log metrics
    * @internal
    */
-  public recordLog(logRecord: LogRecord): void {
+  public recordLog(logRecord: SdkLogRecord): void {
     const columns: TraceData | ExceptionData = getLogData(logRecord);
     if (isExceptionData(columns)) {
       this.totalExceptionCount++;