Closed
Bug 1043880
Opened 10 years ago
Closed 10 years ago
Give Treeherder URLs in Try responses / emails.
Categories
(Release Engineering :: General, defect, P1)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jgraham, Assigned: emorley)
References
Details
(Whiteboard: [treeherder])
Attachments
(2 files)
2.01 KB,
patch
|
nthomas
:
review+
|
Details | Diff | Splinter Review |
3.21 KB,
patch
|
nthomas
:
review+
|
Details | Diff | Splinter Review |
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•10 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•10 years ago
|
||
Attachment #8462621 -
Flags: review?(nthomas)
Assignee | ||
Updated•10 years ago
|
Blocks: treeherder-sheriff-transition
Updated•10 years ago
|
Attachment #8462621 -
Flags: review?(nthomas) → review+
Assignee | ||
Comment 3•10 years ago
|
||
Thank you :-) remote: https://hg.mozilla.org/build/buildbotcustom/rev/d4ec48d7d255
Comment 4•10 years ago
|
||
something here is now in production
Assignee | ||
Comment 5•10 years ago
|
||
(Bug 1044995 is for the hghook response)
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•10 years ago
|
Priority: -- → P1
Assignee | ||
Comment 6•10 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•10 years ago
|
||
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)
Updated•10 years ago
|
Attachment #8465363 -
Flags: review?(nthomas) → review+
Assignee | ||
Comment 8•10 years ago
|
||
Ty :-) remote: https://hg.mozilla.org/build/buildbotcustom/rev/a9df69af8fdd
Comment 9•10 years ago
|
||
something(s) here went to production today
Assignee | ||
Updated•10 years ago
|
Status: REOPENED → RESOLVED
Closed: 10 years ago → 10 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 10•10 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
Comment 11•10 years ago
|
||
In production: https://hg.mozilla.org/build/buildbotcustom/rev/713f1073fb8c
Updated•7 years ago
|
Component: Tools → General
You need to log in
before you can comment on or make changes to this bug.
Description
•