From afa5b0af39eed507e118a39e547cf237c1302e2f Mon Sep 17 00:00:00 2001 From: Arne Beer Date: Sun, 25 Aug 2024 00:06:34 +0200 Subject: [PATCH] change: Default log level to warning --- CHANGELOG.md | 3 ++- pueue/src/bin/pueue.rs | 8 ++++---- pueue/src/bin/pueued.rs | 8 ++++---- pueue/src/client/client.rs | 4 ++-- 4 files changed, 12 insertions(+), 11 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 1c423e9e..e502c0e9 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -43,7 +43,8 @@ TLDR: The new task state representation is more verbose but significantly cleane **Important: The Pueue daemon needs to be restarted and the state will be wiped clean.** - **Breaking**: Streamlined `pueue log` parameters to behave the same way as `start`, `pause` or `kill`. [#509](https://github.com/Nukesor/pueue/issues/509) - **Breaking**: Remove the `--children` commandline flags, that have been deprecated and no longer serve any function since `v3.0.0`. -- **Breaking**: Send log output to `stderr` instead of `stdout` [#562](https://github.com/Nukesor/pueue/issues/562). +- Send log output to `stderr` instead of `stdout` [#562](https://github.com/Nukesor/pueue/issues/562). +- Change default log level from error to warning [#562](https://github.com/Nukesor/pueue/issues/562). ### Add diff --git a/pueue/src/bin/pueue.rs b/pueue/src/bin/pueue.rs index ab97f629..6fb23a28 100644 --- a/pueue/src/bin/pueue.rs +++ b/pueue/src/bin/pueue.rs @@ -35,10 +35,10 @@ async fn main() -> Result<()> { // Init the logger and set the verbosity level depending on the `-v` flags. let level = match opt.verbose { - 0 => LevelFilter::Error, - 1 => LevelFilter::Warn, - 2 => LevelFilter::Info, - _ => LevelFilter::Debug, + 0 => LevelFilter::Warn, + 1 => LevelFilter::Info, + 2 => LevelFilter::Debug, + _ => LevelFilter::Trace, }; // Try to initialize the logger with the timezone set to the Local time of the machine. diff --git a/pueue/src/bin/pueued.rs b/pueue/src/bin/pueued.rs index b637ba39..6ba11eb7 100644 --- a/pueue/src/bin/pueued.rs +++ b/pueue/src/bin/pueued.rs @@ -19,10 +19,10 @@ async fn main() -> Result<()> { // Set the verbosity level of the logger. let level = match opt.verbose { - 0 => LevelFilter::Error, - 1 => LevelFilter::Warn, - 2 => LevelFilter::Info, - _ => LevelFilter::Debug, + 0 => LevelFilter::Warn, + 1 => LevelFilter::Info, + 2 => LevelFilter::Debug, + _ => LevelFilter::Trace, }; // Try to initialize the logger with the timezone set to the Local time of the machine. diff --git a/pueue/src/client/client.rs b/pueue/src/client/client.rs index 7dd145b6..d67b8c11 100644 --- a/pueue/src/client/client.rs +++ b/pueue/src/client/client.rs @@ -5,7 +5,7 @@ use std::{borrow::Cow, collections::HashMap}; use anyhow::{bail, Context, Result}; use clap::crate_version; use crossterm::tty::IsTty; -use log::error; +use log::{error, warn}; use pueue_lib::network::message::*; use pueue_lib::network::protocol::*; @@ -110,7 +110,7 @@ impl Client { }; if show_warning { - error!( + warn!( "Different daemon version detected '{version}'. Consider restarting the daemon." ); }