Feature/hide not found (#44)

* Hide "No files for wildcard" warnings to debug

* rework logging to store all wb-diag-collect logs

debian/changelog

@@ -1,3 +1,9 @@
+wb-diag-collect (1.8.12) stable; urgency=medium
+
+  * Hide "No files for wildcard" warnings to debug
+
+ -- Vladimir Romanov <v.romanov@wirenboard.com>  Tue, 06 Aug 2024 16:58:21 +0300
+
 wb-diag-collect (1.8.11) stable; urgency=medium
 
   * Hide PIN from *.nmconnection files

wb/diag/collector.py

@@ -19,10 +19,10 @@ def collect(self, options, output_directory, output_filename):
             try:
                 self.log_stream = StringIO()
                 self.log_stream_handler = logging.StreamHandler(self.log_stream)
+                self.log_stream_handler.setLevel(logging.DEBUG)
                 self.log_stream_handler.setFormatter(
                     logging.Formatter("%(asctime)s [%(levelname)s] %(message)s")
                 )
-                self.logger.setLevel(self.logger.getEffectiveLevel())
                 self.logger.addHandler(self.log_stream_handler)
 
                 self.copy_files(tmpdir, options["files"])
@@ -70,7 +70,7 @@ def apply_file_wildcard(self, wildcard: str, timeout):
 
         try:
             if proc.wait(timeout) != 0:
-                self.logger.warning("No files for wildcard %s", wildcard)
+                self.logger.debug("No files for wildcard %s", wildcard)
                 return []
 
             file_paths = []

wb/diag/diag_collect.py

@@ -18,13 +18,7 @@ class ResultCode(IntEnum):
 
 
 logger = logging.getLogger(__name__)
-
-console_handler = logging.StreamHandler()
-console_handler.setLevel(logging.DEBUG)
-console_handler.setFormatter(logging.Formatter("%(asctime)s [%(levelname)s] %(message)s"))
-logger.addHandler(console_handler)
-
-logger.setLevel(logging.INFO)
+logger.setLevel(logging.DEBUG)
 
 
 def main(argv=sys.argv):
@@ -44,8 +38,13 @@ def main(argv=sys.argv):
     args = parser.parse_args(argv[1:])
     conf_path = args.config
 
+    console_handler = logging.StreamHandler()
+    console_handler.setFormatter(logging.Formatter("%(asctime)s [%(levelname)s] %(message)s"))
+    console_loglevel = logging.INFO
     if args.debug:
-        logger.setLevel(logging.DEBUG)
+        console_loglevel = logging.DEBUG
+    console_handler.setLevel(console_loglevel)
+    logger.addHandler(console_handler)
 
     try:
         with open(conf_path or DEFAULT_CONF_PATH) as f: