search

open-telemetry / opentelemetry.io

The OpenTelemetry website and documentation
https://opentelemetry.io
Creative Commons Attribution 4.0 International
520 stars 1.11k forks source link

Rework baggage page to be less trace-specific #4692

Closed cartermp closed 3 months ago

cartermp commented 3 months ago

Should fix https://github.com/open-telemetry/opentelemetry.io/issues/3674 and https://github.com/open-telemetry/opentelemetry.io/issues/1878#issuecomment-2158675491

Tried to capture how Baggage isn't trace-specific while still honoring the reality that it's usually used the most in conjunction with tracing.

cc @cijothomas

cartermp commented 3 months ago

/fix:format

opentelemetrybot commented 3 months ago

You triggered fix:format action run at https://github.com/open-telemetry/opentelemetry.io/actions/runs/9529813909

austinlparker commented 3 months ago

Would it be worth calling out somehow that baggage is only intended for Observability data, and that it's not recommended to use it to side channel values for other purposes (control flow, etc)?

cartermp commented 3 months ago

@austinlparker added

cartermp commented 3 months ago

/fix:format

opentelemetrybot commented 3 months ago

You triggered fix:format action run at https://github.com/open-telemetry/opentelemetry.io/actions/runs/9531320659

cartermp commented 3 months ago

/fix:format

opentelemetrybot commented 3 months ago

You triggered fix:format action run at https://github.com/open-telemetry/opentelemetry.io/actions/runs/9682963027

cartermp commented 3 months ago

@theletterf any final thoughts?