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@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@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski@gmail.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo@redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan@redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>