*** ddieterly has joined #openstack-searchlight | 00:05 | |
*** yingjun has joined #openstack-searchlight | 00:24 | |
*** yingjun has quit IRC | 00:35 | |
*** yingjun has joined #openstack-searchlight | 00:35 | |
*** yingjun has quit IRC | 00:40 | |
*** yingjun has joined #openstack-searchlight | 00:44 | |
openstackgerrit | Rick 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/333649 | 00:50 |
---|---|---|
*** ddieterly has quit IRC | 00:54 | |
*** yingjun has quit IRC | 00:59 | |
*** yingjun has joined #openstack-searchlight | 01:00 | |
*** yingjun has quit IRC | 01:04 | |
openstackgerrit | Rick 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/333649 | 01:10 |
*** yingjun has joined #openstack-searchlight | 01:21 | |
*** TravT has quit IRC | 01:54 | |
*** TravT has joined #openstack-searchlight | 02:12 | |
*** lei-zh has joined #openstack-searchlight | 02:24 | |
*** shu-mutou has joined #openstack-searchlight | 02:34 | |
*** yingjun has quit IRC | 04:21 | |
*** yingjun has joined #openstack-searchlight | 04:21 | |
*** yingjun has quit IRC | 04:26 | |
*** yingjun has joined #openstack-searchlight | 04:37 | |
*** GB21 has joined #openstack-searchlight | 05:06 | |
*** yingjun has quit IRC | 05:09 | |
*** yingjun has joined #openstack-searchlight | 05:10 | |
*** yingjun has quit IRC | 05:14 | |
openstackgerrit | Rick 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/333649 | 05:43 |
*** yingjun has joined #openstack-searchlight | 06:01 | |
*** pcaruana has joined #openstack-searchlight | 06:37 | |
*** GB21 has quit IRC | 07:25 | |
*** GB21 has joined #openstack-searchlight | 07:46 | |
*** shu-mutou has quit IRC | 08:54 | |
*** lcastell_ has joined #openstack-searchlight | 09:03 | |
*** lcastell has quit IRC | 09:08 | |
*** lcastell_ is now known as lcastell | 09:08 | |
*** GB21 has quit IRC | 09:29 | |
*** yingjun has quit IRC | 09:38 | |
*** lcastell has quit IRC | 09:43 | |
*** lcastell has joined #openstack-searchlight | 09:45 | |
*** GB21 has joined #openstack-searchlight | 09:58 | |
*** david-lyle has quit IRC | 10:09 | |
*** david-lyle_ has joined #openstack-searchlight | 10:09 | |
*** lei-zh has quit IRC | 10:14 | |
*** ddieterly has joined #openstack-searchlight | 11:17 | |
*** GB21 has quit IRC | 11:47 | |
*** GB21 has joined #openstack-searchlight | 11:54 | |
*** ddieterly has quit IRC | 12:00 | |
*** yingjun has joined #openstack-searchlight | 12:27 | |
*** yingjun has quit IRC | 12:35 | |
*** ddieterly has joined #openstack-searchlight | 12:35 | |
*** yingjun has joined #openstack-searchlight | 12:35 | |
*** GB21 has quit IRC | 12:44 | |
*** yingjun has quit IRC | 12:47 | |
*** yingjun has joined #openstack-searchlight | 12:47 | |
*** yingjun has quit IRC | 12:52 | |
*** ddieterly has quit IRC | 12:55 | |
*** ddieterly has joined #openstack-searchlight | 13:37 | |
*** yingjun has joined #openstack-searchlight | 13:41 | |
*** itisha has joined #openstack-searchlight | 14:01 | |
openstackgerrit | Matt Borland proposed openstack/searchlight-ui: Handle action results https://review.openstack.org/317106 | 14:04 |
*** GB21 has joined #openstack-searchlight | 14:13 | |
*** sjmc7 has joined #openstack-searchlight | 14:18 | |
*** yingjun has quit IRC | 14:27 | |
*** yingjun has joined #openstack-searchlight | 14:28 | |
*** matt-borland has joined #openstack-searchlight | 14:28 | |
matt-borland | Hey searchlighters! I believe due to the npm gate problems, we introduced some lint/test failures into searchlight-ui. | 14:29 |
matt-borland | I 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-borland | https://review.openstack.org/#/c/335041/ fixes the problems, and also has a +2 :) | 14:30 |
GB21 | hey matt-borland, TravT and sjmc7 | 14:31 |
matt-borland | yeah...what's up? | 14:31 |
matt-borland | did you get Horizon working? | 14:31 |
matt-borland | Also GB21, is that your real name? :) I never caught it I don't believe. | 14:32 |
GB21 | yepp, I got it in my local, but this command ./tools/with_venv.sh pip install -i ../searchlight-ui/', didn't work | 14:32 |
GB21 | matt-borland, hahaha, no; my name is Geetika Batra | 14:32 |
*** yingjun has quit IRC | 14:32 | |
GB21 | :P | 14:32 |
matt-borland | do you have the searchlight-ui repo checked out relative to your horizon directory? | 14:33 |
matt-borland | what is the error on your command? | 14:33 |
GB21 | yepp | 14:33 |
GB21 | Well, I followed everything u said last time | 14:34 |
matt-borland | uh oh | 14:34 |
matt-borland | some of that was for devstack, so would be different in this case | 14:34 |
matt-borland | a couple of questions: | 14:34 |
matt-borland | 1) does Horizon run on its own? | 14:34 |
matt-borland | (can you log in, etc. regardless of Searchlight) | 14:35 |
matt-borland | if YES, then it should be pretty straightforward | 14:35 |
GB21 | well, I checked horizon, it was running on its own | 14:35 |
matt-borland | cool | 14:35 |
GB21 | but, I don't know if it was connected to searchlight UI | 14:36 |
matt-borland | OK, so inside of searchlight-ui, did you do 'python setup.py sdist' ? | 14:36 |
GB21 | yepp | 14:37 |
GB21 | I did | 14:37 |
matt-borland | did you copy the enabled file from searchlight-ui to .../local/enabled? | 14:37 |
GB21 | yepp | 14:37 |
GB21 | I did that also | 14:38 |
matt-borland | then, when you changed dir to ./horizon, | 14:38 |
matt-borland | and you ran './tools/with_venv.sh pip install -e ../searchlight-ui/' what did you get? | 14:39 |
matt-borland | you said it failed here | 14:39 |
GB21 | I get "You must give atleast one requirement to install" | 14:41 |
GB21 | I thought that it may be beacuse my searchlight-ui directory is in /opt/stack but horizon is in my home dir | 14:41 |
GB21 | I tried changing the paths | 14:42 |
GB21 | but no luck | 14:42 |
matt-borland | that is a strange error | 14:43 |
GB21 | it says something related to pip | 14:43 |
GB21 | I guess | 14:43 |
matt-borland | it really doesn't sound like the command was right. | 14:43 |
GB21 | yeah | 14:43 |
GB21 | I then tried manually creating the virtual env | 14:44 |
GB21 | by source ./tools/with_venv.sh | 14:44 |
GB21 | right now it says no bin file | 14:45 |
matt-borland | that really makes no sense at all. Can you paste your results into a pastebin or something? | 14:46 |
GB21 | sure | 14:46 |
GB21 | plz gimme a moment | 14:46 |
*** pcaruana has quit IRC | 14:46 | |
*** yingjun has joined #openstack-searchlight | 14:47 | |
GB21 | http://paste.openstack.org/show/524277/ | 14:50 |
GB21 | matt-borland, ^ | 14:50 |
*** david-lyle_ is now known as david-lyle | 14:51 | |
matt-borland | GB21, ummmm | 14:54 |
matt-borland | may I ask you to review the command you typed. | 14:54 |
matt-borland | and review that against the command you were asked to type. | 14:55 |
matt-borland | :) | 14:55 |
GB21 | sure matt-borland , './tools/with_venv.sh pip install -i ../searchlight-ui/' | 14:56 |
TravT | GB21 | 14:57 |
GB21 | this was the command , right | 14:57 |
TravT | is should be -3 | 14:57 |
TravT | -e | 14:57 |
GB21 | hi TravT | 14:57 |
*** RickA-HP has joined #openstack-searchlight | 14:57 | |
TravT | See https://github.com/openstack/searchlight-ui#setup-local-dev-environment | 14:57 |
GB21 | umm, my bad | 14:57 |
GB21 | I will correct it | 14:57 |
TravT | and should be the correct path | 14:57 |
TravT | i think above you said searchlight is in /opt/stack/searchlight-ui | 14:58 |
TravT | so from your horizon dir | 14:58 |
TravT | ./tools/with_venv.sh pip install -e /opt/stack/searchlight-ui | 14:58 |
GB21 | it works,:D | 14:58 |
GB21 | thanks | 14:58 |
TravT | :) | 14:58 |
matt-borland | :) | 14:58 |
TravT | Courtesy 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, tyr | 14:59 |
matt-borland | old.mcdonald.had.a.farm -e -i -e -i -o | 14:59 |
GB21 | :D | 14:59 |
*** gb21_ has joined #openstack-searchlight | 15:24 | |
*** GB21 has quit IRC | 15:25 | |
*** gb21_ is now known as GB21 | 15:25 | |
*** yingjun has quit IRC | 16:02 | |
*** yingjun has joined #openstack-searchlight | 16:02 | |
*** yingjun has quit IRC | 16:07 | |
GB21 | TravT; I wanted to discuss ur review on my recent patch | 16:20 |
GB21 | well, I spent a lot of time on it today | 16:21 |
GB21 | https://review.openstack.org/#/c/315409/8 | 16:21 |
GB21 | actually, we can get the tenant id by nova.flavor_access.list(flavor = *some flavor object*) | 16:22 |
GB21 | tenant 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 |
TravT | hi GB21 | 16:24 |
GB21 | hi TravT | 16:25 |
TravT | so, then are you thinking you'd list the flavors | 16:26 |
TravT | then do a callback for the flavors to get their tenants? | 16:26 |
GB21 | well, what I did was, called the flavor access list and updated the dictionary of the corresponding flavor | 16:27 |
TravT | okay | 16:27 |
GB21 | this will cause me to change my way of listing flavor | 16:27 |
GB21 | also, while serializing I would have to use __dict__ instead of to_dict | 16:28 |
GB21 | because on updating that flavor, the tenant id does not appear in to_dict() method | 16:28 |
GB21 | so, it is becoming very complex | 16:29 |
GB21 | and there are some other parameters also present in __dict__, which I thought that I should pop | 16:30 |
TravT | just a sec, lemme look at your patch | 16:30 |
GB21 | e.g _info | 16:30 |
GB21 | sure | 16:31 |
TravT | okay, 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 |
TravT | i do think on response that we would need to exclude the members from the result | 16:38 |
TravT | unless admin, perhaps | 16:38 |
GB21 | well, the thing is, I tried separating the flavors as members from that of public flavors, the is_public attribute fails for the flavors of members | 16:39 |
GB21 | flavor of members --> flavor with members | 16:39 |
GB21 | so, separating them out is also getting tricky here | 16:39 |
TravT | i'm not sure i understand why you separated out the members of public flavors? | 16:41 |
TravT | do you have a paste with the json structures you are testing with? | 16:41 |
TravT | i 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 | |
GB21 | for public flavors, the nova client throws an error if I call it with nova.flavor_access.list(flavor=*some public flavor*) | 16:43 |
GB21 | I have been trying it in ipython | 16:43 |
GB21 | to be sure | 16:43 |
TravT | ah, okay, i see | 16:43 |
TravT | so, you are saying that you get the flavors | 16:44 |
TravT | the for each flavor | 16:44 |
TravT | you call access list if not is_public? | 16:44 |
GB21 | well is_public is also throwing error if it is not a public flavor | 16:44 |
GB21 | it should return me false | 16:45 |
GB21 | but it just throws error | 16:45 |
*** ddieterly[away] is now known as ddieterly | 16:45 | |
TravT | how are you accessing it? | 16:45 |
GB21 | gimme a moment, I will paste u the examples, I am trying with | 16:46 |
TravT | Just added one note on your current patch | 17:01 |
TravT | You 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-L103 | 17:01 |
TravT | not sure how you are getting the is_public | 17:04 |
TravT | but in addition to the above | 17:04 |
TravT | i did this: | 17:04 |
TravT | if not getattr(flavor, 'is_public', False): | 17:04 |
TravT | #Get members | 17:04 |
TravT | and it worked as expected | 17:04 |
TravT | i have to run shortly for a meeting. | 17:05 |
GB21 | sure, bye | 17:12 |
*** ddieterly is now known as ddieterly[away] | 17:20 | |
*** matt-borland has quit IRC | 17:23 | |
*** TravT has quit IRC | 17:39 | |
*** sjmc7 has quit IRC | 17:42 | |
*** sjmc7 has joined #openstack-searchlight | 17:47 | |
*** ddieterly[away] is now known as ddieterly | 18:12 | |
*** GB21 has quit IRC | 18:33 | |
*** TravT has joined #openstack-searchlight | 18:52 | |
*** TravT has quit IRC | 18:54 | |
*** TravT has joined #openstack-searchlight | 18:55 | |
openstackgerrit | Steve McLellan proposed openstack/searchlight: Fix --type handling for indexing parent plugins https://review.openstack.org/332242 | 19:36 |
*** matt-borland has joined #openstack-searchlight | 19:45 | |
matt-borland | If anyone can review https://review.openstack.org/#/c/335041/, it has a +2 on it; it allows for linking to client-side views | 19:46 |
matt-borland | (which makes things nice and fast) :) | 19:46 |
*** ddieterly is now known as ddieterly[away] | 19:53 | |
matt-borland | thanks david-lyle :) | 20:01 |
*** ddieterly[away] is now known as ddieterly | 20:02 | |
*** sjmc7 has quit IRC | 20:07 | |
openstackgerrit | Merged openstack/searchlight-ui: Link search hits to registered details views if available https://review.openstack.org/335041 | 20:10 |
*** ddieterly is now known as ddieterly[away] | 20:26 | |
*** TravT has quit IRC | 20:47 | |
*** ddieterly[away] is now known as ddieterly | 20:48 | |
*** TravT has joined #openstack-searchlight | 21:14 | |
*** TravT has quit IRC | 21:14 | |
*** TravT has joined #openstack-searchlight | 21:15 | |
*** TravT has quit IRC | 21:16 | |
*** TravT has joined #openstack-searchlight | 21:17 | |
*** ddieterly is now known as ddieterly[away] | 21:18 | |
*** ddieterly[away] is now known as ddieterly | 21:20 | |
*** notmyname has quit IRC | 21:28 | |
*** notmyname_ has joined #openstack-searchlight | 21:28 | |
*** lcastell_ has joined #openstack-searchlight | 21:28 | |
*** notmyname_ is now known as notmyname | 21:29 | |
*** lcastell has quit IRC | 21:29 | |
*** lcastell_ is now known as lcastell | 21:29 | |
*** sjmc7 has joined #openstack-searchlight | 21:39 | |
*** matt-borland has quit IRC | 21:45 | |
*** ddieterly is now known as ddieterly[away] | 22:05 | |
*** TravT has quit IRC | 22:12 | |
*** TravT has joined #openstack-searchlight | 22:14 | |
openstackgerrit | Travis Tripp proposed openstack/searchlight: Remove uneccessary line in resources test script https://review.openstack.org/336306 | 22:22 |
openstackgerrit | Travis Tripp proposed openstack/searchlight: Fix broken incrementer https://review.openstack.org/336306 | 22:31 |
*** ddieterly[away] is now known as ddieterly | 22:36 | |
*** ddieterly is now known as ddieterly[away] | 23:02 | |
*** ddieterly[away] has quit IRC | 23:02 | |
*** sjmc7 has quit IRC | 23:12 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!