autotest backchannel.py fails to identify some Ethernet adapter |
|||||
Issue description
The backchannel.py script in autotest currently uses the 'Port' property reported by ethtool to decide if a network interface is an adapter. However, that doesn't work for some Ethernet adapters as we've observed in the test lab.
One example is the Linksys Linksys USB3GIG USB 3.0 Gigabit Ethernet Adapter (USB 13b1:0041), which has not much information in the ethtool report to determine if it's an Ethernet adapter:
Settings for eth0:
Current message level: 0x00000007 (7)
drv probe link
Link detected: yes
This issue leads to some test failures like:
INIT_ERROR: An ethernet connection is required between the test server and the device under test.
,
Nov 14
This seems to be mainly effecting daisy, peach_pit and peppy for cellular tests in chromeos2 and chromeos6 labs. By board https://stainless.corp.google.com/search?view=matrix&row=test&col=board_model&first_date=2018-10-18&last_date=2018-11-14&reason=An+ethernet+connection+is+required+between&exclude_cts=true&exclude_not_run=false&exclude_non_release=true&exclude_au=true&exclude_acts=true&exclude_retried=true&exclude_non_production=false By host https://stainless.corp.google.com/search?view=matrix&row=test&col=hostname&first_date=2018-10-18&last_date=2018-11-14&reason=An+ethernet+connection+is+required+between&exclude_cts=true&exclude_not_run=false&exclude_non_release=true&exclude_au=true&exclude_acts=true&exclude_retried=true&exclude_non_production=false Is there a need to replace these dongles or will it be fixed in the SW?
,
Nov 15
I think Ben is looking to work around this: https://chromium-review.googlesource.com/c/chromiumos/third_party/autotest/+/1334498
,
Nov 15
,
Nov 16
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/third_party/autotest/+/032d0e18dc1cc7a686ca8ea9f227738b148bb9b2 commit 032d0e18dc1cc7a686ca8ea9f227738b148bb9b2 Author: Ben Chan <benchan@chromium.org> Date: Fri Nov 16 13:02:18 2018 autotest: backchannel: handle Ethernet adapters with unknown port type backchannel.py currently uses the port type reported by ethtool to determine if a network interface is Ethernet. However, ethtool fails to report the port type for some Ethernet adapters. This CL adds a fallback to check against a list of known Ethernet adapters based on their VID/PID. BUG= chromium:905353 TEST=Run pseudomodem tests on a DUT with Linksys USB3GIG Ethernet adapter. Change-Id: I31ca12c91bea27653f8b04d28c4b546fa3ad43ff Reviewed-on: https://chromium-review.googlesource.com/1334498 Commit-Ready: Ben Chan <benchan@chromium.org> Tested-by: Ben Chan <benchan@chromium.org> Reviewed-by: Brian Norris <briannorris@chromium.org> [modify] https://crrev.com/032d0e18dc1cc7a686ca8ea9f227738b148bb9b2/client/cros/backchannel.py
,
Nov 16
Should be fixed? No new results yet, so feel free to reopen if this issue pops back up. |
|||||
►
Sign in to add a comment |
|||||
Comment 1 by benchan@chromium.org
, Nov 14