Closed
Bug 893249
Opened 11 years ago
Closed 11 years ago
Firefox enters "Not Responding" mode during test runs
Categories
(Add-on SDK Graveyard :: General, defect, P1)
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: irakli, Unassigned)
References
Details
Attachments
(2 files)
In the journey of trying to figure why net module fails on tinderbox I experimented with very slow VM. I discovered that our tests super fragile when running on a slow machine. I believe that's due the fact that Firefox freezes several times showing up as "Not Responding" in the task manager and then resumes, if it's frozen enough time then we timeout.
We should figure a way to run tests such that firefox won't enter "Not Responding" mode to be more reliable. I also believe that random oranges that we see every now and then are caused by this.
Reporter | ||
Comment 1•11 years ago
|
||
Pointer to Github pull-request
Reporter | ||
Updated•11 years ago
|
Attachment #775955 -
Flags: review?(jsantell)
Comment 2•11 years ago
|
||
Commit pushed to master at https://github.com/mozilla/addon-sdk
https://github.com/mozilla/addon-sdk/commit/4d783655b311a5d1450436f4c9a22e0bd16f2dd0
Merge pull request #1107 from Gozala/bug/zombie-test@893249
Bug 893249 - Run GC after test case to avoid freezing a process. r=@jsantell
Comment 3•11 years ago
|
||
Comment on attachment 775955 [details]
Pointer to Github pull request: https://github.com/mozilla/addon-sdk/pull/1107
If this doesn't affect test speed in a non-negligible way, looks good
Attachment #775955 -
Flags: review?(jsantell) → review+
Comment 4•11 years ago
|
||
Commit pushed to master at https://github.com/mozilla/addon-sdk
https://github.com/mozilla/addon-sdk/commit/0cbc96c4559333b197ed8ea9ac6da51a6566d132
Merge pull request #1111 from Gozala/bug/zombie-test@893249
Bug 893249 - Increase overall test run timeout to accommodate GC cycles between test runs. r=@KWierso
Reporter | ||
Comment 5•11 years ago
|
||
Taras suggested we could use about:telemetry to find out reasons why firefox get's frozen.
Reporter | ||
Comment 6•11 years ago
|
||
Pointer to Github pull-request
Reporter | ||
Updated•11 years ago
|
Attachment #777238 -
Flags: review?(poirot.alex)
Comment 7•11 years ago
|
||
Comment on attachment 777238 [details]
Pointer to Github pull request: https://github.com/mozilla/addon-sdk/pull/1119
I think you need some additional cleanup on python side:
https://github.com/mozilla/addon-sdk/blob/master/python-lib/cuddlefish/runner.py#L484-L497
There is also this another thing, we we redirect app stdout to a file on python side:
https://github.com/mozilla/addon-sdk/blob/master/python-lib/cuddlefish/runner.py#L526-L539
(yes, we were doing it on both sides!)
Attachment #777238 -
Flags: review?(poirot.alex) → review+
Comment 8•11 years ago
|
||
Commit pushed to master at https://github.com/mozilla/addon-sdk
https://github.com/mozilla/addon-sdk/commit/e7a5dc9f99239b84dec5abe6ed59154c8496a1ee
Merge pull request #1119 from Gozala/bug/dump@893249
Bug 893249 - Just dump output instead of writing to file.
Updated•11 years ago
|
Priority: -- → P1
Depends on: 897683
I'm going through the list of open bugs that github robot has commented in. Is this bug fixed, Irakli?
Flags: needinfo?(rFobic)
Reporter | ||
Comment 10•11 years ago
|
||
Not really, I guess we could close it for a time being and re-open again if it'll still be relevant.
Status: NEW → RESOLVED
Closed: 11 years ago
Flags: needinfo?(rFobic)
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•