openstack/os-vif
Code Issues Proposed changes

Make plugin loading more consistent with logging guidelines.

Browse Source 
This moves the detailed plugin loading messages (classes + names) to
the debug level, and provides a singular INFO log of the names of all
plugins loaded by name.

Change-Id: Ia580149202705303881d861b9eaa4702ffb10ace
This commit is contained in:
Sean Dague 2016-09-21 11:30:18 -04:00
parent 8b1e906f2c
commit 974f737cb1
1 changed files with 6 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
os_vif/__init__.py
View File

@ -40,13 +40,16 @@ def initialize(reset=False):
if reset or (_EXT_MANAGER is None):
_EXT_MANAGER = extension.ExtensionManager(namespace='os_vif',
invoke_on_load=False)
loaded_plugins = []
for plugin_name in _EXT_MANAGER.names():
cls = _EXT_MANAGER[plugin_name].plugin
obj = cls.load(plugin_name)
LOG.info(_LI("Loaded VIF plugin class '%(cls)s' "
"with name '%(plugin_name)s'"),
{'cls': cls, 'plugin_name': plugin_name})
LOG.debug(("Loaded VIF plugin class '%(cls)s' "
"with name '%(plugin_name)s'"),
{'cls': cls, 'plugin_name': plugin_name})
loaded_plugins.append(plugin_name)
_EXT_MANAGER[plugin_name].obj = obj
LOG.info(_LI("Loaded VIF plugins: %s"), ", ".join(loaded_plugins))
def plug(vif, instance_info):