From 2e5148cb27a9413a96596ae2fd43bdaba518231e Mon Sep 17 00:00:00 2001 From: Thanh Ha Date: Sun, 26 Aug 2018 20:58:29 -0400 Subject: [PATCH] Update config subsystem to use logger Use our logger service in the config subsystem. Change-Id: I47235cf9accd362ba1355ff77e15757543d30da1 Signed-off-by: Thanh Ha --- lftools/cli/config.py | 8 ++++++-- lftools/config.py | 8 +++++--- 2 files changed, 11 insertions(+), 5 deletions(-) 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) -- 2.16.6