From 2180e6b820ee14103d11af925738173599b61ff7 Mon Sep 17 00:00:00 2001 From: Florian Klink Date: Wed, 25 Sep 2024 22:26:41 +0200 Subject: [PATCH] fix(tvix/castore): don't return object_store::has with INFO level This otherwise spams the logs quite a bit, for example when uploading to nar-bridge with the new check from cl/12497. Change-Id: Idc2bcc513caea6fae38ae04489e4e3ee7be64bce Reviewed-on: https://cl.tvl.fyi/c/depot/+/12510 Reviewed-by: yuka Autosubmit: flokli Tested-by: BuildkiteCI --- tvix/castore/src/blobservice/object_store.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tvix/castore/src/blobservice/object_store.rs b/tvix/castore/src/blobservice/object_store.rs index 5bb05cf26..b688ebafc 100644 --- a/tvix/castore/src/blobservice/object_store.rs +++ b/tvix/castore/src/blobservice/object_store.rs @@ -92,7 +92,7 @@ fn derive_chunk_path(base_path: &Path, digest: &B3Digest) -> Path { #[async_trait] impl BlobService for ObjectStoreBlobService { - #[instrument(skip_all, ret, err, fields(blob.digest=%digest))] + #[instrument(skip_all, ret(level = Level::TRACE), err, fields(blob.digest=%digest))] async fn has(&self, digest: &B3Digest) -> io::Result { // TODO: clarify if this should work for chunks or not, and explicitly // document in the proto docs.