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

Unified Diff: masters/master.chromium.memory.full/notifier_cfg.py

Issue 2415043002: Remove DrMemory slaves from chromium.memory.full master. (Closed)
Patch Set: Created 4 years, 2 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 | « masters/master.chromium.memory.full/master.cfg ('k') | masters/master.chromium.memory.full/slaves.cfg » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: masters/master.chromium.memory.full/notifier_cfg.py
diff --git a/masters/master.chromium.memory.full/notifier_cfg.py b/masters/master.chromium.memory.full/notifier_cfg.py
index 3f0bfb2b6a0ae6590f67396c37a5caa5038e0b7d..5404c6c9dfac17ca996ff76b38db921e858b63c2 100644
--- a/masters/master.chromium.memory.full/notifier_cfg.py
+++ b/masters/master.chromium.memory.full/notifier_cfg.py
@@ -81,60 +81,3 @@ def Update(config, active_master, c):
'http://dev.chromium.org/developers/how-tos/using-valgrind/threadsanitizer\n'
'\nBy the way, the current memory sheriff is on the CC list.'
))
-
- # Set up a separate notifier for the Dr.Memory bots.
- # TODO(timurrrr); merge this with the main notifier once DrMemory is verified
- # to send few e-mails.
- drm_categories_steps = {
- 'drmemory_tester': [
- # Please keep this list sorted.
- 'memory test: base_unittests',
- 'memory test: components',
- 'memory test: content',
- 'memory test: crypto',
- 'memory test: display_unittests',
- 'memory test: extensions_unittests',
- 'memory test: ipc_tests',
- 'memory test: media',
- 'memory test: net',
- 'memory test: printing',
- 'memory test: remoting',
- 'memory test: sql',
- 'memory test: sync',
- 'memory test: ui_base_unittests',
- 'memory test: unit', # unit_tests might be sharded
- 'memory test: unit_1',
- 'memory test: unit_2',
- 'memory test: url',
- ],
- }
- c['status'].append(failures_notifier.FailuresNotifier(
- fromaddr=active_master.from_address,
- categories_steps=drm_categories_steps,
- exclusions=exclusions,
- relayhost=config.Master.smtp,
- subject='drmemory buildbot %(result)s in %(projectName)s on '
- '%(builder)s, revision %(revision)s',
- sendToInterestedUsers=True,
- extraRecipients=(
- active_master.tree_closing_notification_recipients +
- # Also send e-mails to the Dr.Memory team.
- ['bruening+drmfailure@google.com',
- 'zhaoqin+drmfailure@google.com',
- ]),
- lookup=master_utils.FilterDomain(),
- forgiving_steps=forgiving_steps,
- use_getname=True,
- public_html='../master.chromium/public_html',
- sheriffs=['sheriff_memory'],
- status_header='Failure notification for "%(steps)s" on "%(builder)s".\n\n'
- 'NOTE: This bot is in testing mode, but most of the failures are '
- 'expected to indicate real bugs.\n\n'
- 'Please see if the failures are related to your commit and take '
- 'appropriate actions (e.g. revert, update suppressions, notify '
- 'sheriff, etc.).\n\n'
- 'For more info on Dr.Memory waterfall please see these links:\n'
- 'http://dev.chromium.org/developers/tree-sheriffs/sheriff-details-chromium/memory-sheriff\n'
- 'http://dev.chromium.org/developers/how-tos/using-drmemory\n'
- '\nBy the way, the current memory sheriff is on the CC list.'
- ))
« no previous file with comments | « masters/master.chromium.memory.full/master.cfg ('k') | masters/master.chromium.memory.full/slaves.cfg » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698