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