mirrors/github.com_openSUSE_osc
1
0
Fork 0
mirror of https://github.com/openSUSE/osc.git synced 2024-09-20 09:16:16 +02:00
Code

show repository state and details. It can be broken and the user must be able to see the reason.

Browse Source 
That was always a problem, but with the new option for disabled publishing
due to package failures it becomes more critical
This commit is contained in:
Adrian Schröter 2020-10-05 10:08:52 +02:00 committed by Daniel Mach
parent 8da29dc766
commit 389a37fbd0
1 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
osc/core.py
View File

@ -5788,7 +5788,7 @@ def result_xml_to_dicts(xml):
rmap['repostate'] = node.get('code') rmap['repostate'] = node.get('code')
rmap['pkg'] = rmap['package'] = rmap['pac'] = '' rmap['pkg'] = rmap['package'] = rmap['pac'] = ''
rmap['code'] = node.get('code') rmap['code'] = node.get('code')
rmap['details'] = '' rmap['details'] = node.get('details')
# the way we currently use this function, there should be # the way we currently use this function, there should be
# always a status element # always a status element
snodes = node.findall('status') snodes = node.findall('status')
@ -5809,6 +5809,10 @@ def result_xml_to_dicts(xml):
details = statusnode.find('details') details = statusnode.find('details')
if details is not None: if details is not None:
smap['details'] = details.text smap['details'] = details.text
if rmap['code'] == 'broken':
# real error just becomes visible in details/verbose
smap['code'] = rmap['code']
smap['details'] = "repository: " + rmap['details']
yield smap, is_multi yield smap, is_multi
@ -5951,6 +5955,8 @@ def get_prj_results(apiurl, prj, hide_legend=False, csv=False, status_filter=Non
state = "outdated" state = "outdated"
else: else:
state = node.get('state') state = node.get('state')
if node.get('details'):
state += ' details: ' + node.get('details')
tg = (node.get('repository'), node.get('arch'), state) tg = (node.get('repository'), node.get('arch'), state)
targets.append(tg) targets.append(tg)
for pacnode in node.findall('status'): for pacnode in node.findall('status'):