[prev in list] [next in list] [prev in thread] [next in thread] 

List:       vdsm-patches
Subject:    Change in vdsm[master]: domainMonitor: Log unhandled exceptions in domain monitor th...
From:       danken () redhat ! com
Date:       2013-11-29 13:11:42
Message-ID: 201311291311.rATDBgAn008854 () gerrit ! ovirt ! org
[Download RAW message or body]

Dan Kenigsberg 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
Relates-To: https://bugzilla.redhat.com/1034741
Signed-off-by: Nir Soffer <nsoffer@redhat.com>
Reviewed-on: http://gerrit.ovirt.org/21487
Reviewed-by: Federico Simoncelli <fsimonce@redhat.com>
---
M vdsm/storage/domainMonitor.py
1 file changed, 2 insertions(+), 0 deletions(-)

Approvals:
  Nir Soffer: Verified
  Federico Simoncelli: Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/21487
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I743c59bffa99ab9868f1878ca96b7d979a703efb
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsoffer@redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken@redhat.com>
Gerrit-Reviewer: Daniel Erez <derez@redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce@redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer@redhat.com>
Gerrit-Reviewer: Sergey Gotliv <sgotliv@redhat.com>
Gerrit-Reviewer: Vered Volansky <vvolansk@redhat.com>
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

[prev in list] [next in list] [prev in thread] [next in thread]