Wednesday, 2016-09-28

« Tuesday, 2016-09-27 Index Thursday, 2016-09-29 »
*** ddieterly has quit IRC00:13
*** TravT has joined #openstack-searchlight00:23
*** TravT has quit IRC00:23
*** TravT has joined #openstack-searchlight00:23
*** ddieterly has joined #openstack-searchlight00:46
*** shu-mutou-AWAY is now known as shu-mutou00:52
*** ddieterly has quit IRC01:01
*** ddieterly has joined #openstack-searchlight01:49
*** ddieterly has quit IRC01:52
*** ddieterly has joined #openstack-searchlight02:21
*** ddieterly has quit IRC02:25
*** shu-mutou has quit IRC02:44
*** shu-mutou has joined #openstack-searchlight02:44
*** david-lyle has quit IRC03:04
openstackgerritLi Yingjun proposed openstack/searchlight: Fix dot issue in field name  https://review.openstack.org/37204203:33
openstackgerritLi Yingjun proposed openstack/searchlight: Fix dot issue in field name  https://review.openstack.org/37204203:35
*** pcaruana has joined #openstack-searchlight06:37
openstackgerritMerged openstack/searchlight: Return __searchlight-user-role with highlighting  https://review.openstack.org/37450107:57
*** openstackgerrit has quit IRC08:48
*** openstackgerrit has joined #openstack-searchlight08:49
*** shu-mutou is now known as shu-mutou-AWAY09:25
openstackgerritYuriy Zveryanskyy proposed openstack/searchlight: Allow listener to process different priorities of notifications  https://review.openstack.org/36446110:03
*** bathri-s has joined #openstack-searchlight10:59
*** bathri-s has quit IRC12:48
*** david-lyle has joined #openstack-searchlight12:57
*** matt-borland has joined #openstack-searchlight13:14
*** ddieterly has joined #openstack-searchlight13:52
*** sjmc7 has joined #openstack-searchlight14:14
*** AlexeyAbashkin has joined #openstack-searchlight14:38
*** ddieterly has quit IRC14:56
*** ddieterly has joined #openstack-searchlight14:56
openstackgerritSteve McLellan proposed openstack/searchlight-ui: Honor 'nested' in facets  https://review.openstack.org/36767715:40
openstackgerritOpenStack Proposal Bot proposed openstack/searchlight: Updated from global requirements  https://review.openstack.org/37757216:11
*** ddieterly is now known as ddieterly[away]16:18
sjmc7rosmaita: 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 swamped16:19
rosmaitasjmc7: no, that's fine16:20
sjmc7thanks for all the hard work on it, it looks great16:20
rosmaitacool16:20
rosmaitai never did add the sorting example TravT suggested16:20
rosmaitaif rick has a demo searchlight handy he could maybe add it16:21
RickA-HP_I'll take a look.16:21
rosmaitathanks16:21
TravTrosmaita: thanks for all the hard work.16:21
rosmaitanp16:22
TravTRickA-HP_: sjmc7: I would really love to see a quick aggregation example in there as well16:22
TravTbut was going to suggest it as a follow-on patch16:22
RickA-HP_TravT: Do you have a quick example of the query?16:22
*** ddieterly[away] is now known as ddieterly16:22
TravTi do16:22
TravTi'm trying to find the example i gave steve in a code review16:25
TravTi think the specific agg i used has changed a bit with the nested change16:25
sjmc7let me find the one in the tests16:25
TravThttps://review.openstack.org/#/c/365137/16:26
RickA-HP_TravT: Thanks!16:26
*** RickA-HP_ has quit IRC16:26
TravThttp://paste.openstack.org/show/583342/16:26
TravThmm16:26
TravTlost rick a16:26
*** RickA-HP has joined #openstack-searchlight16:26
TravThttp://paste.openstack.org/show/583342/16:27
TravTRickA-HP:16:27
TravTi think that one works with the nested fields change16:27
TravTthe example is16:27
sjmc7the one in the tests is16:27
sjmc7"aggregations": {16:27
sjmc7                "names": {"terms": {"field": "name"}},16:27
sjmc7                "earliest": {"min": {"field": "created_at"}}16:27
sjmc7            }16:27
TravTmy paste example is:16:27
TravTPlease show me my instances grouped by security groups, flavors, and images16:28
TravTwell, let me reword that.16:28
TravTplease show me the count of instances by security groups, flavors, and images16:28
TravTand the ES link you'd include is16:29
TravThttps://www.elastic.co/guide/en/elasticsearch/reference/current/search-aggregations.html16:29
TravTits kind of funny16:30
RickA-HPWhat's funny?16:31
TravTi think aggregations are one of the most powerful thing of ES, which we didn't really emphasize early on16:31
TravTthe concern of just getting the data and doing basic search16:31
TravTkept me from thinking too much on that front16:31
openstackgerritSteve McLellan proposed openstack/searchlight: Use upper-constraints for tox envs  https://review.openstack.org/37877016:31
RickA-HPTo gain traction, we need to start relatively simple. And then advance to more powerful capabilities!16:32
TravTyep...16:32
TravTand that aggregation example i gave just touches on the most basic example16:33
TravTbut should at least seed the idea16:33
TravTand give the link over to the power of ES16:33
RickA-HPOnce we publish the API doc, I'll make sure the Horizon folks read it :)16:33
sjmc7TravT: RickA-HP : can you rubber stamp https://review.openstack.org/#/c/378770/ please ? we should’ve done it ages ago16:33
RickA-HPLet me get me ink out.16:34
sjmc7then i can pin it in stable/newton16:34
sjmc7right 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 system16:35
sjmc7anything else anyone knows about?16:35
TravTwhat is the error?16:40
TravTthere is a known error which happens16:40
sjmc7TypeError: Cannot create property 'allFacetDefinitions' on string '_type=OS::Nova::Server'16:40
sjmc7    at search (0a4be0081a72.js:1228)16:40
TravTdue to magic search being a little to black magic16:40
sjmc7yeah, it seems to function ok16:41
* TravT trying to remember if that is the issue16:41
TravTif you see it without your patch16:41
TravTthen that is it16:41
sjmc7:D16:41
TravTsjmc7: you do have a lint error16:42
sjmc7yeah, i see that now16:42
sjmc7the massive list of warnings bamboozled me16:42
sjmc7aside from that, does it look ok?16:42
TravTyeah16:42
TravTi copied pasted the actual error into a review for you16:42
TravTdouble checking it your actual code now16:42
TravTbut i have to present my screen in a call i'm in right now16:42
TravTshow off some snazzy power point16:43
sjmc7ok. i’ll try and track down matt or tyr16:44
sjmc7thanks for looking at it16:44
TravTi'm pretty sure that error is the known error16:45
TravTwhich doesn't have an obvious work around16:45
TravTit is related to eventing between widgets16:45
RickA-HPsjmc7: 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-HPWe don't have the exposure using these new packages from all of the testing we've done in the past months.16:46
sjmc7RickA-HP: no, that shouldn’t cause any problems. i’ve put a review up against stable/newton too - https://review.openstack.org/#/c/37877516:46
sjmc7the stable pin is more important since that’s where the issues tend to creep in16:46
openstackgerritOpenStack Proposal Bot proposed openstack/python-searchlightclient: Updated from global requirements  https://review.openstack.org/37756317:01
openstackgerritOpenStack Proposal Bot proposed openstack/searchlight: Updated from global requirements  https://review.openstack.org/37757217:01
openstackgerritMerged openstack/searchlight: Use upper-constraints for tox envs  https://review.openstack.org/37877017:06
*** ddieterly is now known as ddieterly[away]17:15
*** sjmc7 has quit IRC18:10
*** ddieterly[away] is now known as ddieterly18:12
*** sjmc7 has joined #openstack-searchlight18:16
*** ddieterly is now known as ddieterly[away]18:33
*** TravT_ has joined #openstack-searchlight18:44
*** TravT has quit IRC18:47
*** TravT_ has quit IRC18:48
*** TravT has joined #openstack-searchlight19:04
*** TravT has quit IRC19:09
openstackgerritRick Aulino proposed openstack/searchlight: api-ref: Add Searchlight api-ref  https://review.openstack.org/35728719:09
*** ddieterly[away] is now known as ddieterly19:14
*** shu-mutou-AWAY has quit IRC19:20
*** TravT has joined #openstack-searchlight19:31
*** TravT has quit IRC19:32
*** TravT has joined #openstack-searchlight19:36
*** TravT has quit IRC19:36
*** TravT has joined #openstack-searchlight19:36
*** TravT has quit IRC19:37
*** TravT has joined #openstack-searchlight19:38
*** TravT has quit IRC19:47
*** ddieterly is now known as ddieterly[away]20:09
*** ddieterly[away] is now known as ddieterly20:17
*** ddieterly is now known as ddieterly[away]20:34
*** ddieterly[away] is now known as ddieterly20:34
*** ddieterly is now known as ddieterly[away]20:43
openstackgerritRick Aulino proposed openstack/searchlight: api-ref: Add Searchlight api-ref  https://review.openstack.org/35728720:52
*** ddieterly[away] is now known as ddieterly21:06
*** matt-borland has quit IRC21:32
*** sjmc7 has quit IRC22:38
*** ddieterly has quit IRC22:59
*** TravT has joined #openstack-searchlight23:28
openstackgerritMerged openstack/searchlight: api-ref: Add Searchlight api-ref  https://review.openstack.org/35728723:38
openstackgerritMerged openstack/searchlight-ui: Honor 'nested' in facets  https://review.openstack.org/36767723:56
*** TravT_ has joined #openstack-searchlight23:59
« Tuesday, 2016-09-27 Index Thursday, 2016-09-29 »

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!