Merge pull request #105 from VitaliStupin/logger-handler-fix

Fix duplicate logging
nordic-institute · May 21, 2024 · a3d181a · a3d181a
2 parents 3a826e2 + 9f0c807
commit a3d181a
Show file tree

Hide file tree

Showing 6 changed files with 6 additions and 6 deletions.
diff --git a/analysis_module/opmon_analyzer/logger_manager.py b/analysis_module/opmon_analyzer/logger_manager.py
@@ -58,7 +58,7 @@ def _create_file_handler(self):
 
     def _handler_is_set(self, handlers):
         for handler in handlers:
-            if handler is WatchedFileHandler and os.path.abspath(self.log_path) == handler.baseFilename:
+            if isinstance(handler, WatchedFileHandler) and os.path.abspath(self.log_path) == handler.baseFilename:
                 return True
         return False
 

diff --git a/analysis_ui_module/opmon_analyzer_ui/logger_manager.py b/analysis_ui_module/opmon_analyzer_ui/logger_manager.py
@@ -58,7 +58,7 @@ def _create_file_handler(self):
 
     def _handler_is_set(self, handlers):
         for handler in handlers:
-            if handler is WatchedFileHandler and os.path.abspath(self.log_path) == handler.baseFilename:
+            if isinstance(handler, WatchedFileHandler) and os.path.abspath(self.log_path) == handler.baseFilename:
                 return True
         return False
 

diff --git a/anonymizer_module/opmon_anonymizer/utils/logger_manager.py b/anonymizer_module/opmon_anonymizer/utils/logger_manager.py
@@ -58,7 +58,7 @@ def _create_file_handler(self):
 
     def _handler_is_set(self, handlers):
         for handler in handlers:
-            if handler is WatchedFileHandler and os.path.abspath(self.log_path) == handler.baseFilename:
+            if isinstance(handler, WatchedFileHandler) and os.path.abspath(self.log_path) == handler.baseFilename:
                 return True
         return False
 

diff --git a/collector_module/opmon_collector/logger_manager.py b/collector_module/opmon_collector/logger_manager.py
@@ -58,7 +58,7 @@ def _create_file_handler(self):
 
     def _handler_is_set(self, handlers):
         for handler in handlers:
-            if handler is WatchedFileHandler and os.path.abspath(self.log_path) == handler.baseFilename:
+            if isinstance(handler, WatchedFileHandler) and os.path.abspath(self.log_path) == handler.baseFilename:
                 return True
         return False
 

diff --git a/corrector_module/opmon_corrector/logger_manager.py b/corrector_module/opmon_corrector/logger_manager.py
@@ -58,7 +58,7 @@ def _create_file_handler(self):
 
     def _handler_is_set(self, handlers):
         for handler in handlers:
-            if handler is WatchedFileHandler and os.path.abspath(self.log_path) == handler.baseFilename:
+            if isinstance(handler, WatchedFileHandler) and os.path.abspath(self.log_path) == handler.baseFilename:
                 return True
         return False
 

diff --git a/reports_module/opmon_reports/logger_manager.py b/reports_module/opmon_reports/logger_manager.py
@@ -58,7 +58,7 @@ def _create_file_handler(self):
 
     def _handler_is_set(self, handlers):
         for handler in handlers:
-            if handler is WatchedFileHandler and os.path.abspath(self.log_path) == handler.baseFilename:
+            if isinstance(handler, WatchedFileHandler) and os.path.abspath(self.log_path) == handler.baseFilename:
                 return True
         return False