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@redhat.com --- 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@redhat.com
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@redhat.com Gerrit-Reviewer: Federico Simoncelli fsimonce@redhat.com
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@redhat.com Gerrit-Reviewer: Federico Simoncelli fsimonce@redhat.com
vdsm-patches@lists.fedorahosted.org