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>
(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(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>