[MP2P] Still log to stdout when logfile is specified

A previous commit made logging to stdout alternative to logging
to a file. This patch restores stdout logging.
It also ensures the formatter is applied to the root logger for
consistent output.

Change-Id: Ie479006c865e823484aa7b71fea29356e22cf7cd
This commit is contained in:
Salvatore 2021-05-01 08:54:23 +00:00
parent 8e65ed6f9c
commit ea2d0a120c
1 changed files with 8 additions and 9 deletions

View File

@ -1669,27 +1669,26 @@ def MP2Policy_migration(resource, event, trigger, **kwargs):
LOG.setLevel(logging.DEBUG) LOG.setLevel(logging.DEBUG)
else: else:
LOG.setLevel(logging.INFO) LOG.setLevel(logging.INFO)
formatter = logging.Formatter('%(asctime)s %(levelname)s %(message)s')
root_logger = logging.getLogger(None)
start_migration_service = False start_migration_service = False
handler = logging.StreamHandler()
if kwargs.get('property'): if kwargs.get('property'):
properties = admin_utils.parse_multi_keyval_opt(kwargs['property']) properties = admin_utils.parse_multi_keyval_opt(kwargs['property'])
logfile = properties.get('logfile', None) logfile = properties.get('logfile', None)
if logfile: if logfile:
handler = logging.FileHandler(logfile) handler = logging.FileHandler(logfile)
root_logger.addHandler(handler)
start_service_flag = properties.get('start-migration-service', 'False') start_service_flag = properties.get('start-migration-service', 'False')
if start_service_flag.lower() == 'true': if start_service_flag.lower() == 'true':
start_migration_service = True start_migration_service = True
formatter = logging.Formatter('%(asctime)s %(levelname)s %(message)s') # Apply formatter to every handler
handler.setFormatter(formatter) handlers = root_logger.handlers
LOG.addHandler(handler) for handler in handlers:
# Remove handler from root logger to avoid duplication handler.setFormatter(formatter)
root_logger = logging.getLogger(None)
for root_handler in root_logger.handlers[:]:
root_logger.removeHandler(root_handler)
nsxlib = _get_nsxlib_from_config(verbose) nsxlib = _get_nsxlib_from_config(verbose)
nsxpolicy = p_utils.get_connected_nsxpolicy( nsxpolicy = p_utils.get_connected_nsxpolicy(
conf_path=cfg.CONF.nsx_v3, verbose=verbose) conf_path=cfg.CONF.nsx_v3, verbose=verbose)