yudaike/openSUSE-release-tools
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

repo_checker: Fix the review loop

Browse Source 
The break on openQA failures is a left over from 93ee829260a6abf094cbbc31e26eb21bf45e8f15
where we stopped the loop over subprojects. We don't want to break out of
the review loop on openQA failures
This commit is contained in:
Stephan Kulow 2018-07-19 08:41:16 +02:00
parent d16163483a
commit 44f93dfa7c
1 changed files with 3 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
repo_checker.py
View File

@ -164,12 +164,9 @@ class RepoChecker(ReviewBot.ReviewBot):
# Not in a "ready" state. # Not in a "ready" state.
openQA_only = False # Not relevant so set to False. openQA_only = False # Not relevant so set to False.
if status and str(status['overall_state']) == 'failed': if status and str(status['overall_state']) == 'failed':
# Exception to the rule is openQA only in failed state. # Exception to the rule is openQA only in failed state,
openQA_only = True # Broken packages so not just openQA.
if len(status['broken_packages']): openQA_only = (len(status['broken_packages']) == 0)
# Broken packages so not just openQA.
openQA_only = False
break
if not self.force and not openQA_only: if not self.force and not openQA_only:
self.logger.debug('{}: {} not ready'.format(request.reqid, group)) self.logger.debug('{}: {} not ready'.format(request.reqid, group))