Thursday, 2016-03-03

*** yingjun has quit IRC00:01
*** lakshmiS has quit IRC00:06
*** bpokorny has quit IRC00:21
*** sjmc7 has quit IRC00:29
*** bpokorny has joined #openstack-searchlight00:33
*** sjmc7 has joined #openstack-searchlight00:54
*** bpokorny_ has joined #openstack-searchlight00:58
*** bpokorny has quit IRC01:01
*** yingjun has joined #openstack-searchlight01:08
*** sjmc7 has quit IRC01:08
openstackgerritMerged openstack/searchlight: Remove KEYSTONE_CATALOG_BACKEND in devstack plugin  https://review.openstack.org/27968601:14
*** bpokorny_ has quit IRC01:41
*** bpokorny has joined #openstack-searchlight01:42
*** openstackgerrit has quit IRC01:47
*** openstackgerrit has joined #openstack-searchlight01:48
openstackgerritLi Yingjun proposed openstack/python-searchlightclient: Update README  https://review.openstack.org/28755501:56
*** bpokorny has quit IRC03:08
*** briancline has quit IRC03:26
*** briancline has joined #openstack-searchlight03:26
*** lakshmiS has joined #openstack-searchlight03:34
*** yingjun has quit IRC04:41
*** yingjun has joined #openstack-searchlight04:42
*** lakshmiS has quit IRC06:30
*** pcaruana has joined #openstack-searchlight07:45
*** itisha has joined #openstack-searchlight07:59
*** GB21 has joined #openstack-searchlight08:17
*** yingjun has quit IRC09:32
*** yingjun has joined #openstack-searchlight09:32
*** yingjun has quit IRC09:37
*** openstackgerrit_ has joined #openstack-searchlight09:54
*** openstackgerrit_ has quit IRC10:02
*** qwebirc37016 has joined #openstack-searchlight10:19
qwebirc37016Hello guys needed some info here10:19
qwebirc37016i can see that search light is supported by nova glance10:19
qwebirc37016but i cannot see anywhere in the resource list whether heat network also supported by search light10:20
qwebirc37016Is anyone there??10:59
*** GB21 has quit IRC11:00
*** robcresswell has joined #openstack-searchlight11:01
*** GB21 has joined #openstack-searchlight11:31
*** openstackgerrit has quit IRC12:03
*** openstackgerrit has joined #openstack-searchlight12:04
*** yingjun has joined #openstack-searchlight12:08
*** itisha has quit IRC12:09
*** GB21 has quit IRC12:30
*** TravT_ has joined #openstack-searchlight13:30
*** TravT has quit IRC13:33
*** GB21 has joined #openstack-searchlight13:58
*** yingjun has quit IRC14:07
*** yingjun has joined #openstack-searchlight14:07
*** GB21 has quit IRC14:11
*** yingjun has quit IRC14:11
*** GB21 has joined #openstack-searchlight14:21
*** sjmc7 has joined #openstack-searchlight14:54
*** GB21 has quit IRC14:55
*** yingjun has joined #openstack-searchlight14:58
TravT_Courtesy Searchlight meeting reminder in #openstack-meeting-4: lakshmiS, nikhil_k, rosmaita, TravT, david-lyle, kragniz, sjmc7, briancline, lei-zh, yingjun, RickA-HP15:00
nikhilo/15:00
*** lei-zh has joined #openstack-searchlight15:00
*** lakshmiS has joined #openstack-searchlight15:02
*** sigmavirus24_awa is now known as sigmavirus2415:18
openstackgerritLei Zhang proposed openstack/searchlight: Remove version parameter from index helper save method  https://review.openstack.org/28729315:50
openstackgerritLei Zhang proposed openstack/searchlight: Fix role based indexing for designate  https://review.openstack.org/28090815:50
*** lakshmiS has quit IRC16:00
TravT_qwebirc37016: the heat plugin has not been completed yet for searchlight16:05
TravT_also, btw that most people active in this channel are in the US.16:06
TravT_there are a few in China as well16:06
*** yingjun has quit IRC16:12
*** yingjun has joined #openstack-searchlight16:13
*** yingjun has quit IRC16:17
*** lei-zh has quit IRC16:31
*** exploreshaifali has joined #openstack-searchlight16:45
*** bpokorny has joined #openstack-searchlight16:49
*** lakshmiS has joined #openstack-searchlight16:51
*** exploreshaifali has quit IRC17:04
lakshmiSnikhil: can we put the bug# link under every location or is it based on who agrees to get assigned to under that location?17:13
lakshmiS^ on bugsmash etherpad17:13
nikhillakshmiS: you are free to put that in every location, but I doubt it that common etherpad is being used for the same17:14
nikhillakshmiS: the organizers want list of bugs solved in the event17:14
nikhillakshmiS: so, I recommend adding a section to "Open for all locations" at the top of the page and just add once17:14
lakshmiSyeah that makes sense for someone to pick it up17:15
nikhilI think that may be more useful than dividing it17:15
lakshmiSyup17:15
nikhilI am a little mad17:15
nikhilat the organizer that he divided17:15
nikhilthe etherpad in the first place17:16
nikhilthe collaboration is lost17:16
lakshmiSwhat did you have in mind17:16
openstackgerritMichael Krotscheck proposed openstack/searchlight: Moved CORS middleware configuration into oslo-config-generator  https://review.openstack.org/28735417:50
krotscheckSorry about ^^ - had to fix whitespace.17:50
sjmc7hi lakshmiS - with https://review.openstack.org/#/c/280908, i’m able to do initial recordset indexing ok18:04
lakshmiSthat is for notification handler on initial domain creation18:06
lakshmiSwhich triggers initial recordsets creation for SOA,...18:06
lakshmiSto recreate try deleting searchlight index from es and create a new domain on designate18:07
sjmc7before initial indexing?18:12
sjmc7that won’t work - there won’t be any mappings18:12
sjmc7i wouldn’t expect it to work18:13
sjmc7did you try it with the mappings applied but no initial data?18:13
lakshmiSit does create zone document18:14
sjmc7sure. but in general you’ll get weird behavior18:14
lakshmiSyes there are no mappings when you delete index.18:14
sjmc7elasticsearch will decide a mapping for itself18:15
lakshmiSyeah its a corner case and nothing prevents listener from trying to index data18:15
sjmc7ok. but i don’t think it only applies to this patch - it affects all resources18:15
lakshmiSyes thats true. just happened be found here while i was testing it18:16
lakshmiSand specially since it has child plugin mappings18:16
openstackgerritSteve McLellan proposed openstack/searchlight: Fix role based indexing for designate  https://review.openstack.org/28090818:17
*** exploreshaifali has joined #openstack-searchlight18:19
*** TravT_ has quit IRC18:30
krotscheckI made a whitespace change on https://review.openstack.org/287354, and suddenly all the glance tests are broken.18:49
krotscheckSomething about class 'mock.mock.Mock' can't be encoded.18:49
sjmc7krotscheck: probably a dependency change somewhere18:49
sjmc7will take a look in a few minutes18:49
krotschecksjmc7: I figured as much. I've got a before/after log, lemme see if I can compare the freezes.18:50
sjmc7ta18:50
sjmc7it’s release time, everything’s changing :)18:50
krotschecksjmc7: Glanceclient was released, 1.2.0 to 2.0.018:57
sjmc7raaar!18:58
sjmc7joy. thanks for checking; i’ll fire up the debugger once i’m done stuffing my face18:58
* krotscheck gets a perverted kick out of being the bearer of bad news.18:59
sjmc7:D18:59
sjmc7https://bugs.launchpad.net/searchlight/+bug/1552875 krotscheck19:15
openstackLaunchpad bug 1552875 in OpenStack Search (Searchlight) "Tests failing after glanceclient 2.0.0 release" [Critical,Confirmed]19:15
krotschecksjmc7: Yes19:15
krotscheckSee it19:15
sjmc7TravT: i’ll get a fix up ASAP; don’t take m3 before we get this fixed19:16
sjmc7tag19:16
openstackgerritSteve McLellan proposed openstack/searchlight: Remove pre-1.0 glance auth error handling  https://review.openstack.org/28807919:34
sjmc7krotscheck: ^19:34
sjmc7we should’ve done that ages ago19:35
krotscheckSweet, I19:37
krotscheckI'll rebase after lunch19:37
sjmc7ok. we’ll get it merged today unless i blundered19:37
sjmc7lakshmiS: good catch on the neutron rbac query19:44
sjmc7i’m not certain about the restriction on ports though19:44
sjmc7are you sure ports belonging to other tenants on shared networks are visible?19:45
sjmc7that seems suspect to me19:45
*** exploreshaifali has quit IRC19:48
lakshmiSsjmc7: back from lunch.20:02
lakshmiSport doesnt seem to show up on shared networks20:02
lakshmiSits only network filter then20:03
lakshmiSthere are no subnets being indexed right now but in future when then get added they are visible for non-tenant users20:03
lakshmiSs/then/they20:04
sjmc7yep20:04
sjmc7ok, added it for networks. adding a test for that now20:04
*** TravT has joined #openstack-searchlight20:22
*** TravT has quit IRC20:31
*** TravT has joined #openstack-searchlight20:36
*** itisha has joined #openstack-searchlight20:55
openstackgerritSteve McLellan proposed openstack/searchlight: Neutron plugin  https://review.openstack.org/28204321:08
openstackgerritMichael Krotscheck proposed openstack/searchlight: Moved CORS middleware configuration into oslo-config-generator  https://review.openstack.org/28735421:22
sjmc7krotscheck: looking at the CORS patch; this is after the mailing list discussion over the last week or so?21:24
krotschecksjmc7: Yep21:43
krotscheckLooks like the rebase on top of yours fixed it21:43
sjmc7ok. same change is going in on other projects as well?21:43
*** TravT has quit IRC21:44
openstackgerritSteve McLellan proposed openstack/searchlight: Neutron plugin  https://review.openstack.org/28204321:48
*** TravT has joined #openstack-searchlight22:13
sjmc7krotscheck: still there? i don’t see the cors middleware adding its options to the config parser anywhere22:26
krotscheckYep22:27
krotschecksec.22:27
krotscheckit _should_22:27
sjmc7set_defaults only does anything if the option exists (rather than exploding) so i don’t think it’s doing anything22:27
krotscheck*blink*22:28
sjmc7or maybe we’re missing a function call to add them somewhere22:28
krotscheckThat seems like a bloody obvious oversight on my part22:28
sjmc7:)22:29
sjmc7checking hjow other oslo libs add their options22:29
sjmc7ah, they have a ‘_register’ function in opts.py22:29
sjmc7oslo_middleware doesn’t seem to have one22:30
sjmc7if you look in oslo_policy, for instance22:30
sjmc7its set_defaults registers the options first, so they only get registered if you intend to use them which makes sense22:30
krotschecksjmc7: blah blah freature freeze I have a patch i believe22:32
krotscheckhttps://review.openstack.org/#/c/285368/22:33
sjmc7:D  if it’s a problem in SL it’s no big deal to treat this as a bug fix. changes to oslo, well, good luck :)22:33
krotscheckyeaaah22:33
openstackgerritMerged openstack/searchlight: Remove pre-1.0 glance auth error handling  https://review.openstack.org/28807922:33
* krotscheck facepalms22:33
openstackgerritMerged openstack/searchlight: Neutron plugin  https://review.openstack.org/28204322:33
sjmc7i think you’ll still need to register the options somewhere22:33
sjmc7typically config.register_opts(<some list>, <group name>)22:34
krotschecksjmc7: Waaait a sec22:35
krotscheckNo22:35
krotscheckWhat the cfg.set_defaults does is override the values in cfg.CONF with new values22:35
krotscheckSorry22:35
krotscheckno22:35
sjmc7i am definitely willing to admit i’m wrong22:35
sjmc7but i am debugging it and i don’t see it setting those values22:35
krotscheckIt overrides the values in (opts passed to method)22:35
sjmc7ah, hmm..22:36
*** lakshmiS has quit IRC22:36
sjmc7yeah, i may be mistaken. however, in that case i don’t know why they’re not getting set22:36
krotschecksjmc7: Are you looking at the default options in oslo_middleware.cors.CORS, rather than cfg.CONF?22:39
krotscheckSorry- CORS_OPTS22:39
sjmc7ah, they’re separate? then yes, possibly22:39
sjmc7in which case, my mistake22:40
krotschecksjmc7: Righto.22:41
openstackgerritSteve McLellan proposed openstack/searchlight: Fix role based indexing for designate  https://review.openstack.org/28090822:42
openstackgerritMerged openstack/searchlight: Fix role based indexing for designate  https://review.openstack.org/28090822:55
*** GB21 has joined #openstack-searchlight23:09
*** itisha has quit IRC23:39

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