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

danken at redhat.com danken at redhat.com
Mon Aug 13 11:13:12 UTC 2012


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 at 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 at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Omri Hochman <hoomri at gmail.com>
Gerrit-Reviewer: Royce Lv <lvroyce at linux.vnet.ibm.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list