Closed
Bug 1010415
Opened 11 years ago
Closed 10 years ago
Intermittent notification_resend_test.js | mozChromeNotifications: Sending one notification, remove from tray, resend
Categories
(Firefox OS Graveyard :: Gaia::System, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
2.1 S4 (12sep)
People
(Reporter: RyanVM, Assigned: gerard-majax)
References
Details
(Keywords: intermittent-failure, Whiteboard: [systemsfe])
Attachments
(2 files, 1 obsolete file)
https://tbpl.mozilla.org/php/getParsedLog.php?id=39665712&tree=B2g-Inbound
b2g_ubuntu64_vm b2g-inbound opt test gaia-integration on 2014-05-14 09:19:40 PDT for push bd5ec747c11e
slave: tst-linux64-spot-617
09:35:01 INFO - TEST-START | mozChromeNotifications: Sending one notification, remove from tray, resend
09:35:33 INFO - Screenshot:
09:35:33 INFO - TEST-UNEXPECTED-FAIL | /builds/slave/test/gaia/apps/system/test/marionette/notification_resend_test.js | mozChromeNotifications: Sending one notification, remove from tray, resend
09:35:33 INFO - TEST-END | mozChromeNotifications: Sending one notification, remove from tray, resend
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment 11•10 years ago
|
||
Disabling this test so we can turn on Gij on TBPL: https://github.com/mozilla-b2g/gaia/commit/a12ea45a5aca843105ea3188df146d48e84cf673
Adding leave-open to prevent auto-closing of this bug by sheriffs.
Here is a recent run failure of this test: https://tbpl.mozilla.org/php/getParsedLog.php?id=43553081&tree=B2g-Inbound
Ni? on Alex for awareness.
Flags: needinfo?(lissyx+mozillians)
Keywords: leave-open
Comment 12•10 years ago
|
||
Originally I had thought that the "Error: Polling socket recv() timeout!" error was framework related, but I've seen this same test fail multiple times with that, and other tests don't seem to fail so much. So we should investigate to see what that means.
Assignee | ||
Comment 13•10 years ago
|
||
I don't understand, is it an intermittent test or a permanent fail ?!
Flags: needinfo?(lissyx+mozillians)
Comment 14•10 years ago
|
||
(In reply to Alexandre LISSY :gerard-majax from comment #13)
> I don't understand, is it an intermittent test or a permanent fail ?!
This is an intermittent test. We currently have way too many intermittent tests and need to get down to about 1% to re-enable tests on TBPL. In order to do so we will be disabling a lot of tests unfortunately. After we are back up and running on TBPL I will be more than happy to help people debug these issues.
Regarding this specific test, I'm still not sure if it's an issue with the test, or a problem with the harness. We'll need to do a bit of investigation to figure that out.
Assignee | ||
Comment 15•10 years ago
|
||
(In reply to Kevin Grandon :kgrandon from comment #14)
> (In reply to Alexandre LISSY :gerard-majax from comment #13)
> > I don't understand, is it an intermittent test or a permanent fail ?!
>
> This is an intermittent test. We currently have way too many intermittent
> tests and need to get down to about 1% to re-enable tests on TBPL. In order
> to do so we will be disabling a lot of tests unfortunately. After we are
> back up and running on TBPL I will be more than happy to help people debug
> these issues.
>
> Regarding this specific test, I'm still not sure if it's an issue with the
> test, or a problem with the harness. We'll need to do a bit of investigation
> to figure that out.
Thanks. I spent some time looking at the test, I found some weirdness. I have no idea if it could explain the issue, but I filed bug 1037371 and I have a PR that makes use of as much as possible of addEventListener() inside executeAsyncScript() instead of mixing executeScript() and executeAsyncScript().
We'll see if it helps.
Meanwhile, I was not able to reproduce the issue locally.
Updated•10 years ago
|
Comment 16•10 years ago
|
||
Let's see if this test is still intermittent in tbpl.
Comment 17•10 years ago
|
||
Assignee | ||
Comment 18•10 years ago
|
||
Michael, it seems we now have a permafail. At least if we reproduce locally, we can investigate ... :)
https://tbpl.mozilla.org/?rev=99315d2bd079b7d14ae7b40b4edb6d5fbe45f4a4&tree=Gaia-Try&showall=1
Assignee | ||
Comment 19•10 years ago
|
||
1) mozChromeNotifications: Sending one notification, remove from tray, resend:
JavaScriptError: (17) NotFoundError: Node was not found
Remote Stack:
execute_async_script @undefined, line undefined
inline javascript, line 72
src: " nodes = container.removeChild(nodes[0]).querySelectorAll(selector);"
at Error.MarionetteError (/home/alex/codaz/Mozilla/b2g/gaia/node_modules/marionette-client/lib/marionette/error.js:67:13)
at Object.Client._handleCallback (/home/alex/codaz/Mozilla/b2g/gaia/node_modules/marionette-client/lib/marionette/client.js:476:19)
at /home/alex/codaz/Mozilla/b2g/gaia/node_modules/marionette-client/lib/marionette/client.js:510:21
at TcpSync.send (/home/alex/codaz/Mozilla/b2g/gaia/node_modules/marionette-client/lib/marionette/drivers/tcp-sync.js:153:10)
at Object.send (/home/alex/codaz/Mozilla/b2g/gaia/node_modules/marionette-client/lib/marionette/client.js:457:36)
at Object.Client._sendCommand (/home/alex/codaz/Mozilla/b2g/gaia/node_modules/marionette-client/lib/marionette/client.js:503:19)
at Object._executeScript (/home/alex/codaz/Mozilla/b2g/gaia/node_modules/marionette-client/lib/marionette/client.js:1468:19)
at Object.executeAsyncScript (/home/alex/codaz/Mozilla/b2g/gaia/node_modules/marionette-client/lib/marionette/client.js:1249:19)
at Context.<anonymous> (/home/alex/codaz/Mozilla/b2g/gaia/apps/system/test/marionette/notification_resend_test.js:192:20)
at Test.Runnable.run (/home/alex/codaz/Mozilla/b2g/gaia/node_modules/mocha/lib/runnable.js:196:15)
at Runner.runTest (/home/alex/codaz/Mozilla/b2g/gaia/node_modules/mocha/lib/runner.js:373:10)
at /home/alex/codaz/Mozilla/b2g/gaia/node_modules/mocha/lib/runner.js:451:12
at next (/home/alex/codaz/Mozilla/b2g/gaia/node_modules/mocha/lib/runner.js:298:14)
at /home/alex/codaz/Mozilla/b2g/gaia/node_modules/mocha/lib/runner.js:308:7
at next (/home/alex/codaz/Mozilla/b2g/gaia/node_modules/mocha/lib/runner.js:246:23)
at /home/alex/codaz/Mozilla/b2g/gaia/node_modules/mocha/lib/runner.js:270:7
at done (/home/alex/codaz/Mozilla/b2g/gaia/node_modules/mocha/lib/runnable.js:185:5)
at /home/alex/codaz/Mozilla/b2g/gaia/node_modules/mocha/lib/runnable.js:199:9
at Object.executeHook (/home/alex/codaz/Mozilla/b2g/gaia/node_modules/marionette-client/lib/marionette/client.js:369:18)
at process._tickCallback (node.js:419:13)
Assignee | ||
Comment 20•10 years ago
|
||
Michael, this simple change does fix the test locally :)
Assignee: nobody → lissyx+mozillians
Attachment #8476054 -
Attachment is obsolete: true
Status: NEW → ASSIGNED
Attachment #8485534 -
Flags: review?(mhenretty)
Assignee | ||
Comment 21•10 years ago
|
||
Assignee | ||
Comment 22•10 years ago
|
||
So the gaia try with ~50 retriggers expose no failure at all of the notification test, but there are other failures.
Flags: needinfo?(mhenretty)
Updated•10 years ago
|
Attachment #8485534 -
Flags: review?(mhenretty) → review+
Flags: needinfo?(mhenretty)
Comment 23•10 years ago
|
||
master: https://github.com/mozilla-b2g/gaia/commit/c5ad0797cc83b9a6e6519e19fa8f401c021979bb
I'm going to look into some of those other notification errors.
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Updated•10 years ago
|
Whiteboard: [systemsfe]
Target Milestone: --- → 2.1 S4 (12sep)
Comment 24•7 years ago
|
||
Removing leave-open keyword from resolved bugs, per :sylvestre.
Keywords: leave-open
You need to log in
before you can comment on or make changes to this bug.
Description
•