Change in ovirt-hosted-engine-ha[master]: Fix the email flood caused by not handling the yield logic p...

[gpadgett at redhat.com]

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