Closed
Bug 617755
Opened 14 years ago
Closed 14 years ago
Notify orange factor db when bugs are starred
Categories
(Tree Management Graveyard :: TBPL, defect)
Tree Management Graveyard
TBPL
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jgriffin, Unassigned)
References
Details
Attachments
(1 file)
(deleted),
patch
|
mstange
:
review+
|
Details | Diff | Splinter Review |
When a bug is starred in tbpl, we'd like tbpl to send an XHR request to the orange factor database (see https://wiki.mozilla.org/Auto-tools/Projects/WarOnOrange), to record this data.
The request should contain enough information that the bug can be tied to a specific buildbot log; this will be possible once bug 617368 is in production.
Comment 1•14 years ago
|
||
(In reply to comment #0)
> The request should contain enough information that the bug can be tied to a
> specific buildbot log; this will be possible once bug 617368 is in production.
It already is!
Reporter | ||
Comment 2•14 years ago
|
||
>> The request should contain enough information that the bug can be tied to a
>> specific buildbot log; this will be possible once bug 617368 is in production.
> It already is!
I see that, thanks!
Reporter | ||
Comment 3•14 years ago
|
||
This patch updates _postOneComment() in tbpl to additionally post some data to a php page hosted by the automation team which will send the data to an elastic search database that is being used for the 'war on orange' project. This database will help us associate starred bug numbers with data scraped from buildbot logs.
It tests ok locally; there are some firewall issues which prevent it from running properly in production at the moment (it will just fail silently), filed as bug 618170.
Attachment #496683 -
Flags: review?(mstange)
Comment 4•14 years ago
|
||
Comment on attachment 496683 [details] [diff] [review]
tinderboxpushlog patch
diff --git a/js/AddCommentUI.js b/js/AddCommentUI.js
_postOneComment: function AddCommentUI__postOneComment(email, comment, machineResult, callback) {
+ if (comment.toLowerCase().indexOf("bug") > -1) {
+ machinename = "";
Missing a "var" here.
Attachment #496683 -
Flags: review?(mstange) → review+
Reporter | ||
Comment 5•14 years ago
|
||
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 6•14 years ago
|
||
Btw, the firewall problem has been resolved and this tests OK now.
Comment 7•14 years ago
|
||
I've updated the server.
Reporter | ||
Updated•14 years ago
|
Assignee: jgriffin → nobody
Assignee | ||
Updated•10 years ago
|
Product: Webtools → Tree Management
Assignee | ||
Updated•10 years ago
|
Product: Tree Management → Tree Management Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•