Updated recipes command to use chartjson

Updated recipes command to use chartjson

This is from https://codereview.chromium.org/595793005/

BUG=420044
Committed: https://crrev.com/5ae725e82a80bb1f8d155b85ecc4361cfbf083c5
Cr-Commit-Position: refs/heads/master@{#298891}

[sullivan, 2014-10-08 16:57:03]

sullivan@chromium.org changed reviewers:
+ tonyg@chromium.org

[sullivan, 2014-10-08 16:57:03]

https://codereview.chromium.org/634933002/ has been live for a full day with no
issues. This moves the --output-format=chartjson part of that CL to the proper
place, and also adds --upload-results so that the perf dashboard can link to any
files telemetry generates (results.html, traces, etc)

[tonyg, 2014-10-09 01:13:46]

lgtm

After this lands, please just double check that we get a data point on the perf
dashboard for both an android and a desktop bot.

[chromium-reviews, 2014-10-09 14:09:58]

I realized that the android recipes bots are
calling build/android/test_runner.py to run telemetry instead of the actual
telemetry commands:

http://build.chromium.org/p/chromium.perf/builders/Android%20Nexus7v2%20Perf/...

So my previous CL to make chartjson work with recipes and this cl only
apply to desktop.

I will check that data points are still submitting on both types of builds
after submitting, though.

Is there work already in progress to move android builds to the new recipes
that desktop builds are using? If so, they will pick up my changes
automatically. If not, I will look into changing android test_runner.py to
also use chartjson.

On Wed, Oct 8, 2014 at 9:13 PM, <tonyg@chromium.org> wrote:

> lgtm
>
> After this lands, please just double check that we get a data point on the
> perf
> dashboard for both an android and a desktop bot.
>
> https://codereview.chromium.org/641693003/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[sullivan, 2014-10-09 14:10:02]

The CQ bit was checked by sullivan@chromium.org

[commit-bot: I haz the power, 2014-10-09 14:12:19]

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

[commit-bot: I haz the power, 2014-10-09 16:16:50]

Committed patchset #1 (id:1) as 9642f5bbd2105b048262c4eeda0104e791724951

[commit-bot: I haz the power, 2014-10-09 16:17:23]

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

[chromium-reviews, 2014-10-09 17:02:25]

Verified data points are being sent to perf dashboard on desktop. Example:
http://build.chromium.org/p/chromium.perf/builders/Linux%20Perf%20%283%29/bui...

On Thu, Oct 9, 2014 at 12:17 PM, <commit-bot@chromium.org> wrote:

> Patchset 1 (id:??) landed as
> https://crrev.com/5ae725e82a80bb1f8d155b85ecc4361cfbf083c5
> Cr-Commit-Position: refs/heads/master@{#298891}
>
> https://codereview.chromium.org/641693003/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[sullivan, 2014-10-10 15:27:52]

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

The reason for reverting is: Broke android recipes; see crbug.com/422174.