*** sballe has quit IRC | 00:22 | |
*** bitblt has quit IRC | 01:45 | |
*** sigmavirus24_awa is now known as sigmavirus24 | 01:49 | |
*** TravT has joined #openstack-searchlight | 01:51 | |
*** TravT is now known as TravT_away | 01:51 | |
*** sigmavirus24 is now known as sigmavirus24_awa | 02:17 | |
*** ig0r_ has joined #openstack-searchlight | 05:51 | |
*** ig0r__ has quit IRC | 05:55 | |
*** lakshmiS has joined #openstack-searchlight | 06:59 | |
*** TravT_away has quit IRC | 07:08 | |
*** krykowski has joined #openstack-searchlight | 08:54 | |
*** TravT_away has joined #openstack-searchlight | 09:01 | |
*** lakshmiS has quit IRC | 09:13 | |
*** pserebryakov has joined #openstack-searchlight | 12:50 | |
*** sigmavirus24_awa is now known as sigmavirus24 | 13:47 | |
*** lakshmiS has joined #openstack-searchlight | 13:53 | |
*** pserebryakov has quit IRC | 14:00 | |
*** TravT_away is now known as TravT | 14:01 | |
*** lakshmiS has quit IRC | 14:56 | |
*** lakshmiS has joined #openstack-searchlight | 14:59 | |
nikhil_k | Do we have a meeting toay TravT , lakshmiS sjmc7 ? | 15:03 |
---|---|---|
lakshmiS | yeah it should be. let me ping travT | 15:03 |
*** TravT has quit IRC | 15:03 | |
*** TravT has joined #openstack-searchlight | 15:04 | |
sjmc7 | yeah | 15:04 |
nikhil_k | :) | 15:05 |
openstackgerrit | Merged openstack/searchlight: Don't retrieve image members for public images https://review.openstack.org/196754 | 15:40 |
openstackgerrit | Merged openstack/searchlight: Don't retrieve image members for public images https://review.openstack.org/196754 | 15:40 |
TravT | lakshmiS sjmc7 | 15:59 |
lakshmiS | yup | 16:00 |
sjmc7 | yessir | 16:00 |
TravT | there was some discussion on parameterizing the notification topic | 16:01 |
TravT | just wanted to follow up on that | 16:01 |
sjmc7 | ok | 16:01 |
lakshmiS | so i guess the blueprint will cover parameterizing other aspects of plugin too | 16:02 |
sjmc7 | yeah.. ekarlso suggested [plugin:glance] type config | 16:02 |
sjmc7 | and i'd also have [plugin:common] or something | 16:02 |
TravT | sjmc7, you are just talking about making a config section in each plugin | 16:02 |
TravT | i think there are a number of things that should be parameterized | 16:03 |
sjmc7 | if you look at glance/images.py, there's some hardcoded stuff | 16:03 |
TravT | even the ES index and type could be overridden IMO. | 16:03 |
sjmc7 | yep | 16:03 |
*** bitblt has joined #openstack-searchlight | 16:03 | |
TravT | let's add on to that BP all the things that should be parameterized | 16:04 |
TravT | https://blueprints.launchpad.net/searchlight/+spec/plugin-config-options | 16:04 |
*** wko has joined #openstack-searchlight | 16:04 | |
lakshmiS | we also should handle plugin specific file requiremetns like glance requires properties-protection file | 16:05 |
TravT | ahh, good point | 16:05 |
sjmc7 | right | 16:06 |
TravT | so in other words, we all agree that it's a good thing | 16:08 |
*** wko has quit IRC | 16:08 | |
lakshmiS | yeah definetly | 16:08 |
TravT | so, let's brainstorm common items here | 16:09 |
sjmc7 | client auth stuff arguably could be in either place, though i'd like to standardize that more (maybe use the keystone session stuff) | 16:10 |
lakshmiS | does 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 types | 16:11 |
TravT | i wonder if even the mapping should be configurable | 16:11 |
TravT | at least overrideable | 16:12 |
lakshmiS | we could follow heat like resource type namings which each plugin exposes as supported types. | 16:13 |
*** TravT_ has joined #openstack-searchlight | 16:13 | |
TravT_ | okay, looks like my home internet isn't going to cooperate today | 16:13 |
TravT_ | test | 16:14 |
lakshmiS | need to avoid two plugins supporting similar resource type though.. | 16:14 |
TravT_ | i'm dropping on and off... :( | 16:14 |
sjmc7 | i don't want to go too crazy for the sake of it | 16:15 |
lakshmiS | :) | 16:15 |
TravT_ | ugh | 16:15 |
*** TravT has quit IRC | 16:16 | |
*** TravT_ is now known as TravT | 16:16 | |
TravT | lakshmiS i think we need to use the HEAT resource type names | 16:17 |
TravT | that was the intent from the beginning. | 16:17 |
TravT | we used the same in Glance metadefs | 16:17 |
lakshmiS | yup | 16:17 |
sjmc7 | os::glance::image ? | 16:20 |
lakshmiS | yes | 16:21 |
TravT | actually OS::GLANCE::IMAGE | 16:21 |
sjmc7 | what did you intend to use that for? | 16:24 |
sigmavirus24 | Os::gLaNcE::IMaGe | 16:25 |
sigmavirus24 | ^ my vote | 16:25 |
sjmc7 | :) | 16:26 |
sjmc7 | one small issue with config is making sure oslo.config finds the config declarations | 16:28 |
lakshmiS | sjmc7: you mean finding the plugin specific config file? | 16:30 |
sjmc7 | no, parsing the options | 16:31 |
sjmc7 | it's probably too late by the time stevedore finds them | 16:31 |
*** bitblt has quit IRC | 16:45 | |
*** sballe has joined #openstack-searchlight | 16:55 | |
ekarlso | yoyo | 17:12 |
ekarlso | sorry for missing the meeting :p | 17:12 |
ekarlso | hands full with tidying up the house and mkdinner() | 17:12 |
*** TravT_ has joined #openstack-searchlight | 17:13 | |
*** TravT_ has joined #openstack-searchlight | 17:14 | |
*** TravT has quit IRC | 17:16 | |
*** TravT_ is now known as TravT | 17:25 | |
*** lakshmiS has left #openstack-searchlight | 17:34 | |
ekarlso | TravT: / sjmc7 did you come up with a pattern for config file ? | 17:59 |
TravT | ekarlso: sorry, i've gotten pulled into a call with somebody... | 18:27 |
TravT | real quick: only agreement was that it needs to be done, but no specifics | 18:27 |
sjmc7 | ekarlso - no, though i'm ok with your suggestion | 18:30 |
ekarlso | sjmc7: / TravT ok i'll look at putting up a patch :p | 18:40 |
TravT | +1 | 18:41 |
sjmc7 | my experience with oslo.config has not been great, but remember plugins are loaded by stevedore sometime after other config is read in | 18:42 |
ekarlso | let's see sjmc7 :) | 18:42 |
sjmc7 | some things that we thought could go in are notification topic names, index/type names | 18:43 |
*** TravT is now known as TravT_away | 19:03 | |
openstackgerrit | Steve McLellan proposed openstack/searchlight: Clear cached os-client auth on unauthorized https://review.openstack.org/200225 | 19:41 |
*** TravT_away has quit IRC | 19:44 | |
*** TravT_away has joined #openstack-searchlight | 20:00 | |
*** TravT_away is now known as TravT | 20:21 | |
ekarlso | oii | 20:34 |
ekarlso | 2015-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 |
ekarlso | because of the do_upsert thing I think | 20:34 |
ekarlso | any ideas ? | 20:34 |
sjmc7 | ekarlso i think it's because you're using engine.create | 20:36 |
sjmc7 | doc_as_upsert is part of the update API | 20:36 |
sjmc7 | sorry, i missed that in review | 20:36 |
ekarlso | sjmc7: so what should I be doing ? :p | 21:05 |
sjmc7 | engine.update. added a review comment | 21:06 |
*** sigmavirus24 is now known as emacsraptor | 21:10 | |
*** emacsraptor is now known as sigmavirus24 | 21:10 | |
sjmc7 | :) | 21:10 |
*** sigmavirus24 is now known as sigmavirus24_awa | 21:59 | |
*** TravT has quit IRC | 23:05 | |
*** TravT has joined #openstack-searchlight | 23:08 | |
*** openstack has joined #openstack-searchlight | 23:41 | |
openstackgerrit | Endre Karlson proposed openstack/searchlight: Add Designate plugin. https://review.openstack.org/199099 | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!