Thursday, 2016-06-30

*** ddieterly has joined #openstack-searchlight00:05
*** yingjun has joined #openstack-searchlight00:24
*** yingjun has quit IRC00:35
*** yingjun has joined #openstack-searchlight00:35
*** yingjun has quit IRC00:40
*** yingjun has joined #openstack-searchlight00:44
openstackgerritRick Aulino proposed openstack/searchlight: To speed up Searchlight indexing and re-indexing we should use threads instead of inexing resource types sequentially.  https://review.openstack.org/33364900:50
*** ddieterly has quit IRC00:54
*** yingjun has quit IRC00:59
*** yingjun has joined #openstack-searchlight01:00
*** yingjun has quit IRC01:04
openstackgerritRick Aulino proposed openstack/searchlight: To speed up Searchlight indexing and re-indexing we should use threads instead of inexing resource types sequentially.  https://review.openstack.org/33364901:10
*** yingjun has joined #openstack-searchlight01:21
*** TravT has quit IRC01:54
*** TravT has joined #openstack-searchlight02:12
*** lei-zh has joined #openstack-searchlight02:24
*** shu-mutou has joined #openstack-searchlight02:34
*** yingjun has quit IRC04:21
*** yingjun has joined #openstack-searchlight04:21
*** yingjun has quit IRC04:26
*** yingjun has joined #openstack-searchlight04:37
*** GB21 has joined #openstack-searchlight05:06
*** yingjun has quit IRC05:09
*** yingjun has joined #openstack-searchlight05:10
*** yingjun has quit IRC05:14
openstackgerritRick Aulino proposed openstack/searchlight: To speed up Searchlight indexing and re-indexing we should use threads instead of inexing resource types sequentially.  https://review.openstack.org/33364905:43
*** yingjun has joined #openstack-searchlight06:01
*** pcaruana has joined #openstack-searchlight06:37
*** GB21 has quit IRC07:25
*** GB21 has joined #openstack-searchlight07:46
*** shu-mutou has quit IRC08:54
*** lcastell_ has joined #openstack-searchlight09:03
*** lcastell has quit IRC09:08
*** lcastell_ is now known as lcastell09:08
*** GB21 has quit IRC09:29
*** yingjun has quit IRC09:38
*** lcastell has quit IRC09:43
*** lcastell has joined #openstack-searchlight09:45
*** GB21 has joined #openstack-searchlight09:58
*** david-lyle has quit IRC10:09
*** david-lyle_ has joined #openstack-searchlight10:09
*** lei-zh has quit IRC10:14
*** ddieterly has joined #openstack-searchlight11:17
*** GB21 has quit IRC11:47
*** GB21 has joined #openstack-searchlight11:54
*** ddieterly has quit IRC12:00
*** yingjun has joined #openstack-searchlight12:27
*** yingjun has quit IRC12:35
*** ddieterly has joined #openstack-searchlight12:35
*** yingjun has joined #openstack-searchlight12:35
*** GB21 has quit IRC12:44
*** yingjun has quit IRC12:47
*** yingjun has joined #openstack-searchlight12:47
*** yingjun has quit IRC12:52
*** ddieterly has quit IRC12:55
*** ddieterly has joined #openstack-searchlight13:37
*** yingjun has joined #openstack-searchlight13:41
*** itisha has joined #openstack-searchlight14:01
openstackgerritMatt Borland proposed openstack/searchlight-ui: Handle action results  https://review.openstack.org/31710614:04
*** GB21 has joined #openstack-searchlight14:13
*** sjmc7 has joined #openstack-searchlight14:18
*** yingjun has quit IRC14:27
*** yingjun has joined #openstack-searchlight14:28
*** matt-borland has joined #openstack-searchlight14:28
matt-borlandHey searchlighters!  I believe due to the npm gate problems, we introduced some lint/test failures into searchlight-ui.14:29
matt-borlandI have a patch that fixes that AND has some other features, but wondering if I should just patch the problems...if I can get that approved quickly.14:30
matt-borlandhttps://review.openstack.org/#/c/335041/ fixes the problems, and also has a +2 :)14:30
GB21hey matt-borland, TravT and sjmc714:31
matt-borlandyeah...what's up?14:31
matt-borlanddid you get Horizon working?14:31
matt-borlandAlso GB21, is that your real name? :)  I never caught it I don't believe.14:32
GB21yepp, I got it in my local, but this command ./tools/with_venv.sh pip install -i ../searchlight-ui/', didn't work14:32
GB21matt-borland, hahaha, no; my name is Geetika Batra14:32
*** yingjun has quit IRC14:32
GB21:P14:32
matt-borlanddo you have the searchlight-ui repo checked out relative to your horizon directory?14:33
matt-borlandwhat is the error on your command?14:33
GB21yepp14:33
GB21Well, I followed everything u said last time14:34
matt-borlanduh oh14:34
matt-borlandsome of that was for devstack, so would be different in this case14:34
matt-borlanda couple of questions:14:34
matt-borland1) does Horizon run on its own?14:34
matt-borland(can you log in, etc. regardless of Searchlight)14:35
matt-borlandif YES, then it should be pretty straightforward14:35
GB21well, I checked horizon, it was running on its own14:35
matt-borlandcool14:35
GB21but, I don't know if it was connected to searchlight UI14:36
matt-borlandOK, so inside of searchlight-ui, did you do 'python setup.py sdist' ?14:36
GB21yepp14:37
GB21I did14:37
matt-borlanddid you copy the enabled file from searchlight-ui to .../local/enabled?14:37
GB21yepp14:37
GB21I did that also14:38
matt-borlandthen, when you changed dir to ./horizon,14:38
matt-borlandand you ran './tools/with_venv.sh pip install -e ../searchlight-ui/'  what did you get?14:39
matt-borlandyou said it failed here14:39
GB21I get "You must give atleast one requirement to install"14:41
GB21I thought that it may be beacuse my searchlight-ui directory is in /opt/stack but horizon is in my home dir14:41
GB21I tried changing the paths14:42
GB21but no luck14:42
matt-borlandthat is a strange error14:43
GB21it says something related to pip14:43
GB21I guess14:43
matt-borlandit really doesn't sound like the command was right.14:43
GB21yeah14:43
GB21I then tried manually creating the virtual env14:44
GB21by source ./tools/with_venv.sh14:44
GB21right now it says no bin file14:45
matt-borlandthat really makes no sense at all.  Can you paste your results into a pastebin or something?14:46
GB21sure14:46
GB21plz gimme a moment14:46
*** pcaruana has quit IRC14:46
*** yingjun has joined #openstack-searchlight14:47
GB21http://paste.openstack.org/show/524277/14:50
GB21matt-borland, ^14:50
*** david-lyle_ is now known as david-lyle14:51
matt-borlandGB21, ummmm14:54
matt-borlandmay I ask you to review the command you typed.14:54
matt-borlandand review that against the command you were asked to type.14:55
matt-borland:)14:55
GB21sure matt-borland , './tools/with_venv.sh pip install -i ../searchlight-ui/'14:56
TravTGB2114:57
GB21this was the command , right14:57
TravTis should be -314:57
TravT-e14:57
GB21hi TravT14:57
*** RickA-HP has joined #openstack-searchlight14:57
TravTSee https://github.com/openstack/searchlight-ui#setup-local-dev-environment14:57
GB21umm, my bad14:57
GB21I will correct it14:57
TravTand should be the correct path14:57
TravTi think above you said searchlight is in /opt/stack/searchlight-ui14:58
TravTso from your horizon dir14:58
TravT./tools/with_venv.sh pip install -e /opt/stack/searchlight-ui14:58
GB21it works,:D14:58
GB21thanks14:58
TravT:)14:58
matt-borland:)14:58
TravTCourtesy Searchlight meeting reminder in 1 minute in #openstack-meeting-4: lakshmiS, nikhil_k, rosmaita, TravT, david-lyle, sjmc7, itisha, GB21, briancline, lei-zh, yingjun, RickA-HP, robcresswell, matt-borland, tyr14:59
matt-borlandold.mcdonald.had.a.farm -e -i -e -i -o14:59
GB21:D14:59
*** gb21_ has joined #openstack-searchlight15:24
*** GB21 has quit IRC15:25
*** gb21_ is now known as GB2115:25
*** yingjun has quit IRC16:02
*** yingjun has joined #openstack-searchlight16:02
*** yingjun has quit IRC16:07
GB21TravT;  I wanted to discuss ur review on my recent patch16:20
GB21well, I spent a lot of time on it today16:21
GB21https://review.openstack.org/#/c/315409/816:21
GB21actually, we can get the tenant id by nova.flavor_access.list(flavor = *some flavor object*)16:22
GB21tenant id is not visible in if we look for the attributes of flavor objects when accessed from the list i.e(nova.flavors.list()[index])16:23
TravThi GB2116:24
GB21hi TravT16:25
TravTso, then are you thinking you'd list the flavors16:26
TravTthen do a callback for the flavors to get their tenants?16:26
GB21well, what I did was, called the flavor access list and updated the dictionary of the corresponding flavor16:27
TravTokay16:27
GB21this will cause me to change my way of listing flavor16:27
GB21also, while serializing I would have to use __dict__ instead of to_dict16:28
GB21because on updating that flavor, the tenant id does not appear in to_dict() method16:28
GB21so, it is becoming very complex16:29
GB21and there are some other parameters also present in __dict__, which I thought that I should pop16:30
TravTjust a sec, lemme look at your patch16:30
GB21e.g _info16:30
GB21sure16:31
TravTokay, well, if you don't want to hack the client flavor object, you could just create the map of flavor_id to members separate and merge them in at serialization.16:35
TravTi do think on response that we would need to exclude the members from the result16:38
TravTunless admin, perhaps16:38
GB21well, the thing is, I tried separating the flavors as members from that of public flavors, the is_public attribute fails for the flavors of members16:39
GB21flavor of members --> flavor with members16:39
GB21so, separating them out is also getting tricky here16:39
TravTi'm not sure i understand why you separated out the members of public flavors?16:41
TravTdo you have a paste with the json structures you are testing with?16:41
TravTi know members are irrelevant on public flavors, but it should really matter to us, i think?16:42
*** ddieterly is now known as ddieterly[away]16:42
GB21for public flavors, the nova client throws an error if I call it with nova.flavor_access.list(flavor=*some public flavor*)16:43
GB21I have been trying it in ipython16:43
GB21to be sure16:43
TravTah, okay, i see16:43
TravTso, you are saying that you get the flavors16:44
TravTthe for each flavor16:44
TravTyou call access list if not is_public?16:44
GB21well is_public is also throwing error if it is not a public flavor16:44
GB21it should return me false16:45
GB21but it just throws error16:45
*** ddieterly[away] is now known as ddieterly16:45
TravThow are you accessing it?16:45
GB21gimme a moment, I will paste u the examples, I am trying with16:46
TravTJust added one note on your current patch17:01
TravTYou need to specify is_public=None in the argument list to get all flavors: https://github.com/openstack/python-novaclient/blob/master/novaclient/v2/flavors.py#L101-L10317:01
TravTnot sure how you are getting the is_public17:04
TravTbut in addition to the above17:04
TravTi did this:17:04
TravT    if not getattr(flavor, 'is_public', False):17:04
TravT        #Get members17:04
TravTand it worked as expected17:04
TravTi have to run shortly for a meeting.17:05
GB21sure, bye17:12
*** ddieterly is now known as ddieterly[away]17:20
*** matt-borland has quit IRC17:23
*** TravT has quit IRC17:39
*** sjmc7 has quit IRC17:42
*** sjmc7 has joined #openstack-searchlight17:47
*** ddieterly[away] is now known as ddieterly18:12
*** GB21 has quit IRC18:33
*** TravT has joined #openstack-searchlight18:52
*** TravT has quit IRC18:54
*** TravT has joined #openstack-searchlight18:55
openstackgerritSteve McLellan proposed openstack/searchlight: Fix --type handling for indexing parent plugins  https://review.openstack.org/33224219:36
*** matt-borland has joined #openstack-searchlight19:45
matt-borlandIf anyone can review https://review.openstack.org/#/c/335041/, it has a +2 on it; it allows for linking to client-side views19:46
matt-borland(which makes things nice and fast) :)19:46
*** ddieterly is now known as ddieterly[away]19:53
matt-borlandthanks david-lyle :)20:01
*** ddieterly[away] is now known as ddieterly20:02
*** sjmc7 has quit IRC20:07
openstackgerritMerged openstack/searchlight-ui: Link search hits to registered details views if available  https://review.openstack.org/33504120:10
*** ddieterly is now known as ddieterly[away]20:26
*** TravT has quit IRC20:47
*** ddieterly[away] is now known as ddieterly20:48
*** TravT has joined #openstack-searchlight21:14
*** TravT has quit IRC21:14
*** TravT has joined #openstack-searchlight21:15
*** TravT has quit IRC21:16
*** TravT has joined #openstack-searchlight21:17
*** ddieterly is now known as ddieterly[away]21:18
*** ddieterly[away] is now known as ddieterly21:20
*** notmyname has quit IRC21:28
*** notmyname_ has joined #openstack-searchlight21:28
*** lcastell_ has joined #openstack-searchlight21:28
*** notmyname_ is now known as notmyname21:29
*** lcastell has quit IRC21:29
*** lcastell_ is now known as lcastell21:29
*** sjmc7 has joined #openstack-searchlight21:39
*** matt-borland has quit IRC21:45
*** ddieterly is now known as ddieterly[away]22:05
*** TravT has quit IRC22:12
*** TravT has joined #openstack-searchlight22:14
openstackgerritTravis Tripp proposed openstack/searchlight: Remove uneccessary line in resources test script  https://review.openstack.org/33630622:22
openstackgerritTravis Tripp proposed openstack/searchlight: Fix broken incrementer  https://review.openstack.org/33630622:31
*** ddieterly[away] is now known as ddieterly22:36
*** ddieterly is now known as ddieterly[away]23:02
*** ddieterly[away] has quit IRC23:02
*** sjmc7 has quit IRC23:12

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