Set proper result (purple/red) for "Uncaught Exception" and "Failure reason" steps

Set proper result (purple/red) for "Uncaught Exception" and "Failure reason" steps

BUG=none

Committed: https://github.com/luci/recipes-py/commit/6bf9ca763de94c641328900358642ec44cf3932e

[Paweł Hajdan Jr., 2016-07-12 09:59:23]

phajdan.jr@chromium.org changed reviewers:
+ estaab@chromium.org, iannucci@chromium.org, martiniss@chromium.org

[Paweł Hajdan Jr., 2016-07-12 09:59:24]

See e.g.
https://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/...

I think if "Uncaught Exception" is not purple it's quite confusing.

[Paweł Hajdan Jr., 2016-07-12 09:59:27]

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

[commit-bot: I haz the power, 2016-07-12 09:59:37]

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-07-12 10:01:27]

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

[commit-bot: I haz the power, 2016-07-12 10:01:28]

Dry run: This issue passed the CQ dry run.

[martiniss, 2016-07-12 19:23:25]

lgtm

[Paweł Hajdan Jr., 2016-07-13 09:01:23]

The CQ bit was checked by phajdan.jr@chromium.org

[commit-bot: I haz the power, 2016-07-13 09:01:27]

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-07-13 09:03:50]

Description was changed from

==========
Set proper result (purple/red) for "Uncaught Exception" and "Failure reason"
steps

BUG=none
==========

to

==========
Set proper result (purple/red) for "Uncaught Exception" and "Failure reason"
steps

BUG=none

Committed:
https://github.com/luci/recipes-py/commit/6bf9ca763de94c641328900358642ec44cf...
==========

[commit-bot: I haz the power, 2016-07-13 09:03:51]

Committed patchset #1 (id:1) as
https://github.com/luci/recipes-py/commit/6bf9ca763de94c641328900358642ec44cf...

[martiniss, 2016-07-20 22:46:32]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2168653005/ by martiniss@chromium.org.

The reason for reverting is: Broke sheriff-o-matic
(https://screenshot.googleplex.com/td2M1aXN8S4); it caused a bunch of duplicate
alerts to show up.

I believe we can cause the step to turn purple without having it show up as a
failing step, which incorrectly triggers a lot of our sheriffing tools..