Change in ovirt-engine[master]: packaging: engine-backup: Fix failure notifications

sbonazzo at redhat.com sbonazzo at redhat.com
Mon Nov 16 10:00:19 UTC 2015


Sandro Bonazzola has submitted this change and it was merged.

Change subject: packaging: engine-backup: Fix failure notifications
......................................................................


packaging: engine-backup: Fix failure notifications

Always check and set FAILURE_NOTIFIED before trying to notify.

Without this patch, in certain conditions, we could enter an endless
loop while trying to notify the engine about a failure.

Change-Id: Ieade3725a9a932c98cfc4e443e4f0c6b96d041d5
Bug-Url: https://bugzilla.redhat.com/1240923
Signed-off-by: Yedidyah Bar David <didi at redhat.com>
---
M packaging/bin/engine-backup.sh
1 file changed, 3 insertions(+), 3 deletions(-)

Approvals:
  Sandro Bonazzola: Looks good to me, approved; Passed CI tests
  Yedidyah Bar David: Verified



-- 
To view, visit https://gerrit.ovirt.org/48591
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ieade3725a9a932c98cfc4e443e4f0c6b96d041d5
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yedidyah Bar David <didi at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list