Change in vdsm[master]: [WIP2/2] BZ#844656 Respawn the domain monitor when needed

[Federico Simoncelli]

Federico Simoncelli has uploaded a new change for review.

Change subject: [WIP2/2] BZ#844656 Respawn the domain monitor when needed
......................................................................

[WIP2/2] BZ#844656 Respawn the domain monitor when needed

In the domain monitor there are few lines that are not covered by a
try/except clause but that might be enough for the thread to die without
leaving any valuable information in the logs. This patch adds logging
for the possible exceptions and respawn the domain monitor if it exists
unexpectedly.

Change-Id: I6ea67ddbc31d9f2bf7c00fa0fef3e59dc0639e1e
Signed-off-by: Federico Simoncelli <fsimonce(a)redhat.com&gt;
---
M vdsm/storage/domainMonitor.py
1 file changed, 111 insertions(+), 97 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/88/6788/1
--
To view, visit http://gerrit.ovirt.org/6788
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I6ea67ddbc31d9f2bf7c00fa0fef3e59dc0639e1e
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <fsimonce(a)redhat.com&gt;

[Federico Simoncelli]

Federico Simoncelli has abandoned this change.

Change subject: [WIP2/2] BZ#844656 Respawn the domain monitor when needed
......................................................................


Patch Set 1: Abandoned

Squashed into: http://gerrit.ovirt.org/#/c/6787/

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

Gerrit-MessageType: abandon
Gerrit-Change-Id: I6ea67ddbc31d9f2bf7c00fa0fef3e59dc0639e1e
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <fsimonce(a)redhat.com&gt;
Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com&gt;

[Federico Simoncelli]

Federico Simoncelli has posted comments on this change.

Change subject: [WIP2/2] BZ#844656 Respawn the domain monitor when needed
......................................................................


Patch Set 1: (1 inline comment)

....................................................
File vdsm/storage/domainMonitor.py
Line 95:             pass
s/pass/return/

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I6ea67ddbc31d9f2bf7c00fa0fef3e59dc0639e1e
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <fsimonce(a)redhat.com&gt;
Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com&gt;