Fix duplicate logging

Currently check `handler is WatchedFileHandler` always fails and a new handler is added every time _setup_logger is called.

For example corrector logs every line three times.

A fix for opendata was added with pull request #14

This commit makes the same change in all the other services.

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
 

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
 

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
 

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
 

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
 

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