Closed Bug 1091541 Opened 10 years ago Closed 8 years ago

[backend] Alert Manager - needs to fix treeherder links for android

Categories

(Testing :: General, defect)

Product:
Testing
Component:
General
Platform:
ARM
Android
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED WONTFIX

People

(Reporter: jmaher, Unassigned, Mentored)

References

Details

(Whiteboard: [good next 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 the links we generate to view the jobs which were run on tbpl.  We craft a link that has revisions, platform, test so that we can view the builder which ran the job for a set of revisions.

Here is what we get for android right now:
https://tbpl.mozilla.org/?tree=Fx-Team&fromchange=cf419403a2eb&tochange=8546940128b3&jobname=Android%204.0%20Tegra%20fx-team%20pgo%20pgo%20talos%20other

what is of concern is:
jobname=Android 4.0 Tegra fx-team pgo pgo talos other

what we need to have it say is:
jobname=Android 4.0 Panda fx-team talos remote-tspaint

the changes needed are:
1) Platform name from "Android 4.0 Tegra" -> "Android 4.0 Panda" (https://github.com/jmaher/alert_manager/blob/master/managed_settings.py#L37)

2) Test name from "other" -> "remote-tspaint".  this is trickier as we receive an internal test name of "Ts, Paint" (https://github.com/jmaher/alert_manager/blob/master/managed_settings.py#L89).  On all platforms this maps to "other", but on Android, it needs to be "remote-tspaint".
Mentor: jmaher, dminor
Hi Joel,

I'd like to work on this bug. I'm new to Alert Manager so it might take me some time to understand the code first. I'll post any questions that I have here.

Thanks,
Sherry
wonderful!  Do ask questions when you have them in the bug here or on IRC.  I am jmaher in #ateam on irc.mozilla.org.
Hi Joel,

I'm afraid I won't have time to work on this bug until after New Year's. Sorry about that!
no worries Sherry.  If this isn't fixed before then, I will be happy to help you get to a completed pull request!
Summary: [backend] Alert Manager - needs to fix tbpl links for android → [backend] Alert Manager - needs to fix treeherder links for android
alert manager is has been replaced by perfherder
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.