From 87bda3ae7aaf459cd30672164587a3fe246d209a Mon Sep 17 00:00:00 2001 From: Florian Klink Date: Wed, 14 Feb 2024 08:58:10 +0700 Subject: feat(tvix/glue): tune instrumentations in TvixStoreIO Print store paths with their ToString implementation for brevity, and don't log the sucessful return value of read_to_string. Change-Id: I01b6838398acd66b8818095622f361fcca26fa77 Reviewed-on: https://cl.tvl.fyi/c/depot/+/10854 Autosubmit: flokli Reviewed-by: raitobezarius Tested-by: BuildkiteCI --- tvix/glue/src/tvix_store_io.rs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'tvix/glue/src') diff --git a/tvix/glue/src/tvix_store_io.rs b/tvix/glue/src/tvix_store_io.rs index ed06eba89686..a3e4474a66f7 100644 --- a/tvix/glue/src/tvix_store_io.rs +++ b/tvix/glue/src/tvix_store_io.rs @@ -74,7 +74,7 @@ impl TvixStoreIO { /// /// In case there is no PathInfo yet, this means we need to build it /// (which currently is stubbed out still). - #[instrument(skip(self), ret, err)] + #[instrument(skip(self, store_path), fields(store_path=%store_path), ret, err)] async fn store_path_to_node( &self, store_path: &StorePath, @@ -138,7 +138,7 @@ impl EvalIO for TvixStoreIO { } } - #[instrument(skip(self), ret, err)] + #[instrument(skip(self), err)] fn read_to_string(&self, path: &Path) -> io::Result { if let Ok((store_path, sub_path)) = StorePath::from_absolute_path_full(&path.to_string_lossy()) -- cgit 1.4.1