Skip to content

Commit

Permalink
Merge pull request #1249 from jberry-suse/repo_checker-openqa-only
Browse files Browse the repository at this point in the history
repo_checker: review failed stagings with only openQA failures.
  • Loading branch information
jberry-suse committed Nov 9, 2017
2 parents 1efadc5 + 48fe383 commit 3e191ca
Showing 1 changed file with 14 additions and 2 deletions.
16 changes: 14 additions & 2 deletions repo_checker.py
Original file line number Diff line number Diff line change
Expand Up @@ -125,8 +125,20 @@ def prepare_review(self):
# Corrupted requests may reference non-existent projects and will
# thus return a None status which should be considered not ready.
if not status or str(status['overall_state']) not in ('testing', 'review', 'acceptable'):
self.logger.debug('{}: {} not ready'.format(request.reqid, group))
continue
# Not in a "ready" state.
openQA_only = False # Not relevant so set to False.
if str(status['overall_state']) == 'failed':
# Exception to the rule is openQA only in failed state.
openQA_only = True
for project in api.project_status_walk(status):
if len(project['broken_packages']):
# Broken packages so not just openQA.
openQA_only = False
break

if not openQA_only:
self.logger.debug('{}: {} not ready'.format(request.reqid, group))
continue

# Only interested if request is in consistent state.
selected = api.project_status_requests('selected')
Expand Down

0 comments on commit 3e191ca

Please sign in to comment.