Intermittent test_response.html | Test timed out.

RESOLVED FIXED in Firefox 49

Status

()

RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: KWierso, Assigned: bkelly)

Tracking

({intermittent-failure})

49 Branch
mozilla49
intermittent-failure
Points:
---

Firefox Tracking Flags

(firefox49 fixed)

Details

Attachments

(2 attachments, 1 obsolete attachment)

(Assignee)

Comment 1

3 years ago
Log is gone already?
(Reporter)

Comment 2

3 years ago
Something with Treeherder's log parser is acting up, here's the full log: http://archive.mozilla.org/pub/mobile/tinderbox-builds/fx-team-android-api-15-debug/1464182559/fx-team_ubuntu64_vm_armv7_large-debug_test-mochitest-16-bm67-tests1-linux64-build11.txt.gz

Just ctrl-f for `test_response.html | Test timed out.`
(Assignee)

Comment 3

3 years ago
I recently expanded these tests to run all of the test cases in nested workers.  It looks like that pushed this test case right to the edge of the timeout on android debug.  Here is a passing case:

16:55:04     INFO -  69 INFO TEST-START | dom/tests/mochitest/fetch/test_response.html
16:55:28     INFO -  70 INFO TEST-OK | dom/tests/mochitest/fetch/test_response.html | took 29352ms
Assignee: nobody → bkelly
Blocks: 1273070
Status: NEW → ASSIGNED
(Assignee)

Comment 4

3 years ago
Created attachment 8756672 [details] [diff] [review]
Lengthen timeout for all tests using fetch_test_framework.js. r=asuth

https://treeherder.mozilla.org/#/jobs?repo=try&revision=9be09eaf9ce0
(Assignee)

Comment 5

3 years ago
Created attachment 8756673 [details] [diff] [review]
Lengthen timeout for all tests using fetch_test_framework.js. r=asuth

Improved the comment.
Attachment #8756672 - Attachment is obsolete: true
Attachment #8756673 - Flags: review+
(Assignee)

Comment 6

3 years ago
Thanks for the review!  I was waiting to see if it worked in automation, though.  Will do some retrievers later this morning.
(Assignee)

Updated

3 years ago
See Also: → bug 1275817
(Assignee)

Comment 7

3 years ago
There may be something else going on here.  I got this failure in try:

https://treeherder.mozilla.org/logviewer.html#?job_id=21510818&repo=try#L893

You have to look at the full log, which shows this:

08:39:21     INFO -  05-26 08:31:27.950 I/Gecko   ( 2215): [2215] WARNING: Failed to add feature: file /builds/slave/try-and-api-15-d-0000000000000/build/src/dom/fetch/Fetch.cpp, line 942
08:39:21     INFO -  05-26 08:31:27.950 I/Gecko   ( 2215): [2215] WARNING: 'aRv.Failed()', file /builds/slave/try-and-api-15-d-0000000000000/build/src/dom/fetch/Fetch.cpp, line 1211
08:39:21     INFO -  05-26 08:31:27.960 I/Gecko   ( 2215): [2215] WARNING: '!worker->AddFeature(mFeature)', file /builds/slave/try-and-api-15-d-0000000000000/build/src/dom/promise/Promise.cpp, line 2735
08:39:21     INFO -  05-26 08:31:27.970 E/GeckoConsole( 2215): [JavaScript Error: "NS_ERROR_FAILURE" {file: "http://mochi.test:8888/tests/dom/tests/mochitest/fetch/test_request.js" line: 401}]
08:39:21     INFO -  05-26 08:31:28.050 I/Gecko   ( 2215): [2215] WARNING: Failed to add feature: file /builds/slave/try-and-api-15-d-0000000000000/build/src/dom/fetch/Fetch.cpp, line 942
08:39:21     INFO -  05-26 08:31:28.060 I/Gecko   ( 2215): [2215] WARNING: 'aRv.Failed()', file /builds/slave/try-and-api-15-d-0000000000000/build/src/dom/fetch/Fetch.cpp, line 1211
08:39:21     INFO -  05-26 08:31:28.060 I/Gecko   ( 2215): [2215] WARNING: '!worker->AddFeature(mFeature)', file /builds/slave/try-and-api-15-d-0000000000000/build/src/dom/promise/Promise.cpp, line 2735
08:39:21     INFO -  05-26 08:31:28.110 E/GeckoConsole( 2215): [JavaScript Error: "NS_ERROR_FAILURE" {file: "http://mochi.test:8888/tests/dom/tests/mochitest/fetch/test_request.js" line: 415}]
08:39:21     INFO -  05-26 08:31:28.171 I/Gecko   ( 2215): [2215] WARNING: Failed to add feature: file /builds/slave/try-and-api-15-d-0000000000000/build/src/dom/fetch/Fetch.cpp, line 942
08:39:21     INFO -  05-26 08:31:28.190 I/Gecko   ( 2215): [2215] WARNING: 'aRv.Failed()', file /builds/slave/try-and-api-15-d-0000000000000/build/src/dom/fetch/Fetch.cpp, line 1211
08:39:21     INFO -  05-26 08:31:28.190 I/Gecko   ( 2215): [2215] WARNING: '!worker->AddFeature(mFeature)', file /builds/slave/try-and-api-15-d-0000000000000/build/src/dom/promise/Promise.cpp, line 2735
08:39:21     INFO -  05-26 08:31:28.200 E/GeckoConsole( 2215): [JavaScript Error: "NS_ERROR_FAILURE" {file: "http://mochi.test:8888/tests/dom/tests/mochitest/fetch/test_request.js" line: 428}]
08:39:21     INFO -  05-26 08:31:28.280 I/Gecko   ( 2215): [2215] WARNING: Failed to add feature: file /builds/slave/try-and-api-15-d-0000000000000/build/src/dom/fetch/Fetch.cpp, line 942
08:39:21     INFO -  05-26 08:31:28.290 I/Gecko   ( 2215): [2215] WARNING: 'aRv.Failed()', file /builds/slave/try-and-api-15-d-0000000000000/build/src/dom/fetch/Fetch.cpp, line 1211
08:39:21     INFO -  05-26 08:31:28.290 I/Gecko   ( 2215): [2215] WARNING: '!worker->AddFeature(mFeature)', file /builds/slave/try-and-api-15-d-0000000000000/build/src/dom/promise/Promise.cpp, line 2735
08:39:21     INFO -  05-26 08:31:28.300 E/GeckoConsole( 2215): [JavaScript Error: "NS_ERROR_FAILURE" {file: "http://mochi.test:8888/tests/dom/tests/mochitest/fetch/test_request.js" line: 456}]
08:39:21     INFO -  05-26 08:31:28.330 I/Gecko   ( 2215): [2215] WARNING: Failed to add feature: file /builds/slave/try-and-api-15-d-0000000000000/build/src/dom/fetch/Fetch.cpp, line 942
08:39:21     INFO -  05-26 08:31:28.330 I/Gecko   ( 2215): [2215] WARNING: 'aRv.Failed()', file /builds/slave/try-and-api-15-d-0000000000000/build/src/dom/fetch/Fetch.cpp, line 1211
08:39:21     INFO -  05-26 08:31:28.330 I/Gecko   ( 2215): [2215] WARNING: '!worker->AddFeature(mFeature)', file /builds/slave/try-and-api-15-d-0000000000000/build/src/dom/promise/Promise.cpp, line 2735
08:39:21     INFO -  05-26 08:31:28.330 E/GeckoConsole( 2215): [JavaScript Error: "NS_ERROR_FAILURE" {file: "http://mochi.test:8888/tests/dom/tests/mochitest/fetch/test_request.js" line: 465}]

I need to investigate further.
(Assignee)

Comment 8

3 years ago
I think the nested worker test case doesn't actually hold the nested worker alive.  This lets it get GC'd which aborts our various async tests.
(Assignee)

Comment 9

3 years ago
Created attachment 8756978 [details] [diff] [review]
P2 Hold nested worker alive until outer worker is collected. r=asuth

https://treeherder.mozilla.org/#/jobs?repo=try&revision=a978bbd68ecb
(Assignee)

Comment 10

3 years ago
Comment on attachment 8756978 [details] [diff] [review]
P2 Hold nested worker alive until outer worker is collected. r=asuth

The try looks like its still runing M(16) jobs, but they actually all completed successfully if you look in buildapi:

https://secure.pub.build.mozilla.org/buildapi/self-serve/try/rev/a978bbd68ecb49fa836f1d43be437002510cea00
Attachment #8756978 - Flags: review?(bugmail)
Attachment #8756978 - Flags: review?(bugmail) → review+

Comment 12

3 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/a6ffa86cfef8
https://hg.mozilla.org/mozilla-central/rev/250efc2d1015
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
status-firefox49: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla49
You need to log in before you can comment on or make changes to this bug.