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.

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

Update the baggage spec to include opaque metadata. #1066

Merged
merged 7 commits into from
Oct 9, 2020

Conversation

jkwatson
Copy link
Contributor

@jkwatson jkwatson commented Oct 6, 2020

Also update the description of the propagator to clarify metadata propagation.

Fixes #1050

Changes

Adds the concept of baggage entry metadata, and explains how to propagate the W3C baggage.

@jkwatson jkwatson requested review from a team October 6, 2020 17:41
CHANGELOG.md Outdated Show resolved Hide resolved
specification/baggage/api.md Show resolved Hide resolved
specification/baggage/api.md Outdated Show resolved Hide resolved
specification/baggage/api.md Show resolved Hide resolved
@jkwatson
Copy link
Contributor Author

jkwatson commented Oct 7, 2020

Note: I added some additional implementation notes on the propagator, after working on an implementation.

@bogdandrutu
Copy link
Member

Please rebase

@jkwatson
Copy link
Contributor Author

jkwatson commented Oct 7, 2020

Please rebase

done

@carlosalberto carlosalberto added area:api Cross language API specification issue release:required-for-ga Must be resolved before GA release, or nice to have before GA spec:baggage Related to the specification/baggage directory spec:context Related to the specification/context directory labels Oct 7, 2020
@arminru arminru merged commit ad9a913 into open-telemetry:master Oct 9, 2020
carlosalberto pushed a commit to carlosalberto/opentelemetry-specification that referenced this pull request Oct 31, 2024
)

Co-authored-by: Armin Ruech <armin.ruech@dynatrace.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:api Cross language API specification issue release:required-for-ga Must be resolved before GA release, or nice to have before GA spec:baggage Related to the specification/baggage directory spec:context Related to the specification/context directory
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Baggage specification is not in line with the W3C specification; propagator mapping is not defined
6 participants