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

Issue 799792 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

script enterprise_CFM_USBPeripheralHotplugDetect reports mismatch speaker although c930 shows up in both cfm and cros

Project Member Reported by mzhuo@chromium.org, Jan 7 2018

Issue description

Ver:R65-10280.0.0
Panther running meeting mode

What steps will reproduce the problem?
(1)run script enterprise_CFM_USBPeripheralHotplugDetect.c930 on chromeos1-row3-rack5-host2
(2) script report failure although no failure found. 

Test log
--------
https://stainless.corp.google.com/search?exclude_retried=true&first_date=20180102&master_builder_name=&builder_name_number=&shard=&exclude_acts=true&builder_name=&master_builder_name_number=&owner=&retry=&exclude_cts=true&exclude_non_production=true&hostname=&board=&test=%5Eenterprise_CFM_USBPeripheralHotplugDetect%5C.c930%24&exclude_not_run=false&build=%5ER65%5C-10280%5C.0%5C.0%24&status=FAIL&status=ERROR&status=ABORT&reason=&waterfall=&suite=%5Ehotrod%24&last_date=20180105&exclude_non_release=true&exclude_au=true&view=list

01/05 10:10:10.263 DEBUG|              test:0420| after_iteration_hooks completed
01/05 10:10:10.263 WARNI|              test:0637| The test failed with the following exception
Traceback (most recent call last):
  File "/usr/local/autotest/client/common_lib/test.py", line 631, in _exec
    _call_test_function(self.execute, *p_args, **p_dargs)
  File "/usr/local/autotest/client/common_lib/test.py", line 831, in _call_test_function
    return func(*args, **dargs)
  File "/usr/local/autotest/client/common_lib/test.py", line 495, in execute
    dargs)
  File "/usr/local/autotest/client/common_lib/test.py", line 362, in _call_run_once_with_retry
    postprocess_profiled_run, args, dargs)
  File "/usr/local/autotest/client/common_lib/test.py", line 400, in _call_run_once
    self.run_once(*args, **dargs)
  File "/usr/local/autotest/server/site_tests/enterprise_CFM_USBPeripheralHotplugDetect/enterprise_CFM_USBPeripheralHotplugDetect.py", line 257, in run_once
    len(detect_crash.get_crash_files()), crash_identified_at))
TestFail: Peripherals do not match.
Diff: {'Microphone': set(['Logitech Webcam C930e: USB Audio:0,0'])} 
 Cros: <enterprise_CFM_USBPeripheralHotplugDetect._Peripherals object at 0x7f63e8dc8210> 
 CfM: <enterprise_CFM_USBPeripheralHotplugDetect._Peripherals object at 0x7f63e8df7c10> 
.No of Crashes: 4. Crashes: New Warning or Crash Detected before plugging in usb peripherals., New Warning or Crash detected after device enrolled into CFM.
01/05 10:10:10.264 DEBUG|              test:0642| Running cleanup for test.

01/05 10:09:11.307 DEBUG|             utils:0280| [client] 127.0.0.1 - - [05/Jan/2018 10:09:10] "POST /RPC2 HTTP/1.1" 200 -
01/05 10:09:11.307 DEBUG|remote_facade_fact:0141| RPC 'cfm_main_screen.get_camera_devices'((), {}) returns ['Logitech Webcam C930e (046d:0843)'].
01/05 10:09:11.308 INFO |enterprise_CFM_USB:0169| Reported CfM peripherals: <enterprise_CFM_USBPeripheralHotplugDetect._Peripherals object at 0x7f63e8df7c10>
01/05 10:09:11.308 DEBUG|enterprise_CFM_USB:0228| Peripherals detected by hotrod: <enterprise_CFM_USBPeripheralHotplugDetect._Peripherals object at 0x7f63e8df7c10>
01/05 10:09:11.328 DEBUG|          ssh_host:0301| Running (ssh) 'ls /var/spool' from 'run_once|is_new_crash_present|get_new_crash_files|find_crash_files|run|run_very_slowly'
01/05 10:09:11.711 DEBUG|             utils:0280| [stdout] crash
01/05 10:09:11.712 DEBUG|             utils:0280| [stdout] cron-lite

01/05 10:09:11.304 DEBUG|             utils:0280| [client] 10:09:10 DEBUG| got [{
01/05 10:09:11.304 DEBUG|             utils:0280| [client]   "id": 23, 
01/05 10:09:11.304 DEBUG|             utils:0280| [client]   "result": {
01/05 10:09:11.305 DEBUG|             utils:0280| [client]     "result": {
01/05 10:09:11.305 DEBUG|             utils:0280| [client]       "type": "object", 
01/05 10:09:11.306 DEBUG|             utils:0280| [client]       "value": [
01/05 10:09:11.306 DEBUG|             utils:0280| [client]         "Logitech Webcam C930e (046d:0843)"
01/05 10:09:11.306 DEBUG|             utils:0280| [client]       ]
01/05 10:09:11.306 DEBUG|             utils:0280| [client]     }
01/05 10:09:11.306 DEBUG|             utils:0280| [client]   }

1/05 10:07:50.899 DEBUG|             utils:0280| [stdout] T:  Bus=01 Lev=01 Prnt=01 Port=05 Cnt=04 Dev#=  5 Spd=480 MxCh= 0
01/05 10:07:50.916 DEBUG|             utils:0280| [stdout] D:  Ver= 2.00 Cls=ef(misc ) Sub=02 Prot=01 MxPS=64 #Cfgs=  1
01/05 10:07:50.928 DEBUG|             utils:0280| [stdout] P:  Vendor=046d ProdID=0843 Rev=00.13
01/05 10:07:50.931 DEBUG|             utils:0280| [stdout] S:  Product=Logitech Webcam C930e
01/05 10:07:50.934 DEBUG|             utils:0280| [stdout] S:  SerialNumber=F5E9F5FE
01/05 10:07:50.944 DEBUG|             utils:0280| [stdout] C:  #Ifs= 4 Cfg#= 1 Atr=80 MxPwr=500mA
01/05 10:07:50.967 DEBUG|             utils:0280| [stdout] I:  If#= 0 Alt= 0 #EPs= 1 Cls=0e(video) Sub=01 Prot=00 Driver=uvcvideo
01/05 10:07:50.990 DEBUG|             utils:0280| [stdout] I:  If#= 1 Alt= 0 #EPs= 0 Cls=0e(video) Sub=02 Prot=00 Driver=uvcvideo
01/05 10:07:51.013 DEBUG|             utils:0280| [stdout] I:  If#= 2 Alt= 0 #EPs= 0 Cls=01(audio) Sub=01 Prot=00 Driver=snd-usb-audio
01/05 10:07:51.036 DEBUG|             utils:0280| [stdout] I:  If#= 3 Alt= 0 #EPs= 0 Cls=01(audio) Sub=02 Prot=00 Driver=snd-usb-audio
01/05 10:07:51.057 DEBUG|             utils:0280| [stdout] 
 

Comment 1 by tovep@chromium.org, Jan 12 2018

Owner: dtosic@chromium.org

Comment 2 by dtosic@google.com, Jan 12 2018

Status: Started (was: Untriaged)

Comment 4 by dtosic@chromium.org, Jan 12 2018

Digged a bit around, it's a test error :/

The hangouts name contains the suffix "USB Audio:x,y" and (x,y) depend on which port the camera is plugged in to. We do an explicit match (instead of partial or regex) so the test fails since the camera is always expected on (0,0).

I will prepare a fix on Monday.
Project Member

Comment 5 by bugdroid1@chromium.org, Jan 12 2018

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

commit 09b84daec5753d4e2acaa9e08eec3404ce4a5f60
Author: Denis Tosic <dtosic@chromium.org>
Date: Fri Jan 12 22:31:14 2018

[autotest][cfm] Improve logging for USBHotplugDetect

Properly overrides the __str__() for the _Peripherals() class
and changes some log levels to INFO for easier inspection.

BUG= chromium:799792 
TEST=locally

Change-Id: I444ec5b491b9b7491ab69328f38ba51737ef3ad9
Reviewed-on: https://chromium-review.googlesource.com/864154
Commit-Ready: Denis Tosic <dtosic@chromium.org>
Tested-by: Denis Tosic <dtosic@chromium.org>
Reviewed-by: Denis Tosic <dtosic@chromium.org>

[modify] https://crrev.com/09b84daec5753d4e2acaa9e08eec3404ce4a5f60/server/site_tests/enterprise_CFM_USBPeripheralHotplugDetect/enterprise_CFM_USBPeripheralHotplugDetect.py

Comment 7 by dtosic@google.com, Jan 16 2018

Status: Fixed (was: Started)

Sign in to add a comment