The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/c29b4c7ec10db41f38e0361febe9846a95629b5a

commit c29b4c7ec10db41f38e0361febe9846a95629b5a
Author: Aviv Keshet <akeshet@chromium.org>
Date: Thu Dec 15 06:27:35 2016

autotest: delete some email alerts; replace some with monarch metrics

For email alerts that seem (based on searching my email) to never be
sent, I simply deleted them.

For those that are sent sometimes and seem easily amenable to a monarch
metric instead, I changed them to a metric.

This is a first step; there are still many remaining unneccesary email
alerts.

BUG=chromium:672726
TEST=None

Change-Id: Ib1d3715e618623faa16f3faaceabf4218dbad49a
Reviewed-on: https://chromium-review.googlesource.com/420468
Commit-Ready: Aviv Keshet <akeshet@chromium.org>
Tested-by: Aviv Keshet <akeshet@chromium.org>
Reviewed-by: Aviv Keshet <akeshet@chromium.org>

[modify] https://crrev.com/c29b4c7ec10db41f38e0361febe9846a95629b5a/scheduler/monitor_db_cleanup.py
[modify] https://crrev.com/c29b4c7ec10db41f38e0361febe9846a95629b5a/scheduler/drone_manager.py
[modify] https://crrev.com/c29b4c7ec10db41f38e0361febe9846a95629b5a/scheduler/monitor_db.py
[modify] https://crrev.com/c29b4c7ec10db41f38e0361febe9846a95629b5a/scheduler/scheduler_config.py
[modify] https://crrev.com/c29b4c7ec10db41f38e0361febe9846a95629b5a/global_config.ini

aviv, what work remains on this bug? 

There still work left here. We're done when this query returns empty: https://cs.corp.google.com/search/?q=email_manager&m=100&sq=package:chromeos+file:src/third_party/autotest/files&type=cs

sounds like pprahbu@ is owning, assigning to him 

I talk too much :(

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/82c508c92026bad3c2165ac858e2578bc6db479a

commit 82c508c92026bad3c2165ac858e2578bc6db479a
Author: Aviv Keshet <akeshet@chromium.org>
Date: Tue Jan 03 22:10:17 2017

autotest: turn pidfile email alert into a monarch metrics

BUG=chromium:672726
TEST=None

Change-Id: I784bd962e9667b729684caf0effb6ddf892e2252
Reviewed-on: https://chromium-review.googlesource.com/424673
Commit-Ready: Aviv Keshet <akeshet@chromium.org>
Tested-by: Aviv Keshet <akeshet@chromium.org>
Reviewed-by: Prathmesh Prabhu <pprabhu@chromium.org>

[modify] https://crrev.com/82c508c92026bad3c2165ac858e2578bc6db479a/scheduler/pidfile_monitor.py

 Issue 671254  has been merged into this issue.

I'm tempted to say we just delete email_manager for stacktraces entirely.

If there is no consumer of the stacktrace alerts, and monarch can detect repeated service crash/restart cycle, delete email_manager for stacktraces entirely is all reasonable.

> I'm tempted to say we just delete email_manager for stacktraces entirely.

Yes, please.

http://shortn/_DcIXLquqwR