open-telemetry / opentelemetry-rust-contrib

OpenTelemetry Contrib Packages for Rust
Apache License 2.0
38 stars 36 forks source link

grpc-trace-bin header support #37

Open xd009642 opened 4 years ago

xd009642 commented 4 years ago

As well as traceparent for context propagation there's also grpc-trace-bin and x-cloud-trace-context, my current work system is using grpc-trace-bin so it would be nice if it was as easy to propagate the traces as it was with traceparent or the b3 headers

jtescher commented 4 years ago

The spec discussion for this is currently happening in https://github.com/open-telemetry/opentelemetry-specification/issues/639

hdost commented 1 year ago

There's no reason this could not be implemented as an extension crate, it doesn't need to be built into the rust sdk/api.

hdost commented 8 months ago

So i think while this would provide some compatibility with open census, this doesn't seem like it will be in the root. We split the repos recently. I think we can transfer this to the contrib repo. Spec https://graphql-faas.github.io/opencensus-specs/trace/gRPC.html if anyone wants to implement.

RichardChukwu commented 1 week ago

Hi @xd009642 @hdost Is this being actively worked on at the moment please?

xd009642 commented 1 week ago

No, but I'm also not sure if it's relevant anymore? It might have gotten in elsewhere and this issue just not been cleaned up

RichardChukwu commented 1 week ago

Ok. How do we confirm that?