Log iSCSI target output on error.

Add the command output on error to aid in debugging. Change-Id: Ib505eaeba7f1ca0e54e99dec8ebbbf71df3e5a51 Fixes: bug 1186991
openstack · Jun 4, 2013 · f7730d3 · f7730d3
1 parent a003e4a
commit f7730d3
Showing 1 changed file with 9 additions and 6 deletions.
diff --git a/cinder/brick/iscsi/iscsi.py b/cinder/brick/iscsi/iscsi.py
@@ -167,9 +167,10 @@ def create_iscsi_target(self, name, tid, lun, path,
                                        '--update',
                                        name,
                                        run_as_root=True)
-        except exception.ProcessExecutionError, e:
+        except exception.ProcessExecutionError as e:
             LOG.error(_("Failed to create iscsi target for volume "
-                        "id:%(vol_id)s.") % locals())
+                        "id:%(vol_id)s: %(e)s")
+                      % {'vol_id': vol_id, 'e': str(e)})
 
             #Don't forget to remove the persistent file we created
             os.unlink(volume_path)
@@ -205,9 +206,10 @@ def remove_iscsi_target(self, tid, lun, vol_id, **kwargs):
                           '--delete',
                           iqn,
                           run_as_root=True)
-        except exception.ProcessExecutionError, e:
+        except exception.ProcessExecutionError as e:
             LOG.error(_("Failed to remove iscsi target for volume "
-                        "id:%(vol_id)s.") % locals())
+                        "id:%(vol_id)s: %(e)s")
+                      % {'vol_id': vol_id, 'e': str(e)})
             raise exception.ISCSITargetRemoveFailed(volume_id=vol_id)
 
         os.unlink(volume_path)
@@ -259,10 +261,11 @@ def create_iscsi_target(self, name, tid, lun, path,
                     f = open(conf_file, 'a+')
                     f.write(volume_conf)
                     f.close()
-            except exception.ProcessExecutionError, e:
+            except exception.ProcessExecutionError as e:
                 vol_id = name.split(':')[1]
                 LOG.error(_("Failed to create iscsi target for volume "
-                            "id:%(vol_id)s.") % locals())
+                            "id:%(vol_id)s: %(e)s")
+                          % {'vol_id': vol_id, 'e': str(e)})
                 raise exception.ISCSITargetCreateFailed(volume_id=vol_id)
         return tid