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 <[email protected]> Reviewed-on: http://gerrit.ovirt.org/21487 Reviewed-by: Federico Simoncelli <[email protected]> Reviewed-on: http://gerrit.ovirt.org/22214 Reviewed-by: Yaniv Bronhaim <[email protected]> Tested-by: Yaniv Bronhaim <[email protected]> --- 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 <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: [email protected] _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
