Closed Bug 1499319 Opened 6 years ago Closed 4 years ago

Allow for setting states from inside Perfherder popup

Categories

(Tree Management :: Perfherder, enhancement, P3)

enhancement

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: igoldan, Unassigned)

Details

Attachments

(1 file)

By states we mean that it would be nice to add in "infra", "test change" - then we could be tricky with the graphs and ignore infra/test changes to show the real browser performance over time.
Similar to what the devtools team does in their graphs.
Attachment #9018178 - Attachment is patch: true
Attachment #9018178 - Attachment mime type: text/x-github-pull-request → text/plain
Attachment #9018178 - Flags: feedback?(jmaher)
Attachment #9018178 - Flags: feedback?(jmaher)
Given the upcoming work for switching Perfherder to React, this bug has a lower priority.
Priority: P1 → P2
From yesterday's discussion with :wlach, we concluded to avoid using the crosshair approach for selecting downstreams or reassigned alerts. Instead, we will employ popups very similar to those we use in the alert summary view.
Also, to avoid cluttering the black popups (when you click on data point), we will use a single carret (either right or left) that will expand it with the "Mark as" buttons.
Assignee: igoldan → nobody
Priority: P2 → P3

I'm not sure this is of much use, with the exception of marking an alert as invalid. Closing this for now; if any Perf sheriff has other opinions, we can reopen this.

Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: