Give Treeherder URLs in Try responses / emails.

RESOLVED FIXED

Status

Release Engineering
General
P1
normal
RESOLVED FIXED
3 years ago
4 months ago

People

(Reporter: jgraham, Assigned: emorley)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [treeherder])

Attachments

(2 attachments)

(Reporter)

Description

3 years ago
Since we are now at the stage where dogfooding Treeherder seems to be desirable, it would be nice to start putting those urls in reponses and emails for Try pushes, initially marked as experimental.
(Assignee)

Comment 1

3 years ago
Need changes in two places:

http://mxr.mozilla.org/build-central/source/buildbotcustom/status/generators.py#22
21 Results will be displayed on TBPL as they come in:
22 https://tbpl.mozilla.org/?tree=%(tree)s&rev=%(revision)s

http://mxr.mozilla.org/build-central/source/buildbotcustom/bin/try_mailer.py#129
129     text += "For an overview of all results see <a href=\"https://tbpl.mozilla.org/?tree=%(tree)s&rev=%(revision)s\">TBPL</a>.\n" % locals()
Assignee: nobody → emorley
Status: NEW → ASSIGNED
OS: Linux → All
Hardware: x86_64 → All
Whiteboard: [treeherder]
(Assignee)

Comment 2

3 years ago
Created attachment 8462621 [details] [diff] [review]
Provide treeherder URLs alongside TBPL ones
Attachment #8462621 - Flags: review?(nthomas)
(Assignee)

Updated

3 years ago
Blocks: 1030636

Updated

3 years ago
Attachment #8462621 - Flags: review?(nthomas) → review+
(Assignee)

Comment 3

3 years ago
Thank you :-)

remote:   https://hg.mozilla.org/build/buildbotcustom/rev/d4ec48d7d255
something here is now in production
(Assignee)

Comment 5

3 years ago
(Bug 1044995 is for the hghook response)
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
(Assignee)

Updated

3 years ago
Priority: -- → P1
(Assignee)

Updated

3 years ago
Depends on: 1046225
(Assignee)

Comment 6

3 years ago
The URLs being generated use uppercase repo names, and treeherder doesn't like that (bug 1046225). I initially thought we should just change treeherder, but the Proper Case names are actually just hardcoded in the try mailer files, and not something that comes from buildbot. So we just just fix this.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Assignee)

Comment 7

3 years ago
Created attachment 8465363 [details] [diff] [review]
Use lowercase repo names for Treeherder URLs

This gives treeherder it's lowercase repo names, and once TBPL is EOL we can clean these files up a bit.
Attachment #8465363 - Flags: review?(nthomas)
(Assignee)

Updated

3 years ago
No longer depends on: 1046225

Updated

3 years ago
Attachment #8465363 - Flags: review?(nthomas) → review+
(Assignee)

Comment 8

3 years ago
Ty :-)

remote:   https://hg.mozilla.org/build/buildbotcustom/rev/a9df69af8fdd
something(s) here went to production today
(Assignee)

Updated

3 years ago
Status: REOPENED → RESOLVED
Last Resolved: 3 years ago3 years ago
Resolution: --- → FIXED
(Assignee)

Updated

3 years ago
Blocks: 1072159
(Assignee)

Comment 10

3 years ago
Updated to shorter URL form (without the /ui/), now that bug 1063411 is fixed:
remote:   https://hg.mozilla.org/build/buildbotcustom/rev/713f1073fb8c
In production: https://hg.mozilla.org/build/buildbotcustom/rev/713f1073fb8c
Component: Tools → General
Product: Release Engineering → Release Engineering
You need to log in before you can comment on or make changes to this bug.