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

Issue 781132 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Nov 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug

Blocked on:
issue 783780



Sign in to add a comment

autotest cannot run on remote host with specified port.

Project Member Reported by deanliao@chromium.org, Nov 3 2017

Issue description

When I use test_that to run Telemetry autotest on a remote DUT with specific port (say VM localhost:9222), test_that failed. 
Command:
test_that --args local=True localhost:9222 telemetry_Benchmarks.speedometer 

Error log:
11/03 02:44:40.327 DEBUG|             utils:0212| Running 'ssh -o StrictHostKeyChecking=no -o UserKnownHostsFile=/dev/null -o BatchMode=yes -o ConnectTimeout=30 -o Ser\
verAliveInterval=900 -o ServerAliveCountMax=3 -o ConnectionAttempts=4 -o Protocol=2 -x -a -l root localhost python /usr/local/telemetry/src/tools/perf/run_benchmark --\
verbose --output-format=chartjson --output-dir=/usr/local/telemetry/src --browser=system speedometer'                                                                   
11/03 02:44:40.423 DEBUG|  telemetry_runner:0285| Error occurred executing.
11/03 02:44:40.423 DEBUG|  telemetry_runner:0291| Completed with exit code: 255.
stdout:
stderr:Warning: Permanently added 'localhost' (RSA) to the list of known hosts.
Permission denied (publickey,keyboard-interactive).

It is because that the command running run_benchmark on DUT doesn't specify port. It is a telemetry_runner's bug.
 
Cc: achuith@chromium.org
Project Member

Comment 2 by bugdroid1@chromium.org, Nov 9 2017

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

commit f28e0ade98eef77f1ca8f0238978b5443c390bdb
Author: Dean Liao <deanliao@chromium.org>
Date: Thu Nov 09 08:38:13 2017

[telemetry_runner] Connect SSH host with non-default port.

Adds host_port and rsync_options for ssh_host so that
ssh_host users can display host info more precisely, and
can get rsync options with customized port being set.

BUG= chromium:781132 
TEST=Test manually:
1. Start a VM
2. test_that --args local=True localhost:9222 telemetry_Benchmarks.speedometer
3. test_that --args local=True $DUT_IP telemetry_Benchmarks.speedometer (for regular SSH port)
4. test_that $DUT_IP login_GaiaLogin (for non-telemetry tests)
5. cros tryjob -g 756077 --hwtest guado_moblab  (moblab verification).

Change-Id: I68ca9c429ec76d59ec89c7a76d5029f818101542
Reviewed-on: https://chromium-review.googlesource.com/756077
Commit-Ready: Shuo-Peng Liao <deanliao@chromium.org>
Tested-by: Shuo-Peng Liao <deanliao@chromium.org>
Reviewed-by: Chung-yih Wang <cywang@chromium.org>

[modify] https://crrev.com/f28e0ade98eef77f1ca8f0238978b5443c390bdb/server/hosts/abstract_ssh.py
[modify] https://crrev.com/f28e0ade98eef77f1ca8f0238978b5443c390bdb/server/hosts/cros_host.py
[modify] https://crrev.com/f28e0ade98eef77f1ca8f0238978b5443c390bdb/server/cros/telemetry_runner.py
[modify] https://crrev.com/f28e0ade98eef77f1ca8f0238978b5443c390bdb/server/hosts/ssh_host.py
[modify] https://crrev.com/f28e0ade98eef77f1ca8f0238978b5443c390bdb/server/hosts/servo_host.py

Status: Fixed (was: Started)
Blockedon: 783780
Project Member

Comment 6 by bugdroid1@chromium.org, Nov 22 2017

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

commit e4773c7a7e839e47dcb4214a263f972235f27c94
Author: Dean Liao <deanliao@chromium.org>
Date: Wed Nov 22 05:36:16 2017

Raise TestError when it fails to retrieve test results.

BUG= chromium:781132 
TEST=None

Change-Id: I47f77e15b0c0a0f409b20a85dbc9cea0de260b56
Reviewed-on: https://chromium-review.googlesource.com/760183
Commit-Ready: Shuo-Peng Liao <deanliao@chromium.org>
Tested-by: Shuo-Peng Liao <deanliao@chromium.org>
Reviewed-by: Chung-yih Wang <cywang@chromium.org>

[modify] https://crrev.com/e4773c7a7e839e47dcb4214a263f972235f27c94/server/cros/telemetry_runner.py

Comment 7 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Comment 8 by dchan@chromium.org, Jan 23 2018

Status: Fixed (was: Archived)

Sign in to add a comment