Merge lp:~seb128/ubuntu-archive-scripts/display-broken-binaries into lp:ubuntu-archive-scripts

[Sebastien Bacher]

Displays the broken binaries information in the teams report

The current report has empty informations for packages making others uninstallable, with the change it does list the broken binaries information (on one line to keep it concise)

Example output https://people.canonical.com/~seb128/reportbinaries.html

(see gstreamer1.0 section)

[lp:~seb128/ubuntu-archive-scripts/display-broken-binaries]

291. By Sebastien Bacher on 2020-09-07

Displays the broken binaries information in the teams report.

[Iain Lane (laney)]

Logic looks good I think, just asking for a couple of tweaks. Thanks!

review: Needs Fixing

[Sebastien Bacher (seb128)]

Thanks for the review, I merged trunk and pushed the suggested changes

[lp:~seb128/ubuntu-archive-scripts/display-broken-binaries]

292. By Sebastien Bacher on 2020-09-17

rebase on trunk

293. By Sebastien Bacher on 2020-09-17

simply the code a bit as suggested in the review

[Iain Lane (laney)]

Thanks Seb!

For the upstream question, I think maybe "release.debian.org" on the BTS.

review: Approve

=== modified file 'generate-team-p-m'
--- generate-team-p-m	2020-08-13 09:36:50 +0000
+++ generate-team-p-m	2020-09-17 19:51:08 +0000
@@ -192,6 +192,7 @@
     data = attr.ib(default=None) # data for package_in_proposed
     unsatdepends = attr.ib(default=None) # [string]
     unsatbuilddep = attr.ib(default=None) # [string]
+    brokenbin = attr.ib(default=None) # [string]
 
     _age = attr.ib(default=None)
 
@@ -301,7 +302,9 @@
                     for p in packages:
                         unsatdbd[p].append(arch)
                 prob.unsatbuilddep = ['{}: {}'.format(p, ', '.join(sorted(arches))) for p, arches in sorted(unsatdbd.items())]
-
+        if 'implicit-deps' in item['policy_info']['implicit-deps']:
+                if item['policy_info']['implicit-deps']['implicit-deps']['broken-binaries']:
+                        prob.brokenbin = item['policy_info']['implicit-deps']['implicit-deps']['broken-binaries']
 
     package_to_problems = defaultdict(list)
 

=== modified file 'templates/team-report.html'
--- templates/team-report.html	2020-08-13 09:36:50 +0000
+++ templates/team-report.html	2020-09-17 19:51:08 +0000
@@ -113,6 +113,9 @@
             </ul>
           </li>
           {% endif %}
+          {% if prob.brokenbin %}
+          <li>Binaries broken by the update: {{ prob.brokenbin | join(' ') }}</li>
+          {% endif %}
           {% if d["policy_info"]["block-bugs"] %}
           <li>Blocked by bug:
           {% for bug in d["policy_info"]["block-bugs"]|map("int")|sort|map("string") %}