Closed
Bug 782633
Opened 12 years ago
Closed 12 years ago
Intermittent TEST-UNEXPECTED-FAIL | tab process nnn | automationutils.processLeakLog() | missing output line for total leaks!
Categories
(Testing :: Mochitest, defect)
Tracking
(firefox18 wontfix, firefox19 fixed, firefox20 fixed, firefox-esr17 fixed, b2g18 fixed)
RESOLVED
FIXED
mozilla21
People
(Reporter: emorley, Assigned: emorley)
References
Details
(Keywords: intermittent-failure)
Attachments
(1 file)
(deleted),
patch
|
ted
:
review+
|
Details | Diff | Splinter Review |
Not really sure what to file this one as...
Rev4 MacOSX Snow Leopard 10.6 mozilla-inbound debug test mochitests-2/5 on 2012-08-14 03:34:35 PDT for push 59707ed19e48
slave: talos-r4-snow-044
https://tbpl.mozilla.org/php/getParsedLog.php?id=14366225&tree=Mozilla-Inbound
{
[Parent 529] WARNING: not an nsIRDFRemoteDataSource: 'remote != nullptr', file ../../../../rdf/datasource/src/nsLocalStore.cpp, line 279
--DOMWINDOW == 1 (0x122d38ac0) [serial = 3] [outer = 0x0] [url = chrome://browser/content/hiddenWindow.xul]
--DOMWINDOW == 0 (0x122d3b720) [serial = 4] [outer = 0x0] [url = chrome://browser/content/hiddenWindow.xul]
[Parent 529] WARNING: not an nsIRDFRemoteDataSource: 'remote != nullptr', file ../../../../rdf/datasource/src/nsLocalStore.cpp, line 279
[Parent 529] WARNING: NS_ENSURE_TRUE(mTextInputHandler) failed: file ../../../widget/cocoa/nsChildView.mm, line 4070
nsStringStats
=> mAllocCount: 2302636
=> mReallocCount: 106561
=> mFreeCount: 2302636
=> mShareCount: 4690847
=> mAdoptCount: 208698
=> mAdoptFreeCount: 208698
INFO | automation.py | Application ran for: 0:18:35.995106
INFO | automation.py | Reading PID log: /var/folders/Hs/HsDn6a9SG8idoIya6p9mtE+++TI/-Tmp-/tmpUiWBqXpidlog
== BloatView: ALL (cumulative) LEAK AND BLOAT STATISTICS, default process 529
|<----------------Class--------------->|<-----Bytes------>|<----------------Objects---------------->|<--------------References-------------->|
Per-Inst Leaked Total Rem Mean StdDev Total Rem Mean StdDev
0 TOTAL 22 0 130053207 0 (37632.00 +/- 71412.98) 297676800 18446744073709551614 (61969108727.09 +/- 0.00)
nsTraceRefcntImpl::DumpStatistics: 1299 entries
TEST-PASS | automationutils.processLeakLog() | no leaks detected!
TEST-UNEXPECTED-FAIL | tab process 555 | automationutils.processLeakLog() | missing output line for total leaks!
== BloatView: ALL (cumulative) LEAK AND BLOAT STATISTICS, tab process 558
|<----------------Class--------------->|<-----Bytes------>|<----------------Objects---------------->|<--------------References-------------->|
Per-Inst Leaked Total Rem Mean StdDev Total Rem Mean StdDev
0 TOTAL 30 0 62013 0 ( 1012.43 +/- 1195.63) 93059 0 ( 1191.63 +/- 2479.43)
nsTraceRefcntImpl::DumpStatistics: 540 entries
TEST-PASS | tab process 558 | automationutils.processLeakLog() | no leaks detected!
== BloatView: ALL (cumulative) LEAK AND BLOAT STATISTICS, tab process 559
|<----------------Class--------------->|<-----Bytes------>|<----------------Objects---------------->|<--------------References-------------->|
Per-Inst Leaked Total Rem Mean StdDev Total Rem Mean StdDev
0 TOTAL 19 0 7066748 0 ( 3373.42 +/- 1878.13) 4719591 0 ( 1271.50 +/- 2913.44)
nsTraceRefcntImpl::DumpStatistics: 745 entries
TEST-PASS | tab process 559 | automationutils.processLeakLog() | no leaks detected!
INFO | runtests.py | Running tests: end.
program finished with exit code 0
}
Comment 1•12 years ago
|
||
Comment 2•12 years ago
|
||
Comment hidden (Legacy TBPL/Treeherder Robot) |
Assignee | ||
Comment 4•12 years ago
|
||
Comment 5•12 years ago
|
||
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment 7•12 years ago
|
||
Comment 8•12 years ago
|
||
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment 11•12 years ago
|
||
Updated•12 years ago
|
Summary: Intermittent TEST-UNEXPECTED-FAIL | tab process 555 | automationutils.processLeakLog() | missing output line for total leaks! → Intermittent TEST-UNEXPECTED-FAIL | tab process nnn | automationutils.processLeakLog() | missing output line for total leaks!
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment 13•12 years ago
|
||
Comment 14•12 years ago
|
||
Comment 15•12 years ago
|
||
Assignee | ||
Comment 16•12 years ago
|
||
Assignee | ||
Comment 17•12 years ago
|
||
Assignee | ||
Comment 18•12 years ago
|
||
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment 22•12 years ago
|
||
Comment 23•12 years ago
|
||
Comment 24•12 years ago
|
||
Comment 25•12 years ago
|
||
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment 29•12 years ago
|
||
Comment 30•12 years ago
|
||
Assignee | ||
Comment 31•12 years ago
|
||
vporof, bbondy, paul:
The starrings you made via TBPL above were mis-stars. Whilst this bug gets suggested quite a bit of the time, the case to which it refers will never be able to be picked up by the tbpl parser - it always requires manual starring.
Please check when starring on TBPL that the full summary matches (in this case, the "tab process nnn" part is key) - since mis-stars can obscure real failures. Thanks! :-)
Comment 32•12 years ago
|
||
Comment 33•12 years ago
|
||
OS: Mac OS X → All
Comment 34•12 years ago
|
||
Comment 35•12 years ago
|
||
Comment 36•12 years ago
|
||
Comment 37•12 years ago
|
||
Assignee | ||
Comment 38•12 years ago
|
||
Assignee | ||
Comment 39•12 years ago
|
||
Comment 40•12 years ago
|
||
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment 42•12 years ago
|
||
Comment 43•12 years ago
|
||
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment 45•12 years ago
|
||
Assignee | ||
Comment 46•12 years ago
|
||
Comment 47•12 years ago
|
||
Assignee | ||
Comment 48•12 years ago
|
||
Assignee | ||
Comment 49•12 years ago
|
||
Comment 50•12 years ago
|
||
Comment 51•12 years ago
|
||
Comment 52•12 years ago
|
||
Comment 53•12 years ago
|
||
Comment 54•12 years ago
|
||
Comment 55•12 years ago
|
||
Comment 56•12 years ago
|
||
Comment 57•12 years ago
|
||
Assignee | ||
Comment 58•12 years ago
|
||
Comment 59•12 years ago
|
||
Comment 60•12 years ago
|
||
Comment 61•12 years ago
|
||
Comment 62•12 years ago
|
||
Comment 63•12 years ago
|
||
Comment 64•12 years ago
|
||
Comment 65•12 years ago
|
||
Comment 66•12 years ago
|
||
Assignee | ||
Comment 67•12 years ago
|
||
Comment 68•12 years ago
|
||
Assignee | ||
Comment 69•12 years ago
|
||
Assignee | ||
Comment 70•12 years ago
|
||
Assignee | ||
Comment 71•12 years ago
|
||
Assignee | ||
Comment 72•12 years ago
|
||
Comment 73•12 years ago
|
||
Comment 74•12 years ago
|
||
Comment 75•12 years ago
|
||
Comment 76•12 years ago
|
||
Comment 77•12 years ago
|
||
Assignee | ||
Comment 78•12 years ago
|
||
Assignee | ||
Comment 79•12 years ago
|
||
Comment 80•12 years ago
|
||
Comment 81•12 years ago
|
||
Comment 82•12 years ago
|
||
Comment 83•12 years ago
|
||
Comment 84•12 years ago
|
||
Comment 85•12 years ago
|
||
Comment 86•12 years ago
|
||
Comment 87•12 years ago
|
||
Assignee | ||
Comment 88•12 years ago
|
||
Assignee | ||
Comment 89•12 years ago
|
||
Comment 90•12 years ago
|
||
Comment 91•12 years ago
|
||
Assignee | ||
Comment 92•12 years ago
|
||
Comment 93•12 years ago
|
||
Comment 94•12 years ago
|
||
Assignee | ||
Comment 95•12 years ago
|
||
Comment 96•12 years ago
|
||
Assignee | ||
Comment 97•12 years ago
|
||
Comment 98•12 years ago
|
||
Comment 99•12 years ago
|
||
Comment 100•12 years ago
|
||
Comment 101•12 years ago
|
||
Assignee | ||
Comment 102•12 years ago
|
||
Comment 103•12 years ago
|
||
Comment 104•12 years ago
|
||
Comment 105•12 years ago
|
||
Comment 106•12 years ago
|
||
Assignee | ||
Comment 107•12 years ago
|
||
Comment 108•12 years ago
|
||
Comment 109•12 years ago
|
||
Comment 110•12 years ago
|
||
Comment 111•12 years ago
|
||
Comment 112•12 years ago
|
||
Comment 113•12 years ago
|
||
Comment 114•12 years ago
|
||
Comment 115•12 years ago
|
||
Comment 116•12 years ago
|
||
Comment 117•12 years ago
|
||
Comment 118•12 years ago
|
||
Assignee | ||
Comment 119•12 years ago
|
||
Assignee | ||
Comment 120•12 years ago
|
||
Comment 121•12 years ago
|
||
Assignee | ||
Comment 122•12 years ago
|
||
Comment 123•12 years ago
|
||
Comment 124•12 years ago
|
||
Comment 125•12 years ago
|
||
Comment 126•12 years ago
|
||
Comment 127•12 years ago
|
||
Comment 128•12 years ago
|
||
Comment 129•12 years ago
|
||
Comment 130•12 years ago
|
||
Comment 131•12 years ago
|
||
Comment 132•12 years ago
|
||
Updated•12 years ago
|
Comment 133•12 years ago
|
||
Comment 134•12 years ago
|
||
Comment 135•12 years ago
|
||
Comment 136•12 years ago
|
||
Assignee | ||
Comment 137•12 years ago
|
||
Comment 138•12 years ago
|
||
Comment 139•12 years ago
|
||
Comment 140•12 years ago
|
||
Comment 141•12 years ago
|
||
Comment 142•12 years ago
|
||
Comment 143•12 years ago
|
||
Comment 144•12 years ago
|
||
Comment 145•12 years ago
|
||
Comment 146•12 years ago
|
||
Comment 147•12 years ago
|
||
Comment 148•12 years ago
|
||
Updated•12 years ago
|
Keywords: intermittent-failure
Comment 149•12 years ago
|
||
Comment 150•12 years ago
|
||
Comment 151•12 years ago
|
||
Comment 152•12 years ago
|
||
Comment 153•12 years ago
|
||
status-firefox20:
--- → affected
Comment 154•12 years ago
|
||
Comment 155•12 years ago
|
||
Updated•12 years ago
|
Whiteboard: [orange]
Assignee | ||
Comment 156•12 years ago
|
||
Assignee | ||
Comment 157•12 years ago
|
||
Assignee | ||
Comment 158•12 years ago
|
||
Comment 159•12 years ago
|
||
https://tbpl.mozilla.org/php/getParsedLog.php?id=17369860&tree=Try
The problem here is that we don't "join" all content processes on shutdown, i.e. wait for them to terminate. And, not surprisingly, sometimes they don't shut down before the parent process does.
Comment 161•12 years ago
|
||
Can we do that for debug builds only? That's the only place we're doing leak checks anyway.
Oh, golly, yes. I didn't mean to even suggest anything otherwise :).
Comment 163•12 years ago
|
||
Assignee | ||
Comment 164•12 years ago
|
||
Assignee | ||
Comment 165•12 years ago
|
||
Assignee | ||
Comment 166•12 years ago
|
||
Assignee | ||
Comment 167•12 years ago
|
||
Assignee | ||
Comment 168•12 years ago
|
||
Assignee | ||
Comment 169•12 years ago
|
||
Comment 171•12 years ago
|
||
Assignee | ||
Comment 173•12 years ago
|
||
Assignee | ||
Comment 174•12 years ago
|
||
Assignee | ||
Comment 175•12 years ago
|
||
Assignee | ||
Comment 176•12 years ago
|
||
Comment 177•12 years ago
|
||
Comment 178•12 years ago
|
||
Comment 179•12 years ago
|
||
Assignee | ||
Comment 180•12 years ago
|
||
Assignee | ||
Comment 181•12 years ago
|
||
Assignee | ||
Comment 182•12 years ago
|
||
Comment 183•12 years ago
|
||
Assignee | ||
Comment 184•12 years ago
|
||
Assignee | ||
Comment 185•12 years ago
|
||
Assignee | ||
Comment 186•12 years ago
|
||
Assignee | ||
Comment 187•12 years ago
|
||
Assignee | ||
Comment 188•12 years ago
|
||
Assignee | ||
Comment 189•12 years ago
|
||
Assignee | ||
Comment 190•12 years ago
|
||
Assignee | ||
Comment 194•12 years ago
|
||
Assignee | ||
Comment 195•12 years ago
|
||
Comment 196•12 years ago
|
||
Comment 197•12 years ago
|
||
Assignee | ||
Comment 202•12 years ago
|
||
Assignee | ||
Comment 203•12 years ago
|
||
Assignee | ||
Comment 204•12 years ago
|
||
Assignee | ||
Comment 206•12 years ago
|
||
Assignee | ||
Comment 207•12 years ago
|
||
Assignee | ||
Comment 208•12 years ago
|
||
Assignee | ||
Comment 209•12 years ago
|
||
Assignee | ||
Comment 210•12 years ago
|
||
Assignee | ||
Comment 211•12 years ago
|
||
Assignee | ||
Comment 212•12 years ago
|
||
Assignee | ||
Comment 213•12 years ago
|
||
Assignee | ||
Comment 214•12 years ago
|
||
Assignee | ||
Comment 215•12 years ago
|
||
Assignee | ||
Comment 216•12 years ago
|
||
Assignee | ||
Comment 217•12 years ago
|
||
Assignee | ||
Comment 218•12 years ago
|
||
Assignee | ||
Comment 219•12 years ago
|
||
Assignee | ||
Comment 220•12 years ago
|
||
Assignee | ||
Comment 221•12 years ago
|
||
Assignee | ||
Comment 222•12 years ago
|
||
Comment 223•12 years ago
|
||
Comment 224•12 years ago
|
||
Comment 225•12 years ago
|
||
Comment 226•12 years ago
|
||
Comment 227•12 years ago
|
||
Comment 228•12 years ago
|
||
Assignee | ||
Comment 229•12 years ago
|
||
Assignee | ||
Comment 230•12 years ago
|
||
Assignee | ||
Comment 231•12 years ago
|
||
Assignee | ||
Comment 232•12 years ago
|
||
Assignee | ||
Comment 234•12 years ago
|
||
Assignee | ||
Comment 235•12 years ago
|
||
Comment 236•12 years ago
|
||
Comment 237•12 years ago
|
||
Assignee | ||
Comment 238•12 years ago
|
||
Assignee | ||
Comment 239•12 years ago
|
||
Assignee | ||
Comment 240•12 years ago
|
||
Assignee | ||
Comment 241•12 years ago
|
||
Assignee | ||
Comment 242•12 years ago
|
||
Assignee | ||
Comment 243•12 years ago
|
||
Assignee | ||
Comment 244•12 years ago
|
||
Assignee | ||
Comment 245•12 years ago
|
||
Assignee | ||
Comment 246•12 years ago
|
||
Chris, I don't suppose you'd be able to try making the change you suggested in comment 160 - this has become pretty frequent on TBPL now (and currently requires manual starring, so is time consuming for sheriffs + confusing for devs using Try etc).
Cheers :-)
Flags: needinfo?(jones.chris.g)
Assignee | ||
Comment 247•12 years ago
|
||
Comment 248•12 years ago
|
||
Comment 249•12 years ago
|
||
Comment 250•12 years ago
|
||
Comment 251•12 years ago
|
||
Comment 252•12 years ago
|
||
Comment 253•12 years ago
|
||
Comment 254•12 years ago
|
||
Luckily the mechanism we need from this is coming, so hopefully in the near future.
Depends on: 821192
Flags: needinfo?(jones.chris.g)
Comment 256•12 years ago
|
||
Comment 257•12 years ago
|
||
Comment 258•12 years ago
|
||
Comment 259•12 years ago
|
||
Comment 260•12 years ago
|
||
Comment 261•12 years ago
|
||
Comment 262•12 years ago
|
||
Comment 263•12 years ago
|
||
Comment 264•12 years ago
|
||
Comment 265•12 years ago
|
||
Comment 266•12 years ago
|
||
Comment 267•12 years ago
|
||
Comment 268•12 years ago
|
||
Comment 269•12 years ago
|
||
Comment 270•12 years ago
|
||
Comment 271•12 years ago
|
||
Comment 272•12 years ago
|
||
Comment 273•12 years ago
|
||
Comment 274•12 years ago
|
||
Comment 275•12 years ago
|
||
Comment 276•12 years ago
|
||
Comment 277•12 years ago
|
||
Comment 278•12 years ago
|
||
Comment 279•12 years ago
|
||
Comment 280•12 years ago
|
||
Comment 281•12 years ago
|
||
Comment 282•12 years ago
|
||
Assignee | ||
Comment 283•12 years ago
|
||
(In reply to Chris Jones [:cjones] [:warhammer] from comment #255)
> Luckily the mechanism we need from this is coming, so hopefully in the near
> future.
Now that bug 821192 has landed, could I ask a massive favour and get you to finish off the necessary work here? :-)
Flags: needinfo?(jones.chris.g)
Comment 284•12 years ago
|
||
Comment 285•12 years ago
|
||
Comment 286•12 years ago
|
||
Comment 287•12 years ago
|
||
Comment 288•12 years ago
|
||
Comment 289•12 years ago
|
||
Bug 821192 ended up not providing the fix for this, unfortunately :/.
Flags: needinfo?(jones.chris.g)
Comment 291•12 years ago
|
||
Comment 292•12 years ago
|
||
Comment 293•12 years ago
|
||
Comment 294•12 years ago
|
||
Comment 295•12 years ago
|
||
Comment 296•12 years ago
|
||
Comment 297•12 years ago
|
||
Comment 298•12 years ago
|
||
Comment 299•12 years ago
|
||
Comment 300•12 years ago
|
||
Comment 301•12 years ago
|
||
Comment 302•12 years ago
|
||
Comment 303•12 years ago
|
||
Comment 304•12 years ago
|
||
Comment 305•12 years ago
|
||
Assignee | ||
Comment 306•12 years ago
|
||
Assignee | ||
Comment 307•12 years ago
|
||
Assignee | ||
Comment 308•12 years ago
|
||
Assignee | ||
Comment 309•12 years ago
|
||
Comment 310•12 years ago
|
||
Comment 311•12 years ago
|
||
Comment 312•12 years ago
|
||
Comment 313•12 years ago
|
||
Comment 314•12 years ago
|
||
Comment 315•12 years ago
|
||
Comment 316•12 years ago
|
||
Comment 317•12 years ago
|
||
Comment 318•12 years ago
|
||
Assignee | ||
Comment 319•12 years ago
|
||
Comment 320•12 years ago
|
||
Comment 321•12 years ago
|
||
Comment 322•12 years ago
|
||
Comment 323•12 years ago
|
||
Comment 324•12 years ago
|
||
Comment 325•12 years ago
|
||
Comment 326•12 years ago
|
||
Comment 327•12 years ago
|
||
Comment 328•12 years ago
|
||
Assignee | ||
Comment 329•12 years ago
|
||
Comment 330•12 years ago
|
||
Comment 331•12 years ago
|
||
Comment 332•12 years ago
|
||
Comment 333•12 years ago
|
||
Comment 334•12 years ago
|
||
Comment 335•12 years ago
|
||
Comment 336•12 years ago
|
||
Comment 337•12 years ago
|
||
Comment 338•12 years ago
|
||
Comment 339•12 years ago
|
||
Comment 340•12 years ago
|
||
Comment 341•12 years ago
|
||
Comment 342•12 years ago
|
||
Comment 343•12 years ago
|
||
Comment 344•12 years ago
|
||
Comment 345•12 years ago
|
||
Assignee | ||
Comment 346•12 years ago
|
||
Assignee | ||
Comment 347•12 years ago
|
||
Assignee | ||
Comment 348•12 years ago
|
||
Assignee | ||
Comment 349•12 years ago
|
||
Assignee | ||
Comment 350•12 years ago
|
||
Assignee | ||
Comment 351•12 years ago
|
||
Comment 352•12 years ago
|
||
Assignee | ||
Comment 353•12 years ago
|
||
Assignee | ||
Comment 354•12 years ago
|
||
Comment 355•12 years ago
|
||
Assignee | ||
Comment 356•12 years ago
|
||
Assignee | ||
Comment 357•12 years ago
|
||
Comment 358•12 years ago
|
||
Comment 359•12 years ago
|
||
Comment 360•12 years ago
|
||
Comment 361•12 years ago
|
||
Comment 362•12 years ago
|
||
Comment 363•12 years ago
|
||
Comment 364•12 years ago
|
||
Comment 365•12 years ago
|
||
Comment 366•12 years ago
|
||
Comment 367•12 years ago
|
||
Comment 368•12 years ago
|
||
Assignee | ||
Comment 369•12 years ago
|
||
Assignee | ||
Comment 370•12 years ago
|
||
Assignee | ||
Comment 371•12 years ago
|
||
Comment 372•12 years ago
|
||
Comment 373•12 years ago
|
||
Comment 374•12 years ago
|
||
Comment 375•12 years ago
|
||
Comment 376•12 years ago
|
||
Assignee | ||
Comment 377•12 years ago
|
||
Assignee | ||
Comment 378•12 years ago
|
||
Comment 379•12 years ago
|
||
Comment 380•12 years ago
|
||
Comment 381•12 years ago
|
||
Comment 382•12 years ago
|
||
Assignee | ||
Comment 383•12 years ago
|
||
Comment 384•12 years ago
|
||
Assignee | ||
Comment 385•12 years ago
|
||
(Sent via email to auto-tools@, posting here so everyone can see it):
On desktop TBPL unit test runs, automationutils.processSingleLeakFile()
currently checks for the presence of a leak log, and gives a
"TEST-UNEXPECTED-FAIL %s| automationutils.processLeakLog() | missing
output line for total leaks!"" if one is not found:
http://mxr.mozilla.org/mozilla-central/source/build/automationutils.py#387
As it is currently implemented, this causes several problems:
1) WhilstprocessSingleLeakFile() checks for purposeful crashes
(http://mxr.mozilla.org/mozilla-central/source/build/automationutils.py#329),
if we have an unexpected crash (eg intermittent crashes seen on TBPL)
the TBPL annotated summary output includes both the PROCESS-CRASH line
and the "missing output line for total leaks!" line. From my first
glance, there doesn't seem to be an easy way for processSingleLeakFile()
to tell if there was a PROCESS-CRASH since it won't be present in the
leak log (and the callers of processLeakLog() don't have any idea as to
crash state either).
2) Content processes are not always fully shutdown before the parent is,
causing bug 782633, which is both not-automatically starable (so time
consuming for sheriffs) & of late, pretty frequent.
My initial though was to adjust the failure string so that it would be
TBPL-starable (bug 828239), however this:
a) Doesn't resolve #1.
b) Still means we have a frequent intermittent failure bug that there
doesn't appear to be much platform interest in fixing.
As an alternative, would anyone object to me
s/TEST-UNEXPECTED-FAIL/WARNING/ so this isn't a fatal failure (at least
until bug 782633 is resolved)?
Comment 386•12 years ago
|
||
Comment 387•12 years ago
|
||
Comment 388•12 years ago
|
||
Comment 389•12 years ago
|
||
Assignee | ||
Comment 390•12 years ago
|
||
Assignee | ||
Comment 391•12 years ago
|
||
Assignee | ||
Comment 392•12 years ago
|
||
Assignee | ||
Comment 393•12 years ago
|
||
Comment 394•12 years ago
|
||
Comment 395•12 years ago
|
||
Comment 396•12 years ago
|
||
Assignee | ||
Comment 397•12 years ago
|
||
Comment 398•12 years ago
|
||
Comment 399•12 years ago
|
||
Assignee | ||
Comment 400•12 years ago
|
||
Assignee | ||
Comment 401•12 years ago
|
||
Assignee | ||
Comment 402•12 years ago
|
||
Assignee | ||
Comment 403•12 years ago
|
||
No objections to the email/bug comment, so just going to do this to stop me being driven nuts by this failure.
Assignee | ||
Comment 404•12 years ago
|
||
Note: Happy to file a followup for someone to fix this for real & fix the false positive for PROCESS-CRASHes and revert back to TEST-UNEXPECTED-FAIL when done.
Comment 405•12 years ago
|
||
Comment on attachment 701201 [details] [diff] [review]
Patch v1
Review of attachment 701201 [details] [diff] [review]:
-----------------------------------------------------------------
I guess the only thing we'd miss out on here is if a child process failed to write a leak log, right?
Attachment #701201 -
Flags: review?(ted) → review+
Comment 406•12 years ago
|
||
Comment 407•12 years ago
|
||
Assignee | ||
Comment 408•12 years ago
|
||
Target Milestone: --- → mozilla21
Comment 409•12 years ago
|
||
Comment 410•12 years ago
|
||
Comment 411•12 years ago
|
||
Comment 412•12 years ago
|
||
https://tbpl.mozilla.org/php/getParsedLog.php?id=18746404&tree=Mozilla-Inbound
Traceback (most recent call last):
File "/home/cltbld/talos-slave/test/build/mochitest/runtests.py", line 960, in <module>
main()
File "/home/cltbld/talos-slave/test/build/mochitest/runtests.py", line 957, in main
sys.exit(mochitest.runTests(options))
File "/home/cltbld/talos-slave/test/build/mochitest/runtests.py", line 751, in runTests
processLeakLog(self.leak_report_file, options.leakThreshold)
File "/home/cltbld/talos-slave/test/build/mochitest/automationutils.py", line 419, in processLeakLog
processSingleLeakFile(thisFile, processPID, processType, leakThreshold)
File "/home/cltbld/talos-slave/test/build/mochitest/automationutils.py", line 388, in processSingleLeakFile
processString)
TypeError: not all arguments converted during string formatting
Assignee | ||
Comment 413•12 years ago
|
||
Bah I'm an idiot. Followup to fix:
https://tbpl.mozilla.org/php/getParsedLog.php?id=18746404&tree=Mozilla-Inbound
{
Traceback (most recent call last):
File "/home/cltbld/talos-slave/test/build/mochitest/runtests.py", line 960, in <module>
main()
File "/home/cltbld/talos-slave/test/build/mochitest/runtests.py", line 957, in main
sys.exit(mochitest.runTests(options))
File "/home/cltbld/talos-slave/test/build/mochitest/runtests.py", line 751, in runTests
processLeakLog(self.leak_report_file, options.leakThreshold)
File "/home/cltbld/talos-slave/test/build/mochitest/automationutils.py", line 419, in processLeakLog
processSingleLeakFile(thisFile, processPID, processType, leakThreshold)
File "/home/cltbld/talos-slave/test/build/mochitest/automationutils.py", line 388, in processSingleLeakFile
processString)
TypeError: not all arguments converted during string formatting
}
https://hg.mozilla.org/integration/mozilla-inbound/rev/d8599591d07c
Comment 414•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/c4db97d21a09
https://hg.mozilla.org/mozilla-central/rev/d8599591d07c
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Comment 415•12 years ago
|
||
Comment 416•12 years ago
|
||
Comment 417•12 years ago
|
||
Comment 418•12 years ago
|
||
Comment 419•12 years ago
|
||
Comment 420•12 years ago
|
||
Comment 421•12 years ago
|
||
Comment 422•12 years ago
|
||
Comment 423•12 years ago
|
||
Comment 424•12 years ago
|
||
Comment 425•12 years ago
|
||
Assignee | ||
Comment 426•12 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/2b6825a8e723
https://hg.mozilla.org/releases/mozilla-beta/rev/6b6ad5721d02
https://hg.mozilla.org/releases/mozilla-esr17/rev/b1a7fae2cf18
https://hg.mozilla.org/releases/mozilla-b2g18/rev/67503cae6a5a
You need to log in
before you can comment on or make changes to this bug.
Description
•