From 2da1bdd637c2284859f70fbbf06503e29436eca7 Mon Sep 17 00:00:00 2001 From: stack Date: Fri, 2 Sep 2016 03:35:25 -0700 Subject: [PATCH] Correction in log massage format syntax Dictionary key used wrongly as '%(key)' instead of '%(key)s' while formatting log message. Change-Id: I2e31bfdac1021ee00bb001283c4f4ea9c273bc4c Closes-bug: #1619584 --- cinder/volume/drivers/hpe/hpe_3par_common.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/cinder/volume/drivers/hpe/hpe_3par_common.py b/cinder/volume/drivers/hpe/hpe_3par_common.py index feeae61a10a..eafbb90da1f 100644 --- a/cinder/volume/drivers/hpe/hpe_3par_common.py +++ b/cinder/volume/drivers/hpe/hpe_3par_common.py @@ -593,7 +593,7 @@ class HPE3PARCommon(object): volume_update['status'] = 'deleted' except Exception as ex: LOG.error(_LE("There was an error deleting volume %(id)s: " - "%(error)."), + "%(error)s."), {'id': volume.id, 'error': six.text_type(ex)}) volume_update['status'] = 'error' @@ -693,7 +693,7 @@ class HPE3PARCommon(object): snapshot_update['status'] = fields.SnapshotStatus.ERROR except Exception as ex: LOG.error(_LE("There was an error deleting snapshot %(id)s: " - "%(error)."), + "%(error)s."), {'id': snapshot['id'], 'error': six.text_type(ex)}) snapshot_update['status'] = fields.SnapshotStatus.ERROR