Skip to content

Support latest Azure SDK shim#15068

Merged
trask merged 5 commits intoopen-telemetry:mainfrom
trask:azure-sdk-update
Oct 23, 2025
Merged

Support latest Azure SDK shim#15068
trask merged 5 commits intoopen-telemetry:mainfrom
trask:azure-sdk-update

Conversation

@trask
Copy link
Copy Markdown
Member

@trask trask commented Oct 21, 2025

Resolves #15049

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

straight copy from 1.36

Comment on lines +58 to +59
// LibraryTelemetryOptions was introduced in azure-core 1.53
return hasClassesNamed("com.azure.core.util.LibraryTelemetryOptions")
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this class is straight copy from 1.36 except for this method

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

straight copy from 1.36

.hasAttributes(Attributes.empty()),
span ->
span.hasKind(SpanKind.CLIENT)
.hasName("GET")
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

small change from 1.36

Comment on lines +134 to +137
LibraryTelemetryOptions options = new LibraryTelemetryOptions("test-lib");
options.setLibraryVersion("test-version");
options.setResourceProviderNamespace("otel.tests");
return azProvider.createTracer(options, new TracingOptions());
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LibraryTelemetryOptions is new in 1.53

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

straight copy from 1.36

@otelbot-java-instrumentation
Copy link
Copy Markdown
Contributor

🔧 The result from spotlessApply was committed to the PR branch.

@trask trask marked this pull request as ready for review October 21, 2025 03:35
@trask trask requested a review from a team as a code owner October 21, 2025 03:35
@trask trask merged commit abe5cc9 into open-telemetry:main Oct 23, 2025
151 of 153 checks passed
@trask trask deleted the azure-sdk-update branch October 23, 2025 15:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Version bump request: azure-core-tracing-opentelemetry

2 participants