-
Notifications
You must be signed in to change notification settings - Fork 887
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
Add Tracer#Enabled, Instrument#Enabled and corresponding SDK behavior #4063
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
MrAlias
approved these changes
May 28, 2024
trask
reviewed
May 28, 2024
trask
approved these changes
May 28, 2024
trask
reviewed
May 28, 2024
Co-authored-by: Trask Stalnaker <trask.stalnaker@gmail.com>
reyang
reviewed
May 29, 2024
reyang
reviewed
May 31, 2024
reyang
approved these changes
May 31, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM from feature parity perspective.
reyang
changed the title
Add Tracer#Enabled, Meter#Enabled and corresponding SDK behavior
Add Tracer#Enabled, Instrument#Enabled and corresponding SDK behavior
May 31, 2024
carlosalberto
approved these changes
May 31, 2024
MrAlias
approved these changes
May 31, 2024
lmolkova
reviewed
May 31, 2024
lmolkova
approved these changes
Jun 3, 2024
jack-berg
commented
Jun 3, 2024
jack-berg
added a commit
to jack-berg/opentelemetry-specification
that referenced
this pull request
Jun 4, 2024
jack-berg
added a commit
to jack-berg/opentelemetry-specification
that referenced
this pull request
Jun 4, 2024
reyang
pushed a commit
that referenced
this pull request
Jun 6, 2024
This was referenced Jun 12, 2024
This was referenced Sep 3, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Builds off #4020 to extend the
Enabled
API toTracer
and metricsInstrument
.Adds language to Log SDK, Metrics SDK, and Trace SDK for how scope config is used to resolve the
Enabled
operation.This was discussed in #3867 but punted on in #3877 as discussed here.