Merge lp:~brian-murray/ubuntu-archive-tools/stop-phasing into lp:ubuntu-archive-tools

Proposed by Brian Murray
Status: Merged
Approved by: Łukasz Zemczak
Approved revision: 1401
Merged at revision: 1401
Proposed branch: lp:~brian-murray/ubuntu-archive-tools/stop-phasing
Merge into: lp:ubuntu-archive-tools
Diff against target: 35 lines (+11/-7)
1 file modified
phased-updater (+11/-7)
To merge this branch: bzr merge lp:~brian-murray/ubuntu-archive-tools/stop-phasing
Reviewer Review Type Date Requested Status
Łukasz Zemczak Approve
Review via email: mp+393328@code.launchpad.net

Description of the change

The phased updater only respects phasing set to 0% if there is a regression or rate increase in the Error Tracker, work around that by hard coding packages that shouldn't be phased.

To post a comment you must log in.
Revision history for this message
Łukasz Zemczak (sil2100) wrote :

I think for now this would work.

But after we land this I would like us to consider two options:
 1) Making the STOPPED_PACKAGES list work per versions, so the packagelist being ['glibc/SOMEVERSION'] etc.
 2) Maybe moving this to a separate file? Like, some exceptions file hosted elsewhere like with the phase exceptions for resetting regressions?

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'phased-updater'
2--- phased-updater 2020-09-10 10:32:02 +0000
3+++ phased-updater 2020-11-04 17:01:44 +0000
4@@ -650,13 +650,16 @@
5 last_pup = None
6 max_pup = 0
7 if last_pup == 0:
8- for allpb in archive.getPublishedBinaries(
9- exact_match=True, pocket='Updates',
10- binary_name=pbs[0].binary_package_name):
11- if allpb.distro_arch_series.distroseries == release:
12- if allpb.phased_update_percentage > 0:
13- max_pup = allpb.phased_update_percentage
14- break
15+ if rname in STOPPED_PACKAGES:
16+ if src_pkg in STOPPED_PACKAGES[rname]:
17+ continue
18+ for allpb in archive.getPublishedBinaries(
19+ exact_match=True, pocket='Updates',
20+ binary_name=pbs[0].binary_package_name):
21+ if allpb.distro_arch_series.distroseries == release:
22+ if allpb.phased_update_percentage > 0:
23+ max_pup = allpb.phased_update_percentage
24+ break
25 if max_pup and last_pup == 0:
26 rate_increase = crash_rate_increase(release, src_pkg, version, max_pup)
27 else:
28@@ -734,6 +737,7 @@
29 EMAIL_SENDER = 'brian.murray@ubuntu.com'
30 PUP_INCREMENT = 10
31 SLOW_PACKAGES = {'secureboot-db'}
32+ STOPPED_PACKAGES = {'bionic': ['glibc']}
33 PUP_SLOW_INCREMENT = 1
34 REPORT_FILE = 'phased-updates.html'
35 parser = OptionParser(usage="usage: %prog [options]")

Subscribers

People subscribed via source and target branches