From ae9ff35c32cbcfc865e651c92ddb5ad03158140b Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Marijan=20Petri=C4=8Devi=C4=87?= Date: Tue, 8 Oct 2024 12:42:33 -0500 Subject: [PATCH] refactor(tvix/cli/args): remove log-level argument RUST_LOG allows for a more granular tracing configuration Change-Id: Iea29fc2b154345eb7479dfd8bbea6abfd4716341 Reviewed-on: https://cl.tvl.fyi/c/depot/+/12585 Tested-by: BuildkiteCI Reviewed-by: flokli --- tvix/cli/src/args.rs | 8 -------- tvix/cli/src/main.rs | 1 - tvix/docs/src/TODO.md | 3 --- 3 files changed, 12 deletions(-) diff --git a/tvix/cli/src/args.rs b/tvix/cli/src/args.rs index 36f9a6a26..d47258f10 100644 --- a/tvix/cli/src/args.rs +++ b/tvix/cli/src/args.rs @@ -1,7 +1,6 @@ use std::path::PathBuf; use clap::Parser; -use tracing::Level; use tvix_store::utils::ServiceUrlsMemory; /// Provides a CLI interface to trigger evaluation using tvix-eval. @@ -14,13 +13,6 @@ use tvix_store::utils::ServiceUrlsMemory; /// The CLI interface is not stable and subject to change. #[derive(Parser, Clone)] pub struct Args { - /// A global log level to use when printing logs. - /// It's also possible to set `RUST_LOG` according to - /// `tracing_subscriber::filter::EnvFilter`, which will always have - /// priority. - #[arg(long, default_value_t=Level::INFO)] - pub log_level: Level, - /// Path to a script to evaluate pub script: Option, diff --git a/tvix/cli/src/main.rs b/tvix/cli/src/main.rs index 379dd915c..0bac75e0f 100644 --- a/tvix/cli/src/main.rs +++ b/tvix/cli/src/main.rs @@ -53,7 +53,6 @@ fn main() { let args = Args::parse(); let _ = tvix_tracing::TracingBuilder::default() - .level(args.log_level) .enable_progressbar() .build() .expect("unable to set up tracing subscriber"); diff --git a/tvix/docs/src/TODO.md b/tvix/docs/src/TODO.md index 9f8feef9e..7ed207581 100644 --- a/tvix/docs/src/TODO.md +++ b/tvix/docs/src/TODO.md @@ -262,9 +262,6 @@ This requires some more designing. Some goals: - Some work ongoing on the worker operation parsing (griff, picnoir) ### O11Y - - 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 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