Open Bug 1268247 Opened 4 years ago Updated 5 months ago

Alert management UI doesn't really scale to uplifts

Categories

(Tree Management :: Perfherder, enhancement, P3)

enhancement

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.
Priority: -- → P3
Type: defect → enhancement

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

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.

Flags: needinfo?(marian.raiciof)
Flags: needinfo?(fstrugariu)
Flags: needinfo?(aionescu)

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

Flags: needinfo?(fstrugariu)

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

Flags: needinfo?(aionescu)
Flags: needinfo?(marian.raiciof)
You need to log in before you can comment on or make changes to this bug.