Changing env var that flags OTLPSpanExporter usage

Author: rodrigobr-msft

libraries/microsoft-agents-a365-observability-core/microsoft_agents_a365/observability/core/config.py

@@ -1,8 +1,8 @@
 # Copyright (c) Microsoft Corporation.
 # Licensed under the MIT License.
 
-import logging
 import os
+import logging
 import threading
 from collections.abc import Callable
 from typing import Any, Optional
@@ -184,7 +184,7 @@ def _configure_internal(
         self._span_processors["batch"] = batch_processor
         self._span_processors["agent"] = agent_processor
 
-        if os.environ.get("OTEL_EXPORTER_OTLP_ENDPOINT"):
+        if os.environ.get("ENABLE_OTLP_EXPORTER", "").lower() == "true":
             # The OTLPSpanExporter is auto configured from the environment variables
             otlp_exporter = OTLPSpanExporter()
             tracer_provider.add_span_processor(

tests/observability/core/test_agent365.py

@@ -206,9 +206,9 @@ def test_configure_uses_existing_tracer_provider(self, mock_get_provider, mock_i
         "microsoft_agents_a365.observability.core.config.is_agent365_exporter_enabled",
         return_value=False,
     )
-    @patch.dict("os.environ", {"OTEL_EXPORTER_OTLP_ENDPOINT": "http://localhost:4318"}, clear=True)
+    @patch.dict("os.environ", {"ENABLE_OTLP_EXPORTER": "true"}, clear=True)
     def test_otlp_exporter_initialized_when_env_var_set(self, mock_is_enabled, mock_otlp_exporter):
-        """Test that OTLPSpanExporter is initialized when OTEL_EXPORTER_OTLP_ENDPOINT is set."""
+        """Test that OTLPSpanExporter is initialized when ENABLE_OTLP_EXPORTER is set."""
 
         result = configure(
             service_name="test-service",
@@ -227,7 +227,7 @@ def test_otlp_exporter_initialized_when_env_var_set(self, mock_is_enabled, mock_
     def test_otlp_exporter_not_initialized_when_env_var_not_set(
         self, mock_is_enabled, mock_otlp_exporter
     ):
-        """Test that OTLPSpanExporter is NOT initialized when OTEL_EXPORTER_OTLP_ENDPOINT is not set."""
+        """Test that OTLPSpanExporter is NOT initialized when ENABLE_OTLP_EXPORTER is not set."""
 
         result = configure(
             service_name="test-service",