Closed
Bug 1373346
Opened 8 years ago
Closed 10 months ago
Intermittent dom/push/test/test_data.html | Test timed out
Categories
(Core :: DOM: Push Subscriptions, defect, P3)
Core
DOM: Push Subscriptions
Tracking
()
RESOLVED
INACTIVE
People
(Reporter: aryx, Unassigned)
References
Details
(Keywords: intermittent-failure, Whiteboard: [stockwell disabled],dom-lws-bugdash-triage )
Attachments
(2 files)
|
789 bytes,
patch
|
gbrown
:
review+
|
Details | Diff | Splinter Review |
|
1.13 KB,
patch
|
gbrown
:
review+
|
Details | Diff | Splinter Review |
https://treeherder.mozilla.org/logviewer.html#?job_id=107248100&repo=mozilla-inbound
08:51:28 INFO - 4464 INFO SpawnTask.js | Entering test compareJSONSubscription
08:51:28 INFO - 4465 INFO TEST-PASS | dom/push/test/test_data.html | Wrong endpoint
08:51:28 INFO - 4466 INFO TEST-PASS | dom/push/test/test_data.html | Mismatched Base64-encoded key: p256dh
08:51:28 INFO - 4467 INFO TEST-PASS | dom/push/test/test_data.html | Mismatched Base64-encoded key: auth
08:51:28 INFO - 4468 INFO SpawnTask.js | Leaving test compareJSONSubscription
08:51:28 INFO - 4469 INFO SpawnTask.js | Entering test comparePublicKey
08:51:28 INFO - 4470 INFO TEST-PASS | dom/push/test/test_data.html | Key share should be 65 octets
08:51:28 INFO - 4471 INFO TEST-PASS | dom/push/test/test_data.html | Mismatched key share
08:51:28 INFO - 4472 INFO TEST-PASS | dom/push/test/test_data.html | 16 - Auth secret should be 16 octets
08:51:28 INFO - 4473 INFO TEST-PASS | dom/push/test/test_data.html | Mismatched auth secret
08:51:28 INFO - 4474 INFO SpawnTask.js | Leaving test comparePublicKey
08:51:28 INFO - 4475 INFO SpawnTask.js | Entering test sendPushMessageFromPage
08:51:28 INFO - Buffered messages finished
08:51:28 ERROR - 4476 INFO TEST-UNEXPECTED-FAIL | dom/push/test/test_data.html | Test timed out.
08:51:28 INFO - reportError@SimpleTest/TestRunner.js:121:7
08:51:28 INFO - TestRunner._checkForHangs@SimpleTest/TestRunner.js:142:7
08:51:28 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
08:51:28 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
08:51:28 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
08:51:28 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
08:51:28 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
08:51:28 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
08:51:28 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
08:51:28 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
08:51:28 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
08:51:28 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
08:51:28 INFO - TestRunner.runTests@SimpleTest/TestRunner.js:380:5
08:51:28 INFO - RunSet.runtests@SimpleTest/setup.js:194:3
08:51:28 INFO - RunSet.runall@SimpleTest/setup.js:173:5
08:51:28 INFO - hookupTests@SimpleTest/setup.js:266:5
08:51:28 INFO - parseTestManifest@http://mochi.test:8888/manifestLibrary.js:36:5
08:51:28 INFO - getTestManifest/req.onload@http://mochi.test:8888/manifestLibrary.js:49:11
08:51:28 INFO - EventHandlerNonNull*getTestManifest@http://mochi.test:8888/manifestLibrary.js:45:3
08:51:28 INFO - hookup@SimpleTest/setup.js:246:5
08:51:28 INFO - EventHandlerNonNull*@http://mochi.test:8888/tests?autorun=1&closeWhenDone=1&consoleLevel=INFO&manifestFile=tests.json&dumpOutputDirectory=c%3A%5Cusers%5Cgenericworker%5Cappdata%5Clocal%5Ctemp&cleanupCrashes=true:11:1
08:51:29 INFO - Not taking screenshot here: see the one that was previously logged
08:51:29 ERROR - 4477 INFO TEST-UNEXPECTED-FAIL | dom/push/test/test_data.html | This test left a service worker registered without cleaning it up
| Comment hidden (Intermittent Failures Robot) |
| Comment hidden (Intermittent Failures Robot) |
Comment 4•8 years ago
|
||
64 failures since June 15th, all on opt/pgo, not debug and in e10s tests only.
I have done some retriggers:
https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&filter-searchStr=mochitest-e10s-2%20win%20opt&tochange=cb10c0fd0c6a585a13f1796f17fa77aa0cc6202c&fromchange=dfee29c06556ebe2f4ac43cf0744be9e93390d1d&selectedJob=107235629
Possibly this will help narrow down the root cause.
here is a log:
https://queue.taskcluster.net/v1/task/RObwNtCqStylF0lrbFup0w/runs/0/artifacts/public/logs/live_backing.log
and the text related to the failure:
08:46:28 INFO - 4457 INFO TEST-START | dom/push/test/test_data.html
08:46:59 INFO - GECKO(1980) | Unable to read VR Path Registry from C:\Users\GenericWorker\AppData\Local\openvr\openvrpaths.vrpath
08:46:59 INFO - GECKO(1980) | [Parent 1980] WARNING: pipe error: 109: file z:/build/build/src/ipc/chromium/src/chrome/common/ipc_channel_win.cc, line 346
08:51:28 INFO - TEST-INFO | started process screenshot
08:51:28 INFO - TEST-INFO | screenshot: exit 0
08:51:28 INFO - Buffered messages logged at 08:46:29
08:51:28 INFO - 4458 INFO SpawnTask.js | Entering test start
08:51:28 INFO - 4459 INFO SpawnTask.js | Leaving test start
08:51:28 INFO - 4460 INFO SpawnTask.js | Entering test createControlledIFrame
08:51:28 INFO - 4461 INFO SpawnTask.js | Leaving test createControlledIFrame
08:51:28 INFO - 4462 INFO SpawnTask.js | Entering test subscribe
08:51:28 INFO - 4463 INFO SpawnTask.js | Leaving test subscribe
08:51:28 INFO - 4464 INFO SpawnTask.js | Entering test compareJSONSubscription
08:51:28 INFO - 4465 INFO TEST-PASS | dom/push/test/test_data.html | Wrong endpoint
08:51:28 INFO - 4466 INFO TEST-PASS | dom/push/test/test_data.html | Mismatched Base64-encoded key: p256dh
08:51:28 INFO - 4467 INFO TEST-PASS | dom/push/test/test_data.html | Mismatched Base64-encoded key: auth
08:51:28 INFO - 4468 INFO SpawnTask.js | Leaving test compareJSONSubscription
08:51:28 INFO - 4469 INFO SpawnTask.js | Entering test comparePublicKey
08:51:28 INFO - 4470 INFO TEST-PASS | dom/push/test/test_data.html | Key share should be 65 octets
08:51:28 INFO - 4471 INFO TEST-PASS | dom/push/test/test_data.html | Mismatched key share
08:51:28 INFO - 4472 INFO TEST-PASS | dom/push/test/test_data.html | 16 - Auth secret should be 16 octets
08:51:28 INFO - 4473 INFO TEST-PASS | dom/push/test/test_data.html | Mismatched auth secret
08:51:28 INFO - 4474 INFO SpawnTask.js | Leaving test comparePublicKey
08:51:28 INFO - 4475 INFO SpawnTask.js | Entering test sendPushMessageFromPage
08:51:28 INFO - Buffered messages finished
08:51:28 ERROR - 4476 INFO TEST-UNEXPECTED-FAIL | dom/push/test/test_data.html | Test timed out.
08:51:28 INFO - reportError@SimpleTest/TestRunner.js:121:7
08:51:28 INFO - TestRunner._checkForHangs@SimpleTest/TestRunner.js:142:7
08:51:28 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
08:51:28 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
08:51:28 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
08:51:28 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
08:51:28 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
08:51:28 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
08:51:28 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
08:51:28 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
08:51:28 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
08:51:28 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
08:51:28 INFO - TestRunner.runTests@SimpleTest/TestRunner.js:380:5
08:51:28 INFO - RunSet.runtests@SimpleTest/setup.js:194:3
08:51:28 INFO - RunSet.runall@SimpleTest/setup.js:173:5
08:51:28 INFO - hookupTests@SimpleTest/setup.js:266:5
08:51:28 INFO - parseTestManifest@http://mochi.test:8888/manifestLibrary.js:36:5
08:51:28 INFO - getTestManifest/req.onload@http://mochi.test:8888/manifestLibrary.js:49:11
08:51:28 INFO - EventHandlerNonNull*getTestManifest@http://mochi.test:8888/manifestLibrary.js:45:3
08:51:28 INFO - hookup@SimpleTest/setup.js:246:5
08:51:28 INFO - EventHandlerNonNull*@http://mochi.test:8888/tests?autorun=1&closeWhenDone=1&consoleLevel=INFO&manifestFile=tests.json&dumpOutputDirectory=c%3A%5Cusers%5Cgenericworker%5Cappdata%5Clocal%5Ctemp&cleanupCrashes=true:11:1
08:51:29 INFO - Not taking screenshot here: see the one that was previously logged
08:51:29 ERROR - 4477 INFO TEST-UNEXPECTED-FAIL | dom/push/test/test_data.html | This test left a service worker registered without cleaning it up
08:51:29 INFO - afterCleanup@SimpleTest/SimpleTest.js:1196:17
08:51:29 INFO - executeCleanupFunction@SimpleTest/SimpleTest.js:1218:13
08:51:29 INFO - promise callback*executeCleanupFunction@SimpleTest/SimpleTest.js:1230:13
08:51:29 INFO - SimpleTest.finish@SimpleTest/SimpleTest.js:1237:5
08:51:29 INFO - killTest@SimpleTest/TestRunner.js:130:7
08:51:29 INFO - delayedKillTest@SimpleTest/TestRunner.js:157:47
08:51:29 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:157:7
08:51:29 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
08:51:29 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
08:51:29 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
08:51:29 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
08:51:29 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
08:51:29 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
08:51:29 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
08:51:29 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
08:51:29 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
08:51:29 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
08:51:29 INFO - TestRunner.runTests@SimpleTest/TestRunner.js:380:5
08:51:29 INFO - RunSet.runtests@SimpleTest/setup.js:194:3
08:51:29 INFO - RunSet.runall@SimpleTest/setup.js:173:5
08:51:29 INFO - hookupTests@SimpleTest/setup.js:266:5
08:51:29 INFO - parseTestManifest@http://mochi.test:8888/manifestLibrary.js:36:5
08:51:29 INFO - getTestManifest/req.onload@http://mochi.test:8888/manifestLibrary.js:49:11
08:51:29 INFO - EventHandlerNonNull*getTestManifest@http://mochi.test:8888/manifestLibrary.js:45:3
08:51:29 INFO - hookup@SimpleTest/setup.js:246:5
08:51:29 INFO - EventHandlerNonNull*@http://mochi.test:8888/tests?autorun=1&closeWhenDone=1&consoleLevel=INFO&manifestFile=tests.json&dumpOutputDirectory=c%3A%5Cusers%5Cgenericworker%5Cappdata%5Clocal%5Ctemp&cleanupCrashes=true:11:1
08:51:29 INFO - GECKO(1980) | MEMORY STAT | vsize 401MB | vsizeMaxContiguous 571MB | residentFast 25MB | heapAllocated 12MB
08:51:29 INFO - 4478 INFO TEST-OK | dom/push/test/test_data.html | took 301143ms
:overholt- as the triage owner can you find someone from the push notifications team to look into this and get a resolution in the next week?
Flags: needinfo?(overholt)
Whiteboard: [stockwell needswork]
Comment 5•8 years ago
|
||
I wouldn't be surprised if this were related to the recent changes with setTimeout but Kit will know best.
Flags: needinfo?(overholt) → needinfo?(kit)
| Comment hidden (Intermittent Failures Robot) |
Comment 7•8 years ago
|
||
I won't have cycles to look at this until after the work week, sorry. :-( This test will need some more logging, via `dom.push.loglevel = "all"` and `info()`, too. I can't tell if it's stuck waiting for the worker, failing to deliver the message through the mock push service, or something else. Keeping ni? until then.
Comment 8•8 years ago
|
||
as the failure rate is 60+/day, this is close to perma failing, lets disable until there is time to look at a real fix.
Updated•8 years ago
|
Attachment #8879235 -
Flags: review?(gbrown)
Comment 9•8 years ago
|
||
Comment on attachment 8879235 [details] [diff] [review]
temporarily disable on win opt
Review of attachment 8879235 [details] [diff] [review]:
-----------------------------------------------------------------
::: dom/push/test/mochitest.ini
@@ +18,5 @@
> [test_unregister.html]
> [test_multiple_register_different_scope.html]
> [test_subscription_change.html]
> [test_data.html]
> +skip-if = os=="win" && !debug # Bug 1373346
I'm not sure it is necessary, but we usually have spaces around the ==, so
skip-if = os == "win" && !debug
Attachment #8879235 -
Flags: review?(gbrown) → review+
Comment 10•8 years ago
|
||
Pushed by jmaher@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/bde143a9c8b2
Intermittent dom/push/test/test_data.html - temporarily disable. r=gbrown
Keywords: leave-open
| Comment hidden (Intermittent Failures Robot) |
Comment 12•8 years ago
|
||
joel, geoff, this is still perma failing
Flags: needinfo?(jmaher)
Flags: needinfo?(gbrown)
Comment 13•8 years ago
|
||
| bugherder | ||
Comment 14•8 years ago
|
||
pushing to try:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=793a732ef2e7478a5d3bc23f88dd0398905301ba
on my second round, appears there is a lot broken here.
Updated•8 years ago
|
Flags: needinfo?(gbrown)
Comment 15•8 years ago
|
||
here are some try pushes:
https://treeherder.mozilla.org/#/jobs?repo=try&author=jmaher@mozilla.com&fromchange=2c8538bc0960af169ca0eb2ebd45d3662f91bbd0&tochange=686017e0c1d7ca22b654686f899524eb2beec921
Flags: needinfo?(jmaher)
Attachment #8879628 -
Flags: review?(gbrown)
Updated•8 years ago
|
Attachment #8879628 -
Flags: review?(gbrown) → review+
Comment 16•8 years ago
|
||
Pushed by jmaher@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/c4d6c50a1434
dom/push/test has many perma failing mochitests, disable for now. r=gbrown
| Comment hidden (Intermittent Failures Robot) |
Comment 18•8 years ago
|
||
| bugherder | ||
Updated•8 years ago
|
Whiteboard: [stockwell needswork] → [stockwell disabled]
| Comment hidden (Intermittent Failures Robot) |
Comment 21•8 years ago
|
||
This started up again, now on Windows Debug.
Comment 22•8 years ago
|
||
Pushed by gbrown@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/4579ac092409
Skip some dom/push tests on Windows Debug too; r=me,test-only
| Comment hidden (Intermittent Failures Robot) |
Comment 24•8 years ago
|
||
| bugherder | ||
| Comment hidden (Intermittent Failures Robot) |
Updated•8 years ago
|
Priority: -- → P3
Comment 26•7 years ago
|
||
(In reply to Lina Cambridge (she/her) [:lina] from comment #7)
> I won't have cycles to look at this until after the work week, sorry. :-(
"After the work week" turned into "after a year". :-/
Blocks: 1206969
Flags: needinfo?(lina)
Comment 27•6 years ago
|
||
:perry, is this bug obsolete or did we just disable the failing tests?
Flags: needinfo?(perry)
Comment 28•6 years ago
|
||
Looks like failing tests were disabled. I'm not sure if any thing has happened so that this could be re-enabled...
Flags: needinfo?(perry)
| Comment hidden (Intermittent Failures Robot) |
| Comment hidden (Intermittent Failures Robot) |
Updated•3 years ago
|
Severity: normal → S3
| Comment hidden (Intermittent Failures Robot) |
| Comment hidden (Intermittent Failures Robot) |
Comment 33•1 year ago
|
||
Bulk closing intermittent bugs for DOM LWS that have not seen new instances since at least 5 months.
Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → INCOMPLETE
Updated•1 year ago
|
Keywords: leave-open
Comment 34•1 year ago
|
||
Taking back the bulk change.
Status: RESOLVED → UNCONFIRMED
Ever confirmed: false
Resolution: INCOMPLETE → ---
Updated•1 year ago
|
Status: UNCONFIRMED → NEW
Ever confirmed: true
Updated•1 year ago
|
Keywords: leave-open
Comment 35•10 months ago
•
|
||
Not sure why this should be left open. The bot can handle it just as well. (It's not disabled right now)
Status: NEW → RESOLVED
Closed: 1 year ago → 10 months ago
Keywords: leave-open
Resolution: --- → INACTIVE
Updated•10 months ago
|
Whiteboard: [stockwell disabled] → [stockwell disabled],dom-lws-bugdash-triage
You need to log in
before you can comment on or make changes to this bug.
Description
•