Bug 1459185 Fix filter for email in release-notify-promote r=jlorenzo a=release
authorSimon Fraser <sfraser@mozilla.com>
Mon, 07 May 2018 10:05:00 +1200
changeset 795865 4a43592f14df511247ffcfde4a850003f274f272
parent 795864 5c4f8510ccf7e32740818dcfb5f87ad1f931d5ae
child 795867 eca6dd107bf79270da8093873ea90313f198a55c
child 812296 3202d5534730cfd762a7d3c33720afd3bdbfd930
push id110108
push userbmo:tom@mozilla.com
push dateWed, 16 May 2018 18:43:56 +0000
reviewersjlorenzo, release
bugs1459185
milestone60.0.1
Bug 1459185 Fix filter for email in release-notify-promote r=jlorenzo a=release Summary: dustin's updating the documentation in https://github.com/taskcluster/taskcluster-notify/pull/33/files Reviewers: jlorenzo Reviewed By: jlorenzo Bug #: 1459185 Differential Revision: https://phabricator.services.mozilla.com/D1132
taskcluster/taskgraph/transforms/release_notifications.py
--- a/taskcluster/taskgraph/transforms/release_notifications.py
+++ b/taskcluster/taskgraph/transforms/release_notifications.py
@@ -34,17 +34,17 @@ def add_notifications(config, jobs):
             subject = notifications['subject'].format(**format_kwargs)
             message = notifications['message'].format(**format_kwargs)
             # Don't need this any more
             del job['notifications']
 
             # We only send mail on success to avoid messages like 'blah is in the
             # candidates dir' when cancelling graphs, dummy job failure, etc
             job.setdefault('routes', []).extend(
-                ['notify.email.{}.on-success'.format(email) for email in emails]
+                ['notify.email.{}.on-completed'.format(email) for email in emails]
             )
 
             # Customize the email subject to include release name and build number
             job.setdefault('extra', {}).update(
                 {
                    'notify': {
                        'email': {
                             'subject': subject,