open-telemetry / opentelemetry-dotnet

The OpenTelemetry .NET Client
https://opentelemetry.io
Apache License 2.0
3.16k stars 748 forks source link

Can .NET support "No explicit parent Span/SpanContext allowed" spec item? #1676

Open Oberon00 opened 3 years ago

Oberon00 commented 3 years ago

The tracing API specification was changed on September 23 to only allow a full Context as explicit parent, and a SpanContext alone or Span alone is no longer allowed. Spec references:

(How) is .NET going to implement this change?

Relatedly, I see the spec compliance matrix has "unknown" listed in this row https://github.com/open-telemetry/opentelemetry-specification/blob/master/spec-compliance-matrix.md#traces "No explicit parent Span/SpanContext allowed".

cijothomas commented 3 years ago

Thanks for raising this. Can't think of an easy fix here in Otel repo. We need to plan and propose something for the next .NET cycle, as this won't be a trivial change. Keeping this issue open.

github-actions[bot] commented 2 weeks ago

This issue was marked stale due to lack of activity and will be closed in 7 days. Commenting will instruct the bot to automatically remove the label. This bot runs once per day.

cijothomas commented 2 weeks ago

Un-stale. This issue is unresolved. Due to the use of async-local everywhere to pass around implicit Context, I think no-one is actually hitting any issue, but its still better to keep issue as open given this is a spec requirement that .NET is not able to comply with.