[backend] Alert Manager - add support for new test cold load time

RESOLVED WONTFIX

Status

Testing
General
RESOLVED WONTFIX
4 years ago
3 years ago

People

(Reporter: jmaher, Unassigned, Mentored)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [good first bug][lang=python])

Alert Manager provides us with a UI to manage the talos performance regression alerts generated by our graph server.  There are always improvements needed, and little details to polish up.

This bug is about support for a new test, specifically "TS Paint, Cold Load Time".  Our parser searches for known keywords and will only understand alerts with known tests.

We need to add two entries:
https://github.com/jmaher/alert_manager/blob/master/managed_settings.py#L59

and:
https://github.com/jmaher/alert_manager/blob/master/managed_settings.py#L116

The official name is "TS Paint, Cold Load Time" and it maps to the tbpl job name of 'xperf' (example xperf job is here: https://github.com/jmaher/alert_manager/blob/master/managed_settings.py#L77)
(In reply to Joel Maher (:jmaher) from comment #0)
> Alert Manager provides us with a UI to manage the talos performance
> regression alerts generated by our graph server.  There are always
> improvements needed, and little details to polish up.
> 
> This bug is about support for a new test, specifically "TS Paint, Cold Load
> Time".  Our parser searches for known keywords and will only understand
> alerts with known tests.
> 
> We need to add two entries:
> https://github.com/jmaher/alert_manager/blob/master/managed_settings.py#L59
> 
> and:
> https://github.com/jmaher/alert_manager/blob/master/managed_settings.py#L116
> 
> The official name is "TS Paint, Cold Load Time" and it maps to the tbpl job
> name of 'xperf' (example xperf job is here:
> https://github.com/jmaher/alert_manager/blob/master/managed_settings.py#L77)

Hi :jmaher,
I would like to work on this bug. But this is my first bug to solve and I am very new in this world of open source. Can you guide me exactly what steps are to be followed here.

thanks
Hi Guatam, this is a simple bug - we made a decision 2 days ago to disable this test from running, so lets pick a different bug, would bug 1033634 be of interest to you?  To get started clone alert manager from github: https://github.com/jmaher/alert_manager/, follow the instructions to get setup, and then ideally we can work through any parts of the code or bug request which don't make sense.

One thing - no need to be shy.  Ask questions in the bugs, on irc (I am :jmaher in #ateam), or send me a direct email.
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.