Skip to content

tracer: quiet routine OpenTelemetry exporter errors#1075

Merged
keegancsmith merged 1 commit into
mainfrom
k/quiet-otel-exporter-errors
Jun 22, 2026
Merged

tracer: quiet routine OpenTelemetry exporter errors#1075
keegancsmith merged 1 commit into
mainfrom
k/quiet-otel-exporter-errors

Conversation

@keegancsmith

Copy link
Copy Markdown
Member

Zoekt's OpenTelemetry tracer currently installs an SDK error handler that writes exporter errors directly with the standard logger. In environments where tracing is enabled but no collector is listening, such as local development, the exporter reports periodic connection failures that are expected but noisy.

This routes those SDK errors through sourcegraph/log at debug level instead, matching the behavior of Sourcegraph's monolith tracer. The diagnostics remain available when debug logging is enabled, while normal logs stay focused on actionable service output.

Zoekt can run with OpenTelemetry enabled in environments where no local collector is available. The SDK then reports exporter connection failures periodically, which are useful diagnostics only when debug logging is enabled.

Route those errors through sourcegraph/log at debug level, matching Sourcegraph's tracer behavior, so normal local development does not get spammed by expected collector connection failures.

Amp-Thread-ID: https://ampcode.com/threads/T-019eef33-d3d1-757a-9828-8df7e7b6f889
Co-authored-by: Amp <amp@ampcode.com>
@keegancsmith keegancsmith marked this pull request as ready for review June 22, 2026 12:11
@keegancsmith keegancsmith changed the title fix/tracer: quiet routine OpenTelemetry exporter errors tracer: quiet routine OpenTelemetry exporter errors Jun 22, 2026
@keegancsmith keegancsmith merged commit f80c7e0 into main Jun 22, 2026
7 checks passed
@keegancsmith keegancsmith deleted the k/quiet-otel-exporter-errors branch June 22, 2026 12:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants