Change in vdsm[master]: adding log print before svdsm dies

Yaniv Bronhaim has uploaded a new change for review.

Change subject: adding log print before svdsm dies
......................................................................

adding log print before svdsm dies

Change-Id: I306836e6faec2c1e424ad445279427598625ed80
Signed-off-by: Yaniv Bronhaim 
---
M vdsm/supervdsmServer.py
1 file changed, 3 insertions(+), 2 deletions(-)

git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/14/9814/1

diff --git a/vdsm/supervdsmServer.py b/vdsm/supervdsmServer.py
index 1601ba5..25e9422 100755
--- a/vdsm/supervdsmServer.py
+++ b/vdsm/supervdsmServer.py
@@ -293,13 +293,14 @@
         return mkimage.removeFs(path)

-def __pokeParent(parentPid, address):
+def __pokeParent(parentPid, address, log):
     try:
         while True:
             os.kill(parentPid, 0)
             sleep(2)
     except Exception:
         utils.rmFile(address)
+        log.debug("Killing SuperVdsm Process")
         os.kill(os.getpid(), signal.SIGTERM)

@@ -350,7 +351,7 @@
         log.debug("Setting up keep alive thread")

monThread = threading.Thread(target=__pokeParent,
-                                     args=[int(parentPid), address])
+                                     args=[int(parentPid), address, log])
         monThread.setDaemon(True)
         monThread.start()

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I306836e6faec2c1e424ad445279427598625ed80
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim

[Jenkins CI RO]

oVirt Jenkins CI Server has posted comments on this change.

Change subject: adding log print before svdsm dies
......................................................................

Patch Set 1:

Build Started http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/270/ (1/2)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I306836e6faec2c1e424ad445279427598625ed80
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim 
Gerrit-Reviewer: oVirt Jenkins CI Server

[Jenkins CI RO]

oVirt Jenkins CI Server has posted comments on this change.

Change subject: adding log print before svdsm dies
......................................................................

Patch Set 1:

Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_manual_gerrit/304/ (2/2)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I306836e6faec2c1e424ad445279427598625ed80
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim 
Gerrit-Reviewer: oVirt Jenkins CI Server

[Jenkins CI RO]

oVirt Jenkins CI Server has posted comments on this change.

Change subject: adding log print before svdsm dies
......................................................................

Patch Set 1:

Build Successful

http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/270/ : SUCCESS

http://jenkins.ovirt.org/job/vdsm_unit_tests_manual_gerrit/304/ : SUCCESS

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I306836e6faec2c1e424ad445279427598625ed80
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim 
Gerrit-Reviewer: oVirt Jenkins CI Server

Yaniv Bronhaim has posted comments on this change.

Change subject: adding log print before svdsm dies
......................................................................

Patch Set 1: Verified

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I306836e6faec2c1e424ad445279427598625ed80
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Royce Lv 
Gerrit-Reviewer: Saggi Mizrahi 
Gerrit-Reviewer: Yaniv Bronhaim 
Gerrit-Reviewer: oVirt Jenkins CI Server

Saggi Mizrahi has posted comments on this change.

Change subject: adding log print before svdsm dies
......................................................................

Patch Set 1: Looks good to me, approved

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I306836e6faec2c1e424ad445279427598625ed80
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Royce Lv 
Gerrit-Reviewer: Saggi Mizrahi 
Gerrit-Reviewer: Yaniv Bronhaim 
Gerrit-Reviewer: oVirt Jenkins CI Server

[Dan Kenigsberg]

Dan Kenigsberg has submitted this change and it was merged.

Change subject: adding log print before svdsm dies
......................................................................

adding log print before svdsm dies

Change-Id: I306836e6faec2c1e424ad445279427598625ed80
Signed-off-by: Yaniv Bronhaim 
---
M vdsm/supervdsmServer.py
1 file changed, 3 insertions(+), 2 deletions(-)

Approvals:
  Yaniv Bronhaim: Verified
  Saggi Mizrahi: Looks good to me, approved
  Dan Kenigsberg:

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I306836e6faec2c1e424ad445279427598625ed80
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Royce Lv 
Gerrit-Reviewer: Saggi Mizrahi 
Gerrit-Reviewer: Yaniv Bronhaim 
Gerrit-Reviewer: oVirt Jenkins CI Server