fix: support 'none' value for OTEL exporters to disable telemetry

- Filter "none" from OTEL_TRACES_EXPORTER and OTEL_METRICS_EXPORTER
- Return None from get_trace_provider/get_metric_provider when no exporters
- Prevents NotImplementedError on plugin startup with OTEL disabled
- Bump version 0.0.41 → 0.0.42

Fixes plugin crash when OTEL_TRACES_EXPORTER=none or OTEL_METRICS_EXPORTER=none

Author: micmarty-deepsense

CHANGELOG.md

@@ -1,3 +1,7 @@
+## 0.0.42
+
+* **Support "none" value for OTEL_TRACES_EXPORTER and OTEL_METRICS_EXPORTER** - Filter "none" from exporter lists and return None when no exporters configured to properly disable OpenTelemetry instrumentation
+
 ## 0.0.39
 
 * **Remove wrap_error logic as exceptions are categorized in unstructured-ingest**

unstructured_platform_plugins/__version__.py

@@ -1 +1 @@
-__version__ = "0.0.41"  # pragma: no cover
+__version__ = "0.0.42"  # pragma: no cover

unstructured_platform_plugins/etl_uvicorn/otel.py

@@ -30,18 +30,23 @@ def get_settings() -> OtelSettings:
     service_name = os.environ.get(OTEL_SERVICE_NAME, "unknown_service")
     trace_exporters = os.environ.get(OTEL_TRACES_EXPORTER)
     trace_exporters = trace_exporters.split(",") if trace_exporters else []
+    trace_exporters = [e for e in trace_exporters if e != "none"]
 
     metric_exporters = os.environ.get(OTEL_METRICS_EXPORTER)
     metric_exporters = metric_exporters.split(",") if metric_exporters else []
+    metric_exporters = [e for e in metric_exporters if e != "none"]
     return OtelSettings(
         service_name=service_name,
         trace_exporters=trace_exporters,
         metric_exporters=metric_exporters,
     )
 
 
-def get_trace_provider() -> TracerProvider:
+def get_trace_provider() -> TracerProvider | None:
     settings = get_settings()
+    if not settings["trace_exporters"]:
+        return None
+
     provider = TracerProvider(resource=Resource({SERVICE_NAME: settings["service_name"]}))
 
     for trace_exporter_type in settings["trace_exporters"]:
@@ -50,8 +55,11 @@ def get_trace_provider() -> TracerProvider:
     return provider
 
 
-def get_metric_provider() -> MeterProvider:
+def get_metric_provider() -> MeterProvider | None:
     settings = get_settings()
+    if not settings["metric_exporters"]:
+        return None
+
     readers = []
     for metric_exporter_type in settings["metric_exporters"]:
         readers.append(_get_metrics_reader(exporter_type=metric_exporter_type))