Merge pull request #1757 from Vogtinator/reportfix
Don't report that a package is ok if the resultlist is empty
This commit is contained in:
commit
c85f26ede8
@ -399,12 +399,18 @@ class ToTestBase(object):
|
|||||||
url = self.api.makeurl(['build', project, '_result'], query)
|
url = self.api.makeurl(['build', project, '_result'], query)
|
||||||
f = self.api.retried_GET(url)
|
f = self.api.retried_GET(url)
|
||||||
root = ET.parse(f).getroot()
|
root = ET.parse(f).getroot()
|
||||||
for repo in root.findall('result'):
|
succeeded = root.findall('result/status[@code="succeeded"]')
|
||||||
status = repo.find('status')
|
# [@code!='succeeded'] is not supported by ET
|
||||||
if status.get('code') != 'succeeded':
|
failed = [status for status in root.findall("result/status") if status.get('code') != 'succeeded']
|
||||||
logger.info(
|
|
||||||
'%s %s %s %s -> %s' % (project, package, repository, arch, status.get('code')))
|
if len(failed) > 0:
|
||||||
return False
|
logger.info(
|
||||||
|
'%s %s %s %s -> %s' % (project, package, repository, arch, failed[0].get('code')))
|
||||||
|
return False
|
||||||
|
|
||||||
|
if len(succeeded) == 0:
|
||||||
|
logger.info('No results for %s %s %s %s' % (project, package, repository, arch))
|
||||||
|
return False
|
||||||
|
|
||||||
maxsize = self.maxsize_for_package(package)
|
maxsize = self.maxsize_for_package(package)
|
||||||
if not maxsize:
|
if not maxsize:
|
||||||
|
Loading…
x
Reference in New Issue
Block a user