Add add_json_log='on_failure' to json module.

Add add_json_log='on_failure' to json module.

BUG=752367
Review-Url: https://codereview.chromium.org/2989423002
Committed: https://github.com/luci/recipes-py/commit/6de52416675972312112322e412f84ae741357f2

[stgao, 2017-08-04 02:49:28]

stgao@chromium.org changed reviewers:
+ iannucci@chromium.org

[iannucci, 2017-08-04 18:21:11]

lgtm

[stgao, 2017-08-04 19:19:57]

Patchset #2 (id:20001) has been deleted

[stgao, 2017-08-04 19:25:24]

Hi Robbie,

Mind another review for the added tests?

Thanks,
Shuotao

[iannucci, 2017-08-04 20:54:18]

lgtm

[stgao, 2017-08-04 20:54:36]

The CQ bit was checked by stgao@chromium.org

[commit-bot: I haz the power, 2017-08-04 20:54:47]

CQ is trying da patch.

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

[stgao, 2017-08-04 20:55:03]

Description was changed from

==========
Add add_json_log='on_failure' to json module.

BUG=752367
==========

to

==========
Add add_json_log='on_failure' to json module.

BUG=752367
==========

[commit-bot: I haz the power, 2017-08-04 21:00:39]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1501880076353460,
"parent_rev": "e183af00ebfb42ec3790682ff35942b86de77914", "commit_rev":
"6de52416675972312112322e412f84ae741357f2"}

[commit-bot: I haz the power, 2017-08-04 21:00:44]

Description was changed from

==========
Add add_json_log='on_failure' to json module.

BUG=752367
==========

to

==========
Add add_json_log='on_failure' to json module.

BUG=752367

Review-Url: https://codereview.chromium.org/2989423002
Committed:
https://github.com/luci/recipes-py/commit/6de52416675972312112322e412f84ae741...
==========

[commit-bot: I haz the power, 2017-08-04 21:00:45]

Committed patchset #2 (id:40001) as
https://github.com/luci/recipes-py/commit/6de52416675972312112322e412f84ae741...