Added event_mon.set_default_event

Added event_mon.set_default_event

This allows to change the template event used to compute all other
events.

BUG=513882
Committed: https://chromium.googlesource.com/infra/infra/+/eb48e8caff172b2621c66d0f7f87f4bfd8be4129

[pgervais, 2015-10-28 22:31:34]

pgervais@chromium.org changed reviewers:
+ dsansome@chromium.org, yyanagisawa@chromium.org

[pgervais, 2015-10-28 22:31:34]

Simple modification, preparing for more.

[dsansome, 2015-10-29 00:51:55]

lgtm

https://chromiumcodereview.appspot.com/1407143009/diff/1/infra_libs/event_mon...
File infra_libs/event_mon/config.py (right):

https://chromiumcodereview.appspot.com/1407143009/diff/1/infra_libs/event_mon...
infra_libs/event_mon/config.py:175: logging.error('A ChromeInfraEvent is
required as the default event. Got %s',
Feels like you ought to raise a TypeError or ValueError since this is a
programmer error, not a user error.

[Yoshisato Yanagisawa, 2015-10-29 02:50:38]

Description was changed from

==========
Added event_mon.set_default_event

This allows to change the template event used to compute all other
events.

BUG=513882
==========

to

==========
Added event_mon.set_default_event

This allows to change the template event used to compute all other
events.

BUG=513882
==========

[Yoshisato Yanagisawa, 2015-10-29 02:50:38]

yyanagisawa@chromium.org changed reviewers:
+ sergeyberezin@chromium.org
- yyanagisawa@chromium.org

[Yoshisato Yanagisawa, 2015-10-29 02:51:14]

I might not be the proper person to review this code.
-myself
+Sergey.

[pgervais, 2015-10-29 17:11:24]

https://chromiumcodereview.appspot.com/1407143009/diff/1/infra_libs/event_mon...
File infra_libs/event_mon/config.py (right):

https://chromiumcodereview.appspot.com/1407143009/diff/1/infra_libs/event_mon...
infra_libs/event_mon/config.py:175: logging.error('A ChromeInfraEvent is
required as the default event. Got %s',
On 2015/10/29 00:51:54, dsansome wrote:
> Feels like you ought to raise a TypeError or ValueError since this is a
> programmer error, not a user error.

Done.

[pgervais, 2015-10-29 17:11:27]

The CQ bit was checked by pgervais@chromium.org

[pgervais, 2015-10-29 17:11:28]

The patchset sent to the CQ was uploaded after l-g-t-m from
dsansome@chromium.org
Link to the patchset: https://chromiumcodereview.appspot.com/1407143009/#ps20001
(title: "set_default_event raises exception")

[commit-bot: I haz the power, 2015-10-29 17:11:38]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1407143009/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1407143009/20001

[commit-bot: I haz the power, 2015-10-29 17:13:28]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/infra/infra/+/eb48e8caff172b2621c66d0f7f87f...