Closed Bug 620598 Opened 15 years ago Closed 13 years ago

Intermittent test_seek.html | Test timed out or application timed out after 330 seconds with no output

Categories

(Core :: Audio/Video, defect)

Product:
Core
Component:
Audio/Video
Platform:
x86
Linux
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: philor, Unassigned)

References

Details

(Keywords: intermittent-failure)

Attachments

(1 file, 1 obsolete file)

Disable 3 tests on Linux
2.08 KB, patch
Details | Diff | Splinter Review
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1292900377.1292902803.30670.gz Rev3 Fedora 12 mozilla-central debug test mochitests-1/5 on 2010/12/20 18:59:37 s: talos-r3-fed-039 58217 INFO TEST-PASS | /tests/content/media/test/test_seek.html | bug516323.indexed.ogv seek test 2: Checking currentTime at end: 4.208000183105469 SEEK-TEST: Finished bug516323.indexed.ogv seek test 2 58218 ERROR TEST-UNEXPECTED-FAIL | /tests/content/media/test/test_seek.html | Test timed out.
That'll teach me to be too precise, since that second one (on the same push, and no I didn't cause them) died at 58243 INFO TEST-PASS | /tests/content/media/test/test_seek.html | bug516323.indexed.ogv seek test 6: Seek 2 58244 INFO TEST-PASS | /tests/content/media/test/test_seek.html | bug516323.indexed.ogv seek test 6: Seek 3 Seek test 6 time sum:39.8459997177124 SEEK-TEST: Finished bug516323.indexed.ogv seek test 6 WARNING: NS_ENSURE_TRUE(mSuspendCount > 0) failed: file /builds/slave/cen-lnx64-dbg/build/netwerk/protocol/http/nsHttpChannel.cpp, line 3561 WARNING: NS_ENSURE_TRUE(mSuspendCount > 0) failed: file /builds/slave/cen-lnx64-dbg/build/netwerk/protocol/http/nsHttpChannel.cpp, line 3561 58245 ERROR TEST-UNEXPECTED-FAIL | /tests/content/media/test/test_seek.html | Test timed out.
Summary: Intermittent test_seek.html | Test timed out after SEEK-TEST: Finished bug516323.indexed.ogv seek test 2 → Intermittent test_seek.html | Test timed out
Hitting this locally, it looks like this is yet another case where we're stuck inside snd_pcm_writei. Save me bug 623444, you're my only hope.
We've had a spike in failures in test_seek since about Feb 15: http://brasstacks.mozilla.com/orangefactor/?display=Bug&endday=2011-02-28&startday=2010-08-01&bugid=620598 This is about when bug 629618 landed: changeset: 62622:301a3058c82a user: Chris Pearce <chris@pearce.org.nz> date: Wed Feb 16 11:08:31 2011 +1300 summary: Bug 629618 - Correct Ogg seek bisection's backoff and termination. r=doublec a=roc Maybe that regressed seeking somehow? *sigh*
Depends on: cubeb
Oh, apparently I already decided to pile the newer "application timed out after 330 seconds with no output" onto this, didn't I?
Summary: Intermittent test_seek.html | Test timed out → Intermittent test_seek.html | Test timed out or application timed out after 330 seconds with no output
Whiteboard: [orange] → [orange][[fixed by 623444?]
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1332602828.1332608666.22825.gz WINNT 5.2 comm-central-trunk debug test mochitests-1/5 on 2012/03/24 08:27:08 s: cn-sea-qm-win2k3-01 { 80294 INFO TEST-PASS | /tests/content/media/test/test_seek.html | [started split.webm-88] Length of array should match number of running tests - 2 should equal 2 SEEK-TEST: Started split.webm seek test 11 80295 ERROR TEST-UNEXPECTED-FAIL | /tests/content/media/test/test_seek.html | Test timed out. Not taking screenshot here: see the one that was previously logged 80296 INFO TEST-PASS | /tests/content/media/test/test_seek.html | split.webm seek test 10: Video currentTime should be around 0.9835: 1.7703000000000002 }
Whiteboard: [orange][[fixed by 623444?] → [orange]
Attached patch Disable tests on some platforms (obsolete) — — Splinter Review
* Disables test_seek.html, test_seek_out_of_range.html & test_timeupdate_small_files.html on Linux only (this bug + bug 661076 + bug 687972). * Disables test_streams_element_capture.html on Linux & Windows (bug 752796). * Removes a redundant |MOCHITEST_FILES +=| (leftover from when those tests were ifdefed.
Attachment #648352 - Flags: review?(cpearce)
Comment on attachment 648352 [details] [diff] [review] Disable tests on some platforms Review of attachment 648352 [details] [diff] [review]: ----------------------------------------------------------------- *Sigh*. OK. Can you please file a bug to re-enable these tests, so we don't forget to do it... Thanks Ed. ::: content/media/test/Makefile.in @@ +137,5 @@ > + $(NULL) > +ifneq ($(OS_ARCH), WINNT) > +# Disabled on Linux & Windows for frequent timeouts (bug 752796) > +MOCHITEST_FILES += \ > + test_streams_element_capture.html \ Roc is working on patches to change the underlying code that this test exercises, so this may be fixed, or at least change in the next few days/weeks. So leave this one enabled for now please. Revisit disabling it if we're still failing in a few weeks. We can always disable it on other branches if we need to...
Attachment #648352 - Flags: review?(cpearce) → review+
(In reply to Chris Pearce (:cpearce) from comment #392) > Can you please file a bug to re-enable these tests, so we don't forget to do > it... We normally leave the bug open with [test disabled] in the whiteboard instead. Happy to file a separate bug in this instance though if you'd prefer? :-) Since the prior version of the patch, I have added: $(warning test_seek.html is disabled on Linux for timeouts. Bug 620598) $(warning test_seek_out_of_range.html is disabled on Linux for timeouts. Bug 661076) $(warning test_timeupdate_small_files.html is disabled on Linux for timeouts. Bug 687972) ...which should serve as a reminder too (during the build), if that helps? > Roc is working on patches to change the underlying code that this test > exercises, so this may be fixed, or at least change in the next few > days/weeks. So leave this one enabled for now please. Revisit disabling it > if we're still failing in a few weeks. We can always disable it on other > branches if we need to... Sure - have removed that part.
Attachment #648352 - Attachment is obsolete: true
(In reply to Ed Morley [:edmorley] from comment #395) > Created attachment 648644 [details] [diff] [review] > Disable 3 tests on Linux > > (In reply to Chris Pearce (:cpearce) from comment #392) > > Can you please file a bug to re-enable these tests, so we don't forget to do > > it... > > We normally leave the bug open with [test disabled] in the whiteboard > instead. Happy to file a separate bug in this instance though if you'd > prefer? :-) Separate bugs would good in this case since there's so much noise in this one. > > Since the prior version of the patch, I have added: > $(warning test_seek.html is disabled on Linux for timeouts. Bug 620598) > $(warning test_seek_out_of_range.html is disabled on Linux for timeouts. Bug > 661076) > $(warning test_timeupdate_small_files.html is disabled on Linux for > timeouts. Bug 687972) > > ...which should serve as a reminder too (during the build), if that helps? Yes please. Thanks again!
Disable test on Linux: https://hg.mozilla.org/integration/mozilla-inbound/rev/3f0b1cc00bc2
Whiteboard: [orange] → [orange][test disabled on linux][leave open]
Blocks: 780490
(In reply to Chris Pearce (:cpearce) from comment #396) > Separate bugs would good in this case since there's so much noise in this > one. Filed bug 780490, bug 780491 & bug 780492.
This was fixed on mozilla-central at 2012-08-18 00:26:45 NZST by bug 761917. This test has now been re-enabled: https://hg.mozilla.org/integration/mozilla-inbound/rev/7a9ef73422de Closing this bug, bug 780490 will track merging the changeset to mozilla-central.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Whiteboard: [orange][test disabled on linux][leave open] → [orange]
Whiteboard: [orange]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: