Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(336)

Unified Diff: dashboard/dashboard/email_template.py

Issue 2138843003: Add support for setting sheriff-specific bug labels. (Closed) Base URL: https://github.com/catapult-project/catapult@master
Patch Set: Fix unit tests Created 4 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « dashboard/dashboard/email_summary_test.py ('k') | dashboard/dashboard/file_bug.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: dashboard/dashboard/email_template.py
diff --git a/dashboard/dashboard/email_template.py b/dashboard/dashboard/email_template.py
index 937bc6c539357f6d454b1de402dc2139c30c4587..c353f45bc135bcb027f1157dc622e9bac1b8cf07 100644
--- a/dashboard/dashboard/email_template.py
+++ b/dashboard/dashboard/email_template.py
@@ -50,7 +50,7 @@ _BUG_REPORT_COMMENT = (
_BUG_REPORT_LINK_URL = (
'https://code.google.com/p/chromium/issues/entry?summary=%s&comment=%s&'
- 'labels=Type-Bug-Regression,Pri-2,Performance-Sheriff,%s')
+ 'labels=Type-Bug-Regression,Pri-2,%s')
_ALL_ALERTS_LINK = (
'<a href="https://chromeperf.appspot.com/alerts?sheriff=%s">'
@@ -299,7 +299,8 @@ def GetAlertInfo(alert, test):
bug_comment = _BUG_REPORT_COMMENT % interpolation_parameters
bug_summary = ('%(percent_changed)s %(change_type)s in %(test_name)s '
'on %(bot)s at %(start)d:%(end)d') % interpolation_parameters
- labels = bug_label_patterns.GetBugLabelsForTest(test)
+ labels = (alert.sheriff.get().labels +
+ bug_label_patterns.GetBugLabelsForTest(test))
bug_url = _BUG_REPORT_LINK_URL % (
urllib.quote(bug_summary), urllib.quote(bug_comment), ','.join(labels))
« no previous file with comments | « dashboard/dashboard/email_summary_test.py ('k') | dashboard/dashboard/file_bug.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698