From: Yuriy Taraday <yorik.sar@gmail.com>
Date: Sun, 19 Jan 2014 19:08:06 +0000 (+0400)
Subject: Fix UnboundLocalError in TgtAdm.update_iscsi_target
X-Git-Url: https://review.fuel-infra.org/gitweb?a=commitdiff_plain;h=82283790b7c5ba6ae0e561ee3302d7fed7983267;p=openstack-build%2Fcinder-build.git

Fix UnboundLocalError in TgtAdm.update_iscsi_target

Closes-Bug: #1270624
Change-Id: I74a057a9603c5d6aefaec6db62228352b739df56
---

diff --git a/cinder/brick/iscsi/iscsi.py b/cinder/brick/iscsi/iscsi.py
index aafc532ea..a3fcc6206 100644
--- a/cinder/brick/iscsi/iscsi.py
+++ b/cinder/brick/iscsi/iscsi.py
@@ -250,8 +250,8 @@ class TgtAdm(TargetAdmin):
         except putils.ProcessExecutionError as e:
             LOG.error(_("Failed to update iscsi target %(name)s: %(e)s") %
                       {'name': name, 'e': str(e)})
-            LOG.debug("StdOut from tgt-admin --update: %s" % out)
-            LOG.debug("StdErr from tgt-admin --update: %s" % err)
+            LOG.debug("StdOut from tgt-admin --update: %s", e.stdout)
+            LOG.debug("StdErr from tgt-admin --update: %s", e.stderr)
             raise exception.ISCSITargetUpdateFailed(name=name)
 
     def remove_iscsi_target(self, tid, lun, vol_id, vol_name, **kwargs):