append /v1/traces to base OTEL_EXPORTER_OTLP_ENDPOINT fallback

Author: liustve

aws-opentelemetry-distro/src/amazon/opentelemetry/distro/aws_opentelemetry_configurator.py

@@ -330,9 +330,11 @@ def _export_unsampled_span_for_agent_observability(trace_provider: TracerProvide
     if not is_agent_observability_enabled():
         return
 
-    traces_endpoint = os.environ.get(OTEL_EXPORTER_OTLP_TRACES_ENDPOINT) or os.environ.get(
-        "OTEL_EXPORTER_OTLP_ENDPOINT"
-    )
+    traces_endpoint = os.environ.get(OTEL_EXPORTER_OTLP_TRACES_ENDPOINT)
+    if not traces_endpoint:
+        base_endpoint = os.environ.get("OTEL_EXPORTER_OTLP_ENDPOINT")
+        if base_endpoint:
+            traces_endpoint = base_endpoint.rstrip("/") + "/v1/traces"
     if not traces_endpoint:
         return
 

aws-opentelemetry-distro/tests/amazon/opentelemetry/distro/test_aws_opentelementry_configurator.py

@@ -1110,6 +1110,7 @@ def test_export_unsampled_span_for_agent_observability(self):
         self.assertEqual(mock_tracer_provider.add_span_processor.call_count, 1)
         processor = mock_tracer_provider.add_span_processor.call_args_list[0].args[0]
         self.assertIsInstance(processor, BatchUnsampledSpanProcessor)
+        self.assertEqual(processor.span_exporter._endpoint, "http://localhost:4318/v1/traces")
 
         os.environ.pop("AGENT_OBSERVABILITY_ENABLED", None)
         os.environ.pop("OTEL_EXPORTER_OTLP_ENDPOINT", None)