-
Notifications
You must be signed in to change notification settings - Fork 685
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
Merge main 2 #2199
Merge main 2 #2199
Conversation
* Add regexes to check keys and values Fixes open-telemetry#2010 * Fix tests * Add changelog * Fix test * Add checks to set_baggage * Fix lint * Add changelog entry * Fix mypy * Fix mypy * Cast to string * WIP * Key value format * Mostly done * Remove old changelog entry * fomat * Correct typing * Fix lint * Fix issues * Add CHANGELOG entry * Make changes as discussed in SIG meeting * Update opentelemetry-api/src/opentelemetry/baggage/__init__.py Co-authored-by: Leighton Chen <[email protected]> Co-authored-by: Diego Hurtado <[email protected]> Co-authored-by: Leighton Chen <[email protected]>
* Adding thread name * Updating CHANGELOG
* Move _SUPPRESS_INTRUMENTATION key from instrumentation to api This will continue to be part of the internal API but live in the API context package. * Move BaseConfigurator from isntrumentation to SDK This allows SDK to not depend on instrumentation package anymore.
…elemetry#2187) This will continue to be part of the internal API but live in the API context package.
@ocelotl this is merging metrics_new into main. To get rid of the merge conflict markers, I think you just need a PR to metrics_new fixing the unresolved files |
Conflicts are in changelog as well, so I fixed them in the same PR |
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
Merge
main
intometrics_new
. Fixes #2194