-
Notifications
You must be signed in to change notification settings - Fork 762
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
Feat support otel sdk disabled env #3485
Feat support otel sdk disabled env #3485
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #3485 +/- ##
==========================================
+ Coverage 93.74% 93.77% +0.02%
==========================================
Files 249 249
Lines 7593 7610 +17
Branches 1582 1587 +5
==========================================
+ Hits 7118 7136 +18
+ Misses 475 474 -1
|
Might be useful to also document it in the README for the SDK package |
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.
Thank you for working on this! LGTM % nits.
comment fix tests for falsy OTEL_SDK_DISABLED values
Looks like there is a lint error. Would you mind fixing it? Thank you! |
Which problem is this PR solving?
Support for the
OTEL_SDK_DISABLED
env variable.Short description of the changes
In the case of a truthy value, the SDK functions(
start
,detectResources
) that might create side effects are set to be no-op.Parsing of boolean env variables.
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
sdk.start
should be a no-op. Should not register a trace/meter provider.sdk.detectResources
should be a no-op. Should not detect resources.Checklist: