Include "recipe_result" (None) on StepFailure

Include "recipe_result" (None) on StepFailure

This prevents the build from being marked as an infra failure,
see https://codereview.chromium.org/2133573004 .

More specifically, we always generate JSON output unless
there was an infra failure.

BUG=chromium:593999
Committed: https://github.com/luci/recipes-py/commit/9d78ba349aba023d424f6bfd483bdf4e5add9de5

[Paweł Hajdan Jr., 2016-07-08 15:10:07]

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

[Paweł Hajdan Jr., 2016-07-08 15:10:11]

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

[commit-bot: I haz the power, 2016-07-08 15:10:20]

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-08 15:14:09]

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

[commit-bot: I haz the power, 2016-07-08 15:14:09]

Dry run: Try jobs failed on following builders:
  Recipes-py Presubmit on master.tryserver.infra (JOB_FAILED,
https://build.chromium.org/p/tryserver.infra/builders/Recipes-py%20Presubmit/...)

[Paweł Hajdan Jr., 2016-07-08 15:16:25]

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

[commit-bot: I haz the power, 2016-07-08 15:16:29]

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-08 15:18:39]

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

[commit-bot: I haz the power, 2016-07-08 15:18:39]

Dry run: This issue passed the CQ dry run.

[martiniss, 2016-07-08 20:29:19]

lgtm

[Paweł Hajdan Jr., 2016-07-11 08:01:57]

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

[commit-bot: I haz the power, 2016-07-11 08:02:08]

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-11 08:04:04]

Description was changed from

==========
Include "recipe_result" (None) on StepFailure

This prevents the build from being marked as an infra failure,
see https://codereview.chromium.org/2133573004 .

More specifically, we always generate JSON output unless
there was an infra failure.

BUG=chromium:593999
==========

to

==========
Include "recipe_result" (None) on StepFailure

This prevents the build from being marked as an infra failure,
see https://codereview.chromium.org/2133573004 .

More specifically, we always generate JSON output unless
there was an infra failure.

BUG=chromium:593999

Committed:
https://github.com/luci/recipes-py/commit/9d78ba349aba023d424f6bfd483bdf4e5ad...
==========

[commit-bot: I haz the power, 2016-07-11 08:04:05]

Committed patchset #2 (id:20001) as
https://github.com/luci/recipes-py/commit/9d78ba349aba023d424f6bfd483bdf4e5ad...