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.

Sign up for GitHub

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

Jump to bottom

Baggage specification is not in line with the W3C specification; propagator mapping is not defined #1050

Closed
jkwatson opened this issue Oct 2, 2020 · 1 comment · Fixed by #1066
Closed

Baggage specification is not in line with the W3C specification; propagator mapping is not defined #1050

jkwatson opened this issue Oct 2, 2020 · 1 comment · Fixed by #1066
Assignees
@jkwatson
Labels
area:api Cross language API specification issue priority:p1 Highest priority level release:required-for-ga Must be resolved before GA release, or nice to have before GA spec:baggage Related to the specification/baggage directory

Comments

@jkwatson
Copy link
Contributor

jkwatson commented Oct 2, 2020

What are you trying to achieve?
I'm starting to work on a W3C Baggage Propagator, and I'm finding that the mapping from W3C baggage headers it not specified for the OpenTelemetry Baggage. In particular, the W3C spec supports metadata associated with each key/value pair, and the OTel spec doesn't not specify any metadata.

What did you expect to see?

A specification of how to map W3C baggage header(s) to the OTel Baggage implementation (and back).

Additional context.
@jkwatson jkwatson added the spec:baggage Related to the specification/baggage directory label Oct 2, 2020
@arminru
Copy link
Member

arminru commented Oct 6, 2020

@open-telemetry/technical-committee should this be required-for-ga?

@arminru arminru added the release:required-for-ga Must be resolved before GA release, or nice to have before GA label Oct 6, 2020
@andrewhsu andrewhsu added priority:p1 Highest priority level area:api Cross language API specification issue labels Oct 6, 2020
@jkwatson jkwatson mentioned this issue Oct 6, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jkwatson jkwatson

Labels
area:api Cross language API specification issue priority:p1 Highest priority level release:required-for-ga Must be resolved before GA release, or nice to have before GA spec:baggage Related to the specification/baggage directory
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update the baggage spec to include opaque metadata. jkwatson/opentelemetry-specification
3 participants
@andrewhsu @jkwatson @arminru