Giuseppe Lavagetto has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/342657 )

Change subject: Decouple logging setup from importing the module
......................................................................

Decouple logging setup from importing the module

Change-Id: I3e5a30abefb0e1bba6e6ee5a3c9e53433d0dcf70
---
M switchdc/log.py
M switchdc/switch.py
2 files changed, 10 insertions(+), 7 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/switchdc 
refs/changes/57/342657/1

diff --git a/switchdc/log.py b/switchdc/log.py
index db644e6..d20d50a 100644
--- a/switchdc/log.py
+++ b/switchdc/log.py
@@ -3,10 +3,11 @@
 logger = logging.getLogger(__name__)
 
 
-_log_formatter = logging.Formatter(
-    fmt='%(asctime)s [%(levelname)s %(filename)s:%(lineno)s in %(funcName)s] 
%(message)s')
-_log_handler = logging.FileHandler('/var/log/switchdc.log')
-_log_handler.setFormatter(_log_formatter)
-logger.addHandler(_log_handler)
-logger.raiseExceptions = False
-logger.setLevel(logging.INFO)
+def setup_logging():
+    _log_formatter = logging.Formatter(
+        fmt='%(asctime)s [%(levelname)s %(filename)s:%(lineno)s in 
%(funcName)s] %(message)s')
+    _log_handler = logging.FileHandler('/var/log/switchdc.log')
+    _log_handler.setFormatter(_log_formatter)
+    logger.addHandler(_log_handler)
+    logger.raiseExceptions = False
+    logger.setLevel(logging.INFO)
diff --git a/switchdc/switch.py b/switchdc/switch.py
index 97b39a0..f38f6d9 100644
--- a/switchdc/switch.py
+++ b/switchdc/switch.py
@@ -3,6 +3,7 @@
 import importlib
 import os
 
+from switchdc import log
 from switchdc.menu import Item, Menu
 
 
@@ -90,6 +91,7 @@
 
 def main():
     """Entry point, run the tool."""
+    log.setup_logging()
     args = parse_args()
     menu = generate_menu(args.dc_from, args.dc_to)
     run(menu, args.dc_from, args.dc_to)

-- 
To view, visit https://gerrit.wikimedia.org/r/342657
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I3e5a30abefb0e1bba6e6ee5a3c9e53433d0dcf70
Gerrit-PatchSet: 1
Gerrit-Project: operations/switchdc
Gerrit-Branch: master
Gerrit-Owner: Giuseppe Lavagetto <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to