Dan Kenigsberg has submitted this change and it was merged.
Change subject: BZ#844656 Respawn the domain monitor when needed ......................................................................
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 exits unexpectedly.
Signed-off-by: Federico Simoncelli fsimonce@redhat.com Change-Id: I47e4fb6883d72578ee8083e46e4f660b656a5ffc --- M vdsm/storage/domainMonitor.py 1 file changed, 113 insertions(+), 85 deletions(-)
Approvals: Ayal Baron: Looks good to me, approved Omri Hochman: Verified Dan Kenigsberg:
-- To view, visit http://gerrit.ovirt.org/6787 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged Gerrit-Change-Id: I47e4fb6883d72578ee8083e46e4f660b656a5ffc Gerrit-PatchSet: 6 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli fsimonce@redhat.com Gerrit-Reviewer: Ayal Baron abaron@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Federico Simoncelli fsimonce@redhat.com Gerrit-Reviewer: Omri Hochman hoomri@gmail.com Gerrit-Reviewer: Royce Lv lvroyce@linux.vnet.ibm.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server