Change in vdsm[ovirt-3.3]: domainMonitor: Log unhandled exceptions in domain monitor th...
ybronhei at redhat.com
ybronhei at redhat.com
Wed Dec 11 11:26:31 UTC 2013
Yaniv Bronhaim has submitted this change and it was merged.
Change subject: domainMonitor: Log unhandled exceptions in domain monitor thread
......................................................................
domainMonitor: Log unhandled exceptions in domain monitor thread
DomainMonitorThread does not handle exceptions in the thread main
function, making it harder to modify the code or understanding
unexpected failures in the field.
The patch log unhandled exceptions so it will never die silently.
Change-Id: I743c59bffa99ab9868f1878ca96b7d979a703efb
Bug-Url: https://bugzilla.redhat.com/1040166
Relates-To: https://bugzilla.redhat.com/1034741
Signed-off-by: Nir Soffer <nsoffer at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/21487
Reviewed-by: Federico Simoncelli <fsimonce at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/22214
Reviewed-by: Yaniv Bronhaim <ybronhei at redhat.com>
Tested-by: Yaniv Bronhaim <ybronhei at redhat.com>
---
M vdsm/storage/domainMonitor.py
1 file changed, 2 insertions(+), 0 deletions(-)
Approvals:
Yaniv Bronhaim: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/22214
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I743c59bffa99ab9868f1878ca96b7d979a703efb
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.3
Gerrit-Owner: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
More information about the vdsm-patches
mailing list