status_logger: record step data

status_logger: record step data

BUG=567699
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=297885

[Paweł Hajdan Jr., 2015-12-08 11:18:08]

phajdan.jr@chromium.org changed reviewers:
+ akuegel@chromium.org

[Paweł Hajdan Jr., 2015-12-08 11:18:14]

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

[commit-bot: I haz the power, 2015-12-08 11:18:17]

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

[commit-bot: I haz the power, 2015-12-08 11:21:08]

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

[commit-bot: I haz the power, 2015-12-08 11:21:09]

Dry run: This issue passed the CQ dry run.

[Adrian Kuegel, 2015-12-08 11:36:08]

LGTM apart from that I don't really know about the monitoring side.

https://codereview.chromium.org/1507993003/diff/1/scripts/master/status_logge...
File scripts/master/status_logger.py (right):

https://codereview.chromium.org/1507993003/diff/1/scripts/master/status_logge...
scripts/master/status_logger.py:208: d['steps'] = steps
I am wondering if this will work. I haven't seen another example where a
dictionary was used as a value and how it will be processed when writing a graph
query.

[Paweł Hajdan Jr., 2015-12-08 11:36:58]

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

[commit-bot: I haz the power, 2015-12-08 11:37:06]

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

[commit-bot: I haz the power, 2015-12-08 11:43:34]

Description was changed from

==========
status_logger: record step data

BUG=567699
==========

to

==========
status_logger: record step data

BUG=567699

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=297885
==========

[commit-bot: I haz the power, 2015-12-08 11:43:35]

Committed patchset #1 (id:1) as
http://src.chromium.org/viewvc/chrome?view=rev&revision=297885