Fixed logger initialization

Erroneously wrapped the `_logger` setting in the new if block. Signed-off-by: James.E.Bell <[email protected]>
target · Dec 31, 2017 · 4f1b71e · 4f1b71e
1 parent a159074
commit 4f1b71e
Showing 1 changed file with 11 additions and 11 deletions.
diff --git a/tgt_grease/core/Logging.py b/tgt_grease/core/Logging.py
@@ -286,18 +286,18 @@ def DefaultLogger(self):
 
         """
         global GREASE_LOG_HANDLER
+        self._logger = logging.getLogger('GREASE')
+        self._logger.setLevel(logging.DEBUG)
+        self._formatter = logging.Formatter(
+            "{"
+            "\"timestamp\": \"%(asctime)s.%(msecs)03d\", "
+            "\"thread\": \"%(threadName)s\", "
+            "\"level\" : \"%(levelname)s\", "
+            "\"message\" : \"%(message)s\"}",
+            "%Y-%m-%d %H:%M:%S"
+        )
+        self._formatter.converter = time.gmtime
         if not GREASE_LOG_HANDLER:
-            self._logger = logging.getLogger('GREASE')
-            self._logger.setLevel(logging.DEBUG)
-            self._formatter = logging.Formatter(
-                "{"
-                "\"timestamp\": \"%(asctime)s.%(msecs)03d\", "
-                "\"thread\": \"%(threadName)s\", "
-                "\"level\" : \"%(levelname)s\", "
-                "\"message\" : \"%(message)s\"}",
-                "%Y-%m-%d %H:%M:%S"
-            )
-            self._formatter.converter = time.gmtime
             if os.path.isdir(self._conf.greaseDir):
                 GREASE_LOG_HANDLER = logging.FileHandler(self._conf.get('Logging', 'file'))
                 GREASE_LOG_HANDLER.setLevel(logging.DEBUG)