Alert management UI doesn't really scale to uplifts
Categories
(Tree Management :: Perfherder, enhancement, P3)
Tracking
(Not tracked)
People
(Reporter: wlach, Unassigned)
Details
The alerts UI mostly works well, but it doesn't really scale to the large quantities of alerts we see in uplifts, e.g.: https://treeherder.mozilla.org/perf.html#/alerts?id=1022 It's difficult to know exactly what to do here, as the premise of the UI is that you can triage groups of alerts together while referring from one to another. Ideas: 1. Provide another filter at the top to hide triaged alerts from a summary (super easy) 2. Paginate each list of alerts somehow (hard to get a good ui here going, I suspect) 3. ?? As triaging alerts during uplift only happens occasionally, I am tempted to just fix (1) for now.
Updated•6 years ago
|
Updated•4 years ago
|
Comment 1•4 years ago
|
||
(In reply to William Lachance (:wlach) (unavailable Dec 7 -> 14 2019) (use needinfo!) from comment #0)
[...]
- Provide another filter at the top to hide triaged alerts from a summary
(super easy)
[...]As triaging alerts during uplift only happens occasionally, I am tempted to
just fix (1) for now.
I believe (1) has been addressed for some time now, as we have the Hide downstream/ reassigned to / invalid checkbox, at the top of the Alerts view.
Comment 2•4 years ago
|
||
Do you have any suggestions for how we can improve the looks of alert summaries with many alerts? Otherwise, we can close this bug as FIXED.
Comment 3•4 years ago
|
||
We could make the alerts have a max height and have a expand/collapse feature to show/hide hidden data.
Comment 4•4 years ago
|
||
(In reply to Florin Strugariu [:Bebe] (needinfo me) from comment #3)
We could make the alerts have a max height and have a expand/collapse feature to show/hide hidden data.
+1
I would make a max height that would show only the alert's items and after expanding show also the reassigned ones. Offcourse, if the reassigned fit in the collapsed view, I would include there.
Also, I would fix the select all
checkbox. I would make it check only the untriaged items on first click, acknowledged ones (if any) on second click, reassigned to/reassigned from on third click. Or just a dropdown to choose on of those variants, along with the default of uncheck all
.
Third proposal would be to filter the alert summary. When opening a bug, there's the limit of 250 character or so. I would make the summary less redundant, like from:
0.28 - 0.4% Base Content JS (linux64, linux64-qr, linux64-shippable, linux64-shippable-qr, macosx1014-64-shippable, windows7-32)
to
0.28 - 0.4% Base Content JS (linux64|-qr, linux64-shippable|-qr, macosx1014-64-shippable, windows7-32)
or
0.28 - 0.4% Base Content JS (linux64[-shippable]|-qr, macosx1014-64-shippable, windows7-32)
or variants. Just to use a pattern that would match a set of plaform names.
Updated•4 years ago
|
Updated•10 months ago
|
Description
•