Closed
Bug 909799
Opened 11 years ago
Closed 11 years ago
Add ability to Ouija slaves graph to display total jobs ran, not just failures
Categories
(Testing :: General, defect)
Testing
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jmaher, Unassigned)
References
Details
(Whiteboard: [good first bug][mentor=dminor][lang=python])
Attachments
(1 file)
(deleted),
patch
|
dminor
:
review+
|
Details | Diff | Splinter Review |
currently the slaves graph shows failures, retries, total (failures+retries). I would like to include passes, and infra as well, then the total would include all of the jobs types.
I might got it all wrong, but that kind of report will not be readable.
Selecting distinct slave names from ouija database where result is "success" and testtype is not a blank string returns more than 1000 rows.
Perhaps, that should be done with slave type (as in bug 909798)?
Reporter | ||
Comment 2•11 years ago
|
||
This should be a single value, not an entire list. The ultimate goal here is to figure out which slaves are failing an abnormal amount so we can pull them from production and reimage them. While we can tell that based on total failures, it also helps to know if we end up with a small number of machines running thousands of jobs with <1% failure rate even though those total number of failures might look high compared to other slaves running much fewer jobs.
(In reply to Joel Maher (:jmaher) from comment #2)
> This should be a single value, not an entire list. The ultimate goal here
> is to figure out which slaves are failing an abnormal amount so we can pull
> them from production and reimage them. While we can tell that based on
> total failures, it also helps to know if we end up with a small number of
> machines running thousands of jobs with <1% failure rate even though those
> total number of failures might look high compared to other slaves running
> much fewer jobs.
So, for every slave listed on slaves page, I need to add additional column that contains "Passes" (this should be the first column after slave name?) and include value of this column for total runs calculation? I don't need to show slaves that have only "success" results?
Reporter | ||
Comment 4•11 years ago
|
||
It should be the last column in the list of non calculated data, as it is useful but not the most critical.
If a slave has no failures, it would be nice to show them as well, a big bonus would be a checkbox that could hide/show non failing slaves.
Attachment #806723 -
Flags: review?(dminor)
Comment 6•11 years ago
|
||
Comment on attachment 806723 [details] [diff] [review]
0011-add-new-column.patch
Review of attachment 806723 [details] [diff] [review]:
-----------------------------------------------------------------
Looks good, thanks!
Attachment #806723 -
Flags: review?(dminor) → review+
Comment 7•11 years ago
|
||
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•