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

Add a section for OTel specific values in TraceState. #1852

Merged
merged 30 commits into from
Oct 15, 2021
Merged
Changes from 6 commits
Commits
Show all changes
30 commits
Select commit Hold shift + click to select a range
e3479a6
Add a section for OTel specific values in TraceState.
carlosalberto Aug 5, 2021
00cb453
Update specification/trace/api.md
carlosalberto Aug 5, 2021
59c30d1
Update specification/trace/api.md
carlosalberto Aug 5, 2021
a9edb5f
Fix sample.
carlosalberto Aug 5, 2021
aa51b8e
Merge branch 'main' into trace_state_otel_values
carlosalberto Aug 11, 2021
ba7b543
Apply feedback.
carlosalberto Aug 11, 2021
885c99a
Use same notation as TraceContext.
carlosalberto Aug 19, 2021
223c5cf
Remove unused OWS mention.
carlosalberto Aug 19, 2021
e067da0
Fix link.
carlosalberto Aug 19, 2021
0fb7420
Merge branch 'main' into trace_state_otel_values
carlosalberto Aug 26, 2021
bc36949
Update specification/trace/sdk.md
carlosalberto Aug 26, 2021
7a839f7
Clarify values.
carlosalberto Aug 26, 2021
85e9a9c
Prevent larger than 256 char values.
carlosalberto Aug 26, 2021
b8dc7bd
Fix typo.
carlosalberto Aug 26, 2021
3e7bf62
Update specification/trace/sdk.md
carlosalberto Aug 26, 2021
24c1dd6
Merge branch 'main' into trace_state_otel_values
carlosalberto Sep 1, 2021
47fde3f
Move trace state handling to its own document.
carlosalberto Sep 2, 2021
75f01b1
Update specification/trace/tracestate-handling.md
carlosalberto Sep 2, 2021
e1a1298
Fix lint.
carlosalberto Sep 2, 2021
7ac0e3c
Apply feedback.
carlosalberto Sep 15, 2021
7254826
Fix typo.
carlosalberto Sep 15, 2021
cfafa77
s/SHOULD/MUST
carlosalberto Sep 15, 2021
238fe23
Merge branch 'main' into trace_state_otel_values
carlosalberto Sep 15, 2021
f16c34c
Merge branch 'main' into trace_state_otel_values
carlosalberto Sep 20, 2021
e84f79c
Merge branch 'main' into trace_state_otel_values
carlosalberto Sep 29, 2021
845ddec
Merge branch 'main' into trace_state_otel_values
carlosalberto Oct 4, 2021
287bd68
Update specification/trace/tracestate-handling.md
carlosalberto Oct 14, 2021
3e8fb99
Merge branch 'main' into trace_state_otel_values
carlosalberto Oct 14, 2021
92c01c2
Merge branch 'main' into trace_state_otel_values
carlosalberto Oct 15, 2021
a04d104
Merge branch 'main' into trace_state_otel_values
jmacd Oct 15, 2021
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
19 changes: 19 additions & 0 deletions specification/trace/sdk.md
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@
* [Id Generator](#id-generators)
* [Span Processor](#span-processor)
* [Span Exporter](#span-exporter)
* [OTel TraceState values](#otel-tracestate-values)

</details>

Expand Down Expand Up @@ -680,5 +681,23 @@ public interface SpanExporter {
}
```

## OTel TraceState values
jmacd marked this conversation as resolved.
Show resolved Hide resolved

When setting [TraceState](api#tracestate) values that are part of the OTel ecosystem,
they MUST all be contained in a single entry using the `ot` key, with the value being
a semicolon separated list of key-value pairs such as:

* `ot=p:8;r:64`
jmacd marked this conversation as resolved.
Show resolved Hide resolved
* `ot=foo:bar;k1:13`

Set values MUST be either updated or added to the `ot` entry in `TraceState`,
in order to preserve existing values belonging to other OTel concerns. For example,
if a given concern K wants to set `k1:13`:
yurishkuro marked this conversation as resolved.
Show resolved Hide resolved

* `ot=p:8;r:64` will become `ot=p:8;r:64;k1:13`.
* `ot=p:8,r:64;k1:7` will become `ot:p8;r:64;k1:13`.
jmacd marked this conversation as resolved.
Show resolved Hide resolved

Keys and values MUST only contain alphanumeric characters.
carlosalberto marked this conversation as resolved.
Show resolved Hide resolved

[trace-flags]: https://www.w3.org/TR/trace-context/#trace-flags
[otep-83]: https://github.com/open-telemetry/oteps/blob/main/text/0083-component.md