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

Issue 630034 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Jan 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

moblab: not having an ethernet dongle should still allow for testing android/brillo

Project Member Reported by sbasi@chromium.org, Jul 20 2016

Issue description

Theres a lot of new people using moblab right now and this might trip people up.

This used to work but is no longer true, bug is available for whoever has time and wants to take a look. Involves upstart scripts and bash.
 

Comment 1 by dshi@chromium.org, Jul 26 2016

Labels: Hotlist-Fixit
Mark as a fixit. 

sbasi, if the bug goes idle for too long, could you please find an owner later?

Comment 2 by sbasi@chromium.org, Sep 2 2016

Issue 643272 has been merged into this issue.
Labels: -Pri-2 Pri-1
Status: Available (was: Untriaged)
This will break our team members' local moblab setups becuase we use a single port and use a network switch to connect to DUTs.

As soon as they update their moblab, it'll loose network connection.

This also means that other people can _loose network connection when they update_. That is a rather bad outcome of an update ;)
More to the point, this could adversely impact our ability to
keep moblab instances up and running in the test lab.  We don't
want to lose network access to a moblab instance just because
there's some bug with detecting the USB dongle.

This should move from "fixit" to "deal with it right away".
Owner: ntang@chromium.org
Status: Assigned (was: Available)
Per c#4, this needs triage and action.

Comment 6 by sbasi@chromium.org, Dec 13 2016

Questions to your messages:
1) Prabhu, please note testing of Chrome OS DUTs via moblab only works through this ethernet dongle. This is because it serves DHCP to the DUTs and the devserver location is hardcoded in the shadow_config as the dongle's ethernet address (http://192.168.231.1) so they won't be able to do provision if not on this subnet.

2) Richard, the moblab DUTs are still ssh'able if this dongle does not come up. However testing/verify will fail as expected. Not having the dongle limits the ability for autotest services to launch not the ability for shill to be running.
Re #6: The big problem here is hidden in (2). Not having the ehternet dongle _is_ limiting shill's ability to run (since moblab-network-bridge-init.conf runs 'shill stop' but never restarts it on failure)

That means that moblab looses network connection to the external world.

Comment 8 by ntang@google.com, Dec 14 2016

by the way, all the guado_moblab DUTs in the lab have ethernet dongle attached?



Comment 9 by ntang@google.com, Dec 16 2016

Cc: cros-peng-moblab@google.com
I am going try to implement a strategy in moblab-network-bridge-init.conf to start shill on external interface first, and then block on usb-dongle for bridge initialization. In this way, the external connection will just work.

A better solution is to trigger bridge initialization when a dongle is plugged in. I am not sure if there is an easy way to do that.

Let me know if you have any concern.

Comment 10 by ntang@google.com, Dec 20 2016

Labels: Proj-Moblab

Comment 11 by ntang@google.com, Dec 20 2016

Labels: M-56
Project Member

Comment 12 by bugdroid1@chromium.org, Dec 23 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/board-overlays/+/f7f559287d577ec174d0de4d2155d731a814842e

commit f7f559287d577ec174d0de4d2155d731a814842e
Author: Michael Tang <ntang@google.com>
Date: Tue Dec 20 17:50:11 2016

Waiting for USB ethernet dongle without stopping the shill.

The upstart script moblab-network-bridge-init.conf will block 5 minutes
to wait for the USB ethnet dongle ready. We used to stop the shill
during the wait. This cause the Moblab to have no external interet
connection during the time. Also if the dongle is not configured, shill
is not restarted. We fix this by only stop the shill when USB dongle is
detected.

BUG= chromium:630034 
TEST=manual.

Change-Id: I72a3ffd99a3dadf06c6bb042a5a19b02645ecd5d
Reviewed-on: https://chromium-review.googlesource.com/422534
Commit-Ready: Michael Tang <ntang@chromium.org>
Tested-by: Michael Tang <ntang@chromium.org>
Reviewed-by: Richard Barnette <jrbarnette@google.com>
Reviewed-by: Michael Tang <ntang@chromium.org>

[rename] https://crrev.com/f7f559287d577ec174d0de4d2155d731a814842e/project-moblab/chromeos-base/chromeos-bsp-moblab/chromeos-bsp-moblab-0.0.5-r47.ebuild
[modify] https://crrev.com/f7f559287d577ec174d0de4d2155d731a814842e/project-moblab/chromeos-base/chromeos-bsp-moblab/files/moblab-network-bridge-init.conf

Project Member

Comment 13 by bugdroid1@chromium.org, Jan 7 2017

Labels: merge-merged-release-R56-9000.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/board-overlays/+/6070f99d275072c37dd441388dd0d0e2044a12ac

commit 6070f99d275072c37dd441388dd0d0e2044a12ac
Author: Michael Tang <ntang@google.com>
Date: Tue Dec 20 17:50:11 2016

Waiting for USB ethernet dongle without stopping the shill.

The upstart script moblab-network-bridge-init.conf will block 5 minutes
to wait for the USB ethnet dongle ready. We used to stop the shill
during the wait. This cause the Moblab to have no external interet
connection during the time. Also if the dongle is not configured, shill
is not restarted. We fix this by only stop the shill when USB dongle is
detected.

BUG= chromium:630034 
TEST=manual.

Change-Id: I72a3ffd99a3dadf06c6bb042a5a19b02645ecd5d
Reviewed-on: https://chromium-review.googlesource.com/422534
Commit-Ready: Michael Tang <ntang@chromium.org>
Tested-by: Michael Tang <ntang@chromium.org>
Reviewed-by: Richard Barnette <jrbarnette@google.com>
Reviewed-by: Michael Tang <ntang@chromium.org>
(cherry picked from commit f7f559287d577ec174d0de4d2155d731a814842e)
Reviewed-on: https://chromium-review.googlesource.com/423353
Commit-Queue: Michael Tang <ntang@chromium.org>
Trybot-Ready: Michael Tang <ntang@chromium.org>

[rename] https://crrev.com/6070f99d275072c37dd441388dd0d0e2044a12ac/project-moblab/chromeos-base/chromeos-bsp-moblab/chromeos-bsp-moblab-0.0.5-r47.ebuild
[modify] https://crrev.com/6070f99d275072c37dd441388dd0d0e2044a12ac/project-moblab/chromeos-base/chromeos-bsp-moblab/files/moblab-network-bridge-init.conf

Components: -Infra>Client>ChromeOS

Comment 15 by sbasi@chromium.org, Jan 12 2017

Michael can we close this?

Comment 16 by ntang@google.com, Jan 13 2017

Status: Fixed (was: Assigned)

Sign in to add a comment