diff options
author | Mate Lakat <mate.lakat@citrix.com> | 2013-06-26 16:23:44 +0100 |
---|---|---|
committer | Mate Lakat <mate.lakat@citrix.com> | 2013-06-28 08:12:03 +0100 |
commit | 6bde383b4e539000a6dd0ec52763b6d09f4c870e (patch) | |
tree | 1b498f669735c96f3b9e194c379c26b6d7ff4194 | |
parent | 35029ec8d584be6eadfbe4ccfb179ac9e99acf03 (diff) | |
download | nova-6bde383b4e539000a6dd0ec52763b6d09f4c870e.tar.gz nova-6bde383b4e539000a6dd0ec52763b6d09f4c870e.tar.xz nova-6bde383b4e539000a6dd0ec52763b6d09f4c870e.zip |
fix invalid logging
Logging parameters were passed as a tuple, yielding:
TypeError: not enough arguments for format string
fixes bug 1194917
Change-Id: Ifec893e1e4551f28849b96c7b1bf3c929b708b4d
-rw-r--r-- | nova/virt/xenapi/volume_utils.py | 12 |
1 files changed, 10 insertions, 2 deletions
diff --git a/nova/virt/xenapi/volume_utils.py b/nova/virt/xenapi/volume_utils.py index 7a0b4a67e..853bc3262 100644 --- a/nova/virt/xenapi/volume_utils.py +++ b/nova/virt/xenapi/volume_utils.py @@ -233,8 +233,16 @@ def parse_volume_info(connection_data): target_host = _get_target_host(target_portal) target_port = _get_target_port(target_portal) target_iqn = connection_data['target_iqn'] - LOG.debug('(vol_id,number,host,port,iqn): (%s,%s,%s,%s)', - (volume_id, target_host, target_port, target_iqn)) + + log_params = { + "vol_id": volume_id, + "host": target_host, + "port": target_port, + "iqn": target_iqn + } + LOG.debug(_('(vol_id,host,port,iqn): ' + '(%(vol_id)s,%(host)s,%(port)s,%(iqn)s)'), log_params) + if (volume_id is None or target_host is None or target_iqn is None): |