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

tnisan at redhat.com tnisan at redhat.com
Mon Nov 30 08:37:31 UTC 2015


Tal Nisan 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>
(cherry picked from commit a1a99f2829ec02ca5a33efad1468592288439265)
---
M packaging/bin/engine-backup.sh
1 file changed, 3 insertions(+), 3 deletions(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ieade3725a9a932c98cfc4e443e4f0c6b96d041d5
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6
Gerrit-Owner: Yedidyah Bar David <didi at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan 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