1536018 - Alert summary's status doesn't properly update

Status: RESOLVED INVALID

(Tree Management :: Perfherder, defect, P3)

Description

[Ionuț Goldan [:igoldan]]

Steps to reproduce:

1. Go to https://treeherder.mozilla.org/perf.html#/alerts?id=19785
2. Make sure alerts are in untriaged state. Reset them if not.
3. Mark them as confirming.
4. Then mark them as acknowledged.

Expected:
Alert summary is in improvement state.
What we actually get:
Alert summary remains in confirming state.

Comment 1

[Sarah Clements [:sclements]]

This will get fixed during the conversion.

Comment 2

[Sarah Clements [:sclements]]

I think this might be a backend logic issue rather than a UI issue. After following the steps you listed, the alertSummary status goes from "untriaged" to "confirming" and then to "investigating".

Comment 3

[Sarah Clements [:sclements]]

Is this still a problem Ionut?

Comment 4

[Ionuț Goldan [:igoldan]]

(In reply to Sarah Clements [:sclements] - PTO Aug 19 - Sept 3 from comment #3)

Is this still a problem Ionut?

You are right about this. Those precise STR no longer reproduce.
But there's still a problem. Please follow this new STR.

Steps to reproduce:

1. Go to https://treeherder.mozilla.org/perf.html#/alerts?id=19785
2. Make sure alerts are in untriaged state. Reset them if not.
3. Mark them as confirming.

Expected:
Alert summary is in confirming state.
What we actually get:
Alert summary remains in untriaged state.

Comment 5

[Sarah Clements [:sclements]]

Ionut, it seems these changes to the "confirming" status will affect this bug. Should this be closed?

Comment 6

[Ionuț Goldan [:igoldan]]

(In reply to Sarah Clements [:sclements] from comment #5)

Ionut, it seems these changes to the "confirming" status will affect this bug. Should this be closed?

You are right. The plan for the PR you posted should fix this problem.

Comment 8

[Ionuț Goldan [:igoldan]]

Marking this INVALID, as we removed the confirming state from Perfherder.