New issue
Advanced search Search tips

Issue 856171 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

elm: video_VideoSeek.vp8.switchres: Error: Seek test is stuck and timeout

Project Member Reported by tfiga@chromium.org, Jun 25 2018

Issue description

Project Member

Comment 1 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 2 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