report-database-stats.py gives bogus cpu utilization
Bug #590684 reported by
Stuart Bishop
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
High
|
Stuart Bishop | ||
Bug Description
The database stats report just does a simple average over the data to calculate cpu utilization per user. Unfortunately, when a script is not running at all no data is stored for that user so the times when a script was not running at all are not counted.
The report needs to calculate the average using the number of samples taken in the time frame, not the number of rows of data for the given user.
Related branches
lp://qastaging/~stub/launchpad/db-stats
- Stuart Bishop (community): Approve
-
Diff: 50 lines (+7/-5)1 file modifiedutilities/report-database-stats.py (+7/-5)
- Robert Collins (community): Approve
- Jelmer Vernooij (community): Abstain
-
Diff: 175 lines (+78/-22)2 files modifiedlib/lp/scripts/helpers.py (+2/-1)
utilities/report-database-stats.py (+76/-21)
| Changed in launchpad-foundations: | |
| status: | New → Triaged |
| importance: | Undecided → High |
| assignee: | nobody → Stuart Bishop (stub) |
Revision history for this message
| Ursula Junque (ursinha) wrote Bug fixed by a commit | #1 |
| Changed in launchpad-foundations: | |
| milestone: | none → 10.06 |
| status: | Triaged → Fix Committed |
| tags: | added: qa-needstesting |
Revision history for this message
| Ursula Junque (ursinha) wrote | #2 |
| 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 | #3 |
| Changed in launchpad-foundations: | |
| milestone: | 10.07 → 10.08 |
| tags: |
added: qa-needstesting removed: qa-ok |
| tags: |
added: qa-ok removed: qa-needstesting |
To post a comment you must log in.
Fixed in db-stable r9433 <http://