Change in ovirt-hosted-engine-ha[master]: Fix the email flood caused by not handling the yield logic p...
Greg Padgett has submitted this change and it was merged.
Change subject: Fix the email flood caused by not handling the yield logic properly
......................................................................
Fix the email flood caused by not handling the yield logic properly
The notification logic handled the internal loop only
(in _perform_engine_actions), but forgot to handle the loop around
that method. That caused an email flood with the same information.
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1032644
Change-Id: I4f35d85472f9e27bad222254a4d5c81b845d6282
Signed-off-by: Martin Sivak <msivak(a)redhat.com>
---
M ovirt_hosted_engine_ha/agent/hosted_engine.py
M ovirt_hosted_engine_ha/broker/notifications.py
2 files changed, 13 insertions(+), 6 deletions(-)
Approvals:
Martin Sivák: Verified
Greg Padgett: Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/21466
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I4f35d85472f9e27bad222254a4d5c81b845d6282
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-hosted-engine-ha
Gerrit-Branch: master
Gerrit-Owner: Martin Sivák <msivak(a)redhat.com>
Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server