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