Thursday, 2015-07-09

*** sballe has quit IRC00:22
*** bitblt has quit IRC01:45
*** sigmavirus24_awa is now known as sigmavirus2401:49
*** TravT has joined #openstack-searchlight01:51
*** TravT is now known as TravT_away01:51
*** sigmavirus24 is now known as sigmavirus24_awa02:17
*** ig0r_ has joined #openstack-searchlight05:51
*** ig0r__ has quit IRC05:55
*** lakshmiS has joined #openstack-searchlight06:59
*** TravT_away has quit IRC07:08
*** krykowski has joined #openstack-searchlight08:54
*** TravT_away has joined #openstack-searchlight09:01
*** lakshmiS has quit IRC09:13
*** pserebryakov has joined #openstack-searchlight12:50
*** sigmavirus24_awa is now known as sigmavirus2413:47
*** lakshmiS has joined #openstack-searchlight13:53
*** pserebryakov has quit IRC14:00
*** TravT_away is now known as TravT14:01
*** lakshmiS has quit IRC14:56
*** lakshmiS has joined #openstack-searchlight14:59
nikhil_kDo we have a meeting toay TravT , lakshmiS sjmc7 ?15:03
lakshmiSyeah it should be. let me ping travT15:03
*** TravT has quit IRC15:03
*** TravT has joined #openstack-searchlight15:04
sjmc7yeah15:04
nikhil_k:)15:05
openstackgerritMerged openstack/searchlight: Don't retrieve image members for public images  https://review.openstack.org/19675415:40
openstackgerritMerged openstack/searchlight: Don't retrieve image members for public images  https://review.openstack.org/19675415:40
TravTlakshmiS sjmc715:59
lakshmiSyup16:00
sjmc7yessir16:00
TravTthere was some discussion on parameterizing the notification topic16:01
TravTjust wanted to follow up on that16:01
sjmc7ok16:01
lakshmiSso i guess the blueprint will cover parameterizing other aspects of plugin too16:02
sjmc7yeah.. ekarlso suggested [plugin:glance] type config16:02
sjmc7and i'd also have [plugin:common] or something16:02
TravTsjmc7, you are just talking about making a config section in each plugin16:02
TravTi think there are a number of things that should be parameterized16:03
sjmc7if you look at glance/images.py, there's some hardcoded stuff16:03
TravTeven the ES index and type could be overridden IMO.16:03
sjmc7yep16:03
*** bitblt has joined #openstack-searchlight16:03
TravTlet's add on to that BP all the things that should be parameterized16:04
TravThttps://blueprints.launchpad.net/searchlight/+spec/plugin-config-options16:04
*** wko has joined #openstack-searchlight16:04
lakshmiSwe also should handle plugin specific file requiremetns like glance requires properties-protection file16:05
TravTahh, good point16:05
sjmc7right16:06
TravTso in other words, we all agree that it's a good thing16:08
*** wko has quit IRC16:08
lakshmiSyeah definetly16:08
TravTso, let's brainstorm common items here16:09
sjmc7client auth stuff arguably could be in either place, though i'd like to standardize that more (maybe use the keystone session stuff)16:10
lakshmiSdoes each plugin handle by project like one for glance, one for nova or is it specific to resource like glance images, glance snapshots ? feel like having one by project is good supporting multiple resource types16:11
TravTi wonder if even the mapping should be configurable16:11
TravTat least overrideable16:12
lakshmiSwe could follow heat like resource type namings which each plugin exposes as supported types.16:13
*** TravT_ has joined #openstack-searchlight16:13
TravT_okay, looks like my home internet isn't going to cooperate today16:13
TravT_test16:14
lakshmiSneed to avoid two plugins supporting similar resource type though..16:14
TravT_i'm dropping on and off... :(16:14
sjmc7i don't want to go too crazy for the sake of it16:15
lakshmiS:)16:15
TravT_ugh16:15
*** TravT has quit IRC16:16
*** TravT_ is now known as TravT16:16
TravTlakshmiS i think we need to use the HEAT resource type names16:17
TravTthat was the intent from the beginning.16:17
TravTwe used the same in Glance metadefs16:17
lakshmiSyup16:17
sjmc7os::glance::image ?16:20
lakshmiSyes16:21
TravTactually OS::GLANCE::IMAGE16:21
sjmc7what did you intend to use that for?16:24
sigmavirus24Os::gLaNcE::IMaGe16:25
sigmavirus24^ my vote16:25
sjmc7:)16:26
sjmc7one small issue with config is making sure oslo.config finds the config declarations16:28
lakshmiSsjmc7: you mean finding the plugin specific config file?16:30
sjmc7no, parsing the options16:31
sjmc7it's probably too late by the time stevedore finds them16:31
*** bitblt has quit IRC16:45
*** sballe has joined #openstack-searchlight16:55
ekarlsoyoyo17:12
ekarlsosorry for missing the meeting :p17:12
ekarlsohands full with tidying up the house and mkdinner()17:12
*** TravT_ has joined #openstack-searchlight17:13
*** TravT_ has joined #openstack-searchlight17:14
*** TravT has quit IRC17:16
*** TravT_ is now known as TravT17:25
*** lakshmiS has left #openstack-searchlight17:34
ekarlsoTravT: / sjmc7 did you come up with a pattern for config file ?17:59
TravTekarlso: sorry, i've gotten pulled into a call with somebody...18:27
TravTreal quick: only agreement was that it needs to be done, but no specifics18:27
sjmc7ekarlso - no, though i'm ok with your suggestion18:30
ekarlsosjmc7: / TravT ok i'll look at putting up a patch :p18:40
TravT+118:41
sjmc7my experience with oslo.config has not been great, but remember plugins are loaded by stevedore sometime after other config is read in18:42
ekarlsolet's see sjmc7 :)18:42
sjmc7some things that we thought could go in are notification topic names, index/type names18:43
*** TravT is now known as TravT_away19:03
openstackgerritSteve McLellan proposed openstack/searchlight: Clear cached os-client auth on unauthorized  https://review.openstack.org/20022519:41
*** TravT_away has quit IRC19:44
*** TravT_away has joined #openstack-searchlight20:00
*** TravT_away is now known as TravT20:21
ekarlsooii20:34
ekarlso2015-07-09 20:33:51.449 TRACE searchlight.elasticsearch.plugins.designate.notification_handlers ConflictError: TransportError(409, u'DocumentAlreadyExistsException[[designate][3] [domain][4fda3fdd-950f-437f-ad0e-44d7a775aa5a]: document already exists]')20:34
ekarlsobecause of the do_upsert thing I think20:34
ekarlsoany ideas ?20:34
sjmc7ekarlso i think it's because you're using engine.create20:36
sjmc7doc_as_upsert is part of the update API20:36
sjmc7sorry, i missed that in review20:36
ekarlsosjmc7: so what should I be doing ? :p21:05
sjmc7engine.update. added a review comment21:06
*** sigmavirus24 is now known as emacsraptor21:10
*** emacsraptor is now known as sigmavirus2421:10
sjmc7:)21:10
*** sigmavirus24 is now known as sigmavirus24_awa21:59
*** TravT has quit IRC23:05
*** TravT has joined #openstack-searchlight23:08
*** openstack has joined #openstack-searchlight23:41
openstackgerritEndre Karlson proposed openstack/searchlight: Add Designate plugin.  https://review.openstack.org/19909923:59

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