Merge pull request #34 from synkd/fix_cli_log_level

Make CLI respect log_level setting
SatelliteQE · Mar 18, 2024 · c230772 · c230772
2 parents 8ff1e29 + 87d5c61
commit c230772
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/manifester/logger.py b/manifester/logger.py
@@ -4,8 +4,6 @@
 
 import logzero
 
-from manifester.settings import settings
-
 
 def setup_logzero(level="info", path="logs/manifester.log", silent=True):
     """Call logzero setup with the given settings."""
@@ -17,13 +15,10 @@ def setup_logzero(level="info", path="logs/manifester.log", silent=True):
     log_level = getattr(logging, level.upper(), logging.INFO)
     # formatter for terminal
     formatter = logzero.LogFormatter(fmt=debug_fmt if log_level is logging.DEBUG else log_fmt)
-    logzero.setup_default_logger(formatter=formatter, disableStderrLogger=silent)
+    logzero.setup_logger(formatter=formatter, disableStderrLogger=silent)
     logzero.loglevel(log_level)
     # formatter for file
     formatter = logzero.LogFormatter(
         fmt=debug_fmt if log_level is logging.DEBUG else log_fmt, color=False
     )
     logzero.logfile(path, loglevel=log_level, maxBytes=1e9, backupCount=3, formatter=formatter)
-
-
-setup_logzero(level=settings.get("log_level", "info"))
diff --git a/manifester/manifester.py b/manifester/manifester.py
@@ -18,8 +18,11 @@
     simple_retry,
     update_inventory,
 )
+from manifester.logger import setup_logzero
 from manifester.settings import settings
 
+setup_logzero(level=settings.get("log_level", "info"))
+
 
 class Manifester:
     """Main Manifester class responsible for generating a manifest from the provided settings."""