Change in ovirt-hosted-engine-ha[ovirt-hosted-engine-ha-1.3]: Fix IOError handling when sending notification emails

msivak at redhat.com msivak at redhat.com
Wed May 4 13:42:59 UTC 2016


Martin Sivák has submitted this change and it was merged.

Change subject: Fix IOError handling when sending notification emails
......................................................................


Fix IOError handling when sending notification emails

IOError and other exceptions from the EnvironmentError group
had the power to crash the whole broker. Those should be logged
and ignored when sending notifications.

Change-Id: Ic8500928ebe6b210ace89784d070bb53c61a33fc
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1332963
Signed-off-by: Martin Sivak <msivak at redhat.com>
(cherry picked from commit cd14809b560668f2baeb9ca1727f3a49a6f957e8)
---
M ovirt_hosted_engine_ha/broker/notifications.py
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Martin Sivák: Verified; Looks good to me, approved
  Simone Tiraboschi: Looks good to me, but someone else must approve
  Jenkins CI: Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/57039
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic8500928ebe6b210ace89784d070bb53c61a33fc
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-hosted-engine-ha
Gerrit-Branch: ovirt-hosted-engine-ha-1.3
Gerrit-Owner: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list