Skip to content
This repository was archived by the owner on Aug 1, 2025. It is now read-only.

Commit 3f4600c

Browse files
[Backport M104] fix: remove OTEL metrics (#8070)
This is not quite used feature, removing for now. ## Test plan - N/A <br> Backport d9c6213 from #8069 Co-authored-by: fengzhou-sg <feng.zhou@sourcegraph.com>
1 parent 5e47a65 commit 3f4600c

File tree

1 file changed

+1
-24
lines changed

1 file changed

+1
-24
lines changed

vscode/src/services/open-telemetry/OpenTelemetryService.node.ts

Lines changed: 1 addition & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -16,9 +16,7 @@ import {
1616
startWith,
1717
} from '@sourcegraph/cody-shared'
1818

19-
import { DiagConsoleLogger, DiagLogLevel, diag, metrics } from '@opentelemetry/api'
20-
import { OTLPMetricExporter } from '@opentelemetry/exporter-metrics-otlp-http'
21-
import { MeterProvider, PeriodicExportingMetricReader } from '@opentelemetry/sdk-metrics'
19+
import { DiagConsoleLogger, DiagLogLevel, diag } from '@opentelemetry/api'
2220
import { BatchSpanProcessor } from '@opentelemetry/sdk-trace-base'
2321
import { externalAuthRefresh } from '@sourcegraph/cody-shared/src/configuration/auth-resolver'
2422
import { isEqual } from 'lodash'
@@ -35,7 +33,6 @@ export interface OpenTelemetryServiceConfig {
3533
extensionVersion?: string
3634
}
3735
export class OpenTelemetryService {
38-
private meterProvider?: MeterProvider
3936
private tracerProvider?: NodeTracerProvider
4037
private spanProcessors: BatchSpanProcessor[] = []
4138
private unloadInstrumentations?: () => void
@@ -65,11 +62,6 @@ export class OpenTelemetryService {
6562
// Register once at startup
6663
this.tracerProvider.register()
6764

68-
this.meterProvider = new MeterProvider({
69-
resource,
70-
})
71-
metrics.setGlobalMeterProvider(this.meterProvider)
72-
7365
this.configSubscription = combineLatest(
7466
resolvedConfig,
7567
featureFlagProvider.evaluatedFeatureFlag(FeatureFlag.CodyAutocompleteTracing),
@@ -85,18 +77,6 @@ export class OpenTelemetryService {
8577
const headers = new Headers()
8678
await addAuthHeaders(auth, headers, traceUrl)
8779

88-
const metricUrl = new URL('/-/debug/otlp/v1/metrics', auth.serverEndpoint)
89-
this.meterProvider?.addMetricReader(
90-
new PeriodicExportingMetricReader({
91-
exporter: new OTLPMetricExporter({
92-
url: metricUrl.toString(),
93-
headers: Object.fromEntries(headers.entries()),
94-
}),
95-
// export metrics every minute
96-
exportIntervalMillis: 60 * 1000,
97-
})
98-
)
99-
10080
const newConfig = {
10181
isTracingEnabled: this.isTracingEnabled,
10282
traceUrl: traceUrl.toString(),
@@ -135,9 +115,6 @@ export class OpenTelemetryService {
135115

136116
public dispose(): void {
137117
this.configSubscription.unsubscribe()
138-
this.meterProvider
139-
?.shutdown()
140-
.catch(error => console.error('Error shutting down meter provider:', error))
141118
this.reset().catch(error => console.error('Error disposing OpenTelemetry:', error))
142119
}
143120

0 commit comments

Comments
 (0)