From Barak Korren <bkorren@redhat.com>:
Barak Korren has submitted this change and it was merged. Change subject: Enhance queue api to also return failure cause ...................................................................... Enhance queue api to also return failure cause Enhanced the API of the ChangeQueue objects to also return the specific change that caused the failure when failed changes are detected. (Up until now we just assumed the causing change will always be the first in the list of changes that are removed due to failure) This will allow adding more sophisticated cause detection code to be added to the change queue objects. Change-Id: Iaf34e047a0a53d3b6c151319f810431ccac73a3a Signed-off-by: Barak Korren <bkorren@redhat.com> Jira-Ticket: https://ovirt-jira.atlassian.net/browse/OVIRT-1077 --- M scripts/change_queue/__init__.py M test/test_change_queue.py 2 files changed, 42 insertions(+), 37 deletions(-) Approvals: Barak Korren: Verified; Looks good to me, approved; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/75122 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iaf34e047a0a53d3b6c151319f810431ccac73a3a Gerrit-PatchSet: 7 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Barak Korren <bkorren@redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk@redhat.com> Gerrit-Reviewer: Barak Korren <bkorren@redhat.com> Gerrit-Reviewer: Daniel Belenky <dbelenky@redhat.com> Gerrit-Reviewer: Evgheni Dereveanchin <ederevea@redhat.com> Gerrit-Reviewer: Eyal Edri <eedri@redhat.com> Gerrit-Reviewer: Gil Shinar <gshinar@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nadav Goldin <ngoldin@redhat.com> Gerrit-Reviewer: Pavel Zhukov <pzhukov@redhat.com> Gerrit-Reviewer: Roy Golan <rgolan@redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo@redhat.com> Gerrit-Reviewer: Shlomo Ben David <sbendavi@redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi@redhat.com> Gerrit-Reviewer: Yuval Turgeman <yturgema@redhat.com>