Closed Bug 1180236 Opened 9 years ago Closed 8 years ago

Intermittent apps/system/test/marionette/browser_chrome_new_window_test.js | Browser Chrome - Open New Window open new window

Categories

(Firefox OS Graveyard :: Gaia, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: philor, Assigned: u520299)

Details

(Keywords: intermittent-failure, Whiteboard: [systemsfe])

Attachments

(2 files, 1 obsolete file)

      No description provided.
Blocks: 1222215
Whiteboard: [systemsfe]
Assignee: nobody → francisco
Status: NEW → ASSIGNED
After expending some time in the bug, was able to reproduce thanks to :mhenretty and :mcav by using the cpulimit script here:

https://groups.google.com/d/msg/mozilla.dev.fxos/wHKnQE1gOxE/QZKfAzePAgAJ

Checking the previous comments and the links to the fails in treeherder like:

https://treeherder.mozilla.org/logviewer.html#?repo=fx-team&job_id=5749386

Locally also got the same error:
Error: timeout exceeded!
      at Object.Client.waitForSync (/Users/arcturus/dev/git/community/gaia/node_modules/marionette-client/lib/marionette/client.js:759:16)
      at Object.Client.waitFor (/Users/arcturus/dev/git/community/gaia/node_modules/marionette-client/lib/marionette/client.js:725:60)
      at Context.<anonymous> (/Users/arcturus/dev/git/community/gaia/apps/system/test/marionette/browser_chrome_new_window_test.js:78:12)

In that line we wait for the Clock app to be displayed in the foreground, and the reason why we don't transition from the new browser chrome window to the clock app is cause the clock app died.

Still need to understand the causes why the clock dies on low cpu availability.
This is one of our worst intermittents, so I'm disabling this for now.

disabled in master: https://github.com/mozilla-b2g/gaia/commit/373a1ef64bca6f6454bb9b8af7f11fbf71dd37c8
Removing the dependency since this test is now disabled.
No longer blocks: 1222215
Michael, did this disable properly?  It seems we're still hitting it frequently.
Flags: needinfo?(mhenretty)
I think so Ben. When I look at the brasstacks link from comment 56, the most recent failure I see is from 12/18, which is not unusual give the amount of time gaia changes take to propagate to mozilla-inbound (~1 day).  If I look at the logs for a recent Gij 19 run on mozilla-central ([1] for instance), I don't see browser_chrome_new_window_test.js getting run. Have you seen a more recent failure?

1.) https://public-artifacts.taskcluster.net/Eoz-WWaJQ0iLzIrFwF3rKw/0/public/logs/live_backing.log
Flags: needinfo?(mhenretty) → needinfo?(bkelly)
Sorry for the noise.  I must have misclicked brasstacks or somehow gotten stale data from it.

Should we go ahead and close this?
Flags: needinfo?(bkelly)
We should leave this open, since it is still a bad test and needs to be fixed, we just have it disabled right now. After we land bug 1222215, we will be going through and fixing and re-enabling these tests [1] one by one.

1.) https://github.com/mozilla-b2g/gaia/blob/master/shared/test/integration/tbpl-manifest.json
Keywords: leave-open
I'll give it a spin. Francisco: If you wanna keep it, just take it again.
Assignee: francisco → tilman
Attachment #8712088 - Flags: review?(mhenretty)
Can you show me a try run with say 40 re-triggers all green? This used to be one of our worst intermittents.
Flags: needinfo?(tilman)
Attachment #8712088 - Flags: review?(mhenretty) → review+
Still intermittent - disabled test again:
https://github.com/tilmankamp/gaia/commit/f3f5831f215977b5ae65c496a98a6b6479478ede
Attachment #8712088 - Attachment is obsolete: true
Comment on attachment 8714092 [details] [review]
[gaia] tilmankamp:u1180236 > mozilla-b2g:master

50/50 green runs:
https://treeherder.mozilla.org/#/jobs?repo=gaia&revision=2e9a757b8c12822d5344ad67a6d541a84a33761e
Attachment #8714092 - Flags: review?(mhenretty)
Comment on attachment 8714092 [details] [review]
[gaia] tilmankamp:u1180236 > mozilla-b2g:master

Left a question on github. Feel free to re-flag me when you've answered it. Thanks!
Attachment #8714092 - Flags: review?(mhenretty)
Comment on attachment 8714092 [details] [review]
[gaia] tilmankamp:u1180236 > mozilla-b2g:master

50/50 green test runs:
https://treeherder.mozilla.org/#/jobs?repo=gaia&revision=3568721142fb1742a47ca4c52c00c6243365d6e8
Attachment #8714092 - Flags: review?(mhenretty)
Attachment #8714092 - Flags: review?(mhenretty) → review+
Merged into master:
https://github.com/tilmankamp/gaia/commit/47185183c76e18605e48ceccce12af04791defd7
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
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.

Attachment

General

Created:
Updated:
Size: