From 7ac9b071025f0518ee0a3350e3d7c8d881efc1ae Mon Sep 17 00:00:00 2001 From: Gorka Eguileor Date: Mon, 26 Jun 2017 17:16:56 +0200 Subject: [PATCH] Stop using internal `oslo_log.log._loggers` dict We are currently using `_loggers` internal dictionary from oslo_log for the dynamic log reconfiguration, which is not ideal. This patch uses the newly added `get_loggers` public method instead. Depends-On: Ife33a5dd044a4565ad6a4d559ef9f0c108f260a3 TrivialFix Change-Id: I9f3277320673de7687f0f387f1f50fdc5ebffd08 --- cinder/utils.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/cinder/utils.py b/cinder/utils.py index 0b18662fd04..453dadd0003 100644 --- a/cinder/utils.py +++ b/cinder/utils.py @@ -1123,7 +1123,7 @@ def set_log_levels(prefix, level_string): level = get_log_method(level_string) prefix = prefix or '' - for k, v in logging._loggers.items(): + for k, v in logging.get_loggers().items(): if k and k.startswith(prefix): v.logger.setLevel(level) @@ -1131,7 +1131,7 @@ def set_log_levels(prefix, level_string): def get_log_levels(prefix): prefix = prefix or '' return {k: logging.logging.getLevelName(v.logger.getEffectiveLevel()) - for k, v in logging._loggers.items() + for k, v in logging.get_loggers().items() if k and k.startswith(prefix)}