| *** bpokorny has quit IRC | 00:02 | |
| openstackgerrit | Travis Tripp proposed openstack/searchlight: Add Devstack install support for searchlightclient https://review.openstack.org/268358 | 00:35 |
|---|---|---|
| *** bpokorny_ has quit IRC | 01:10 | |
| *** bpokorny has joined #openstack-searchlight | 01:10 | |
| *** bpokorny has quit IRC | 01:13 | |
| *** bpokorny has joined #openstack-searchlight | 01:14 | |
| openstackgerrit | Rick Aulino proposed openstack/searchlight-specs: Zero Downtime Reindexing Proposal https://review.openstack.org/245222 | 01:38 |
| *** bpokorny has quit IRC | 02:39 | |
| openstackgerrit | Li Yingjun proposed openstack/python-searchlightclient: Added search resource client and cli https://review.openstack.org/260899 | 02:48 |
| *** lakshmiS has quit IRC | 02:57 | |
| *** akanksha_ has joined #openstack-searchlight | 04:10 | |
| *** bpokorny has joined #openstack-searchlight | 05:07 | |
| *** RickA-HP has quit IRC | 05:13 | |
| *** exploreshaifali has joined #openstack-searchlight | 06:27 | |
| openstackgerrit | Li Yingjun proposed openstack/python-searchlightclient: Add optional fields for source to query cli https://review.openstack.org/271152 | 06:37 |
| *** akanksha_ has quit IRC | 09:07 | |
| *** exploreshaifali has quit IRC | 09:43 | |
| *** exploreshaifali has joined #openstack-searchlight | 11:21 | |
| *** openstackgerrit has quit IRC | 11:47 | |
| *** openstackgerrit has joined #openstack-searchlight | 11:48 | |
| *** openstackgerrit has quit IRC | 12:33 | |
| *** openstackgerrit has joined #openstack-searchlight | 12:33 | |
| *** exploreshaifali has quit IRC | 14:19 | |
| *** pcaruana has joined #openstack-searchlight | 14:49 | |
| *** sigmavirus24_awa is now known as sigmavirus24 | 15:13 | |
| dhellmann | TravT: please ack this release model correction (you can change it back next cycle if you decide you want to, but for now we need to reflect what is actually being done) https://review.openstack.org/271369 | 15:18 |
| *** bpokorny has quit IRC | 15:36 | |
| *** RickA-HP has joined #openstack-searchlight | 15:41 | |
| *** bpokorny has joined #openstack-searchlight | 16:13 | |
| sjmc7 | thanks for your spec reviews, rosmaita - some good feedback | 16:15 |
| TravT | dhellmann: yes, that looks right. adding comment to review now. Thanks! | 16:16 |
| *** RickA-HP has quit IRC | 16:23 | |
| *** RickA-HP has joined #openstack-searchlight | 16:33 | |
| openstackgerrit | Travis Tripp proposed openstack/searchlight: Add Devstack install support for searchlightclient https://review.openstack.org/268358 | 16:36 |
| *** GB21 has joined #openstack-searchlight | 16:44 | |
| openstackgerrit | Merged openstack/python-searchlightclient: Updated from global requirements https://review.openstack.org/270950 | 16:46 |
| sjmc7 | TravT: real quick, looking at the client patch for queries. i noticed that ‘Name’ isn’t included if ‘Source’ is. my preference is for —source just to add columns, not remove, but before i force li to do another revision, is that unreasonable? | 16:47 |
| sjmc7 | he’s been very patient, i don’t want to send it round the circle again unnecessarily | 16:47 |
| openstackgerrit | Merged openstack/searchlight: Updated from global requirements https://review.openstack.org/270952 | 16:47 |
| TravT | sjmc7: i also had noticed that and was having some debate with myself on it as well | 16:48 |
| openstackgerrit | Merged openstack/python-searchlightclient: Remove dash for resource-type in cli https://review.openstack.org/270894 | 16:48 |
| sjmc7 | ok. i think i’ll -1 it then, and approve it over the weekend - he seems to work friday nights my time | 16:48 |
| TravT | it is a little funny to see columns change | 16:48 |
| TravT | there is one other thing i'm thinking about, but thought it could be debated in follow on | 16:48 |
| TravT | maybe we don't need query | 16:49 |
| TravT | just openstack search <query> | 16:49 |
| TravT | rather than openstack search query <query> | 16:49 |
| TravT | but have to think further | 16:49 |
| sjmc7 | i did wonder about that. i don’t know enough about how the CLI parser works | 16:49 |
| TravT | sjmc7: you could consider updating the name field yourself and we could ask another reviewer to take a look. | 16:50 |
| TravT | just a thought to stop annoying li | 16:50 |
| sjmc7 | yeah, that’s better | 16:50 |
| sjmc7 | :) | 16:50 |
| sjmc7 | hmm, it also removes updated_at | 16:51 |
| sjmc7 | problem is, when those are there, the column with thing is even more of an issue | 16:51 |
| sjmc7 | maybe let’s leave it for now | 16:51 |
| TravT | hmm, does it become too wide or something. | 16:51 |
| TravT | on a small terminal window, it probably would look bad. | 16:51 |
| sjmc7 | yeah. you’d need max-width=3 to keep it within 100 chars :) | 16:51 |
| sjmc7 | it’d look like assembler | 16:51 |
| sjmc7 | ok, let’s leave that for now | 16:52 |
| TravT | well, we can debate it in a further patch | 16:52 |
| TravT | since this isn't released | 16:52 |
| sjmc7 | yeah. rest of it looks good | 16:52 |
| TravT | ok, i was just running through it | 16:52 |
| sjmc7 | yeah, i ran a bunch of queries | 16:52 |
| TravT | so if you +2, mine will follow shortly. | 16:52 |
| sjmc7 | just gonna test the python interface | 16:53 |
| TravT | assuming i don't see anything hideous | 16:53 |
| sjmc7 | the only slightly unfortunate thing | 16:54 |
| sjmc7 | is that becaues of our lack of RESTy noun-verb thing | 16:54 |
| sjmc7 | the python client looks like sc.search.search() | 16:54 |
| sjmc7 | rather than e.g. heatclient.stacks.list() | 16:55 |
| sjmc7 | but we can ruminate on it another time | 16:55 |
| openstackgerrit | Merged openstack/python-searchlightclient: Added search resource client and cli https://review.openstack.org/260899 | 17:23 |
| *** lakshmiS has joined #openstack-searchlight | 17:29 | |
| -openstackstatus- NOTICE: Restarting zuul due to a memory leak | 17:51 | |
| *** lakshmiS has quit IRC | 17:51 | |
| *** bpokorny_ has joined #openstack-searchlight | 17:57 | |
| *** bpokorny_ has quit IRC | 17:58 | |
| *** bpokorny_ has joined #openstack-searchlight | 17:59 | |
| *** bpokorny has quit IRC | 18:00 | |
| *** GB21 has quit IRC | 18:36 | |
| *** lakshmiS has joined #openstack-searchlight | 18:56 | |
| *** bpokorny_ has quit IRC | 19:07 | |
| *** bpokorny has joined #openstack-searchlight | 19:07 | |
| *** bpokorny has quit IRC | 19:31 | |
| *** RickA-HP has quit IRC | 19:34 | |
| *** pcaruana has quit IRC | 20:14 | |
| *** bpokorny has joined #openstack-searchlight | 21:24 | |
| openstackgerrit | Merged openstack/searchlight: Replace deprecated library function os.popen() with subprocess https://review.openstack.org/267555 | 21:30 |
| *** pcaruana has joined #openstack-searchlight | 21:48 | |
| sjmc7 | lakshmiS: is the swift patch at a point you could upload it as a WIP? | 21:58 |
| lakshmiS | need to tweak rbac a little bit and then will put it up | 21:58 |
| sjmc7 | ok | 21:59 |
| lakshmiS | if i cant' figure it out by eod, will put the patch minus the rbac | 22:00 |
| sjmc7 | cool. i want to check i can get the middleware to a point we can support everything | 22:02 |
| openstackgerrit | Merged openstack/searchlight: Add python 3.4 to programming languages in setup.cfg https://review.openstack.org/267783 | 22:10 |
| *** TravT has quit IRC | 22:18 | |
| *** TravT has joined #openstack-searchlight | 22:20 | |
| sjmc7 | reviews for https://review.openstack.org/#/c/257516/ would be good, too; it’s going to cause a load of rebasing to other plugins/code | 22:20 |
| *** sigmavirus24 is now known as sigmavirus24_awa | 23:00 | |
| openstackgerrit | Steve McLellan proposed openstack/searchlight: Separate documents by role https://review.openstack.org/257516 | 23:52 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!