| *** ig0r__ has joined #openstack-searchlight | 05:51 | |
| *** ig0r_ has quit IRC | 05:55 | |
| openstackgerrit | Endre Karlson proposed openstack/searchlight: DevStack: Keystone V3 Service/Endpoint Creation https://review.openstack.org/197885 | 12:10 |
|---|---|---|
| *** sigmavirus24_awa is now known as sigmavirus24 | 13:24 | |
| openstackgerrit | Sergey Vilgelm proposed openstack/searchlight: Remove unused oslo incubator files https://review.openstack.org/198738 | 14:11 |
| *** TravT has joined #openstack-searchlight | 14:13 | |
| *** TravT has quit IRC | 14:15 | |
| *** TravT has joined #openstack-searchlight | 14:15 | |
| *** TravT has quit IRC | 14:50 | |
| *** TravT has joined #openstack-searchlight | 14:57 | |
| openstackgerrit | Merged openstack/searchlight: Tidy up searchlight entry points https://review.openstack.org/196750 | 16:36 |
| openstackgerrit | Ian Cordasco proposed openstack/searchlight: Replace exception_to_str with oslo exception_to_unicode https://review.openstack.org/196084 | 17:08 |
| ekarlso | is it possible to enable just certain plugins ? | 17:22 |
| sjmc7 | ekarlso yes, in setup.cfg | 17:26 |
| sjmc7 | they're listed as entry points | 17:26 |
| ekarlso | sjmc7: yeah, but is there a config option like enabled_plugins = glance,foo etc ? | 17:29 |
| sjmc7 | not currently, all via stevedore | 17:33 |
| sigmavirus24 | TravT: thoughts on having the default be "everything stevedore can discover" and then allowing overrides via an (enabled|selected)_plugins option? | 17:40 |
| sigmavirus24 | nikhil_k: same q ^ | 17:40 |
| nikhil_k | ++ | 17:41 |
| TravT | sigmavirus24, that makes sense to me. the other alternative that isn't mutually exclusive would be a disable_plugins option | 17:43 |
| sigmavirus24 | TravT: yeah. I tend to prefer to have opt-in type options rather than opt-out | 17:45 |
| sigmavirus24 | The problem that I've seen people have with things like --enable --disable on CLIs is that they'll pass the same thing to both and get angry no matter what behaviour you choose | 17:45 |
| TravT | yeah, i think you are right. | 17:46 |
| sigmavirus24 | So having both in a config setting makes me really nervous about people doing bad things | 17:46 |
| sigmavirus24 | And it's not even necessarily bad, just things that cause indeterminate behaviour | 17:46 |
| TravT | Let's avoid that confusion then | 17:48 |
| TravT | and go with enabled | 17:48 |
| sigmavirus24 | I can see the idea for wanting disabled_plugins | 17:49 |
| sigmavirus24 | I would just rather they be mutually exclusive options | 17:49 |
| sigmavirus24 | Like, we might grow to have 20 plugins | 17:49 |
| sigmavirus24 | And they want to disable 1 | 17:49 |
| sigmavirus24 | they can A) Uninstall it | 17:49 |
| sigmavirus24 | B) put all other 19 on that one line | 17:49 |
| sigmavirus24 | then when they install another 5, they have to add those 5 | 17:49 |
| sigmavirus24 | We could definitely implement both, but if a deployer supplies both options, we would need to either: A) choose one to obey and issue log warnings that both were supplied. B) Crash | 17:50 |
| sigmavirus24 | where crash is just a hard error on start-up preventing the servcie from even launching | 17:50 |
| * sigmavirus24 doesn't know which is the "better" way | 17:50 | |
| sigmavirus24 | I personally would love config validation at start-up to prevent me from running a jacked service that maybe wasn't going to do what I intended anyway | 17:51 |
| sigmavirus24 | but I can see how this could cause problems with bad deployment technology | 17:51 |
| TravT | what, you don't like having to debug a hundred things later only to discover that you screwed up one line in a config file? | 17:51 |
| sigmavirus24 | As someone who wants to implement searchlight for openstack-ansible, no | 18:50 |
| openstackgerrit | Steve McLellan proposed openstack/searchlight: WIP - Add nova instances plugin https://review.openstack.org/198852 | 18:53 |
| sjmc7 | TravT, i'll make the devstack update for the entrypoints change now | 18:59 |
| TravT | sjmc7, thanks. i've been haggling over things in horizon all morning. was about to kickoff a new devstack, but can wait for your update so I can test it. | 19:03 |
| sjmc7 | ok. be done in a couple of minutes | 19:03 |
| ekarlso | so hmmm | 19:08 |
| ekarlso | the searchlight-index thing | 19:08 |
| ekarlso | that does a list for all things at once ? | 19:09 |
| sjmc7 | currently | 19:09 |
| ekarlso | :/ | 19:09 |
| sjmc7 | TravT or ekarlso, is this right in the devstack plugin? "if is_service_enabled searchlight; then <install, configure, start>" ? | 19:10 |
| sjmc7 | does that mean enable_service searchlight is needed as well as enable_service searchlight-api ? | 19:10 |
| ekarlso | sjmc7: no, enable_service is done in the settings file :p | 19:10 |
| sjmc7 | "# we have to add searchlight to enabled services for screen_it to work" ? | 19:11 |
| ekarlso | ye | 19:19 |
| ekarlso | so you only have to enable_service searchlight-api | 19:20 |
| sjmc7 | hmm. ok | 19:20 |
| sjmc7 | TravT - you want to have to enable searchlight-api and searchlight-listener, or have all or nothing? | 19:24 |
| sjmc7 | i think existing devstack usually allows pretty granular control | 19:24 |
| TravT | sjmc7 i think we should enable both since they are different processes | 19:25 |
| sjmc7 | yep, ok | 19:25 |
| openstackgerrit | Steve McLellan proposed openstack/searchlight: Devstack plugin changes for additional entrypoints https://review.openstack.org/198863 | 19:26 |
| sjmc7 | take a look at that TravT ^ | 19:26 |
| sjmc7 | i've not had a chance to test it properly | 19:26 |
| TravT | ok, i just am at a point to kick off devstack run with that patch | 19:28 |
| openstackgerrit | Travis Tripp proposed openstack/searchlight: Document Listener and Manage Entrypoints Dev Docs https://review.openstack.org/198868 | 19:36 |
| ekarlso | so guys | 19:46 |
| ekarlso | how can I check what's added to the index ? | 19:46 |
| ekarlso | kibana ? | 19:46 |
| sjmc7 | worth looking at the elasticsearch API | 19:47 |
| sjmc7 | if you want everything, curl localhost:9200/_search | 19:47 |
| sjmc7 | will give you ten random results | 19:47 |
| ekarlso | or just kibana I gues.. | 19:47 |
| openstackgerrit | Steve McLellan proposed openstack/searchlight: WIP - Add nova instances plugin https://review.openstack.org/198852 | 19:48 |
| sjmc7 | if you want a web frontend, https://www.elastic.co/guide/en/elasticsearch/client/community/current/front-ends.html | 19:48 |
| ekarlso | elastichq is nice too sjmc7 :) | 19:52 |
| ekarlso | so, wwill it store any data returned by serialize() ? | 20:00 |
| sjmc7 | yes | 20:00 |
| ekarlso | hmmms, I wonder if kafka would be a nice fit for openstack + searchlight | 20:02 |
| ekarlso | since then you could potentially replay updates if the listener service goes boom | 20:02 |
| ekarlso | or resume from position rather | 20:02 |
| sjmc7 | if oslo supports it, sure | 20:03 |
| ekarlso | sjmc7: I found out it was easy to make a generator that just loads n items pr page | 20:08 |
| ekarlso | ;) | 20:08 |
| ekarlso | alot nicer to load 50-100 domains / records then to load all :P | 20:16 |
| openstackgerrit | Merged openstack/searchlight: Replace exception_to_str with oslo exception_to_unicode https://review.openstack.org/196084 | 20:48 |
| openstackgerrit | Steve McLellan proposed openstack/searchlight: Devstack plugin changes for additional entrypoints https://review.openstack.org/198863 | 21:31 |
| openstackgerrit | Steve McLellan proposed openstack/searchlight: Devstack plugin changes for additional entrypoints https://review.openstack.org/198863 | 22:37 |
| openstackgerrit | Travis Tripp proposed openstack/searchlight: Add Manage command Main invocation https://review.openstack.org/198923 | 23:31 |
| openstackgerrit | Travis Tripp proposed openstack/searchlight: Document Listener and Manage Entrypoints Dev Docs https://review.openstack.org/198868 | 23:42 |
| openstackgerrit | Travis Tripp proposed openstack/searchlight: Document Listener and Manage Entrypoints Dev Docs https://review.openstack.org/198868 | 23:44 |
| openstackgerrit | Travis Tripp proposed openstack/searchlight: Don't retrieve image members for public images https://review.openstack.org/196754 | 23:51 |
| *** sigmavirus24 is now known as sigmavirus24_awa | 23:52 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!