1268247 - Alert management UI doesn't really scale to uplifts

Status: RESOLVED DUPLICATE of bug 1725419

(Tree Management :: Perfherder, enhancement, P3)

Description

[William Lachance (:wlach)]

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.

Comment 1

[Ionuț Goldan [:igoldan]]

(In reply to William Lachance (:wlach) (unavailable Dec 7 -> 14 2019) (use needinfo!) from comment #0)

[...]

1. 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

[Ionuț Goldan [:igoldan]]

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

[Florin Strugariu [:Bebe]]

We could make the alerts have a max height and have a expand/collapse feature to show/hide hidden data.

Comment 4

[Alexandru Ionescu (needinfo me) [:alexandrui]]

(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.