Change in jenkins[master]: Enhance queue api to also return failure cause

Code Review gerrit at ovirt.org
Tue May 23 08:30:16 UTC 2017


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


More information about the Engine-commits mailing list