Last Comment Bug 615045 - [SeaMonkey] check: TestTXMgr.cpp (nsITransactionManager tests) intermittently times out in "Aggregate Transaction Stress Test"
: [SeaMonkey] check: TestTXMgr.cpp (nsITransactionManager tests) intermittently...
Status: RESOLVED FIXED
[test which aborts the suite]
: intermittent-failure
Product: Core
Classification: Components
Component: Editor (show other bugs)
: Trunk
: All All
: -- major (vote)
: mozilla2.0b8
Assigned To: Serge Gautherie (:sgautherie)
:
Mentors:
http://mxr.mozilla.org/mozilla-centra...
Depends on:
Blocks: SmTestFail 617967 438871 618695
  Show dependency treegraph
 
Reported: 2010-11-27 04:19 PST by Serge Gautherie (:sgautherie)
Modified: 2012-11-25 19:31 PST (History)
1 user (show)
bugzillamozillaorg_serge_20140323: in‑testsuite+
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---
.14-fixed
.17-fixed


Attachments
(Av1) Add a trivial feedback not to let the user think the test is stuck [Checked in: Comment 4 & 5] (1.60 KB, patch)
2010-12-09 08:53 PST, Serge Gautherie (:sgautherie)
ehsan: review+
Details | Diff | Splinter Review

Description Serge Gautherie (:sgautherie) 2010-11-27 04:19:08 PST
I assume "Aggregate Transaction Stress Test" may just become too slow when the builder is overloaded:
I assume we could add a 'printf()' to get some output.

While there, we should fix the warnings too.


http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1290815606.1290821648.13026.gz&fulltext=1
Linux comm-central-trunk leak test build on 2010/11/26 15:53:26
{
...
WARNING: NS_ENSURE_TRUE(aTransaction) failed: file /builds/slave/comm-central-trunk-linux-debug/build/mozilla/editor/txmgr/src/nsTransactionManager.cpp, line 100
...
WARNING: NS_ENSURE_TRUE(aListener) failed: file /builds/slave/comm-central-trunk-linux-debug/build/mozilla/editor/txmgr/src/nsTransactionManager.cpp, line 632
...
WARNING: NS_ENSURE_TRUE(aListener) failed: file /builds/slave/comm-central-trunk-linux-debug/build/mozilla/editor/txmgr/src/nsTransactionManager.cpp, line 646
...
WARNING: NS_ENSURE_SUCCESS(result, result) failed with result 0x80004005: file /builds/slave/comm-central-trunk-linux-debug/build/mozilla/editor/txmgr/src/nsTransactionItem.cpp, line 318
...
WARNING: NS_ENSURE_SUCCESS(result, result) failed with result 0x80004005: file /builds/slave/comm-central-trunk-linux-debug/build/mozilla/editor/txmgr/src/nsTransactionItem.cpp, line 318
...
WARNING: NS_ENSURE_SUCCESS(result, result) failed with result 0x80004005: file /builds/slave/comm-central-trunk-linux-debug/build/mozilla/editor/txmgr/src/nsTransactionItem.cpp, line 318
...
- Aggregate Transaction Stress Test:
-----------------------------------------------------
Stress test (may take a while) ... 
command timed out: 300 seconds without output, killing pid 3092
}


http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1290818580.1290827447.2055.gz&fulltext=1
WINNT 5.2 comm-central-trunk leak test build on 2010/11/26 16:43:00
{
...
- Begin Batch Transaction Test:
...
Execute 20 batched transient transaWARNING: NS_ENSURE_SUCCESS(result, result) failed with result 0x80004005: file e:/builds/slave/comm-central-trunk-win32-debug/build/mozilla/editor/txmgr/src/nsTransactionItem.cpp, line 318
ctions ... passed
...
- Simple Transaction Stress Test:
-----------------------------------------------------
Stress test (may take a while) ... WARNING: NS_ENSURE_SUCCESS(result, result) failed with result 0x80004005: file e:/builds/slave/comm-central-trunk-win32-debug/build/mozilla/editor/txmgr/src/nsTransactionItem.cpp, line 318
passed
...
- Aggregate Transaction Stress Test:
-----------------------------------------------------
Stress test (may take a while) ... 
command timed out: 300 seconds without output
program finished with exit code 1
}


Expected result is
{
Stress test (may take a while) ... passed
2630250 transactions processed during stress test.
}
Comment 1 Serge Gautherie (:sgautherie) 2010-11-27 04:42:26 PST
Previous failures:

...

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1290442150.1290446556.2253.gz
Linux comm-central-trunk leak test build on 2010/11/22 08:09:10

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1290465478.1290470627.15131.gz
Linux comm-central-trunk leak test build on 2010/11/22 14:37:58

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1290473133.1290487271.25446.gz
WINNT 5.2 comm-central-trunk leak test build on 2010/11/22 16:45:33

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1290698523.1290709813.1983.gz
Linux comm-central-trunk leak test build on 2010/11/25 07:22:03
Comment 2 Serge Gautherie (:sgautherie) 2010-12-04 06:28:28 PST
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1291451566.1291466093.32533.gz
WINNT 5.2 comm-central-trunk leak test build on 2010/12/04 00:32:46
{
command timed out: 300 seconds without output
program finished with exit code 1
elapsedTime=1091.938000
}
Comment 3 Serge Gautherie (:sgautherie) 2010-12-09 08:53:57 PST
Created attachment 496506 [details] [diff] [review]
(Av1) Add a trivial feedback not to let the user think the test is stuck
[Checked in: Comment 4 & 5]
Comment 4 Serge Gautherie (:sgautherie) 2010-12-09 17:02:09 PST
Comment on attachment 496506 [details] [diff] [review]
(Av1) Add a trivial feedback not to let the user think the test is stuck
[Checked in: Comment 4 & 5]

http://hg.mozilla.org/mozilla-central/rev/2abc8100631e
Comment 5 Serge Gautherie (:sgautherie) 2010-12-09 18:20:55 PST
Comment on attachment 496506 [details] [diff] [review]
(Av1) Add a trivial feedback not to let the user think the test is stuck
[Checked in: Comment 4 & 5]

http://hg.mozilla.org/releases/mozilla-1.9.2/rev/ae9e562f8237

http://hg.mozilla.org/releases/mozilla-1.9.1/rev/464ca571e076
Comment 6 Serge Gautherie (:sgautherie) 2010-12-12 08:23:50 PST
(In reply to comment #0)
> While there, we should fix the warnings too.

I filed bug 617967.

Note You need to log in before you can comment on or make changes to this bug.