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

Issue 620177 link

Starred by 1 user

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

Running a servo test with test_that may create an unnecessary tunnel

Reported by jrbarnette@chromium.org, Jun 15 2016

Issue description

It's possible (and still common in some parts) to run
tests that require servo using a desktop servo V2.
In that case, the ServoHost code will create a wholly
unnecessary ssh tunnel to 'localhost'.  Potentially,
that may not even work: ssh to localhost may fail by
requiring a password; it's even possible for a host to
reject connections from the loopback network.

One amusing/infuriating extreme example:  If you have
a desktop servo V3, and manually create your own ssh
tunnel, it works by creating a tunnel to localhost which
then forward to the tunnel on the beaglebone.  Really.
(And yes, I've seen it _work_, but still...)

We should adjust the ServoHost creation code not to
create a tunnel in this case.  The simplest way to do
it is to say "if the servo host is 'localhost', don't
create a tunnel".

 
Cc: kevcheng@chromium.org
Labels: Hotlist-Fixit
Components: -Infra>Client>ChromeOS
Labels: -Hotlist-Fixit
Owner: kevcheng@chromium.org
Removing infra component.
Components: Test Tests>Telemetry
Components: -Tests>Telemetry
Don't think this has anything to do with Telemetry?
Owner: akes...@chromium.org
Bulk-edit, reassigning to Aviv to assign to someone.
Components: Infra>Client>ChromeOS>Test
Owner: ----
Gathered onto a servo v4 hotlist, removing self as owner.

Sign in to add a comment