From Barak Korren <bkorren(a)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(a)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(a)redhat.com>
Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com>
Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com>
Gerrit-Reviewer: Daniel Belenky <dbelenky(a)redhat.com>
Gerrit-Reviewer: Evgheni Dereveanchin <ederevea(a)redhat.com>
Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com>
Gerrit-Reviewer: Gil Shinar <gshinar(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nadav Goldin <ngoldin(a)redhat.com>
Gerrit-Reviewer: Pavel Zhukov <pzhukov(a)redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Shlomo Ben David <sbendavi(a)redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>
Gerrit-Reviewer: Yuval Turgeman <yturgema(a)redhat.com>