[release] Add json output to release tools.

[release] Add json output to release tools.

This will allow callers (e.g. the infra recipe) to check
which steps have been executed and monitor success/failure.

BUG=chromium:559141
LOG=n
NOTRY=true

Committed: https://crrev.com/cded3ea64f3a24ea0cb716204cfbc2d44ad3fa03
Cr-Commit-Position: refs/heads/master@{#32150}

[Michael Achenbach, 2015-11-20 13:54:08]

Description was changed from

==========
[release] Add json output to release tools.

BUG=
==========

to

==========
[release] Add json output to release tools.

BUG=chromium:559141
LOG=n
==========

[Michael Achenbach, 2015-11-20 13:55:49]

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

[Michael Achenbach, 2015-11-20 13:55:49]

PTAL

https://codereview.chromium.org/1463143004/diff/1/tools/release/auto_roll.py
File tools/release/auto_roll.py (left):

https://codereview.chromium.org/1463143004/diff/1/tools/release/auto_roll.py#...
tools/release/auto_roll.py:16: class CheckActiveRoll(Step):
Removing this here as it is already done on the recipe side.

[Michael Hablich, 2015-11-20 14:44:00]

On 2015/11/20 13:55:49, Michael Achenbach wrote:
> PTAL
> 
> https://codereview.chromium.org/1463143004/diff/1/tools/release/auto_roll.py
> File tools/release/auto_roll.py (left):
> 
>
https://codereview.chromium.org/1463143004/diff/1/tools/release/auto_roll.py#...
> tools/release/auto_roll.py:16: class CheckActiveRoll(Step):
> Removing this here as it is already done on the recipe side.

lgtm

For what do you need this?

[Michael Achenbach, 2015-11-20 14:46:57]

Description was changed from

==========
[release] Add json output to release tools.

BUG=chromium:559141
LOG=n
==========

to

==========
[release] Add json output to release tools.

This will allow callers (e.g. the infra recipe) to check
which steps have been executed.

BUG=chromium:559141
LOG=n
==========

[Michael Achenbach, 2015-11-20 14:47:06]

Description was changed from

==========
[release] Add json output to release tools.

This will allow callers (e.g. the infra recipe) to check
which steps have been executed.

BUG=chromium:559141
LOG=n
==========

to

==========
[release] Add json output to release tools.

This will allow callers (e.g. the infra recipe) to check
which steps have been executed.

BUG=chromium:559141
LOG=n
NOTRY=true
==========

[Michael Achenbach, 2015-11-20 14:47:52]

On 2015/11/20 14:44:00, Hablich wrote:
> On 2015/11/20 13:55:49, Michael Achenbach wrote:
> > PTAL
> > 
> > https://codereview.chromium.org/1463143004/diff/1/tools/release/auto_roll.py
> > File tools/release/auto_roll.py (left):
> > 
> >
>
https://codereview.chromium.org/1463143004/diff/1/tools/release/auto_roll.py#...
> > tools/release/auto_roll.py:16: class CheckActiveRoll(Step):
> > Removing this here as it is already done on the recipe side.
> 
> lgtm
> 
> For what do you need this?

Left comment in CL desc.

[Michael Achenbach, 2015-11-20 14:48:08]

Description was changed from

==========
[release] Add json output to release tools.

This will allow callers (e.g. the infra recipe) to check
which steps have been executed.

BUG=chromium:559141
LOG=n
NOTRY=true
==========

to

==========
[release] Add json output to release tools.

This will allow callers (e.g. the infra recipe) to check
which steps have been executed and monitor success/failure.

BUG=chromium:559141
LOG=n
NOTRY=true
==========

[Michael Achenbach, 2015-11-20 14:48:22]

The CQ bit was checked by machenbach@chromium.org

[commit-bot: I haz the power, 2015-11-20 14:48:31]

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

[commit-bot: I haz the power, 2015-11-20 14:49:34]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-11-20 14:50:19]

Patchset 1 (id:??) landed as
https://crrev.com/cded3ea64f3a24ea0cb716204cfbc2d44ad3fa03
Cr-Commit-Position: refs/heads/master@{#32150}