Closed
Bug 1340425
Opened 9 years ago
Closed 7 years ago
Intermittent leakcheck | default process: 4734311 bytes leaked (AbstractThread, AbstractWatcher, Animation, AnimationEffectReadOnly, AnimationEffectTimingReadOnly, ...)
Categories
(Core :: General, defect)
Core
General
Tracking
()
RESOLVED
WORKSFORME
People
(Reporter: intermittent-bug-filer, Assigned: mccr8)
References
Details
(Keywords: intermittent-failure, memory-leak, Whiteboard: [stockwell fixed:product])
Attachments
(1 file)
59 bytes,
text/x-review-board-request
|
smaug
:
review+
lizzard
:
approval-mozilla-beta+
|
Details |
Filed by: philringnalda [at] gmail.com
https://treeherder.mozilla.org/logviewer.html#?job_id=78035070&repo=mozilla-central
https://archive.mozilla.org/pub/firefox/tinderbox-builds/mozilla-central-win32-debug/1487266159/mozilla-central_win7_vm-debug_test-mochitest-browser-chrome-7-bm129-tests1-windows-build131.txt.gz
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
![]() |
||
Comment 6•8 years ago
|
||
osx leaks on May 15 caused by bug 1364911, which was backed out.
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Updated•8 years ago
|
Whiteboard: [stockwell fixed]
Assignee | ||
Updated•8 years ago
|
Assignee | ||
Updated•8 years ago
|
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WORKSFORME
Comment hidden (Intermittent Failures Robot) |
Updated•8 years ago
|
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
![]() |
||
Comment 12•8 years ago
|
||
This bug previously tracked a leak in browser-chrome tests; that was resolved around May 16.
Currently, this intermittent leak is observed in mochitest-a11y, and OF suggests it started around June 21.
https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&filter-searchStr=mochitest-a11y%20debug%20linux&tochange=7bdf1aa4e51482e51c0104845dc249cdc027a809&fromchange=8e1de9b8675384f6307830d381fcd195a432ea83 strongly suggests this leak started with https://hg.mozilla.org/integration/mozilla-inbound/rev/e2efdd8dcd82d74128f47410014d491e76276263, bug 833143. I note https://bugzilla.mozilla.org/show_bug.cgi?id=833143#c41: "this may cause new intermittent leaks".
:mccr8 -- Just checking with you in case this leak is concerning or you can offer insight into resolving it.
Flags: needinfo?(continuation)
Assignee | ||
Comment 13•8 years ago
|
||
That's bad. I'll look into it. (Unfortunately, large leaks like this end up all looking the same as far as Treeherder classification goes.)
Flags: needinfo?(continuation)
Assignee | ||
Updated•8 years ago
|
Assignee: nobody → continuation
Assignee | ||
Comment 14•8 years ago
|
||
Thanks for tracking down what caused this.
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment 21•8 years ago
|
||
I see this has a steady stream of intermittents- not a high failure rate, but steady- :mccr8, I see you assigned yourself to this, is there a chance you had some WIP code to finish up?
Flags: needinfo?(continuation)
Assignee | ||
Comment 22•8 years ago
|
||
Oops, I forgot about this bug. I should just add back in the extra shutdown GC for debug builds and hope that fixes it.
Comment hidden (mozreview-request) |
Comment 24•8 years ago
|
||
mozreview-review |
Comment on attachment 8900428 [details]
Bug 1340425 - Add back the magic GC in DoShutdown() to avoid intermittent leaks.
https://reviewboard.mozilla.org/r/171778/#review177018
Attachment #8900428 -
Flags: review?(bugs) → review+
Assignee | ||
Updated•8 years ago
|
Flags: needinfo?(continuation)
Keywords: leave-open
Comment 25•8 years ago
|
||
Pushed by amccreight@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b4f88e70e596
Add back the magic GC in DoShutdown() to avoid intermittent leaks. r=smaug
![]() |
||
Comment 26•8 years ago
|
||
bugherder |
Assignee | ||
Comment 27•8 years ago
|
||
No occurrences of this failure since I landed my patch.
status-firefox56:
--- → affected
status-firefox57:
--- → fixed
Assignee | ||
Comment 28•8 years ago
|
||
Comment on attachment 8900428 [details]
Bug 1340425 - Add back the magic GC in DoShutdown() to avoid intermittent leaks.
Approval Request Comment
[Feature/Bug causing the regression]: bug 833143
[User impact if declined]: None. This just fixes an intermittent failure on Treeherder.
[Is this code covered by automated tests?]: Yes.
[Has the fix been verified in Nightly?]: Yes.
[Needs manual test from QE? If yes, steps to reproduce]: No.
[List of other uplifts needed for the feature/fix]: None.
[Is the change risky?]: No.
[Why is the change risky/not risky?]: This only affects debug builds.
[String changes made/needed]: None.
Attachment #8900428 -
Flags: approval-mozilla-beta?
Updated•8 years ago
|
Whiteboard: [stockwell fixed] → [stockwell fixed:product]
Comment hidden (Intermittent Failures Robot) |
Comment 30•8 years ago
|
||
Comment on attachment 8900428 [details]
Bug 1340425 - Add back the magic GC in DoShutdown() to avoid intermittent leaks.
Fix for failing tests on debug builds, avoids a leak, please uplift for beta 7.
Attachment #8900428 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 31•8 years ago
|
||
bugherder uplift |
Comment 32•8 years ago
|
||
(In reply to Andrew McCreight [:mccr8] from comment #28)
> [Is this code covered by automated tests?]: Yes.
> [Has the fix been verified in Nightly?]: Yes.
> [Needs manual test from QE? If yes, steps to reproduce]: No.
Setting qe-verify- based on Andrew's assessment on manual testing needs and the fact that this fix has automated coverage.
Flags: qe-verify-
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment 44•7 years ago
|
||
The leave-open keyword is there and there is no activity for 6 months.
:mccr8, maybe it's time to close this bug?
Flags: needinfo?(continuation)
Assignee | ||
Updated•7 years ago
|
Status: REOPENED → RESOLVED
Closed: 8 years ago → 7 years ago
Flags: needinfo?(continuation)
Resolution: --- → WORKSFORME
Updated•7 years ago
|
Keywords: leave-open
You need to log in
before you can comment on or make changes to this bug.
Description
•