Bug 1189376 - Link to perfherder for try pushes that contain talos jobs. r=bhearsum
authorWilliam Lachance <wlachance@mozilla.com>
Fri, 31 Jul 2015 18:38:07 -0400 (2015-07-31)
changeset 4370 ba27e51a5823f0e1e52b2ee446aeb75576f31d98
parent 4366 70a377d206c5484b9db38ea7bba8d81042bc28d9
child 4373 99a9e7140718bff4eb45dbe08fdc583447d19a70
push id3660
push userryanvm@gmail.com
push dateTue, 04 Aug 2015 19:02:44 +0000 (2015-08-04)
reviewersbhearsum
bugs1189376
Bug 1189376 - Link to perfherder for try pushes that contain talos jobs. r=bhearsum
status/generators.py
--- a/status/generators.py
+++ b/status/generators.py
@@ -11,29 +11,37 @@ def buildTryChangeMessage(change, packag
     msgdict['headers'] = {"In-Reply-To": "<%(tree)s-%(revision)s>" % locals(),
                           "References": "<%(tree)s-%(revision)s>" % locals(),
                           }
     msgdict["body"] = """\
 Thank you for your try submission. It's the best!
 
 Results will be displayed on Treeherder as they come in:
 https://treeherder.mozilla.org/#/jobs?repo=%(tree)s&revision=%(revision)s
-
-Once completed, builds and logs will be available at:
-%(packageDir)s
 """ % locals()
 
     commitTitles = change.properties.getProperty('commit_titles')
     if commitTitles:
         title = getSensibleCommitTitle(commitTitles)
         allTitles = '\n  * '.join(commitTitles)
 
+        if ((' -t ' in allTitles or ' --talos ' in allTitles) and
+            '-t none' not in allTitles and '--talos none' not in allTitles):
+            msgdict['body'] += """\
+
+It looks like this submission has talos jobs. You can compare the performance of your push against a baseline revision here:
+https://treeherder.mozilla.org/perf.html?newProject=try&newRevision=%(revision)s
+""" % locals()
+
         msgdict['subject'] += ': %(title)s' % locals()
         msgdict['body'] += """\
 
+Once completed, builds and logs will be available at:
+%(packageDir)s
+
 Summary:
   * %(allTitles)s
 """ % locals()
 
     return msgdict
 
 
 def getSensibleCommitTitle(titles):