This patch fixes logging issue that caused that the logs for a recipe
were appended with content of further recipes in the test run. This
issue was seen only with python2.6. It's type() built-in function did
not worked properly and python documentation recommends to use isinstance()
instead. This issue was not seen with python2.7.
Also, in case of passing recipe directory the typo was discovered and
fixed, root_logger was incorrectly written as logger.
Signed-off-by: Jan Tluka <jtluka(a)redhat.com>
---
lnst/Common/Logs.py | 6 +++---
1 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/lnst/Common/Logs.py b/lnst/Common/Logs.py
index e9c33ea..799f5da 100644
--- a/lnst/Common/Logs.py
+++ b/lnst/Common/Logs.py
@@ -198,7 +198,7 @@ class Logs:
for logger in cls.loggers:
handlers = list(logger.handlers)
for handler in handlers:
- if type(handler) == logging.FileHandler:
+ if isinstance(handler, logging.FileHandler):
logger.removeHandler(handler)
cls.loggers = cls.loggers[:1]
@@ -242,8 +242,8 @@ class Logs:
handlers = list(root_logger.handlers)
for handler in handlers:
- if type(handler) == logging.FileHandler:
- logger.removeHandler(handler)
+ if isinstance(handler, logging.FileHandler):
+ root_logger.removeHandler(handler)
(file_debug, file_info) = cls._create_file_handler(cls.root_path)
root_logger.addHandler(file_debug)
--
1.7.7.6
Show replies by date