use raw link to revisions instead of a revision number in the talos regression template

RESOLVED FIXED

Status

Tree Management
Perfherder
RESOLVED FIXED
a year ago
a year ago

People

(Reporter: jmaher, Assigned: ShrutiJ)

Tracking

Details

Attachments

(1 attachment)

(Reporter)

Description

a year ago
When filing a talos regression, we have a template which generates a bug for us.  This is great, but a complain today in Bug 1352085 seems like an easy fix:

(In reply to igoldan from comment #0)
> Talos has detected a Firefox performance regression from push
> 76ad1c764e5c626b20b87099fe9b822e21dc23e9.

Can you give an actual link to hg.m.o in future? If one person does it, it
saves everyone else having to manually copy, figure out the link, paste and go.


in this case it would be:
https://hg.mozilla.org/mozilla-central/rev/76ad1c764e5c626b20b87099fe9b822e21dc23e9
Shruti, do you want to take this one too? (whenever you have time)

Treeherder's repository object has a function we can use to generate this: getPushLogHref

If you run:

git grep getPushLog

You'll see a bunch of examples of us using it (including several in related code in Perfherder). So I think all we need to do is modify the fixtures here to replace "revision" with "revisionHref":

treeherder/perf/fixtures/performance_bug_templates.yaml

And then pass the argument in from the perfherder alerts code (I'll let you find it)
Assignee: nobody → shrutijasoria1996
(Assignee)

Comment 2

a year ago
I'll take this one too.
Created attachment 8857336 [details] [review]
[treeherder] SJasoria:Bug_1352398 > mozilla:master
(Assignee)

Comment 4

a year ago
Comment on attachment 8857336 [details] [review]
[treeherder] SJasoria:Bug_1352398 > mozilla:master

I hope the changes which I've made are good.
Attachment #8857336 - Flags: review?(wlachance)
Comment on attachment 8857336 [details] [review]
[treeherder] SJasoria:Bug_1352398 > mozilla:master

Could you make one small adjustment? Sorry for not mentioning it earlier.
Attachment #8857336 - Flags: review?(wlachance)

Comment 6

a year ago
Commit pushed to master at https://github.com/mozilla/treeherder

https://github.com/mozilla/treeherder/commit/ec967845072493ad67ded0151ba2a50188e1589e
Bug 1352398 - use raw link to revisions instead of a revision number in the talos regression template (#2337)
Status: NEW → RESOLVED
Last Resolved: a year ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.