Change in vdsm[master]: qemuimg: use deathSignal when using execCmd

Federico Simoncelli fsimonce at redhat.com
Wed Apr 16 21:22:21 UTC 2014


Federico Simoncelli has uploaded a new change for review.

Change subject: qemuimg: use deathSignal when using execCmd
......................................................................

qemuimg: use deathSignal when using execCmd

Change-Id: I9b15b50cc99463bfdc81b44ec98ea3f676edb460
Signed-off-by: Federico Simoncelli <fsimonce at redhat.com>
---
M lib/vdsm/qemuimg.py
1 file changed, 4 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/69/26869/1

diff --git a/lib/vdsm/qemuimg.py b/lib/vdsm/qemuimg.py
index 69912d9..f7277a7 100644
--- a/lib/vdsm/qemuimg.py
+++ b/lib/vdsm/qemuimg.py
@@ -66,7 +66,7 @@
         cmd.extend(("-f", format))
 
     cmd.append(image)
-    rc, out, err = utils.execCmd(cmd)
+    rc, out, err = utils.execCmd(cmd, deathSignal=signal.SIGKILL)
 
     if rc != 0:
         raise QImgError(rc, out, err)
@@ -105,7 +105,7 @@
     if size:
         cmd.append(str(size))
 
-    rc, out, err = utils.execCmd(cmd)
+    rc, out, err = utils.execCmd(cmd, deathSignal=signal.SIGKILL)
 
     if rc != 0:
         raise QImgError(rc, out, err)
@@ -118,7 +118,7 @@
         cmd.extend(("-f", format))
 
     cmd.append(image)
-    rc, out, err = utils.execCmd(cmd)
+    rc, out, err = utils.execCmd(cmd, deathSignal=signal.SIGKILL)
 
     # FIXME: handle different error codes and raise errors accordingly
     if rc != 0:
@@ -163,7 +163,7 @@
         cmd.extend(("-f", format))
 
     cmd.extend((image, str(newSize)))
-    rc, out, err = utils.execCmd(cmd)
+    rc, out, err = utils.execCmd(cmd, deathSignal=signal.SIGKILL)
 
     if rc != 0:
         raise QImgError(rc, out, err)


-- 
To view, visit http://gerrit.ovirt.org/26869
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I9b15b50cc99463bfdc81b44ec98ea3f676edb460
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <fsimonce at redhat.com>


More information about the vdsm-patches mailing list