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

Issue 591646 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Apr 2016
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

[Chameleon_audio] workaround for daisy login freeze problem

Project Member Reported by cychiang@chromium.org, Mar 3 2016

Issue description

Issue 588579 affects all the test on Daisy using telemetry login.
As a workaround, we need to wait 30 seconds after login.
We can add this sleep in multimedia_xmlrpc_server.
Open chrome and login, if board is daisy, sleep 30, then start the server.
Then all the tests using multimedia_xmlrpc_server can avoid the issue.

 

Comment 1 Deleted

Project Member

Comment 2 by bugdroid1@chromium.org, Mar 6 2016

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

commit 8b3c4269e847b8a1bc2f2de9d055b3a807b9bffa
Author: Cheng-Yi Chiang <cychiang@chromium.org>
Date: Thu Mar 03 08:38:34 2016

[autotest] facade_resource: Add a delay after login on daisy

For issue crbug.com/588579, add a 30 seconds delay after login
to avoid test accessing multimedia server during the time window that
Chrome might freeze.
Also, increase the timeout from 60s to 90s in remote_facade_factory.

BUG= chromium:591646 
TEST=run audio_AudioBasicUSBRecord test on daisy.

Change-Id: I1979741e0cdaeeda7528dda8ee8b60f595b4b4db
Reviewed-on: https://chromium-review.googlesource.com/329929
Commit-Ready: Cheng-Yi Chiang <cychiang@chromium.org>
Tested-by: Cheng-Yi Chiang <cychiang@chromium.org>
Reviewed-by: Hsinyu Chao <hychao@chromium.org>
Reviewed-by: Wai-Hong Tam <waihong@chromium.org>
Reviewed-by: Kalin Stoyanov <kalin@chromium.org>

[modify] https://crrev.com/8b3c4269e847b8a1bc2f2de9d055b3a807b9bffa/server/cros/multimedia/remote_facade_factory.py
[modify] https://crrev.com/8b3c4269e847b8a1bc2f2de9d055b3a807b9bffa/client/cros/multimedia/facade_resource.py

Labels: Merge-Request-50
The CL in #2 needs to be merged to R50 to stabilize the test.
Thanks!

Comment 4 by tin...@google.com, Mar 11 2016

Labels: -Merge-Request-50 Merge-Approved-50 Hotlist-Merge-Approved
Your change meets the bar and is auto-approved for M50 (branch: 2661)
Project Member

Comment 5 by bugdroid1@chromium.org, Mar 12 2016

Labels: merge-merged-release-R50-7978.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/0f5dc6eec6ae0edb9b84c6b3baa5cb3e475bf142

commit 0f5dc6eec6ae0edb9b84c6b3baa5cb3e475bf142
Author: Cheng-Yi Chiang <cychiang@chromium.org>
Date: Thu Mar 03 08:38:34 2016

[autotest] facade_resource: Add a delay after login on daisy

For issue crbug.com/588579, add a 30 seconds delay after login
to avoid test accessing multimedia server during the time window that
Chrome might freeze.
Also, increase the timeout from 60s to 90s in remote_facade_factory.

BUG= chromium:591646 
TEST=run audio_AudioBasicUSBRecord test on daisy.

Change-Id: I1979741e0cdaeeda7528dda8ee8b60f595b4b4db
Previous-Reviewed-on: https://chromium-review.googlesource.com/329929
(cherry picked from commit e3cdf4c60020b5b276e836c8766571e80c824648)
Reviewed-on: https://chromium-review.googlesource.com/332347
Reviewed-by: Kalin Stoyanov <kalin@chromium.org>
Reviewed-by: Wai-Hong Tam <waihong@chromium.org>
Commit-Queue: Cheng-Yi Chiang <cychiang@chromium.org>
Tested-by: Cheng-Yi Chiang <cychiang@chromium.org>

[modify] https://crrev.com/0f5dc6eec6ae0edb9b84c6b3baa5cb3e475bf142/server/cros/multimedia/remote_facade_factory.py
[modify] https://crrev.com/0f5dc6eec6ae0edb9b84c6b3baa5cb3e475bf142/client/cros/multimedia/facade_resource.py

Project Member

Comment 6 by sheriffbot@chromium.org, Apr 24 2016

Labels: -Merge-Approved-50
This issue hasn't been updated in the last 6 weeks, so removing its merge approval label. Please re-request a merge if needed.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 7 by ka...@chromium.org, Apr 24 2016

Status: Verified (was: Started)
This is fixed. the workaround works

Sign in to add a comment