Merge lp:~brian-murray/britney/britney1-backlog-report-update into lp:~ubuntu-release/britney/britney1-ubuntu

Proposed by Brian Murray
Status: Merged
Merged at revision: 336
Proposed branch: lp:~brian-murray/britney/britney1-backlog-report-update
Merge into: lp:~ubuntu-release/britney/britney1-ubuntu
Diff against target: 25 lines (+4/-3)
1 file modified
scripts/backlog-report (+4/-3)
To merge this branch: bzr merge lp:~brian-murray/britney/britney1-backlog-report-update
Reviewer Review Type Date Requested Status
Iain Lane Approve
Steve Langasek Pending
Ubuntu Release Team Pending
Review via email: mp+387796@code.launchpad.net

Description of the change

backlog-report has been report 0 for valid candidates, not considered, and total since the new html output of update_excuse was created. This fixes that.

Old:
 $ python2 backlog-report update_excuses.html
time,valid candidates,not considered,total,median age,backlog
1595371220000,0,0,0,6,16687
New:
 $ python2 backlog-report update_excuses.html
time,valid candidates,not considered,total,median age,backlog
1595371220000,3,797,800,6,16687

To post a comment you must log in.
Revision history for this message
Iain Lane (laney) wrote :

this should use the yaml, but ok :)

thanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'scripts/backlog-report'
--- scripts/backlog-report 2017-07-26 18:16:35 +0000
+++ scripts/backlog-report 2020-07-22 00:39:14 +0000
@@ -44,9 +44,9 @@
44 elif age_match:44 elif age_match:
45 ages.append(int(age_match.group(1)))45 ages.append(int(age_match.group(1)))
46 backlog += max(int(age_match.group(1)) - 3, 0)46 backlog += max(int(age_match.group(1)) - 3, 0)
47 elif line.startswith("<li>Valid candidate"):47 elif 'Will attempt migration' in line:
48 valid_candidates += 148 valid_candidates += 1
49 elif line.startswith("<li>Not considered"):49 elif 'BLOCKED' in line:
50 not_considered += 150 not_considered += 1
51 finally:51 finally:
52 binary.close()52 binary.close()
@@ -81,7 +81,8 @@
8181
82 try:82 try:
83 csv_writer = csv.DictWriter(83 csv_writer = csv.DictWriter(
84 output, ["time", "valid candidates", "not considered", "total", "median age", "backlog"])84 output, ["time", "valid candidates", "not considered", "total",
85 "median age", "backlog"])
85 if csv_is_new:86 if csv_is_new:
86 csv_writer.writeheader()87 csv_writer.writeheader()
87 for arg in args:88 for arg in args:

Subscribers

People subscribed via source and target branches