Add conversion types in some strings

Some strings have no conversion types
for 'String Formatting Operations'.

Change-Id: I2c6176954643a93898ff2d06d686793fb0977458
Closes-Bug: #1289230

diff --git a/cinder/brick/iscsi/iscsi.py b/cinder/brick/iscsi/iscsi.py
index 23fa37ff17f5efffff742eef5a1a0180292b2656..754b9e1220de63c2de3bf41ad7ac52d278c2ff6f 100644 (file)
--- a/cinder/brick/iscsi/iscsi.py
+++ b/cinder/brick/iscsi/iscsi.py
@@ -177,7 +177,7 @@ class TgtAdm(TargetAdmin):
         f.write(volume_conf)
         f.close()
         LOG.debug(_('Created volume path %(vp)s,\n'
-                    'content: %(vc)%')
+                    'content: %(vc)s')
                   % {'vp': volume_path, 'vc': volume_conf})
 
         old_persist_file = None

diff --git a/cinder/volume/drivers/ibm/storwize_svc/helpers.py b/cinder/volume/drivers/ibm/storwize_svc/helpers.py
index 20d7cdd7e739a0f433160e594cc223a5fc66cdbb..8b65154cb3c86ade815cef796c890f9138bae454 100644 (file)
--- a/cinder/volume/drivers/ibm/storwize_svc/helpers.py
+++ b/cinder/volume/drivers/ibm/storwize_svc/helpers.py
@@ -339,7 +339,7 @@ class StorwizeHelpers(object):
                     found = True
             if not found:
                 LOG.warning(_('unmap_vol_from_host: No mapping of volume '
-                              '%(vol_name)s to host %(host) found.') %
+                              '%(vol_name)s to host %(host)s found.') %
                             {'vol_name': volume_name, 'host': host_name})
 
         # We now know that the mapping exists

diff --git a/cinder/volume/drivers/nexenta/iscsi.py b/cinder/volume/drivers/nexenta/iscsi.py
index 5db016911dcdd0751ba81501f2b083efdd38ed79..7635ea606eb69991ffe392b498c199d9706533b0 100644 (file)
--- a/cinder/volume/drivers/nexenta/iscsi.py
+++ b/cinder/volume/drivers/nexenta/iscsi.py
@@ -177,7 +177,7 @@ class NexentaISCSIDriver(driver.ISCSIDriver):  # pylint: disable=R0921
             try:
                 self.delete_snapshot({'volume_name': volume, 'name': snapshot})
             except nexenta.NexentaException as exc:
-                LOG.warning(_('Cannot delete snapshot %(origin): %(exc)s'),
+                LOG.warning(_('Cannot delete snapshot %(origin)s: %(exc)s'),
                             {'origin': origin, 'exc': exc})
 
     def create_cloned_volume(self, volume, src_vref):

diff --git a/cinder/zonemanager/drivers/brocade/brcd_fc_san_lookup_service.py b/cinder/zonemanager/drivers/brocade/brcd_fc_san_lookup_service.py
index 9eb02a3e11ae8336f5e0ce0991642f8f76d2d4d9..5c05b52b2ff2c9c59814011d6d5c981a81c9f186 100644 (file)
--- a/cinder/zonemanager/drivers/brocade/brcd_fc_san_lookup_service.py
+++ b/cinder/zonemanager/drivers/brocade/brcd_fc_san_lookup_service.py
@@ -132,7 +132,7 @@ class BrcdFCSanLookupService(FCSanLookupService):
                                     "fabric %s") % fabric_ip)
                 except Exception as e:
                     msg = _("SSH connection failed "
-                            "for %(fabric) with error: %(err)"
+                            "for %(fabric)s with error: %(err)s"
                             ) % {'fabric': fabric_ip, 'err': e}
                     LOG.error(msg)
                     raise exception.FCSanLookupServiceException(message=msg)
@@ -212,7 +212,7 @@ class BrcdFCSanLookupService(FCSanLookupService):
             stdin, stdout, stderr = self.client.exec_command(cmd)
             switch_data = stdout.readlines()
         except paramiko.SSHException as e:
-            msg = (_("SSH Command failed with error '%(err)r' "
+            msg = (_("SSH Command failed with error '%(err)s' "
                      "'%(command)s'") % {'err': e,
                                          'command': cmd})
             LOG.error(msg)