From Dan Kenigsberg <dan...@redhat.com>:

Dan Kenigsberg has uploaded a new change for review.

Change subject: checkimages hook: split line for readability
......................................................................

checkimages hook: split line for readability

Change-Id: Ic7ae180c3386ed7991150cff4ce5e0d589500c1d
Signed-off-by: Dan Kenigsberg <dan...@redhat.com>
---
M vdsm_hooks/checkimages/before_vm_start.py
1 file changed, 2 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/94/80394/1

diff --git a/vdsm_hooks/checkimages/before_vm_start.py 
b/vdsm_hooks/checkimages/before_vm_start.py
index 31b123d..ee307cf 100755
--- a/vdsm_hooks/checkimages/before_vm_start.py
+++ b/vdsm_hooks/checkimages/before_vm_start.py
@@ -84,7 +84,8 @@
         sys.stderr.write('Increate timeout or check image availability.')
         sys.exit(2)
 
-    ((out, err), rc) = (p.communicate(), p.returncode)
+    out, err = p.communicate()
+    rc = p.returncode
 
     if rc == 0:
         sys.stderr.write('checkimages: %s image check returned: %s\n' %


-- 
To view, visit https://gerrit.ovirt.org/80394
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic7ae180c3386ed7991150cff4ce5e0d589500c1d
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg <dan...@redhat.com>
_______________________________________________
vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org
To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org

Reply via email to