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.

Sign up for GitHub

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

Jump to bottom

Consider make tracing a global setting instead of per client #21817

Open
jhendrixMSFT opened this issue Oct 23, 2023 · 1 comment
Open

Consider make tracing a global setting instead of per client #21817

jhendrixMSFT opened this issue Oct 23, 2023 · 1 comment
Labels
Azure.Core design-discussion An area of design currently under discussion and open to team and community feedback. OpenTelemetry OpenTelemetry instrumentation (not Monitor-specific)
Milestone
Backlog

Comments

@jhendrixMSFT
Copy link
Member

Conceptually this makes sense and is similar to how other SDKs (and logging) work.
@jhendrixMSFT jhendrixMSFT added Azure.Core OpenTelemetry OpenTelemetry instrumentation (not Monitor-specific) labels Oct 23, 2023
@jhendrixMSFT jhendrixMSFT self-assigned this Oct 23, 2023
@serbrech
Copy link
Member

Aligned.
For our usage, there is not a case where we would want to enable tracing per client that I'm aware of.
We could imagine a case where we would want to disable it for a specific client/call, but I would call YAGNI on this until someone asks for it

@RickWinter RickWinter added this to the Backlog milestone Aug 9, 2024
@RickWinter RickWinter added the design-discussion An area of design currently under discussion and open to team and community feedback. label Jan 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Azure.Core design-discussion An area of design currently under discussion and open to team and community feedback. OpenTelemetry OpenTelemetry instrumentation (not Monitor-specific)
Projects
None yet
Milestone
Backlog
Development

No branches or pull requests
3 participants
@serbrech @RickWinter @jhendrixMSFT