Merge lp:~brian-murray/ubuntu-archive-tools/sru-report-incomplete into lp:ubuntu-archive-tools

[Brian Murray]

[Łukasz Zemczak (sil2100)]

Generally +1, just two inline things.

review: Needs Fixing

[lp:~brian-murray/ubuntu-archive-tools/sru-report-incomplete]

1193. By Brian Murray on 2018-11-21

better check the package and release for a task, only check tasks when we have a bug

1194. By Brian Murray on 2018-11-21

add in the yellow color to the legend

[Brian Murray (brian-murray)]

I'm now splitting task_link with / to address the package naming concern. I also put the task look up into the try / except for the Launchpad bug.

[Łukasz Zemczak (sil2100)]

Ok, thanks! This is better now. The splitting and analysis of self_link is not the most pretty way to go, but I guess it should work fine. Approving.

review: Approve

=== modified file 'sru-report'
--- sru-report	2018-07-20 18:29:04 +0000
+++ sru-report	2018-11-21 19:00:41 +0000
@@ -185,6 +185,7 @@
     .noborder { border-style: none; }
     a { color: blue; }
     a.messages { color: #999900; font-weight: bold; }
+    a.incomplete { color: yellow; font-weight: bold; }
     a.verified { color: green; font-weight: bold; }
     a.verificationfailed { color: red; font-weight: bold; }
     a.kerneltracking { font-style: italic; }
@@ -220,6 +221,7 @@
 
 <p>Bugs in <span style="color:green;">green</span> are verified,
 bugs in <span style="color:red;">red</span> failed verification,
+bugs in <span style="color:yellow;">yellow</span> are Incomplete,
 bugs in <span style="color: #999900;">golden</span> have received a comment
 since the package was accepted in -proposed,
 bugs in <span style="color: gray;">gray</span> are candidates for removal
@@ -331,10 +333,19 @@
             antique = False
             for b, t in sorted(rpkg['bugs'].iteritems()):
                 cls = ' class="'
+                incomplete = False
                 try:
                     bug = lp.bugs[b]
                     bug_title = bug.title.encode('UTF-8')
                     hover_text = bug_title
+                    for task in bug.bug_tasks:
+                        if task.self_link.split('/')[4] != 'ubuntu':
+                            continue
+                        if pkg == task.self_link.split('/')[7] \
+                                and release == task.self_link.split('/')[5]:
+                            if task.status == 'Incomplete':
+                                incomplete = True
+                                break
                 except KeyError:
                     logging.debug(
                         'bug %d does not exist or is not accessible' % b)
@@ -343,7 +354,9 @@
                 if ('kernel-tracking-bug' in t or
                         'kernel-release-tracking-bug' in t):
                     cls += 'kerneltracking '
-                if ('verification-failed' in t or
+                if incomplete:
+                    cls += ' incomplete'
+                elif ('verification-failed' in t or
                         'verification-failed-%s' % release in t):
                     cls += ' verificationfailed'
                 elif 'verification-done-%s' % release in t: