Friday, 2016-03-18

« Thursday, 2016-03-17 Index Saturday, 2016-03-19 »
*** preetika has quit IRC00:17
openstackgerritTravis Tripp proposed openstack/searchlight: Update Docs for Mitaka Release  https://review.openstack.org/29433300:22
*** yingjun has joined #openstack-searchlight00:40
*** bpokorny_ has joined #openstack-searchlight00:43
*** bpokorny has quit IRC00:46
*** bpokorny_ has quit IRC00:47
*** lakshmiS has joined #openstack-searchlight02:37
openstackgerritMerged openstack/searchlight: Update Docs for Mitaka Release  https://review.openstack.org/29433302:52
openstackgerritSteve McLellan proposed openstack/searchlight: Add neutron subnets and routers  https://review.openstack.org/29377102:55
*** lakshmiS_ has joined #openstack-searchlight04:48
*** lakshmiS has quit IRC04:52
*** lakshmiS_ has quit IRC05:12
*** GB21 has joined #openstack-searchlight05:16
*** itisha has quit IRC05:19
*** pcaruana has quit IRC06:18
openstackgerritLei Zhang proposed openstack/searchlight-specs: Move notification-forwarding for SearchLight BP to Newton  https://review.openstack.org/29441806:34
*** TravT_ has joined #openstack-searchlight07:23
*** TravT has quit IRC07:25
*** GB21 has quit IRC07:25
*** pcaruana has joined #openstack-searchlight07:57
*** GB21 has joined #openstack-searchlight08:07
*** GB21 has quit IRC08:27
*** GB21 has joined #openstack-searchlight08:27
*** GB21 has quit IRC08:45
*** GB21 has joined #openstack-searchlight08:45
*** GB21 has quit IRC08:55
*** preetika has joined #openstack-searchlight08:58
*** GB21 has joined #openstack-searchlight09:23
*** yingjun has quit IRC09:43
*** yingjun has joined #openstack-searchlight09:44
*** yingjun has quit IRC09:48
*** GB21 has quit IRC10:17
*** yingjun has joined #openstack-searchlight11:34
*** yingjun has quit IRC11:35
*** yingjun has joined #openstack-searchlight11:36
*** yingjun has quit IRC11:40
*** preetika has quit IRC12:07
*** openstackgerrit has quit IRC12:48
*** openstackgerrit has joined #openstack-searchlight12:49
*** itisha has joined #openstack-searchlight13:13
*** sigmavirus24_awa is now known as sigmavirus2414:12
*** pcaruana has quit IRC14:38
*** nehap has joined #openstack-searchlight15:01
*** bpokorny has joined #openstack-searchlight15:02
openstackgerritMerged openstack/searchlight-specs: Move notification-forwarding for SearchLight BP to Newton  https://review.openstack.org/29441815:21
sjmc7TravT_: did you want https://review.openstack.org/#/c/294633/ merged before the rc tag?15:30
TravT_it couldn't hurt15:30
*** TravT_ is now known as TravT15:30
TravTFYI, i'm going through the subnet / routers patch right now15:30
sjmc7:D famous last words15:30
*** david-lyle has quit IRC15:32
*** david-lyle has joined #openstack-searchlight15:33
openstackgerritRick Aulino proposed openstack/searchlight: Fix duplicate entries displayed w/"index sync".  https://review.openstack.org/29416815:42
TravTsjmc7 i added a comment on the subnet mapping.15:45
sjmc7yeah, looking at it15:45
sjmc7where do you see created_at?15:45
TravTduring initial indexing.15:45
*** lakshmiS has joined #openstack-searchlight15:45
sjmc7hmmm15:45
TravTi can show you my debugger if you want15:45
sjmc7i can run it. i wonder where i’m getting it from :D15:46
TravTfor ports, there are a bunch of fields not mapped15:46
TravTis that because they don't come through in notifications?15:46
sjmc7umm.. yeah, most likely15:47
sjmc7which ones in particular?15:47
TravTit probably wouldn't hurt to comment on them.15:47
lakshmiSsjmc7: i did notice that port name is empty15:47
sjmc7did you give the port a name?15:47
TravThttp://pasteboard.co/2mpmi659.png15:47
lakshmiSyes15:47
lakshmiSdo you see a port name in results?15:48
lakshmiSlet me verify that again15:48
*** nehap has quit IRC15:48
sjmc7how many questions are in flight? :)15:48
sjmc7the extra binding: fields you have there, i don’t have, so maybe i need to add those15:49
sjmc7looks like OVS versus ‘normal'15:49
TravTyeah, port also has dynamic mapping true15:49
TravTso we should add explicit mappings15:50
sjmc7or set it to false15:50
TravTor both15:50
sjmc7you want me to make port changes as part of this patch?15:50
TravTumm... no, that can be separate15:51
TravTi just have debugger going15:51
sjmc7ok. it looks like there are some fields need adding. i’ll file a separate bug15:51
TravTok.15:52
sjmc7still not seeing created_at anywhere15:52
TravTthat was in subnet15:53
TravTon index list15:53
TravTmaybe they just added it?15:53
sjmc7hhmm, ok. lakshmiS, checking por tname15:53
TravTmy devstack is 4 days old15:53
sjmc7hahaha, that’d be nice of them15:53
lakshmiSsjmc7: it does show when port name is given15:54
lakshmiSbut i got confused by horizon15:54
sjmc7ok15:54
lakshmiSsince it shows names as "(d7cc5540-8eac)" and our index has null for the same15:54
sjmc7my devstack’s about 3 weeks old, so maybe they added it15:54
sjmc7(dates)15:54
lakshmiShorizon is probably setting some default if name is null15:55
sjmc7lakshmiS: yeah, i think it does some things like ‘detached’ if a port’s not connected15:55
TravTyes15:56
TravThttps://github.com/openstack/horizon/blob/master/openstack_dashboard/dashboards/project/networks/ports/tables.py#L67-L7015:56
TravTthat is the data15:56
TravTbut the column name is "name or id"15:57
TravTis -> isn't15:57
lakshmiSeven though it only shows partial id as name15:57
lakshmiSif the id is d7cc5540-8eac-40a9-9dc5-fe488488b328 it shows port name as (d7cc5540-8eac)15:58
TravTprobably just to save room and that is enough to recognize it15:58
lakshmiSoh15:58
TravTi don't know15:58
TravTok, i'm gonna keep on debugging on subnets / routers15:59
lakshmiSwill go with events testing...15:59
*** david-lyle has quit IRC16:00
*** david-lyle has joined #openstack-searchlight16:01
david-lylelakshmiS: horizon shows name or id if name is not set16:02
*** bpokorny has quit IRC16:02
david-lylemy internet died for a minute so someone may have mentioned that already16:02
lakshmiSyeah travis did :)16:02
david-lyleok, good16:02
TravTdavid-lyle thanks, though16:02
*** nehap has joined #openstack-searchlight16:10
*** bpokorny has joined #openstack-searchlight16:13
TravTsjmc7: a couple mapping items in routers as well.16:32
TravTwill now try to view via horizon and run through notifications16:32
*** pcaruana has joined #openstack-searchlight16:47
*** nehap has quit IRC16:50
*** bpokorny_ has joined #openstack-searchlight16:53
sjmc7ok. i’ll fix the ports one as well, that’s reasonably low risk. the dhcp port change is a bit bigger, will need to consider it for RC216:54
*** bpokorny_ has quit IRC16:54
*** bpokorny_ has joined #openstack-searchlight16:54
*** bpokorny has quit IRC16:56
david-lyleTravT: your Manifest.in on searchlight-ui is wrong, do you want a pull request, or wait until the repo is approved and go through gerrit?16:58
david-lyleI'll just put up a pull request16:58
lakshmiSsjmc7: when you say dhcp port change, do you mean the way a default port is created when you create subnet?17:00
lakshmiSwith device type "network:dhcp"17:01
sjmc7yeah, i’m making some changes to deal with that magic port but i think if we’re looking to tag RC1 soon it’s not sensible to rush that in17:01
sjmc7yep17:01
sjmc7it’s registered as a separate bug17:01
lakshmiSok just wanted to confirm17:01
sjmc7yeah. i was hoping to finish it this morning but it’s really fiddly17:01
TravTdavid-lyle: go ahead with pull request17:02
sjmc7it’s not just one port belonging to the subnet, it’s a port that may or may not be there on the network17:02
david-lyleactually TravT nevermind, it's working pbr magic17:07
* david-lyle returns to the shadows17:08
TravTdavid-lyle: i actually got excited to see you looking it!17:15
david-lylejust walking through it now17:16
david-lylereviewed the project config and gov patches17:16
david-lylenow looking at the plugin17:16
openstackgerritSteve McLellan proposed openstack/searchlight: Add missing fields to ports mapping  https://review.openstack.org/29473317:34
sjmc7urgh, sorry TravT , missed the email about the review. will fix those mapping issues17:36
sjmc7TravT: looks like they did add date fields in the last few weeks17:43
sjmc7fancy that.17:43
sjmc7will add them17:44
*** david-lyle_ has joined #openstack-searchlight17:44
*** david-lyle has quit IRC17:45
*** david-lyle_ is now known as david-lyle17:46
*** itisha has quit IRC17:59
sjmc7haha, routers didn’t get them. for reasons!18:04
TravTsjmc718:07
TravTcool18:07
sjmc7putting patch up in a minute, added those other fields18:07
TravTwell, i just fixed 3 issues in searchlight ui on horizon master patch...18:07
*** bpokorny_ has quit IRC18:07
sjmc7ok. any other router/subnet issues?18:07
TravTwill push to new repo.18:07
TravThaven't seen any yet18:07
*** bpokorny has joined #openstack-searchlight18:07
TravTlet me try delete, though18:07
TravTdelete seems to be something that catches us18:07
*** bpokorny has quit IRC18:11
*** bpokorny has joined #openstack-searchlight18:12
sjmc7ok. uploading patch now with those fixes18:14
TravTokay will pull18:16
sjmc7i’ll hold on18:16
sjmc7gah, hold one, one sec18:16
sjmc7fixing one test18:16
TravTokay18:16
sjmc7graggh, VM crashed :(18:21
TravTthat bodes well ;)18:22
sjmc7need to check if subnet delete notifications contain date info18:24
sjmc7i need to correct dates for the network and port plugins too, but not sure about squeaking that in for RC1 if you’re doing it this afternoon18:26
TravTlet me go talk to doug18:27
sjmc7ok. i need to take a quick break soon18:27
sjmc7and i know you’re hoping to take off in a bit18:27
sjmc7the ports one will take maybe 30 minutes18:27
*** itisha has joined #openstack-searchlight18:29
openstackgerritRick Aulino proposed openstack/searchlight: Fix duplicate entries displayed w/"index sync".  https://review.openstack.org/29416818:29
TravTdoug just said it would be okay to tag rc1 early next week.18:30
sjmc7“okay” or “why i oughtta” ? :)18:30
TravTok.18:31
sjmc7i guess for non-devstack-core it’s less of a big deal18:31
TravTbut it is unfortunate18:31
sjmc7ok. let me get this subnet patch up, then i really need to eat something18:31
TravTmaybe makes his job easier today.18:31
sjmc7when are you hoping to duck out?18:31
TravTyeah, i am probably going to leave shortly18:31
TravTlakshmiS: if you could bang on it a bit?18:31
sjmc7i can maybe get the date one done, but the dhcp one i need to test a lot more18:31
lakshmiSwhat am i testing? date fields?18:32
lakshmiSTravT ^18:32
TravTyes18:36
TravTwe discovered neutron added date fields very recently18:36
TravTbut also should do a round of general testing18:36
sjmc7they got added to networks, subnets, ports18:36
sjmc7i’ve added tem to subnets in the subnet/router patch18:36
sjmc7which i will check in imminently18:37
sjmc7filed a bug for nets and ports18:37
openstackgerritSteve McLellan proposed openstack/searchlight: Add neutron subnets and routers  https://review.openstack.org/29377118:46
sjmc7lakshmiS: addressed all the comments in that patch18:46
sjmc7^18:46
lakshmiSok will do another sanity check18:46
sjmc7i’ll approve rick’s change18:47
sjmc7ok, i need to take a quick break18:48
sjmc7TravT: if you did want to do RC1 today i’m ok with that, we’ll need to tag those last two bugs for RC2 though18:50
*** TravT has quit IRC18:50
*** TravT has joined #openstack-searchlight18:51
*** bpokorny_ has joined #openstack-searchlight18:53
*** bpokorny_ has quit IRC18:54
*** bpokorny has quit IRC18:57
*** bpokorny has joined #openstack-searchlight18:58
lakshmiSsjmc7: indexing failed for updated_at field.19:14
lakshmiSwill be back after lunch19:15
openstackgerritMerged openstack/searchlight: Fix duplicate entries displayed w/"index sync".  https://review.openstack.org/29416819:26
*** nehap has joined #openstack-searchlight19:27
sjmc7lakshmiS: when did you last update devstack?19:35
sjmc7they added the date fields sometime in the last 3 weeks19:35
*** sjmc7 has left #openstack-searchlight19:36
*** sjmc7 has joined #openstack-searchlight19:36
lakshmiSsjmc7: that might be the problem then. ignore it19:46
sjmc7try neutron subnet-list (or net-list)19:46
sjmc7if you don’t see created_at/updated_at then it’s more recent than your neutron19:47
sjmc7it’s like they listened to my complaints!19:47
lakshmiS:)19:47
lakshmiSyou are right. its my devstack not upto date19:48
openstackgerritMerged openstack/searchlight: Mitaka doc updates  https://review.openstack.org/29463319:52
lakshmiSmy devstack barks at me after neutron update. will take a while before i get the latest devstack to test again.19:53
openstackgerritSteve McLellan proposed openstack/searchlight: Add neutron subnets and routers  https://review.openstack.org/29377119:55
*** bpokorny has quit IRC20:06
*** bpokorny has joined #openstack-searchlight20:24
sjmc7yeah, i had to restack20:47
*** nehap has quit IRC20:55
openstackgerritSteve McLellan proposed openstack/searchlight: Add neutron subnets and routers  https://review.openstack.org/29377121:06
*** david-lyle has quit IRC21:06
*** david-lyle has joined #openstack-searchlight21:07
sjmc7lakshmiS: travis suggested keeping the code around in case neutron’s older than the change that added dates.. i’m not sure how i feel about keep code around for backwards compatibility like that. thoughts?21:15
lakshmiShmm. werent you previously checking updated_at vs current time.21:16
sjmc7yeah21:16
lakshmiSwe could warn them but its tricky since we may forget to remove it later21:17
sjmc7but there’s some plumbing to pass it around21:17
sjmc7yeah, nobody’ll read warnings21:17
lakshmiS:)21:17
lakshmiSits to remind us21:17
sjmc7i dunno, it seems reasonable that if you’re running mitaka RC1 of SL you’ve got RC1 of neutron21:18
lakshmiSyeah its better not to put any checks in our code21:18
sjmc7ok, i’m gonna take it out. he can add it back if he wants :)21:19
lakshmiSah aaah21:20
lakshmiSi have to drop in few mins. will test once my devstack is done installing. seems the patch is pretty solid now though21:24
*** dhellmann has quit IRC21:26
*** dhellmann has joined #openstack-searchlight21:26
*** lakshmiS has quit IRC21:27
*** sigmavirus24 is now known as sigmavirus24_awa22:00
openstackgerritSteve McLellan proposed openstack/searchlight: Add missing fields to ports mapping  https://review.openstack.org/29473322:43
*** akanksha_ has joined #openstack-searchlight23:54
« Thursday, 2016-03-17 Index Saturday, 2016-03-19 »

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