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)

x86
macOS
defect

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.
Attachment #775955 - Flags: review?(jsantell)
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 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+
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
Taras suggested we could use about:telemetry to find out reasons why firefox get's frozen.
Attachment #777238 - Flags: review?(poirot.alex)
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+
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.
Priority: -- → P1
I'm going through the list of open bugs that github robot has commented in. Is this bug fixed, Irakli?
Flags: needinfo?(rFobic)
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.

Attachment

General

Created:
Updated:
Size: