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 c671c8c..ad08849 100644 (file)
--- 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 c6abc9a..bc0fbb8 100644 (file)
--- 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)