New issue
Advanced search Search tips

Issue 846239 link

Starred by 3 users

Issue metadata

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



Sign in to add a comment

Seek test is stuck and timeout on falco

Project Member Reported by inker@chromium.org, May 24 2018

Issue description

Comment 1 by inker@chromium.org, May 24 2018

Description: Show this description
Labels: videoshortlist
Status: Assigned (was: Untriaged)
Seems to be a one-time occurence. Will try to repro and see if I can find something in the logs.

Comment 5 by tfiga@chromium.org, Jun 26 2018

Labels: cros-media-sheriff
Any update on this? (CrOS Media Sheriff ping)
Triage nag: This Chrome OS bug has an owner but no component. Please add a component so that this can be tracked by the relevant team.
Components: OS>Kernel>Video
Project Member

Comment 9 by bugdroid1@chromium.org, Oct 2

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

commit 390008e8b3c80c03f92afe0bd7d9b76e252b8823
Author: Alexandre Courbot <acourbot@chromium.org>
Date: Tue Oct 02 16:17:30 2018

video_VideoSeek: improve timeout detection

The timeout detection simply consisted in triggering a timeout error if
the test took more than 180 seconds to complete. This can become a short
time to do 100 seeks, especially if the network conditions are not
optimal - as a result, many lab boards were reporting a timeout after
~70 iterations.

Fix this by doing the timeout check not on the whole test, but for each
individual seek: we consider a timeout has occured if we stay more than
60 seconds on the same seek.

BUG= chromium:822582 
BUG=chromium:835679
BUG= chromium:846239 
BUG= chromium:856171 
BUG=chromium:884583
BUG=b:62360942
TEST=Checked that the test completed succesfully on Eve.

Change-Id: I1d53a7a314b4a20e9f2be506a1bdf8e42de0c266
Reviewed-on: https://chromium-review.googlesource.com/1256582
Commit-Ready: Alexandre Courbot <acourbot@chromium.org>
Tested-by: Alexandre Courbot <acourbot@chromium.org>
Reviewed-by: Alexandre Courbot <acourbot@chromium.org>

[modify] https://crrev.com/390008e8b3c80c03f92afe0bd7d9b76e252b8823/client/site_tests/video_VideoSeek/video_VideoSeek.py

Project Member

Comment 10 by bugdroid1@chromium.org, Oct 5

Labels: merge-merged-release-R70-11021.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/05874e747ce606225a1884763c23b119a2fa0dec

commit 05874e747ce606225a1884763c23b119a2fa0dec
Author: Alexandre Courbot <acourbot@chromium.org>
Date: Fri Oct 05 04:39:23 2018

video_VideoSeek: improve timeout detection

The timeout detection simply consisted in triggering a timeout error if
the test took more than 180 seconds to complete. This can become a short
time to do 100 seeks, especially if the network conditions are not
optimal - as a result, many lab boards were reporting a timeout after
~70 iterations.

Fix this by doing the timeout check not on the whole test, but for each
individual seek: we consider a timeout has occured if we stay more than
60 seconds on the same seek.

BUG= chromium:822582 
BUG=chromium:835679
BUG= chromium:846239 
BUG= chromium:856171 
BUG=chromium:884583
BUG=b:62360942
TEST=Checked that the test completed succesfully on Eve.

Change-Id: I1d53a7a314b4a20e9f2be506a1bdf8e42de0c266
Reviewed-on: https://chromium-review.googlesource.com/1256582
Commit-Ready: Alexandre Courbot <acourbot@chromium.org>
Tested-by: Alexandre Courbot <acourbot@chromium.org>
Reviewed-by: Alexandre Courbot <acourbot@chromium.org>
(cherry picked from commit 390008e8b3c80c03f92afe0bd7d9b76e252b8823)
Reviewed-on: https://chromium-review.googlesource.com/c/1264115
Commit-Queue: Alexandre Courbot <acourbot@chromium.org>

[modify] https://crrev.com/05874e747ce606225a1884763c23b119a2fa0dec/client/site_tests/video_VideoSeek/video_VideoSeek.py

Status: Fixed (was: Assigned)
This should be fixed now.

Sign in to add a comment