fix comments, pylint and ruff

Mellanox · Dec 19, 2024 · e5827e5 · e5827e5
1 parent e501ef1
commit e5827e5
Showing 1 changed file with 3 additions and 5 deletions.
diff --git a/plugins/ufm_log_analyzer_plugin/src/loganalyze/logs_extraction/tar_extractor.py b/plugins/ufm_log_analyzer_plugin/src/loganalyze/logs_extraction/tar_extractor.py
@@ -134,7 +134,7 @@ def extract_files(
                         # Read some bytes to verify the file is not corrupted
                         inner_tar_stream.peek(1)  # Peek at the first byte
                     except Exception as e:
-                        log.Logger.info(f"Error reading inner tar file {inner_tar_name}: {e}")
+                        log.Logger.info("Error reading inner tar file %s: %s", inner_tar_name, e)
                         continue  # Skip this file if it's invalid
 
                     inner_file_open_mode = (
@@ -157,10 +157,8 @@ def extract_files(
 
 
                     except EOFError as e:
-                        log.logger.info(f"EOFError in inner tar {inner_tar_name}: {e}")
-                        continue  # Handle the EOFError and continue with the next file
-
-
+                        log.LOGGER.info("EOFError in inner tar %s: %s", inner_tar_name, e)
+                        continue
             # If we got to this point, we might have a simple tar, try to extract from it
             return self._get_files_from_tar(
                 outer_tar, files_to_extract, directories_to_extract, destination