[Core] Make OTel the preferred tracing plugin (#35050)

pvaneck · web-flow · commit 1d4fe36b8053 · 2024-07-01T17:15:06.000-07:00
This changes the order for which we try to use tracing plugins. Now, the
OTel plugin will be checked first.

Signed-off-by: Paul Van Eck &lt;paulvaneck@microsoft.com&gt;
diff --git a/sdk/core/azure-core/CHANGELOG.md b/sdk/core/azure-core/CHANGELOG.md
@@ -10,6 +10,8 @@
 
 ### Other Changes
 
+- The Azure Core OpenTelemetry tracing plugin will now be the preferred tracing plugin over the OpenCensus plugin. If both plugins are installed and `opentelemetry` is imported, then OpenTelemetry will be used to trace Azure SDK operations.  #35050
+
 ## 1.30.2 (2024-06-06)
 
 ### Features Added
diff --git a/sdk/core/azure-core/azure/core/settings.py b/sdk/core/azure-core/azure/core/settings.py
@@ -181,7 +181,7 @@ def convert_tracing_impl(value: Optional[Union[str, Type[AbstractSpan]]]) -> Opt
     """
     if value is None:
         return (
-            _get_opencensus_span_if_opencensus_is_imported() or _get_opentelemetry_span_if_opentelemetry_is_imported()
+            _get_opentelemetry_span_if_opentelemetry_is_imported() or _get_opencensus_span_if_opencensus_is_imported()
         )
 
     if not isinstance(value, str):

Original file line number	Diff line number	Diff line change
`@@ -181,7 +181,7 @@ def convert_tracing_impl(value: Optional[Union[str, Type[AbstractSpan]]]) -> Opt`
`181`	`181`	`"""`
`182`	`182`	`if value is None:`
`183`	`183`	`return (`
`184`		`- _get_opencensus_span_if_opencensus_is_imported() or _get_opentelemetry_span_if_opentelemetry_is_imported()`
	`184`	`+ _get_opentelemetry_span_if_opentelemetry_is_imported() or _get_opencensus_span_if_opencensus_is_imported()`
`185`	`185`	`)`
`186`	`186`
`187`	`187`	`if not isinstance(value, str):`