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

Status: RESOLVED WONTFIX

(Testing :: General, defect)

Description

[Joel Maher ( :jmaher ) (UTC -8)]

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)

Comment 1

[Gautam Anghore (:matuag)]

(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

Comment 2

[Joel Maher ( :jmaher ) (UTC -8)]

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.