Closed
Bug 456274
Opened 16 years ago
Closed 14 years ago
On tinderbox waterfall, wanted leak report for test boxes too
Categories
(Release Engineering :: General, defect, P5)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: sgautherie, Assigned: coop)
References
(Depends on 1 open bug)
Details
(Whiteboard: [unittest][tinderbox][oldbugs][triagefollowup])
Bug 450983 comment 25:
{
From Serge Gautherie 2008-09-09 19:03:26 PDT
As these leaks are not even reported on the waterfall itself [hint !],
[...]
}
***
Build boxes report RLk + Lk + MH + A.
Test boxes should report leaks for each of TUnit + ... + browser.
Currently, a box becomes Orange, but we have to open the (brief) report to discover why :-<
My proposal:
If |leak == threshold| (including "0"), no report ... That's the goal ;-)
If |leak != threshold|, add a report line like
"RLk: leak < threshold" (= green) or "RLk: leak > threshold" (= orange).
Updated•16 years ago
|
Assignee: build → nobody
Component: Tinderbox Configuration → Release Engineering
QA Contact: ccooper → release
Comment 1•16 years ago
|
||
dupe of bug 445596?
Comment 2•16 years ago
|
||
Note that we don't track leaks in any of our test suites other than mochitest.
Reporter | ||
Comment 3•16 years ago
|
||
(In reply to comment #1)
> dupe of bug 445596?
Right:
let's say that this is an enhanced output request, after minimal bug 445596 comment 1 ;->
(In reply to comment #2)
> Note that we don't track leaks in any of our test suites other than mochitest.
Well, let's apply it to where it can, of course ;-)
mochitest + chrome + browser:
Already leak tracked :-)
TUnit:
Could be (a little) harder as ("all") output does to their individual log files,
but it would be worse doing it (as I check them locally), as it does/can have leaks !
reftest + crashtest:
(I haven't run them yet, so I can't say, though I would think it couldn't hurt to monitor them too.)
Depends on: 445596
Comment 4•16 years ago
|
||
Probably worth filing separate bugs for implementing leak tracking in TUnit and reftest.
Reporter | ||
Comment 5•16 years ago
|
||
(In reply to comment #4)
> Probably worth filing separate bugs for implementing leak tracking in TUnit and
> reftest.
I filed bug 456414.
Updated•16 years ago
|
Component: Release Engineering → Release Engineering: Future
Reporter | ||
Comment 6•16 years ago
|
||
CC Marcia, as we briefly spoke about it at "Mozilla Camp Europe 2008".
Assignee | ||
Comment 7•15 years ago
|
||
Mass move of bugs from Release Engineering:Future -> Release Engineering. See
http://coop.deadsquid.com/2010/02/kiss-the-future-goodbye/ for more details.
Component: Release Engineering: Future → Release Engineering
Priority: -- → P3
Updated•15 years ago
|
Whiteboard: [unittest]
Assignee | ||
Updated•15 years ago
|
Priority: P3 → P5
Assignee | ||
Updated•14 years ago
|
Assignee: nobody → ccooper
Assignee | ||
Updated•14 years ago
|
Whiteboard: [unittest] → [unittest][oldbugs][triagefollowup]
Assignee | ||
Updated•14 years ago
|
Whiteboard: [unittest][oldbugs][triagefollowup] → [unittest][tinderbox][oldbugs][triagefollowup]
Assignee | ||
Comment 8•14 years ago
|
||
If and when bug 456414 gets fixed, we'll implement support for it in buildbot, but we won't explicitly be adding support to tinderbox for it.
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → WONTFIX
Updated•11 years ago
|
Product: mozilla.org → Release Engineering
You need to log in
before you can comment on or make changes to this bug.
Description
•