New one: elasticsearch-zlb.webapp.scl3.mozilla.com:9200
* 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)
* 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)
(Note part 2 isn't as bad as it looks - there's only ~2 lines added, one changed and the rest indented).
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.
Attachment #722307 - Flags: review?(ryanvm) → review+
Attachment #722309 - Flags: review?(ryanvm) → review+
Testing blocked on bug 849161, since without that I cannot submit to http://elasticsearch-zlb.webapp.scl3.mozilla.com:9200/
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
In production :-)
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Product: Tree Management → Tree Management Graveyard
You need to log in before you can comment on or make changes to this bug.