ubuntu-archive-tools

Merge lp:~brian-murray/ubuntu-archive-tools/phased-updater-notify-bdmurray into lp:ubuntu-archive-tools

  1. phased-updater-notify-bdmurray
  2. Merge into trunk
Proposed by Brian Murray
Status: Merged
Merged at revision: 794
Proposed branch: lp:~brian-murray/ubuntu-archive-tools/phased-updater-notify-bdmurray
Merge into: lp:ubuntu-archive-tools
Diff against target: 44 lines (+5/-2)
1 file modified
phased-updater (+5/-2)
To merge this branch: bzr merge lp:~brian-murray/ubuntu-archive-tools/phased-updater-notify-bdmurray
Reviewer Review Type Date Requested Status
Ubuntu Package Archive Administrators Pending
Review via email: mp+193950@code.launchpad.net

Description of the change

It was suggested to me that the Reply-To header be set so that it is easier for people to ask for overrides of problems. This branch adds that and also CC's me when an update is stopped.

To post a comment you must log in.

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 2013-10-25 19:32:07 +0000
3+++ phased-updater 2013-11-05 16:11:59 +0000
4@@ -211,6 +211,7 @@
5 notifications[person].append((problem, pkg, pkg_version))
6 except IOError:
7 pass
8+ bdmurray_mail = 'brian@ubuntu.com'
9 b_body = ('Your upload of %s version %s to %s has resulted in %s'
10 'error%s that %s first reported about this version of the '
11 'package. The error%s follow%s:\n\n'
12@@ -261,7 +262,7 @@
13 spph_buckets[rname][spph]['buckets'].remove(notice)
14 if len(spph_buckets[rname][spph]['buckets']) == 0:
15 continue
16- receivers = []
17+ receivers = ['%s' % bdmurray_mail]
18 quantity = len(spph_buckets[rname][spph]['buckets'])
19 msg = MIMEText(
20 b_body % (pkg, version, rname, ('an ', '')[quantity != 1],
21@@ -274,6 +275,7 @@
22 subject = '[%s/%s] Possible Regression' % (rname, pkg)
23 msg['Subject'] = subject
24 msg['From'] = EMAIL_SENDER
25+ msg['Reply-To'] = bdmurray_mail
26 receivers.append(signer_email)
27 msg['To'] = signer_email
28 if changer != signer and changer.is_valid:
29@@ -299,13 +301,14 @@
30 notices = notifications[signer_name]
31 if ('increased-rate', pkg, version) in notices:
32 continue
33- receivers = []
34+ receivers = ['%s' % bdmurray_mail]
35 msg = MIMEText(i_body % (pkg, version, rname,
36 spph_buckets[rname][spph]['rate'][1])
37 + remedy)
38 subject = '[%s/%s] Increase in crash rate' % (rname, pkg)
39 msg['Subject'] = subject
40 msg['From'] = EMAIL_SENDER
41+ msg['Reply-To'] = bdmurray_mail
42 receivers.append(signer_email)
43 msg['To'] = signer_email
44 if changer != signer and changer.is_valid:

Subscribers

People subscribed via source and target branches