From: Thanh Ha <thanh.ha@linuxfoundation.org>
Date: Mon, 27 Aug 2018 00:58:29 +0000 (-0400)
Subject: Update config subsystem to use logger
X-Git-Tag: v0.17.0~4
X-Git-Url: https://gerrit.linuxfoundation.org/infra/gitweb?a=commitdiff_plain;h=refs%2Fchanges%2F97%2F12397%2F4;p=releng%2Flftools.git

Update config subsystem to use logger

Use our logger service in the config subsystem.

Change-Id: I47235cf9accd362ba1355ff77e15757543d30da1
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
---

diff --git a/lftools/cli/config.py b/lftools/cli/config.py
index c671c8ce..ad088492 100644
--- a/lftools/cli/config.py
+++ b/lftools/cli/config.py
@@ -11,11 +11,14 @@
 
 __author__ = 'Thanh Ha'
 
+import logging
 
 import click
 
 from lftools import config
 
+log = logging.getLogger(__name__)
+
 
 @click.group(name='config')
 @click.pass_context
@@ -33,9 +36,9 @@ def get_setting(ctx, section, option):
     result = config.get_setting(section, option)
     if isinstance(result, list):
         for i in result:
-            print('{}: {}'.format(i, config.get_setting(section, i)))
+            log.info('{}: {}'.format(i, config.get_setting(section, i)))
     else:
-        print(result)
+        log.info(result)
 
 
 @click.command(name='set')
@@ -45,6 +48,7 @@ def get_setting(ctx, section, option):
 @click.pass_context
 def set_setting(ctx, section, option, value):
     """Set a setting in the config file."""
+    log.debug('Set config\n[{}]\n{}:{}'.format(section, option, value))
     config.set_setting(section, option, value)
 
 
diff --git a/lftools/config.py b/lftools/config.py
index c6abc9af..bc0fbb85 100644
--- a/lftools/config.py
+++ b/lftools/config.py
@@ -12,11 +12,13 @@
 
 __author__ = 'Thanh Ha'
 
+import logging
 import sys
 
 from six.moves import configparser
 from xdg import XDG_CONFIG_HOME
 
+log = logging.getLogger(__name__)
 LFTOOLS_CONFIG_FILE = '/'.join([XDG_CONFIG_HOME, 'lftools', 'lftools.ini'])
 
 
@@ -35,17 +37,17 @@ def get_setting(section, option=None):
         try:
             return config.get(section, option)
         except configparser.NoOptionError:
-            print('ERROR: Config option does not exist.')
+            log.error('Config option does not exist.')
             sys.exit(1)
         except configparser.NoSectionError:
-            print('ERROR: Config section does not exist.')
+            log.error('Config section does not exist.')
             sys.exit(1)
 
     else:
         try:
             return config.options(section)
         except configparser.NoSectionError:
-            print('ERROR: Config section does not exist.')
+            log.error('Config section does not exist.')
             sys.exit(1)