Change in vdsm[ovirt-3.3]: hsm: Log unhandled exceptions in storage refresh thread

ybronhei at redhat.com ybronhei at redhat.com
Wed Dec 11 11:29:06 UTC 2013


Yaniv Bronhaim has submitted this change and it was merged.

Change subject: hsm: Log unhandled exceptions in storage refresh thread
......................................................................


hsm: Log unhandled exceptions in storage refresh thread

hsm's storage refresh background thread does not handle exceptions,
making it harder then it should to debug stupid errors in underlying
calls or debug real errors in the field.

This patch log a traceback for unhandled exceptions so it will never die
silently again.

Change-Id: I2e69464ba27b7d522c5ceb3a3099949efe7fbc88
Bug-Url: https://bugzilla.redhat.com/1040164
Signed-off-by: Nir Soffer <nsoffer at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/21486
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/22213
Reviewed-by: Yaniv Bronhaim <ybronhei at redhat.com>
Tested-by: Yaniv Bronhaim <ybronhei at redhat.com>
---
M vdsm/storage/hsm.py
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  Yaniv Bronhaim: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I2e69464ba27b7d522c5ceb3a3099949efe7fbc88
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