Merge pull request #1462 from jberry-suse/check_source_in_factory-repo-checker-allow
check_source_in_factory: allow repo-checker reviews.
This commit is contained in:
commit
d2cec14e17
@ -181,9 +181,15 @@ class FactorySourceChecker(ReviewBot.ReviewBot):
|
|||||||
self.logger.error("%s in state review but no reviews?", srref(req.reqid))
|
self.logger.error("%s in state review but no reviews?", srref(req.reqid))
|
||||||
return False
|
return False
|
||||||
for r in req.reviews:
|
for r in req.reviews:
|
||||||
if r.by_project and r.state == 'new' and r.by_project.startswith('openSUSE:Factory:Staging:'):
|
if r.state == 'new':
|
||||||
self.logger.info("%s review by %s ok", srref(req.reqid), r.by_project)
|
if r.by_project and r.by_project.startswith('openSUSE:Factory:Staging:'):
|
||||||
continue
|
self.logger.info("%s review by %s ok", srref(req.reqid), r.by_project)
|
||||||
|
continue
|
||||||
|
|
||||||
|
if r.by_user == 'repo-checker':
|
||||||
|
self.logger.info('%s review by %s ok', srref(req.reqid), r.by_user)
|
||||||
|
continue
|
||||||
|
|
||||||
if r.state != 'accepted':
|
if r.state != 'accepted':
|
||||||
if r.by_user:
|
if r.by_user:
|
||||||
self.logger.info("%s waiting for review by %s", srref(req.reqid), r.by_user)
|
self.logger.info("%s waiting for review by %s", srref(req.reqid), r.by_user)
|
||||||
|
Loading…
x
Reference in New Issue
Block a user