Closed Bug 1678760 Opened 6 months ago Closed 4 months ago

Status Progress failure count differs from other indicators (icons and tabs)

Categories

(Tree Management :: Push Health, task)

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: sclements, Assigned: sclements)

References

(Blocks 1 open bug)

Details

Attachments

(4 files)

This came up during a user review and caused confusion, especially in cases where all of the build/lint/test icons indicate everything has passed but the status progress pie chart suggests there are failures (per the screenshot). It could be that in some cases there is an error summary log missing so we should find a meaningful way to convey those errors and have those results match up.

However, this test has an error summary log, but we are either not parsing it or something else is happening: https://treeherder.mozilla.org/jobs?revision=a8462facf3d844e361cbff3b30d65908316562f0&repo=try&selectedTaskRun=eD8Vrwm9Q_iEgmnNHIFTUg.0

Two things that came up during an informal demo to a dev that should be fixed (see screenshots below):

  1. If someone is looking at an old push in jobs view (that would include pushes beyond our 4 month retention date), the spinner will keep spinning.
  2. If someone looked at that old push in My Pushes (because its in the last n of their pushes) or in push-health/push?revision=<sha> the StatusProgress will show 0% complete and the spinner keeps spinning for the base commit. We should set a default of 100% for anything past 4 months.
Assignee: nobody → sclements
Status: NEW → ASSIGNED
Status: ASSIGNED → RESOLVED
Closed: 4 months ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.