New issue
Advanced search Search tips

Issue 809414 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Jun 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

chameleon: audio_AudioBasicUSBPlayback failure on squawks

Project Member Reported by cychiang@chromium.org, Feb 6 2018

Issue description

https://cros-goldeneye.corp.google.com/chromeos/healthmonitoring/testDetails?testName=audio_AudioBasicUSBPlayback&suite=&daysBack=30&board=&architecture=&boardFamily=rambi&buildConfig=&reason=&version=&milestone=&dut=

Example failure causes:

1. Failure: Wrong input node(s) selected: ['INTERNAL_MIC'] expected: ['USB']	

2. Failure: Wrong input node(s) selected: ['MIC'] expected: ['USB']


3. Failure: Wrong output node(s) selected ['HEADPHONE'] expected: ['USB']


1: it looks like USB detection issue.

2 and 3: 3.5mm jack has greater precedence than USB. We should modify the test to let Chrome select USB.
 
chromeos15-audiobox1-host2: passed 5/5


I was wrong on the precedence.
If USB headphone is plugged, it should be selected.

I can pass audio_AudioBasicUSBPlayback test on chromeos2-row10-rack8-host7.cros 5/5.

Keep monitoring this test...
Labels: -Pri-2 Pri-3
Project Member

Comment 4 by bugdroid1@chromium.org, May 4 2018

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

commit 3fb07c376b3bc0286c092d5a43149c974be24826
Author: paulhsia <paulhsia@chromium.org>
Date: Fri May 04 07:13:51 2018

[autotest] Fix USB detection failures in tests

Try to fix "Wrong input node" failure by increasing delay time after
binding USB from chameleon.

BUG= chromium:809414 
TEST=test_that -b squawks <ip> audio_AudioBasicUSBPlayback
     --args='chameleon_host=$chameleon_ip'

Change-Id: I041935dd5ef857501e43f1fb520a29bd4e330183
Reviewed-on: https://chromium-review.googlesource.com/1041665
Commit-Ready: Chih-Yang Hsia <paulhsia@chromium.org>
Tested-by: Chih-Yang Hsia <paulhsia@chromium.org>
Reviewed-by: Cheng-Yi Chiang <cychiang@chromium.org>

[modify] https://crrev.com/3fb07c376b3bc0286c092d5a43149c974be24826/client/cros/chameleon/audio_widget_link.py

Project Member

Comment 5 by bugdroid1@chromium.org, May 8 2018

Labels: merge-merged-release-R67-10575.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/29abd643e28c26342c8eb45e47afcf93e0e3f815

commit 29abd643e28c26342c8eb45e47afcf93e0e3f815
Author: paulhsia <paulhsia@chromium.org>
Date: Tue May 08 01:55:55 2018

[autotest] Fix USB detection failures in tests

Try to fix "Wrong input node" failure by increasing delay time after
binding USB from chameleon.

BUG= chromium:809414 
TEST=test_that -b squawks <ip> audio_AudioBasicUSBPlayback
     --args='chameleon_host=$chameleon_ip'

Change-Id: I041935dd5ef857501e43f1fb520a29bd4e330183
Previous-Reviewed-on: https://chromium-review.googlesource.com/1041665
(cherry picked from commit 26572ad7598ef76f190ed30643131610b923f5b0)
Reviewed-on: https://chromium-review.googlesource.com/1049225
Reviewed-by: Cheng-Yi Chiang <cychiang@chromium.org>
Commit-Queue: Chih-Yang Hsia <paulhsia@chromium.org>
Tested-by: Chih-Yang Hsia <paulhsia@chromium.org>

[modify] https://crrev.com/29abd643e28c26342c8eb45e47afcf93e0e3f815/client/cros/chameleon/audio_widget_link.py

Project Member

Comment 6 by bugdroid1@chromium.org, Jun 6 2018

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

commit 3dda649ec89264738665f4b8a50011d2e158556a
Author: paulhsia <paulhsia@chromium.org>
Date: Wed Jun 06 12:00:52 2018

[autotest] Select USB device explicitly in audio_AudioBasicUSB tests

Sometimes the state for audio device selection will cause the DUT ignore
a plugged USB device.
In audio_AudioBasicUSBPlayback, audio_AudioBasicUSBRecord and
audio_AudioBasicUSBPlaybackRecord tests, we should select USB device in
tests explicitly since the tests are only for testing the functionality
of playback and record functions.
Testing the selection mechanism for nodes should not be included in these
tests.

BUG= chromium:809414 
BUG= chromium:849594 
TEST=test_that --board=BOARD --args="chameleon_host=CHAMELEON_IP" DUT_IP
audio_AudioBasicPlayback
TEST=test_that --board=BOARD --args="chameleon_host=CHAMELEON_IP" DUT_IP
audio_AudioBasicRecord
TEST=test_that --board=BOARD --args="chameleon_host=CHAMELEON_IP" DUT_IP
audio_AudioBasicPlaybackRecord

Change-Id: I3db2e7d30783a9ea897416f7830fb65f20683b94
Reviewed-on: https://chromium-review.googlesource.com/1086901
Commit-Ready: Chih-Yang Hsia <paulhsia@chromium.org>
Tested-by: Chih-Yang Hsia <paulhsia@chromium.org>
Reviewed-by: Cheng-Yi Chiang <cychiang@chromium.org>

[modify] https://crrev.com/3dda649ec89264738665f4b8a50011d2e158556a/server/site_tests/audio_AudioBasicUSBPlaybackRecord/audio_AudioBasicUSBPlaybackRecord.py
[modify] https://crrev.com/3dda649ec89264738665f4b8a50011d2e158556a/server/site_tests/audio_AudioBasicUSBRecord/audio_AudioBasicUSBRecord.py
[modify] https://crrev.com/3dda649ec89264738665f4b8a50011d2e158556a/server/site_tests/audio_AudioBasicUSBPlayback/audio_AudioBasicUSBPlayback.py

Project Member

Comment 7 by bugdroid1@chromium.org, Jun 6 2018

Labels: merge-merged-release-R68-10718.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/1b86343227020d21c9b54e607155c036a474626a

commit 1b86343227020d21c9b54e607155c036a474626a
Author: paulhsia <paulhsia@chromium.org>
Date: Wed Jun 06 19:20:15 2018

[autotest] Select USB device explicitly in audio_AudioBasicUSB tests

Sometimes the state for audio device selection will cause the DUT ignore
a plugged USB device.
In audio_AudioBasicUSBPlayback, audio_AudioBasicUSBRecord and
audio_AudioBasicUSBPlaybackRecord tests, we should select USB device in
tests explicitly since the tests are only for testing the functionality
of playback and record functions.
Testing the selection mechanism for nodes should not be included in these
tests.

BUG= chromium:809414 
BUG= chromium:849594 
TEST=test_that --board=BOARD --args="chameleon_host=CHAMELEON_IP" DUT_IP
audio_AudioBasicPlayback
TEST=test_that --board=BOARD --args="chameleon_host=CHAMELEON_IP" DUT_IP
audio_AudioBasicRecord
TEST=test_that --board=BOARD --args="chameleon_host=CHAMELEON_IP" DUT_IP
audio_AudioBasicPlaybackRecord

Change-Id: I3db2e7d30783a9ea897416f7830fb65f20683b94
Previous-Reviewed-on: https://chromium-review.googlesource.com/1086901
(cherry picked from commit f7279c2309022ac2abd2bd1e2738fc77fa07aead)
Reviewed-on: https://chromium-review.googlesource.com/1089590
Reviewed-by: Chih-Yang Hsia <paulhsia@chromium.org>
Commit-Queue: Chih-Yang Hsia <paulhsia@chromium.org>
Tested-by: Chih-Yang Hsia <paulhsia@chromium.org>

[modify] https://crrev.com/1b86343227020d21c9b54e607155c036a474626a/server/site_tests/audio_AudioBasicUSBPlaybackRecord/audio_AudioBasicUSBPlaybackRecord.py
[modify] https://crrev.com/1b86343227020d21c9b54e607155c036a474626a/server/site_tests/audio_AudioBasicUSBRecord/audio_AudioBasicUSBRecord.py
[modify] https://crrev.com/1b86343227020d21c9b54e607155c036a474626a/server/site_tests/audio_AudioBasicUSBPlayback/audio_AudioBasicUSBPlayback.py

Owner: paulhsia@chromium.org
Status: Verified (was: Started)
After multiple changes for USB tests, audio_AudioBasicUSBPlayback test is quite stable on squawks now!
Please check from the following link:
https://stainless.corp.google.com/search?exclude_cts=true&exclude_non_release=true&board=%5Esquawks%24&test=%5Eaudio%5C_AudioBasicUSBPlayback%24&view=matrix&col=build&row=board_model&first_date=2018-06-03&last_date=2018-06-09

Sign in to add a comment