oops reports show negative total time
Bug #632022 reported by
Robert Collins
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
Critical
|
Robert Collins | ||
Bug Description
SQL time: 1009611 ms
Non-sql time: -996603 ms
Total time: 13008 ms
Statement Count: 961
This is being caused by the 999999 sentinel in incomplete actions; changing this to treat the action as finishing when the log tuple is gathered will correct it.
Related branches
lp:~lifeless/launchpad/oops
- Michael Hudson-Doyle: Approve
-
Diff: 43 lines (+13/-2)2 files modifiedlib/canonical/launchpad/webapp/adapter.py (+10/-2)
lib/lp/services/timeline/timedaction.py (+3/-0)
- Jeroen T. Vermeulen (community): Approve (code)
- Tim Penhey (community): Approve (release-critical)
- Graham Binns: Pending (release-critical) requested
-
Diff: 407 lines (+127/-31)11 files modifiedlib/canonical/launchpad/doc/librarian.txt (+19/-0)
lib/canonical/launchpad/webapp/errorlog.py (+5/-1)
lib/canonical/librarian/client.py (+26/-7)
lib/canonical/librarian/ftests/test_client.py (+3/-2)
lib/lp/services/mail/sendmail.py (+20/-6)
lib/lp/services/mail/tests/test_sendmail.py (+0/-4)
lib/lp/services/memcache/client.py (+26/-1)
lib/lp/services/memcache/tests/test_memcache_client.py (+19/-6)
lib/lp/services/timeline/tests/test_timedaction.py (+4/-2)
lib/lp/services/timeline/timedaction.py (+1/-1)
lib/lp/testing/tests/test_testcase.py (+4/-1)
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote Bug fixed by a commit | #1 |
| Changed in launchpad-foundations: | |
| assignee: | nobody → Robert Collins (lifeless) |
| milestone: | none → 10.09 |
| tags: | added: qa-needstesting |
| Changed in launchpad-foundations: | |
| status: | New → Fix Committed |
| Changed in launchpad-foundations: | |
| importance: | Undecided → Critical |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in launchpad-foundations: | |
| status: | Fix Committed → Fix Released |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #2 |
| Changed in launchpad-foundations: | |
| milestone: | 10.09 → 10.10 |
| tags: |
added: qa-needstesting removed: qa-ok |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #4 |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #5 |
| tags: |
added: qa-untestable removed: qa-needstesting |
| tags: |
added: qa-ok removed: qa-untestable |
To post a comment you must log in.
Fixed in stable r11514 <http://