Merge lp:~vorlon/britney/britney1-more-stats into lp:~ubuntu-release/britney/britney1-ubuntu

Proposed by Steve Langasek
Status: Merged
Merged at revision: 308
Proposed branch: lp:~vorlon/britney/britney1-more-stats
Merge into: lp:~ubuntu-release/britney/britney1-ubuntu
Diff against target: 25 lines (+3/-0)
1 file modified
scripts/backlog-report (+3/-0)
To merge this branch: bzr merge lp:~vorlon/britney/britney1-more-stats
Reviewer Review Type Date Requested Status
Iain Lane Needs Information
Review via email: mp+327887@code.launchpad.net

Description of the change

Add an additional way to measure the proposed-migration backlog

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

Do you think it's possible that anything could have been reading this and relying on the number of columns it had?

What about forwarding to Debian?

Other than that, I think this is fine.

review: Needs Information
Revision history for this message
Steve Langasek (vorlon) wrote :

> Do you think it's possible that anything could have been
> reading this and relying on the number of columns it had?

It's possible, but we've never made any promises around that, and the last change to the column count was only a year ago.

> What about forwarding to Debian?

Applicability to Debian is not straightforward; our devel-proposed partial suite is in principle something we want to drive down to zero, the same is not true for how Debian manages unstable vs. testing. I think for now we should just carry it as a delta.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'scripts/backlog-report'
2--- scripts/backlog-report 2016-03-28 19:31:05 +0000
3+++ scripts/backlog-report 2017-07-21 15:04:29 +0000
4@@ -26,6 +26,7 @@
5 valid_candidates = 0
6 not_considered = 0
7 ages = []
8+ backlog = 0
9
10 if path.endswith(".gz"):
11 binary = gzip.open(path)
12@@ -42,6 +43,7 @@
13 match.group(1), "%Y.%m.%d %H:%M:%S +0000")
14 elif age_match:
15 ages.append(int(age_match.group(1)))
16+ backlog += max(int(age_match.group(1)) - 3, 0)
17 elif line.startswith("<li>Valid candidate"):
18 valid_candidates += 1
19 elif line.startswith("<li>Not considered"):
20@@ -57,6 +59,7 @@
21 "not considered": not_considered,
22 "total": valid_candidates + not_considered,
23 "median age": ages[len(ages)/2],
24+ "backlog": backlog,
25 })
26
27

Subscribers

People subscribed via source and target branches