*** ddieterly has quit IRC | 00:13 | |
*** TravT has joined #openstack-searchlight | 00:23 | |
*** TravT has quit IRC | 00:23 | |
*** TravT has joined #openstack-searchlight | 00:23 | |
*** ddieterly has joined #openstack-searchlight | 00:46 | |
*** shu-mutou-AWAY is now known as shu-mutou | 00:52 | |
*** ddieterly has quit IRC | 01:01 | |
*** ddieterly has joined #openstack-searchlight | 01:49 | |
*** ddieterly has quit IRC | 01:52 | |
*** ddieterly has joined #openstack-searchlight | 02:21 | |
*** ddieterly has quit IRC | 02:25 | |
*** shu-mutou has quit IRC | 02:44 | |
*** shu-mutou has joined #openstack-searchlight | 02:44 | |
*** david-lyle has quit IRC | 03:04 | |
openstackgerrit | Li Yingjun proposed openstack/searchlight: Fix dot issue in field name https://review.openstack.org/372042 | 03:33 |
---|---|---|
openstackgerrit | Li Yingjun proposed openstack/searchlight: Fix dot issue in field name https://review.openstack.org/372042 | 03:35 |
*** pcaruana has joined #openstack-searchlight | 06:37 | |
openstackgerrit | Merged openstack/searchlight: Return __searchlight-user-role with highlighting https://review.openstack.org/374501 | 07:57 |
*** openstackgerrit has quit IRC | 08:48 | |
*** openstackgerrit has joined #openstack-searchlight | 08:49 | |
*** shu-mutou is now known as shu-mutou-AWAY | 09:25 | |
openstackgerrit | Yuriy Zveryanskyy proposed openstack/searchlight: Allow listener to process different priorities of notifications https://review.openstack.org/364461 | 10:03 |
*** bathri-s has joined #openstack-searchlight | 10:59 | |
*** bathri-s has quit IRC | 12:48 | |
*** david-lyle has joined #openstack-searchlight | 12:57 | |
*** matt-borland has joined #openstack-searchlight | 13:14 | |
*** ddieterly has joined #openstack-searchlight | 13:52 | |
*** sjmc7 has joined #openstack-searchlight | 14:14 | |
*** AlexeyAbashkin has joined #openstack-searchlight | 14:38 | |
*** ddieterly has quit IRC | 14:56 | |
*** ddieterly has joined #openstack-searchlight | 14:56 | |
openstackgerrit | Steve McLellan proposed openstack/searchlight-ui: Honor 'nested' in facets https://review.openstack.org/367677 | 15:40 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/searchlight: Updated from global requirements https://review.openstack.org/377572 | 16:11 |
*** ddieterly is now known as ddieterly[away] | 16:18 | |
sjmc7 | rosmaita: any objections to rick making the final couple of small changes on the api-ref doc so we can get it into RC2? i know you’re swamped | 16:19 |
rosmaita | sjmc7: no, that's fine | 16:20 |
sjmc7 | thanks for all the hard work on it, it looks great | 16:20 |
rosmaita | cool | 16:20 |
rosmaita | i never did add the sorting example TravT suggested | 16:20 |
rosmaita | if rick has a demo searchlight handy he could maybe add it | 16:21 |
RickA-HP_ | I'll take a look. | 16:21 |
rosmaita | thanks | 16:21 |
TravT | rosmaita: thanks for all the hard work. | 16:21 |
rosmaita | np | 16:22 |
TravT | RickA-HP_: sjmc7: I would really love to see a quick aggregation example in there as well | 16:22 |
TravT | but was going to suggest it as a follow-on patch | 16:22 |
RickA-HP_ | TravT: Do you have a quick example of the query? | 16:22 |
*** ddieterly[away] is now known as ddieterly | 16:22 | |
TravT | i do | 16:22 |
TravT | i'm trying to find the example i gave steve in a code review | 16:25 |
TravT | i think the specific agg i used has changed a bit with the nested change | 16:25 |
sjmc7 | let me find the one in the tests | 16:25 |
TravT | https://review.openstack.org/#/c/365137/ | 16:26 |
RickA-HP_ | TravT: Thanks! | 16:26 |
*** RickA-HP_ has quit IRC | 16:26 | |
TravT | http://paste.openstack.org/show/583342/ | 16:26 |
TravT | hmm | 16:26 |
TravT | lost rick a | 16:26 |
*** RickA-HP has joined #openstack-searchlight | 16:26 | |
TravT | http://paste.openstack.org/show/583342/ | 16:27 |
TravT | RickA-HP: | 16:27 |
TravT | i think that one works with the nested fields change | 16:27 |
TravT | the example is | 16:27 |
sjmc7 | the one in the tests is | 16:27 |
sjmc7 | "aggregations": { | 16:27 |
sjmc7 | "names": {"terms": {"field": "name"}}, | 16:27 |
sjmc7 | "earliest": {"min": {"field": "created_at"}} | 16:27 |
sjmc7 | } | 16:27 |
TravT | my paste example is: | 16:27 |
TravT | Please show me my instances grouped by security groups, flavors, and images | 16:28 |
TravT | well, let me reword that. | 16:28 |
TravT | please show me the count of instances by security groups, flavors, and images | 16:28 |
TravT | and the ES link you'd include is | 16:29 |
TravT | https://www.elastic.co/guide/en/elasticsearch/reference/current/search-aggregations.html | 16:29 |
TravT | its kind of funny | 16:30 |
RickA-HP | What's funny? | 16:31 |
TravT | i think aggregations are one of the most powerful thing of ES, which we didn't really emphasize early on | 16:31 |
TravT | the concern of just getting the data and doing basic search | 16:31 |
TravT | kept me from thinking too much on that front | 16:31 |
openstackgerrit | Steve McLellan proposed openstack/searchlight: Use upper-constraints for tox envs https://review.openstack.org/378770 | 16:31 |
RickA-HP | To gain traction, we need to start relatively simple. And then advance to more powerful capabilities! | 16:32 |
TravT | yep... | 16:32 |
TravT | and that aggregation example i gave just touches on the most basic example | 16:33 |
TravT | but should at least seed the idea | 16:33 |
TravT | and give the link over to the power of ES | 16:33 |
RickA-HP | Once we publish the API doc, I'll make sure the Horizon folks read it :) | 16:33 |
sjmc7 | TravT: RickA-HP : can you rubber stamp https://review.openstack.org/#/c/378770/ please ? we should’ve done it ages ago | 16:33 |
RickA-HP | Let me get me ink out. | 16:34 |
sjmc7 | then i can pin it in stable/newton | 16:34 |
sjmc7 | right now, there’s the api-ref doc and the UI nested/object thing which i put a patch up for, plus a mysterious TypeError i’m getting in sl-ui that may or may not be just a problem on my system | 16:35 |
sjmc7 | anything else anyone knows about? | 16:35 |
TravT | what is the error? | 16:40 |
TravT | there is a known error which happens | 16:40 |
sjmc7 | TypeError: Cannot create property 'allFacetDefinitions' on string '_type=OS::Nova::Server' | 16:40 |
sjmc7 | at search (0a4be0081a72.js:1228) | 16:40 |
TravT | due to magic search being a little to black magic | 16:40 |
sjmc7 | yeah, it seems to function ok | 16:41 |
* TravT trying to remember if that is the issue | 16:41 | |
TravT | if you see it without your patch | 16:41 |
TravT | then that is it | 16:41 |
sjmc7 | :D | 16:41 |
TravT | sjmc7: you do have a lint error | 16:42 |
sjmc7 | yeah, i see that now | 16:42 |
sjmc7 | the massive list of warnings bamboozled me | 16:42 |
sjmc7 | aside from that, does it look ok? | 16:42 |
TravT | yeah | 16:42 |
TravT | i copied pasted the actual error into a review for you | 16:42 |
TravT | double checking it your actual code now | 16:42 |
TravT | but i have to present my screen in a call i'm in right now | 16:42 |
TravT | show off some snazzy power point | 16:43 |
sjmc7 | ok. i’ll try and track down matt or tyr | 16:44 |
sjmc7 | thanks for looking at it | 16:44 |
TravT | i'm pretty sure that error is the known error | 16:45 |
TravT | which doesn't have an obvious work around | 16:45 |
TravT | it is related to eventing between widgets | 16:45 |
RickA-HP | sjmc7: The new tox.ini patches results in us downgrading two packages: "cryptography" is now 1.5.1 (instead of 1.5.2) and "urllib3" is now 1.17 instead of 1.18. THis doesn't seem like a problem. | 16:45 |
RickA-HP | We don't have the exposure using these new packages from all of the testing we've done in the past months. | 16:46 |
sjmc7 | RickA-HP: no, that shouldn’t cause any problems. i’ve put a review up against stable/newton too - https://review.openstack.org/#/c/378775 | 16:46 |
sjmc7 | the stable pin is more important since that’s where the issues tend to creep in | 16:46 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-searchlightclient: Updated from global requirements https://review.openstack.org/377563 | 17:01 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/searchlight: Updated from global requirements https://review.openstack.org/377572 | 17:01 |
openstackgerrit | Merged openstack/searchlight: Use upper-constraints for tox envs https://review.openstack.org/378770 | 17:06 |
*** ddieterly is now known as ddieterly[away] | 17:15 | |
*** sjmc7 has quit IRC | 18:10 | |
*** ddieterly[away] is now known as ddieterly | 18:12 | |
*** sjmc7 has joined #openstack-searchlight | 18:16 | |
*** ddieterly is now known as ddieterly[away] | 18:33 | |
*** TravT_ has joined #openstack-searchlight | 18:44 | |
*** TravT has quit IRC | 18:47 | |
*** TravT_ has quit IRC | 18:48 | |
*** TravT has joined #openstack-searchlight | 19:04 | |
*** TravT has quit IRC | 19:09 | |
openstackgerrit | Rick Aulino proposed openstack/searchlight: api-ref: Add Searchlight api-ref https://review.openstack.org/357287 | 19:09 |
*** ddieterly[away] is now known as ddieterly | 19:14 | |
*** shu-mutou-AWAY has quit IRC | 19:20 | |
*** TravT has joined #openstack-searchlight | 19:31 | |
*** TravT has quit IRC | 19:32 | |
*** TravT has joined #openstack-searchlight | 19:36 | |
*** TravT has quit IRC | 19:36 | |
*** TravT has joined #openstack-searchlight | 19:36 | |
*** TravT has quit IRC | 19:37 | |
*** TravT has joined #openstack-searchlight | 19:38 | |
*** TravT has quit IRC | 19:47 | |
*** ddieterly is now known as ddieterly[away] | 20:09 | |
*** ddieterly[away] is now known as ddieterly | 20:17 | |
*** ddieterly is now known as ddieterly[away] | 20:34 | |
*** ddieterly[away] is now known as ddieterly | 20:34 | |
*** ddieterly is now known as ddieterly[away] | 20:43 | |
openstackgerrit | Rick Aulino proposed openstack/searchlight: api-ref: Add Searchlight api-ref https://review.openstack.org/357287 | 20:52 |
*** ddieterly[away] is now known as ddieterly | 21:06 | |
*** matt-borland has quit IRC | 21:32 | |
*** sjmc7 has quit IRC | 22:38 | |
*** ddieterly has quit IRC | 22:59 | |
*** TravT has joined #openstack-searchlight | 23:28 | |
openstackgerrit | Merged openstack/searchlight: api-ref: Add Searchlight api-ref https://review.openstack.org/357287 | 23:38 |
openstackgerrit | Merged openstack/searchlight-ui: Honor 'nested' in facets https://review.openstack.org/367677 | 23:56 |
*** TravT_ has joined #openstack-searchlight | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!