Closed
Bug 848826
Opened 11 years ago
Closed 11 years ago
Make TBPL submit orange bug data to both the old metrics ES instance and the new IT SCL3 one
Categories
(Tree Management Graveyard :: TBPL, defect)
Tree Management Graveyard
TBPL
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: emorley, Assigned: emorley)
References
Details
Attachments
(2 files)
5.48 KB,
patch
|
RyanVM
:
review+
|
Details | Diff | Splinter Review |
3.70 KB,
patch
|
RyanVM
:
review+
|
Details | Diff | Splinter Review |
New one: elasticsearch-zlb.webapp.scl3.mozilla.com:9200
Assignee | ||
Comment 1•11 years ago
|
||
* Changes brace style (bringing them onto the same line) * Fixes whitespace * Moves the "open log file" lines up a bit, ready for part 2. (Not sure what local diff program you have, but I find DiffMerge excellent for seeing the wood for the trees, for whitespace changes like this, btw).
Attachment #722307 -
Flags: review?(ryanvm)
Assignee | ||
Comment 2•11 years ago
|
||
* Makes the server URLs pref an array, with the new ES instance added (from bug 772503 comment 19). * Loops the last part of the file (that actually handles the submission) and submits to each server in the array. Suspect there isn't a huge amount we can do to test locally, since it's always been a bit sketchy for me anyway - so likely just a case of pushing and see (lol).
Attachment #722309 -
Flags: review?(ryanvm)
Assignee | ||
Comment 3•11 years ago
|
||
(Note part 2 isn't as bad as it looks - there's only ~2 lines added, one changed and the rest indented).
Assignee | ||
Comment 4•11 years ago
|
||
I should add that the logging in that file is a bit of a joke, since aiui we have no way of accessing that file on production - plus the TBPL UI drops all failures on the floor (in a similar vein to the now fixed bug 844102); but probably not worth fixing at this point, with TBPL2 on the way.
Updated•11 years ago
|
Attachment #722307 -
Flags: review?(ryanvm) → review+
Updated•11 years ago
|
Attachment #722309 -
Flags: review?(ryanvm) → review+
Assignee | ||
Comment 5•11 years ago
|
||
Testing blocked on bug 849161, since without that I cannot submit to http://elasticsearch-zlb.webapp.scl3.mozilla.com:9200/
Assignee | ||
Comment 6•11 years ago
|
||
I've decided not to wait any longer and just land this on tbpl-dev, since unless we come up with a way to migrate the OF data from the old metrics instance to the new IT instance, we're going to lose all results that haven't been submitted to both during the transition period. https://hg.mozilla.org/webtools/tbpl/rev/c9cc28eb3e52 https://hg.mozilla.org/webtools/tbpl/rev/5ccada7dcb47
Assignee | ||
Comment 7•11 years ago
|
||
In production :-)
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•10 years ago
|
Product: Webtools → Tree Management
Updated•9 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
•