From ef64fedbd929239bbca7db7b18c6a6ebe637b8b6 Mon Sep 17 00:00:00 2001 From: Wesley Aptekar-Cassels Date: Mon, 20 Mar 2023 20:49:52 -0400 Subject: [PATCH] Only enable OTLP exporter when configured This wasn't a problem in the past, since we only enabled automatic instrumentation when this was set up, but it does cause errors when trying to add manual instrumentation. --- bookwyrm/telemetry/open_telemetry.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/bookwyrm/telemetry/open_telemetry.py b/bookwyrm/telemetry/open_telemetry.py index aff68a8b3..e405011d9 100644 --- a/bookwyrm/telemetry/open_telemetry.py +++ b/bookwyrm/telemetry/open_telemetry.py @@ -10,7 +10,7 @@ if settings.OTEL_EXPORTER_CONSOLE: trace.get_tracer_provider().add_span_processor( BatchSpanProcessor(ConsoleSpanExporter()) ) -else: +elif settings.OTEL_EXPORTER_OTLP_ENDPOINT: trace.get_tracer_provider().add_span_processor( BatchSpanProcessor(OTLPSpanExporter()) )