The default bug view has changed. See this FAQ.

Give Treeherder URLs in Try responses / emails.

RESOLVED FIXED

Status

Release Engineering
Tools
P1
normal
RESOLVED FIXED
3 years ago
2 years 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.
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]
Created attachment 8462621 [details] [diff] [review]
Provide treeherder URLs alongside TBPL ones
Attachment #8462621 - Flags: review?(nthomas)
Blocks: 1030636

Updated

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

remote:   https://hg.mozilla.org/build/buildbotcustom/rev/d4ec48d7d255
something here is now in production
(Bug 1044995 is for the hghook response)
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Priority: -- → P1
Depends on: 1046225
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 → ---
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)
No longer depends on: 1046225

Updated

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

remote:   https://hg.mozilla.org/build/buildbotcustom/rev/a9df69af8fdd
something(s) here went to production today
Status: REOPENED → RESOLVED
Last Resolved: 3 years ago3 years ago
Resolution: --- → FIXED
Blocks: 1072159
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
You need to log in before you can comment on or make changes to this bug.