*** yingjun has quit IRC | 00:01 | |
*** lakshmiS has quit IRC | 00:06 | |
*** bpokorny has quit IRC | 00:21 | |
*** sjmc7 has quit IRC | 00:29 | |
*** bpokorny has joined #openstack-searchlight | 00:33 | |
*** sjmc7 has joined #openstack-searchlight | 00:54 | |
*** bpokorny_ has joined #openstack-searchlight | 00:58 | |
*** bpokorny has quit IRC | 01:01 | |
*** yingjun has joined #openstack-searchlight | 01:08 | |
*** sjmc7 has quit IRC | 01:08 | |
openstackgerrit | Merged openstack/searchlight: Remove KEYSTONE_CATALOG_BACKEND in devstack plugin https://review.openstack.org/279686 | 01:14 |
---|---|---|
*** bpokorny_ has quit IRC | 01:41 | |
*** bpokorny has joined #openstack-searchlight | 01:42 | |
*** openstackgerrit has quit IRC | 01:47 | |
*** openstackgerrit has joined #openstack-searchlight | 01:48 | |
openstackgerrit | Li Yingjun proposed openstack/python-searchlightclient: Update README https://review.openstack.org/287555 | 01:56 |
*** bpokorny has quit IRC | 03:08 | |
*** briancline has quit IRC | 03:26 | |
*** briancline has joined #openstack-searchlight | 03:26 | |
*** lakshmiS has joined #openstack-searchlight | 03:34 | |
*** yingjun has quit IRC | 04:41 | |
*** yingjun has joined #openstack-searchlight | 04:42 | |
*** lakshmiS has quit IRC | 06:30 | |
*** pcaruana has joined #openstack-searchlight | 07:45 | |
*** itisha has joined #openstack-searchlight | 07:59 | |
*** GB21 has joined #openstack-searchlight | 08:17 | |
*** yingjun has quit IRC | 09:32 | |
*** yingjun has joined #openstack-searchlight | 09:32 | |
*** yingjun has quit IRC | 09:37 | |
*** openstackgerrit_ has joined #openstack-searchlight | 09:54 | |
*** openstackgerrit_ has quit IRC | 10:02 | |
*** qwebirc37016 has joined #openstack-searchlight | 10:19 | |
qwebirc37016 | Hello guys needed some info here | 10:19 |
qwebirc37016 | i can see that search light is supported by nova glance | 10:19 |
qwebirc37016 | but i cannot see anywhere in the resource list whether heat network also supported by search light | 10:20 |
qwebirc37016 | Is anyone there?? | 10:59 |
*** GB21 has quit IRC | 11:00 | |
*** robcresswell has joined #openstack-searchlight | 11:01 | |
*** GB21 has joined #openstack-searchlight | 11:31 | |
*** openstackgerrit has quit IRC | 12:03 | |
*** openstackgerrit has joined #openstack-searchlight | 12:04 | |
*** yingjun has joined #openstack-searchlight | 12:08 | |
*** itisha has quit IRC | 12:09 | |
*** GB21 has quit IRC | 12:30 | |
*** TravT_ has joined #openstack-searchlight | 13:30 | |
*** TravT has quit IRC | 13:33 | |
*** GB21 has joined #openstack-searchlight | 13:58 | |
*** yingjun has quit IRC | 14:07 | |
*** yingjun has joined #openstack-searchlight | 14:07 | |
*** GB21 has quit IRC | 14:11 | |
*** yingjun has quit IRC | 14:11 | |
*** GB21 has joined #openstack-searchlight | 14:21 | |
*** sjmc7 has joined #openstack-searchlight | 14:54 | |
*** GB21 has quit IRC | 14:55 | |
*** yingjun has joined #openstack-searchlight | 14:58 | |
TravT_ | Courtesy Searchlight meeting reminder in #openstack-meeting-4: lakshmiS, nikhil_k, rosmaita, TravT, david-lyle, kragniz, sjmc7, briancline, lei-zh, yingjun, RickA-HP | 15:00 |
nikhil | o/ | 15:00 |
*** lei-zh has joined #openstack-searchlight | 15:00 | |
*** lakshmiS has joined #openstack-searchlight | 15:02 | |
*** sigmavirus24_awa is now known as sigmavirus24 | 15:18 | |
openstackgerrit | Lei Zhang proposed openstack/searchlight: Remove version parameter from index helper save method https://review.openstack.org/287293 | 15:50 |
openstackgerrit | Lei Zhang proposed openstack/searchlight: Fix role based indexing for designate https://review.openstack.org/280908 | 15:50 |
*** lakshmiS has quit IRC | 16:00 | |
TravT_ | qwebirc37016: the heat plugin has not been completed yet for searchlight | 16: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 well | 16:06 |
*** yingjun has quit IRC | 16:12 | |
*** yingjun has joined #openstack-searchlight | 16:13 | |
*** yingjun has quit IRC | 16:17 | |
*** lei-zh has quit IRC | 16:31 | |
*** exploreshaifali has joined #openstack-searchlight | 16:45 | |
*** bpokorny has joined #openstack-searchlight | 16:49 | |
*** lakshmiS has joined #openstack-searchlight | 16:51 | |
*** exploreshaifali has quit IRC | 17:04 | |
lakshmiS | nikhil: 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 etherpad | 17:13 |
nikhil | lakshmiS: you are free to put that in every location, but I doubt it that common etherpad is being used for the same | 17:14 |
nikhil | lakshmiS: the organizers want list of bugs solved in the event | 17:14 |
nikhil | lakshmiS: so, I recommend adding a section to "Open for all locations" at the top of the page and just add once | 17:14 |
lakshmiS | yeah that makes sense for someone to pick it up | 17:15 |
nikhil | I think that may be more useful than dividing it | 17:15 |
lakshmiS | yup | 17:15 |
nikhil | I am a little mad | 17:15 |
nikhil | at the organizer that he divided | 17:15 |
nikhil | the etherpad in the first place | 17:16 |
nikhil | the collaboration is lost | 17:16 |
lakshmiS | what did you have in mind | 17:16 |
openstackgerrit | Michael Krotscheck proposed openstack/searchlight: Moved CORS middleware configuration into oslo-config-generator https://review.openstack.org/287354 | 17:50 |
krotscheck | Sorry about ^^ - had to fix whitespace. | 17:50 |
sjmc7 | hi lakshmiS - with https://review.openstack.org/#/c/280908, i’m able to do initial recordset indexing ok | 18:04 |
lakshmiS | that is for notification handler on initial domain creation | 18:06 |
lakshmiS | which triggers initial recordsets creation for SOA,... | 18:06 |
lakshmiS | to recreate try deleting searchlight index from es and create a new domain on designate | 18:07 |
sjmc7 | before initial indexing? | 18:12 |
sjmc7 | that won’t work - there won’t be any mappings | 18:12 |
sjmc7 | i wouldn’t expect it to work | 18:13 |
sjmc7 | did you try it with the mappings applied but no initial data? | 18:13 |
lakshmiS | it does create zone document | 18:14 |
sjmc7 | sure. but in general you’ll get weird behavior | 18:14 |
lakshmiS | yes there are no mappings when you delete index. | 18:14 |
sjmc7 | elasticsearch will decide a mapping for itself | 18:15 |
lakshmiS | yeah its a corner case and nothing prevents listener from trying to index data | 18:15 |
sjmc7 | ok. but i don’t think it only applies to this patch - it affects all resources | 18:15 |
lakshmiS | yes thats true. just happened be found here while i was testing it | 18:16 |
lakshmiS | and specially since it has child plugin mappings | 18:16 |
openstackgerrit | Steve McLellan proposed openstack/searchlight: Fix role based indexing for designate https://review.openstack.org/280908 | 18:17 |
*** exploreshaifali has joined #openstack-searchlight | 18:19 | |
*** TravT_ has quit IRC | 18:30 | |
krotscheck | I made a whitespace change on https://review.openstack.org/287354, and suddenly all the glance tests are broken. | 18:49 |
krotscheck | Something about class 'mock.mock.Mock' can't be encoded. | 18:49 |
sjmc7 | krotscheck: probably a dependency change somewhere | 18:49 |
sjmc7 | will take a look in a few minutes | 18:49 |
krotscheck | sjmc7: I figured as much. I've got a before/after log, lemme see if I can compare the freezes. | 18:50 |
sjmc7 | ta | 18:50 |
sjmc7 | it’s release time, everything’s changing :) | 18:50 |
krotscheck | sjmc7: Glanceclient was released, 1.2.0 to 2.0.0 | 18:57 |
sjmc7 | raaar! | 18:58 |
sjmc7 | joy. thanks for checking; i’ll fire up the debugger once i’m done stuffing my face | 18:58 |
* krotscheck gets a perverted kick out of being the bearer of bad news. | 18:59 | |
sjmc7 | :D | 18:59 |
sjmc7 | https://bugs.launchpad.net/searchlight/+bug/1552875 krotscheck | 19:15 |
openstack | Launchpad bug 1552875 in OpenStack Search (Searchlight) "Tests failing after glanceclient 2.0.0 release" [Critical,Confirmed] | 19:15 |
krotscheck | sjmc7: Yes | 19:15 |
krotscheck | See it | 19:15 |
sjmc7 | TravT: i’ll get a fix up ASAP; don’t take m3 before we get this fixed | 19:16 |
sjmc7 | tag | 19:16 |
openstackgerrit | Steve McLellan proposed openstack/searchlight: Remove pre-1.0 glance auth error handling https://review.openstack.org/288079 | 19:34 |
sjmc7 | krotscheck: ^ | 19:34 |
sjmc7 | we should’ve done that ages ago | 19:35 |
krotscheck | Sweet, I | 19:37 |
krotscheck | I'll rebase after lunch | 19:37 |
sjmc7 | ok. we’ll get it merged today unless i blundered | 19:37 |
sjmc7 | lakshmiS: good catch on the neutron rbac query | 19:44 |
sjmc7 | i’m not certain about the restriction on ports though | 19:44 |
sjmc7 | are you sure ports belonging to other tenants on shared networks are visible? | 19:45 |
sjmc7 | that seems suspect to me | 19:45 |
*** exploreshaifali has quit IRC | 19:48 | |
lakshmiS | sjmc7: back from lunch. | 20:02 |
lakshmiS | port doesnt seem to show up on shared networks | 20:02 |
lakshmiS | its only network filter then | 20:03 |
lakshmiS | there are no subnets being indexed right now but in future when then get added they are visible for non-tenant users | 20:03 |
lakshmiS | s/then/they | 20:04 |
sjmc7 | yep | 20:04 |
sjmc7 | ok, added it for networks. adding a test for that now | 20:04 |
*** TravT has joined #openstack-searchlight | 20:22 | |
*** TravT has quit IRC | 20:31 | |
*** TravT has joined #openstack-searchlight | 20:36 | |
*** itisha has joined #openstack-searchlight | 20:55 | |
openstackgerrit | Steve McLellan proposed openstack/searchlight: Neutron plugin https://review.openstack.org/282043 | 21:08 |
openstackgerrit | Michael Krotscheck proposed openstack/searchlight: Moved CORS middleware configuration into oslo-config-generator https://review.openstack.org/287354 | 21:22 |
sjmc7 | krotscheck: looking at the CORS patch; this is after the mailing list discussion over the last week or so? | 21:24 |
krotscheck | sjmc7: Yep | 21:43 |
krotscheck | Looks like the rebase on top of yours fixed it | 21:43 |
sjmc7 | ok. same change is going in on other projects as well? | 21:43 |
*** TravT has quit IRC | 21:44 | |
openstackgerrit | Steve McLellan proposed openstack/searchlight: Neutron plugin https://review.openstack.org/282043 | 21:48 |
*** TravT has joined #openstack-searchlight | 22:13 | |
sjmc7 | krotscheck: still there? i don’t see the cors middleware adding its options to the config parser anywhere | 22:26 |
krotscheck | Yep | 22:27 |
krotscheck | sec. | 22:27 |
krotscheck | it _should_ | 22:27 |
sjmc7 | set_defaults only does anything if the option exists (rather than exploding) so i don’t think it’s doing anything | 22:27 |
krotscheck | *blink* | 22:28 |
sjmc7 | or maybe we’re missing a function call to add them somewhere | 22:28 |
krotscheck | That seems like a bloody obvious oversight on my part | 22:28 |
sjmc7 | :) | 22:29 |
sjmc7 | checking hjow other oslo libs add their options | 22:29 |
sjmc7 | ah, they have a ‘_register’ function in opts.py | 22:29 |
sjmc7 | oslo_middleware doesn’t seem to have one | 22:30 |
sjmc7 | if you look in oslo_policy, for instance | 22:30 |
sjmc7 | its set_defaults registers the options first, so they only get registered if you intend to use them which makes sense | 22:30 |
krotscheck | sjmc7: blah blah freature freeze I have a patch i believe | 22:32 |
krotscheck | https://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 |
krotscheck | yeaaah | 22:33 |
openstackgerrit | Merged openstack/searchlight: Remove pre-1.0 glance auth error handling https://review.openstack.org/288079 | 22:33 |
* krotscheck facepalms | 22:33 | |
openstackgerrit | Merged openstack/searchlight: Neutron plugin https://review.openstack.org/282043 | 22:33 |
sjmc7 | i think you’ll still need to register the options somewhere | 22:33 |
sjmc7 | typically config.register_opts(<some list>, <group name>) | 22:34 |
krotscheck | sjmc7: Waaait a sec | 22:35 |
krotscheck | No | 22:35 |
krotscheck | What the cfg.set_defaults does is override the values in cfg.CONF with new values | 22:35 |
krotscheck | Sorry | 22:35 |
krotscheck | no | 22:35 |
sjmc7 | i am definitely willing to admit i’m wrong | 22:35 |
sjmc7 | but i am debugging it and i don’t see it setting those values | 22:35 |
krotscheck | It overrides the values in (opts passed to method) | 22:35 |
sjmc7 | ah, hmm.. | 22:36 |
*** lakshmiS has quit IRC | 22:36 | |
sjmc7 | yeah, i may be mistaken. however, in that case i don’t know why they’re not getting set | 22:36 |
krotscheck | sjmc7: Are you looking at the default options in oslo_middleware.cors.CORS, rather than cfg.CONF? | 22:39 |
krotscheck | Sorry- CORS_OPTS | 22:39 |
sjmc7 | ah, they’re separate? then yes, possibly | 22:39 |
sjmc7 | in which case, my mistake | 22:40 |
krotscheck | sjmc7: Righto. | 22:41 |
openstackgerrit | Steve McLellan proposed openstack/searchlight: Fix role based indexing for designate https://review.openstack.org/280908 | 22:42 |
openstackgerrit | Merged openstack/searchlight: Fix role based indexing for designate https://review.openstack.org/280908 | 22:55 |
*** GB21 has joined #openstack-searchlight | 23:09 | |
*** itisha has quit IRC | 23:39 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!