Thursday, 2016-06-23

*** ddieterly has quit IRC00:44
*** ddieterly has joined #openstack-searchlight00:47
*** ddieterly has quit IRC00:48
*** ddieterly has joined #openstack-searchlight01:09
*** ddieterly has quit IRC01:31
*** shu-mutou has joined #openstack-searchlight02:19
*** shu-mutou is now known as shu-mutou-alt02:20
*** yingjun has joined #openstack-searchlight02:43
*** yingjun has quit IRC02:54
*** yingjun has joined #openstack-searchlight02:54
*** yingjun has quit IRC02:58
*** yingjun has joined #openstack-searchlight03:14
*** yingjun has quit IRC04:10
*** yingjun has joined #openstack-searchlight04:10
*** yingjun has quit IRC04:15
*** GB21 has joined #openstack-searchlight04:18
*** GB21 has quit IRC05:21
*** GB21 has joined #openstack-searchlight05:37
*** yingjun has joined #openstack-searchlight05:55
*** Kevin_Zheng has joined #openstack-searchlight06:08
yingjunTravT, i will not be able to attend the weekly meeting, i’m be on vacation till 27th.07:13
yingjunwill07:14
*** yingjun has quit IRC07:44
*** yingjun has joined #openstack-searchlight07:44
*** yingjun has quit IRC08:36
*** GB21 has quit IRC08:37
*** GB21 has joined #openstack-searchlight08:46
*** lei-zh has joined #openstack-searchlight09:18
*** pcaruana has joined #openstack-searchlight09:51
*** GB21 has quit IRC09:54
*** lei-zh has quit IRC09:57
*** shu-mutou-alt has quit IRC10:22
*** GB21 has joined #openstack-searchlight10:24
*** ddieterly has joined #openstack-searchlight11:05
*** ekarlso has quit IRC11:26
*** ddieterly has quit IRC11:31
*** ddieterly has joined #openstack-searchlight11:35
*** ddieterly has quit IRC11:38
*** GB21 has quit IRC11:40
*** GB21 has joined #openstack-searchlight11:42
*** ekarlso has joined #openstack-searchlight11:51
*** ddieterly has joined #openstack-searchlight11:51
*** ddieterly has quit IRC11:59
*** GB21 has quit IRC12:07
*** GB21 has joined #openstack-searchlight12:24
*** ddieterly has joined #openstack-searchlight12:24
*** ddieterly has quit IRC12:32
*** M00nr41n has quit IRC12:37
*** GB21 has quit IRC12:37
*** RickA-HP has quit IRC13:07
*** RickA-HP has joined #openstack-searchlight13:08
*** sigmavirus24_awa is now known as sigmavirus2413:09
*** david-lyle_ has joined #openstack-searchlight13:32
*** ddieterly has joined #openstack-searchlight13:33
*** david-lyle has quit IRC13:36
*** itisha has joined #openstack-searchlight13:55
*** ddieterly is now known as ddieterly[away]13:56
TravTthx yingjun13:58
*** ddieterly[away] is now known as ddieterly13:59
*** sjmc7 has joined #openstack-searchlight14:19
RickA-HPTravT: I'm about to promote ES2.3.2 testing in Zuul from experimental. To reduce the comibinatorics, I was going to use Steve's advise and replace Python2.7/ES1.7 testing with Python2.7/ES2.X testing.14:24
RickA-HPWe still have Python3.4/ES1.7 testing.14:24
RickA-HPsjmc7: ^^^14:25
sjmc7my recommendation was to test es 2 with both python versions and test with 1.7 with one python version14:26
RickA-HPWe can do that. It seems Python 2.7 is used more than Python3.4, so we can test Python2.7 with both versions of ES.14:28
sjmc7we sort of half made the decision not to officially support ES 1.x for newton anyway, so testing should focus on 2.x. the problem if we use py34/es2 and py27/es1 is it won’t be obvious if errors happen why they’re happening14:28
sjmc7yes, i don’t think it particularly matters which python version we test with. we should also test with the upgraded python ES client lib at some point14:30
RickA-HPI was thinking Python2.7/ES2.3.2 (main case) and Python3.4/ES1.7 (Lessor case). But we can do Python2.7/ES1.7, Python2.7/ES2.3.2 and Python3.4/ES2.3.2.14:30
RickA-HPI'll also file a separate bug for upgrading the python ES client lib.14:33
sjmc7i think we probably need three14:33
sjmc7i want to be able to tell by looking at the jenkins comment whether something failde because of the ES version or the python version14:34
RickA-HPOk, three it is.14:35
*** pcaruana has quit IRC14:41
*** GB21 has joined #openstack-searchlight14:51
TravThey RickA-HP, sorry, was eating some bfast.14:52
TravTmaybe you can repeat your intentions in the meeting14:53
RickA-HPTravT: Sure, even though "intentions" sound ominous!14:54
GB21Hi TravT and sjmc714:55
sjmc7hi14:56
TravThi GB2114:56
GB21Please take a look at this https://review.openstack.org/#/c/315409/514:56
GB21py34 gate is failing14:56
sjmc7it looks like something isn’t initializing properly. i’ll take a look14:57
openstackgerritTravis Tripp proposed openstack/searchlight: Expand designate functional tests  https://review.openstack.org/32920214:57
TravTsjmc7 ^ just rebased that one to see what results come out of gerrit14:57
GB21 I delete the .testrepository/times.dbm, but it is breaking jenkins gate. Please suggest a workaround.14:58
GB21umm14:58
GB21actually it is working after deleting the .testrepository/times.dbm14:58
GB21that was my commnet on the review14:58
GB21I ran it in my local, it did pass14:59
TravTCourtesy Searchlight meeting reminder in #openstack-meeting-4: lakshmiS, nikhil_k, rosmaita, TravT, david-lyle, sjmc7, itisha, GB21, lei-zh, yingjun, RickA-HP, robcresswell, matt-borland, tyr14:59
*** david-lyle_ is now known as david-lyle14:59
sjmc7i will take a look15:00
GB21thanks15:00
*** ddieterly is now known as ddieterly[away]15:21
*** ddieterly[away] is now known as ddieterly15:24
*** GB21 has quit IRC16:02
openstackgerritSteve McLellan proposed openstack/searchlight: Add totals to facets  https://review.openstack.org/30858216:29
*** ddieterly is now known as ddieterly[away]16:44
*** ddieterly[away] is now known as ddieterly16:48
sjmc7TravT: for cross region searching, can you remind me why we decided on adding a region field to results?16:51
TravTi believe our thought was that if somebody wanted to create a searchlight of searchlights, they could16:52
sjmc7hmm16:53
TravTbasically, we thought if you had this info16:53
TravTyou could set up a searchlight with tribe nodes16:53
TravTacross regions16:53
TravTbut it was a deployment concern16:54
TravTand the region field allowed that.16:54
TravTwell, region field allowed you to know where it came from16:54
TravTlet's see....16:54
sjmc7hmm. i wonder if this is something we should push out since we don’t have a clear need for it now16:55
TravThttp://specs.openstack.org/openstack/searchlight-specs/specs/newton/cross-region-search.html16:55
TravTthat seems to be what is in the proposed change section16:55
sjmc7yeah, i do remember the reasoning now16:55
sjmc7just weighing whether we should add something that likely won’t benefit anyone right now16:57
sjmc7and that we won’t really be testing at all16:58
sjmc7guess it could be a config option so it’s there if anyone wants it16:59
TravTyeah, lots of people want cross region search17:00
TravTand we identified the various issues17:00
TravTand thought this was something we could do to enable it17:00
TravTi wonder if it should just be generalized17:01
TravTe.g.17:01
TravTin a given config file17:01
TravTyou can add metadata to anything indexed17:01
TravTregion would be one usage of it17:01
sjmc7thing is though, if you’re adding a static field to every document, it s seems a bit redundant17:02
sjmc7you know it’ll be there when you search17:02
TravTyes, but for it to work with ES tribe nodes17:02
sjmc7the only case in which it is useful is under the tribe node setup and we’re a ways from anyone setting that up as a solution17:02
*** M00nr41n has joined #openstack-searchlight17:02
TravTit needs to be indexed, right?17:02
sjmc7i dunno. it’s not technically that difficult, it’s just that i’m wary of adding code for vague future needs cos we have to maintain it forever17:03
TravTyeah there is always the yagni, but this did seem to be decision at the summit17:04
sjmc7i remember the sauna room now :)17:05
TravTthere was apple pie17:05
TravTbut no beer17:05
sjmc7i guess there’s not a huge amount of harm adding it, especially if it can be disabled17:06
sjmc7i don’t want to add a general case until there’s a proven need for it17:06
TravThaha, but that's the openstack way!17:07
TravTadd random metadata17:07
sjmc7quite17:07
TravTbtw, i'm not arguing strongly for it17:07
TravTfor the random metadata17:07
TravTjust mentioning17:08
TravTregion field seems to be well defined17:08
sjmc7yeah, but again it’s code that has to be maintained and executed on every indexing operation whether it’s used or not17:09
*** M00nr41n has quit IRC17:09
*** ddieterly is now known as ddieterly[away]17:20
*** M00nr41n has joined #openstack-searchlight17:22
*** GB21 has joined #openstack-searchlight17:26
*** GB21 has quit IRC18:04
*** sjmc7 has quit IRC18:11
*** sjmc7 has joined #openstack-searchlight18:12
RickA-HPIf anyone wants to review the infra changes for testing ES2.3.2: https://review.openstack.org/#/c/333525/18:12
*** ddieterly[away] has quit IRC18:21
*** ddieterly has joined #openstack-searchlight18:24
*** M00nr41n has quit IRC18:29
*** TravT has quit IRC18:49
*** TravT has joined #openstack-searchlight19:01
*** TravT has quit IRC19:01
*** TravT has joined #openstack-searchlight19:02
*** ddieterly is now known as ddieterly[away]19:20
*** ddieterly[away] is now known as ddieterly19:44
*** itisha has quit IRC20:27
*** itisha has joined #openstack-searchlight20:31
*** ddieterly is now known as ddieterly[away]21:28
*** TravT has quit IRC21:58
*** ddieterly[away] is now known as ddieterly22:00
openstackgerritSteve McLellan proposed openstack/searchlight: Reduce number of API calls on nova boot  https://review.openstack.org/31710022:04
*** sjmc7 has quit IRC22:07
*** ddieterly has quit IRC22:18
openstackgerritRick Aulino proposed openstack/searchlight: Add Neutron security group plugin  https://review.openstack.org/32676722:50
Kevin_ZhengTravT: I fell asleep last night22:51
Kevin_Zheng:)22:51
*** shu-mutou-AFK is now known as shu-mutou23:28

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