From 618aacaa61972c3e25b8c996abfa1d4dc475154e Mon Sep 17 00:00:00 2001 From: Simon Hauser Date: Thu, 27 Jun 2024 10:27:29 +0200 Subject: feat(tvix/tracing): http trace propagation Introduces a helper function within tvix-tracing that returns a reqwest tracing middleware that will ingest the traceparent if otlp is enabled. It is feature flagged in tvix-tracing so not every consumer of that library automatically has reqwest in its dependencies. Tested using netcat to verify that the `traceparent` header is there if otlp is enabled and missing if otlp feature is disabled. Change-Id: I5abccae777b725f5ff7382e3686165383c477a39 Reviewed-on: https://cl.tvl.fyi/c/depot/+/11886 Tested-by: BuildkiteCI Reviewed-by: flokli --- tvix/docs/src/TODO.md | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) (limited to 'tvix/docs') diff --git a/tvix/docs/src/TODO.md b/tvix/docs/src/TODO.md index 127fb6f4d0c1..52fa0a9e7480 100644 --- a/tvix/docs/src/TODO.md +++ b/tvix/docs/src/TODO.md @@ -224,7 +224,6 @@ logs etc, but this is something requiring a lot of designing. - Maybe drop `--log-level` entirely, and only use `RUST_LOG` env exclusively? `debug`,`trace` level across all crates is a bit useless, and `RUST_LOG` can be much more granular… - - Trace propagation for HTTP clients too, using - https://www.w3.org/TR/trace-context/ or https://www.w3.org/TR/baggage/, - whichever makes more sense. - Candidates: nix+http(s) protocol, object_store crates. + - Trace propagation for object_store once they support a way to register a + middleware, so we can use that to register a tracing middleware. + https://github.com/apache/arrow-rs/issues/5990 -- cgit 1.4.1