New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 620464 link

Starred by 3 users

Issue metadata

Status: Verified
Owner:
Closed: Jul 2016
Cc:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

[autotest] telemetry_Crosperf needs to stop using stdout for gathering results

Project Member Reported by cmt...@chromium.org, Jun 15 2016

Issue description

Telemetry is deprecating stdout and wants to stop putting test results there.  We (the toolchain team) need to fix telemetry_Crosperf to stop using stdout to read/parse the test results.
 

Comment 1 by cmt...@chromium.org, Jun 15 2016

The recommendation is that we use the (local on the DUT) file:

/usr/local/telemetry/src/tools/perf/results.html

Comment 2 by cmt...@chromium.org, Jun 15 2016

Cc: eakuefner@chromium.org

Comment 3 by lloz...@google.com, Jun 15 2016

please check if we can reuse something from the catapult stack (perf dashboard and bisecting)
also, check the following 2 issues where tihs problem was solved :
https://bugs.chromium.org/p/chromium/issues/detail?id=589619
https://chromium-review.googlesource.com/#/c/331795/3

To be clear, the recommendation is to use results-chart.json (using --output-format=chartjson), not results.html.

Comment 5 by cmt...@chromium.org, Jul 18 2016

Owner: cmt...@chromium.org
Ok, thanks for mentioning that; will do.
Project Member

Comment 6 by bugdroid1@chromium.org, Jul 25 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chromeos/toolchain-utils/+/180fb3868ae7a625cd82427ad29bc6224504c757

commit 180fb3868ae7a625cd82427ad29bc6224504c757
Author: Caroline Tice <cmtice@google.com>
Date: Fri Jul 22 23:00:02 2016

Project Member

Comment 7 by bugdroid1@chromium.org, Jul 25 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chromeos/toolchain-utils/+/180fb3868ae7a625cd82427ad29bc6224504c757

commit 180fb3868ae7a625cd82427ad29bc6224504c757
Author: Caroline Tice <cmtice@google.com>
Date: Fri Jul 22 23:00:02 2016

Comment 8 by cmt...@chromium.org, Jul 26 2016

Cc: sullivan@chromium.org achuith@chromium.org
 Issue 628148  has been merged into this issue.

Comment 9 by cmt...@chromium.org, Jul 26 2016

Status: Fixed (was: Assigned)
Labels: VerifyIn-54
Labels: VerifyIn-55

Comment 12 by dchan@google.com, Nov 19 2016

Labels: VerifyIn-56

Comment 13 by dchan@google.com, Jan 21 2017

Labels: VerifyIn-57

Comment 14 by dchan@google.com, Mar 4 2017

Labels: VerifyIn-58
Status: Verified (was: Fixed)

Sign in to add a comment