Nir Soffer has uploaded a new change for review. Change subject: v2v: Use signal name instead of number ......................................................................
v2v: Use signal name instead of number Signal numbers are architecture specific. We should use only signal names available in the signal module. Change-Id: I38113a16271306495bd146fc735cb289a2267b6e Signed-off-by: Nir Soffer <[email protected]> --- M vdsm/v2v.py 1 file changed, 2 insertions(+), 1 deletion(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/87/43987/1 diff --git a/vdsm/v2v.py b/vdsm/v2v.py index fd72cd0..005d17e 100644 --- a/vdsm/v2v.py +++ b/vdsm/v2v.py @@ -30,6 +30,7 @@ import logging import os import re +import signal import threading import xml.etree.ElementTree as ET @@ -368,7 +369,7 @@ cmd = self._create_command() logging.info('Job %r starting import', self._id) - self._proc = execCmd(cmd, sync=False, deathSignal=15, + self._proc = execCmd(cmd, sync=False, deathSignal=signal.SIGTERM, env=self._execution_environments()) self._proc.blocking = True -- To view, visit https://gerrit.ovirt.org/43987 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I38113a16271306495bd146fc735cb289a2267b6e Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Nir Soffer <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
