New issue
Advanced search Search tips

Issue 792741 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Dec 2017
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

[autotest] telemetry_Crosperf depends on telemetry_runner.py variables that have been deleted.

Project Member Reported by cmt...@chromium.org, Dec 7 2017

Issue description

The telemetry_runner.py module has changed, and telemetry_Crosperf depends on things that are no longer there.  telemetry_Crosperf needs to be updated to work the same way that telemetry_runner.py works.
 
Actually it will be simpler to just copy the missing values into telemetry_Crosperf.py.
Project Member

Comment 2 by bugdroid1@chromium.org, Dec 8 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/84fd5b3695bf07ef41a5d3ddf8048b593f5a9c1c

commit 84fd5b3695bf07ef41a5d3ddf8048b593f5a9c1c
Author: Caroline Tice <cmtice@google.com>
Date: Fri Dec 08 11:59:21 2017

[autotest] Add missing scp variables/values to telemetry_Crosperf.

We used to get these values from telemetry_runner.py, but they have
been removed from there, so adding them in here.

BUG= chromium:792741 
TEST=Failing crosperf tests started working again.

Change-Id: I19ed853635ed3b195c8b789c1fc2aa4bfd1fac00
Reviewed-on: https://chromium-review.googlesource.com/814655
Commit-Ready: Caroline Tice <cmtice@chromium.org>
Tested-by: Caroline Tice <cmtice@chromium.org>
Reviewed-by: Yunlian Jiang <yunlian@chromium.org>

[modify] https://crrev.com/84fd5b3695bf07ef41a5d3ddf8048b593f5a9c1c/server/site_tests/telemetry_Crosperf/telemetry_Crosperf.py

Status: Fixed (was: Untriaged)

Comment 4 by cmt...@chromium.org, May 15 2018

Status: Verified (was: Fixed)

Sign in to add a comment