-
Notifications
You must be signed in to change notification settings - Fork 617
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
Move _SUPPRESS_INTRUMENTATION key from instrumentation to api #2187
Move _SUPPRESS_INTRUMENTATION key from instrumentation to api #2187
Conversation
959a490
to
d4a1f95
Compare
opentelemetry-instrumentation/src/opentelemetry/instrumentation/utils.py
Outdated
Show resolved
Hide resolved
d4a1f95
to
54a3040
Compare
54a3040
to
494ee73
Compare
582b458
to
3737ea5
Compare
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.
@owais
Are you going to be including the tox
changes in this PR as well? (to remove opentelemetry-instrumentation
dependency)
@lzchen let's not merge this for now and wait for the release. I'd like all these related changes to go in together after the next release. |
Marking it as draft to avoid accidental merging 👍 |
This will continue to be part of the internal API but live in the API context package.
3737ea5
to
24444b4
Compare
Description
This will continue to be part of the internal API but live in the API
context package.
Fixes #2184
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Does This PR Require a Contrib Repo Change?
Checklist: