← Back to team overview

launchpad-reviewers team mailing list archive

[Merge] lp:~cjwatson/lpbuildbot/reduce-email into lp:lpbuildbot

 

Colin Watson has proposed merging lp:~cjwatson/lpbuildbot/reduce-email into lp:lpbuildbot.

Commit message:
Stop sending failure emails to the Launchpad team list.

Requested reviews:
  Tom Wardill (twom)

For more details, see:
https://code.launchpad.net/~cjwatson/lpbuildbot/reduce-email/+merge/392035

This is too noisy in practice, and we have other ways to notice when builds are failing.
-- 
Your team Launchpad code reviewers is subscribed to branch lp:lpbuildbot.
=== modified file 'master.cfg'
--- master.cfg	2020-10-05 22:12:06 +0000
+++ master.cfg	2020-10-09 10:46:22 +0000
@@ -215,21 +215,13 @@
              'lp-db-devel-bionic-py3'),
             ]))
 
-# we want to send success email only to interested users, but fail email to
-# interested users and launchpad list.
+# Send email to interested users.
 import lpbuildbot.mail
 c['status'].append(
     lpbuildbot.mail.MailNotifier(
         fromaddr="canonical-launchpad@xxxxxxxxxxxxxxxxxxx",
         lookup=lpbuildbot.mail.PassThroughLookup(),
         builders=['lp-devel-xenial', 'lp-db-devel-xenial']))
-c['status'].append(
-    lpbuildbot.mail.MailNotifier(
-        fromaddr="losas@xxxxxxxxxxxxx",
-        extraRecipients=["canonical-launchpad@xxxxxxxxxxxxxxxxxxx"],
-        mode='failing',
-        sendToInterestedUsers=False,
-        builders=['lp-devel-xenial', 'lp-db-devel-xenial']))
 
 # at the time of this writing, the buildbot IRC client does not support SSL
 # so we would need to modify buildbot to use this.  Supposedly Twisted's IRC


References