Add a config for optional UBSan trybot.

Add a config for optional UBSan trybot.

The plan is to eventually make it a part of CQ, but the first
step is to set it up as optional, measure performance and
other things, and then go through a technical review for getting
the trybot into the CQ.

BUG=612594
Committed: https://chromium.googlesource.com/chromium/tools/build/+/3c0d76d7ccbc0957ab0edf78333b77661b8ffe89

[krasin1, 2016-11-02 23:19:22]

krasin@chromium.org changed reviewers:
+ dpranke@chromium.org

[krasin1, 2016-11-02 23:19:27]

The CQ bit was checked by krasin@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-02 23:19:32]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-02 23:19:33]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-02 23:19:33]

Dry run: No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
Committers are members of the group "project-infra-committers".
Note that this has nothing to do with OWNERS files.

[Dirk Pranke, 2016-11-03 00:15:56]

lgtm

[krasin1, 2016-11-03 00:17:34]

The CQ bit was checked by krasin@chromium.org

[krasin1, 2016-11-03 00:17:44]

On 2016/11/03 00:15:56, Dirk Pranke wrote:
> lgtm

Thank you, Dirk!

[commit-bot: I haz the power, 2016-11-03 00:17:45]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-03 00:27:23]

Description was changed from

==========
Add a config for optional UBSan trybot.

The plan is to eventually make it a part of CQ, but the first
step is to set it up as optional, measure performance and
other things, and then go through a technical review for getting
the trybot into the CQ.

BUG=612594
==========

to

==========
Add a config for optional UBSan trybot.

The plan is to eventually make it a part of CQ, but the first
step is to set it up as optional, measure performance and
other things, and then go through a technical review for getting
the trybot into the CQ.

BUG=612594

Committed:
https://chromium.googlesource.com/chromium/tools/build/+/3c0d76d7ccbc0957ab0e...
==========

[commit-bot: I haz the power, 2016-11-03 00:27:23]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/tools/build/+/3c0d76d7ccbc0957ab0e...

[Michael Achenbach, 2016-11-03 12:52:47]

machenbach@chromium.org changed reviewers:
+ machenbach@chromium.org

[Michael Achenbach, 2016-11-03 12:52:48]

See https://codereview.chromium.org/2477673002/

[krasin1, 2016-11-03 17:25:19]

On 2016/11/03 12:52:48, machenbach (slow) wrote:
> See https://codereview.chromium.org/2477673002/

Thank you for fixing it, and sorry for the breakage. :(

[krasin1, 2016-11-04 03:36:21]

On 2016/11/03 17:25:19, krasin1 wrote:
> On 2016/11/03 12:52:48, machenbach (slow) wrote:
> > See https://codereview.chromium.org/2477673002/
> 
> Thank you for fixing it, and sorry for the breakage. :(

I have been thinking for the whole day trying to reproduce the sequence of
actions which led to the mistake (to avoid it in the future), and it appears to
be the following:

1. I have added the new ubsan trybot to the list of trybots served by CQ trybots
pool near the asan one.
2. I realized it's the wrong place, as I was looking for the slaves serving the
optional bots
3. I have cut the wrong line (asan instead of ubsan) and pasted into the correct
place for the ubsan bot.

That created this diff and asan bot got significantly fewer slaves. And then I
didn't spot it during a self-review.

Apologies again, I will try to better.