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@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@redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett@redhat.com> Gerrit-Reviewer: Martin Sivák <msivak@redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo@redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server