Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FEATURE REQ] Switch to .NET 10 SchemaURL instead of az.schema_url when possible #48535

Open
lmolkova opened this issue Mar 4, 2025 · 2 comments
Assignees
Labels
Azure.Core Client This issue points to a problem in the data-plane of the library. OpenTelemetry OpenTelemetry instrumentation (not Monitor-specific)
Milestone

Comments

@lmolkova
Copy link
Member

lmolkova commented Mar 4, 2025

Library name

Azure.Core

Please describe the feature.

Use Formally defined OTel schema url introduced in dotnet/runtime#113034 instead of custom az.schema_url attribute on .NET 10+.

Using reflection would be challenging (due to AOT), so we might need to wait until we can take dependency on the corresponding DiagnosticSource version (which ships out of band, but we still update it very slowly).

@lmolkova lmolkova added Client This issue points to a problem in the data-plane of the library. OpenTelemetry OpenTelemetry instrumentation (not Monitor-specific) Azure.Core labels Mar 4, 2025
@github-actions github-actions bot added the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label Mar 4, 2025
@jsquire jsquire removed the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label Mar 4, 2025
@jsquire jsquire added this to the Backlog milestone Mar 4, 2025
@jsquire
Copy link
Member

jsquire commented Mar 4, 2025

@m-redding: Would you please take a look at this when you start looking into OTel for SCM?

@m-redding
Copy link
Member

@lmolkova System.Diagnostics.DiagnosticSource is built into the runtime in .NET6+ so we remove the dependency on that package in those targets. I think we should be able to do this once Azure.Core adds a net10.0 target.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Azure.Core Client This issue points to a problem in the data-plane of the library. OpenTelemetry OpenTelemetry instrumentation (not Monitor-specific)
Projects
None yet
Development

No branches or pull requests

3 participants