Change in ovirt-hosted-engine-ha[master]: Fix IOError handling when sending notification emails
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(a)redhat.com>
---
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/57035
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: master
Gerrit-Owner: Martin Sivák <msivak(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>