Skip to content

Commit

Permalink
show repository state and details. It can be broken and the user must…
Browse files Browse the repository at this point in the history
… be able to see the reason.

That was always a problem, but with the new option for disabled publishing
due to package failures it becomes more critical
  • Loading branch information
adrianschroeter authored and dmach committed Mar 16, 2022
1 parent 8da29dc commit 389a37f
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion osc/core.py
Expand Up @@ -5788,7 +5788,7 @@ def result_xml_to_dicts(xml):
rmap['repostate'] = node.get('code')
rmap['pkg'] = rmap['package'] = rmap['pac'] = ''
rmap['code'] = node.get('code')
rmap['details'] = ''
rmap['details'] = node.get('details')
# the way we currently use this function, there should be
# always a status element
snodes = node.findall('status')
Expand All @@ -5809,6 +5809,10 @@ def result_xml_to_dicts(xml):
details = statusnode.find('details')
if details is not None:
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


Expand Down Expand Up @@ -5951,6 +5955,8 @@ def get_prj_results(apiurl, prj, hide_legend=False, csv=False, status_filter=Non
state = "outdated"
else:
state = node.get('state')
if node.get('details'):
state += ' details: ' + node.get('details')
tg = (node.get('repository'), node.get('arch'), state)
targets.append(tg)
for pacnode in node.findall('status'):
Expand Down

0 comments on commit 389a37f

Please sign in to comment.