Baggage not included w/ OpenTelemetry about serilog.enrichers.span HOT 6 CLOSED

My understanding was that the OTEL API was just a wrapper around Activity but with the correct names. Is that incorrect?

from serilog.enrichers.span.

Looks like that's not not the case. There's not much documentation around it but:

open-telemetry/opentelemetry-dotnet#1842

and

https://github.com/open-telemetry/opentelemetry-dotnet/blob/main/src/OpenTelemetry.Api/README.md#baggage-api

The recommended way to add Baggage is to use the Baggage.SetBaggage() API. OpenTelemetry users should not use the Activity.AddBaggage method.

I think the idea here is that in OTEL API Baggage isn't tied to an Activity (Span) context; it's a separate/standalone thing with its own propagator and whatnot.

It's all pretty confusing, because OTEL Baggage is propagated using standard W3C headers (as long you're using the OTEL propagators). And a receiving app using ASP.NET Core's built in diagnostics (without OTEL) will see those headers and set them on Activity.Baggage, where this Serilog Enricher will find them.

But the limitation there is that OTEL Baggage isn't copied into Activity.Baggage until that propagation has occurred. So for an app that does Baggage.SetBaggage(), this Serilog Enricher won't see that data and include it in the logs.

Maybe this is an issue for OTEL .NET to address -- maybe Baggage.SetBaggage() should be copying values into Activity.Baggage() But it's really hard to know what's most correct here...

from serilog.enrichers.span.

The key lines in the second link:

It is important to note that Baggage is not automatically attached to any telemetry. User can explicitly read Baggage and use it to enrich metrics, logs and traces. An example of doing this for traces is shown here.

It sounds like you have to copy baggage manually?

from serilog.enrichers.span.

I think the idea there is that OTEL telemetry data (Traces, Metrics, Logs) is not automatically enriched with Baggage. If you want it, you have to be explicit.

This also underscores how, in the OTEL specification, Baggage is a standalone thing and not tied to Traces/Activities, which is probably why OTEL .NET is trying to move away from Activity.Baggage.

Since this project is a Serilog Enricher, that's why I raised the issue here. But I realize Serilog != OTEL Logging, and I don't know what it'd look like to bring those things together. For now, I can certainly add a tiny OtelBaggageEnricher to Serilog to solve this problem.

from serilog.enrichers.span.

This is certainly confusing. I'm not sure this project should do anything about this though. We'd also need to add a reference to the OTEL package if we decide to. Should we close for now?

from serilog.enrichers.span.

Confusing indeed. Especially when OTEL acts as if Baggage is a separate thing from Trace, but recommends using "W3C format for distributed trace" to propagate it. But these things are in various levels of draft/experimental/beta, so probably premature to decide how Serilog (and this Enricher) should participate in that world, and maybe another package someday to avoid the extra dependency.

I'll go ahead and close. Hope this issue / bread crumb helps someone else!

from serilog.enrichers.span.