*** openstackgerrit has quit IRC | 01:16 | |
*** openstackgerrit has joined #openstack-searchlight | 01:17 | |
*** sigmavirus24_awa has quit IRC | 03:23 | |
*** sigmavirus24_awa has joined #openstack-searchlight | 03:29 | |
*** sigmavirus24_awa has quit IRC | 03:29 | |
*** sigmavirus24_awa has joined #openstack-searchlight | 03:29 | |
*** lakshmiS has joined #openstack-searchlight | 03:47 | |
*** lakshmiS has quit IRC | 06:06 | |
*** lakshmiS has joined #openstack-searchlight | 06:12 | |
openstackgerrit | Endre Karlson proposed openstack/searchlight: Add Designate plugin. https://review.openstack.org/199099 | 09:04 |
---|---|---|
*** openstackgerrit has quit IRC | 10:16 | |
*** openstackgerrit has joined #openstack-searchlight | 10:17 | |
*** lakshmiS has quit IRC | 11:42 | |
*** lakshmiS has joined #openstack-searchlight | 11:48 | |
*** openstackgerrit has quit IRC | 12:16 | |
*** openstackgerrit has joined #openstack-searchlight | 12:17 | |
*** lakshmiS has quit IRC | 12:42 | |
*** lakshmiS has joined #openstack-searchlight | 13:03 | |
*** openstackgerrit has quit IRC | 13:16 | |
*** openstackgerrit has joined #openstack-searchlight | 13:17 | |
*** mugsie has left #openstack-searchlight | 13:40 | |
openstackgerrit | Lakshmi N Sampath proposed openstack/searchlight: Fix for updated/missing fields in python glance client for image members https://review.openstack.org/219168 | 13:43 |
*** lakshmiS has quit IRC | 13:44 | |
*** GB21_ has joined #openstack-searchlight | 13:58 | |
*** GeetikaBatra__ has joined #openstack-searchlight | 13:58 | |
*** GeetikaBatra__ is now known as GB21 | 13:58 | |
*** GB21_ has quit IRC | 13:58 | |
*** sigmavirus24_awa is now known as sigmavirus24 | 14:00 | |
*** david-lyle has quit IRC | 14:19 | |
*** david-lyle has joined #openstack-searchlight | 15:06 | |
*** GB21 has quit IRC | 15:17 | |
*** sballe has joined #openstack-searchlight | 15:59 | |
*** openstackgerrit has quit IRC | 16:01 | |
*** openstackgerrit has joined #openstack-searchlight | 16:02 | |
ekarlso | hey sjmc7 | 17:48 |
sjmc7 | hola | 17:48 |
ekarlso | yah i'll put it up tmrw, been busy with the kid today after work :ยง | 17:49 |
ekarlso | is that ok or too late ? | 17:49 |
sjmc7 | no, i think we're good. we're accepting changes a little later than some other projects i think | 17:49 |
*** openstackgerrit has quit IRC | 18:16 | |
*** openstackgerrit has joined #openstack-searchlight | 18:16 | |
*** david-lyle has quit IRC | 18:29 | |
*** TravT_ has joined #openstack-searchlight | 18:35 | |
*** TravT has quit IRC | 18:37 | |
*** TravT_ has quit IRC | 19:10 | |
*** TravT has joined #openstack-searchlight | 19:14 | |
*** david-lyle has joined #openstack-searchlight | 19:34 | |
*** TravT_ has joined #openstack-searchlight | 19:40 | |
*** TravT has quit IRC | 19:40 | |
*** TravT_ has quit IRC | 19:50 | |
*** TravT has joined #openstack-searchlight | 20:06 | |
*** TravT_ has joined #openstack-searchlight | 20:22 | |
*** TravT has quit IRC | 20:24 | |
*** openstackgerrit has quit IRC | 20:31 | |
*** openstackgerrit has joined #openstack-searchlight | 20:32 | |
*** TravT_ has quit IRC | 20:34 | |
*** TravT has joined #openstack-searchlight | 20:35 | |
ekarlso | sjmc7: has it occurred to you to split the notification handlers into seperate plugins ? | 20:39 |
*** TravT has quit IRC | 20:49 | |
*** TravT has joined #openstack-searchlight | 20:50 | |
ekarlso | TravT: or sjmc7 you around ? | 20:51 |
sjmc7 | ekarlso - how do you mean? | 20:52 |
sjmc7 | i wouldn't want to split the initial indexing out from the listener | 20:52 |
*** TravT has quit IRC | 20:53 | |
ekarlso | sjmc7: uh I mean more like wouldn't it be better for the notification plugin to have opptions for that ? | 20:53 |
ekarlso | like topics etc | 20:53 |
*** TravT has joined #openstack-searchlight | 20:53 | |
sjmc7 | you mean if you have two handlers for a service (like the designate ones) that they probably share the same notification system? | 20:54 |
sjmc7 | because in that case, yes, that has occurred to me :) | 20:55 |
ekarlso | sjmc7: I mean like instead of having hte index plguin have a get_notification_handler or what the method is again we make a searchlight.notification_handlers namespace | 20:57 |
sjmc7 | currently the index plugin is the entry point. i don't want to have to install a nova server indexer and a nova server notification handler | 20:58 |
ekarlso | sjmc7: well ok :p | 20:59 |
ekarlso | sjmc7: I'm wondering how to map the exchange / topic though in cfg opts.. | 21:00 |
TravT | okay, i just got off the phone with my internet provider. let me go back in room logs and try to catch up on this conversation. | 21:00 |
ekarlso | haha TravT it's not that long :p | 21:00 |
sjmc7 | maybe what you're asking is if lots of plugins rely on nova's notifications, you don't want to configure them 20 times? | 21:02 |
ekarlso | sjmc7: nvm that | 21:03 |
ekarlso | how you want the options to look for nova ? | 21:03 |
sjmc7 | ok :) then i still don't get what you're asking | 21:03 |
sjmc7 | something like [OS::Nova::Server]\nexchange_topics=searchlight_indexer,nova | 21:04 |
sjmc7 | i don't know how oslo.config does lists, but however that works | 21:04 |
ekarlso | atm it will be [indexer:os_nova_server] | 21:04 |
sjmc7 | but i can see that'd get a bit crappy as you get lots of plugins that share notifications | 21:05 |
ekarlso | in case you later add more plugin types indexer is a type | 21:05 |
sjmc7 | ok | 21:06 |
TravT | ekarlso, i am not sure on the start of this conversation... but this last part. | 21:07 |
TravT | i *think* it sounds like it makes sense. | 21:07 |
ekarlso | [indexer:server] rather | 21:07 |
ekarlso | it uses the entrypoint name as the plugin name | 21:07 |
ekarlso | btw | 21:07 |
sjmc7 | ok | 21:07 |
sjmc7 | that makes sense... still not getting the question | 21:08 |
TravT | ekarlso: not to change topics, but i just started looking at your designate patch. | 21:08 |
ekarlso | TravT: uhm k | 21:08 |
TravT | https://review.openstack.org/#/c/199099/21..22/searchlight/elasticsearch/plugins/designate/notification_handlers.py | 21:08 |
TravT | line 48 of patchset 21 | 21:08 |
TravT | did you address that? | 21:09 |
ekarlso | TravT: as I noted I think in a previous PS there's a bug atm with designate not emitting status updates on resources changes | 21:09 |
TravT | ok | 21:09 |
TravT | i missed that. | 21:09 |
ekarlso | dunno if it will land in L or most likely M | 21:09 |
TravT | so then i won't consider it a searchlight bug | 21:09 |
TravT | at least for now | 21:09 |
ekarlso | yep | 21:09 |
sjmc7 | ekarlso TravT i just tested a bunch of operations on the deisngate plugin and it all seems good, delete, searching, indexing work ok (aside from that status bug) | 21:11 |
TravT | i saw steve's comments about the explicit indexing of various id fields... i was the one who suggested ekarlso just use the dynamic _id mapping | 21:11 |
TravT | kinda see your point sjmc7, but don't think it would be worth blocking over. | 21:12 |
sjmc7 | yeah, me neither | 21:12 |
TravT | i'm trying out designate now. i just want to check the rbac responses | 21:12 |
TravT | (in addition to the regular flow). | 21:12 |
sjmc7 | i've +2ed although with a note about the 'name' field being analyzed | 21:13 |
TravT | ah, is that the same comment i've called out a few times and opened a bug on? | 21:13 |
TravT | let me grab that bug | 21:13 |
sjmc7 | mebbe. i may look at that today, so i can tell people i'm busy and leave me alone :) | 21:16 |
sjmc7 | yes, i think it is | 21:16 |
*** TravT_ has joined #openstack-searchlight | 21:17 | |
TravT_ | hello world? | 21:17 |
TravT_ | hello world? | 21:17 |
*** TravT_ has quit IRC | 21:19 | |
sjmc7 | hello TravT_ | 21:19 |
*** TravT_ has joined #openstack-searchlight | 21:19 | |
*** TravT has quit IRC | 21:19 | |
*** TravT_ is now known as TravT | 21:20 | |
ekarlso | so sjmc7 a ListOpt is just a csv opt... | 21:27 |
sjmc7 | grr! could split with slashes maybe | 21:28 |
sjmc7 | searchlight_indexer/nova | 21:28 |
sjmc7 | then comma sep those | 21:29 |
ekarlso | sjmc7: MutliStrOpt it seems | 21:29 |
ekarlso | exchange_topics=searchlight_indexer,nova | 21:29 |
ekarlso | exchange_topics=searchlight_indexer,foo | 21:29 |
ekarlso | MultiOpt rather | 21:30 |
sjmc7 | oo, that works too | 21:31 |
*** openstackgerrit has quit IRC | 21:46 | |
*** openstackgerrit has joined #openstack-searchlight | 21:47 | |
ekarlso | can't openstack go kafka already for notifications | 21:48 |
ekarlso | hmmm sjmc7 / TravT do we have bp for the config stuff? | 22:06 |
TravT | yes. | 22:07 |
TravT | let me grab that | 22:07 |
TravT | https://blueprints.launchpad.net/searchlight/+spec/plugin-config-options | 22:07 |
openstackgerrit | Endre Karlson proposed openstack/searchlight: Make use of config options for indexer plugins https://review.openstack.org/223318 | 22:08 |
ekarlso | there u go TravT / sjmc7 | 22:08 |
ekarlso | :P | 22:08 |
TravT | :)\ | 22:08 |
TravT | keeping me busy ekarlso | 22:08 |
ekarlso | TravT: u should +2 that and the client one ;) | 22:08 |
TravT | I'm running through your designate plugin now. | 22:09 |
TravT | :) | 22:09 |
openstackgerrit | Endre Karlson proposed openstack/searchlight: Refactor clients handling to use Sessions. https://review.openstack.org/219271 | 22:09 |
ekarlso | now I can go sleep well soon :D | 22:09 |
TravT | :D | 22:10 |
ekarlso | TravT: where's your horizon thing for SL ? | 22:10 |
TravT | it's coming along... showed it to David on Friday. He liked it. | 22:11 |
TravT | right now it is still a patch on my local box. | 22:11 |
ekarlso | TravT: care to sahre ? :) | 22:14 |
TravT | i can. give me another day, though. | 22:15 |
TravT | :) | 22:15 |
TravT | ekarlso | 22:28 |
TravT | if I create an address record in a domain | 22:28 |
ekarlso | TravT: ya? | 22:29 |
TravT | with ip addr of 123.123.123.123 | 22:29 |
TravT | in horizon, i see this: | 22:29 |
TravT | listener.test30.test. | 22:29 |
TravT | A123.123.123.123 | 22:29 |
ekarlso | ok ? | 22:29 |
TravT | in recordset data, i don't see any data? | 22:29 |
TravT | { | 22:29 |
TravT | "_score": 1, | 22:29 |
TravT | "_type": "OS::Designate::RecordSet", | 22:29 |
TravT | "_id": "28fe5f44-3e1a-419e-88db-9487c877def7", | 22:29 |
TravT | "_source": { | 22:29 |
TravT | "zone_id": "3f95a6fc-988c-41e8-9236-e6a9d63a8ccb", | 22:30 |
TravT | "created_at": "2015-09-14T22:26:40.000000", | 22:30 |
TravT | "description": null, | 22:30 |
TravT | "updated_at": null, | 22:30 |
TravT | "records": [ | 22:30 |
ekarlso | uhms | 22:30 |
TravT | {} | 22:30 |
TravT | ], | 22:30 |
TravT | "version": 1, | 22:30 |
TravT | "ttl": null, | 22:30 |
ekarlso | bug then :| | 22:30 |
TravT | "project_id": "01b1db31e2b74175afd33c759574a69a", | 22:30 |
TravT | "type": "A", | 22:30 |
TravT | "id": "28fe5f44-3e1a-419e-88db-9487c877def7", | 22:30 |
TravT | "name": "listener.test30.test." | 22:30 |
TravT | }, | 22:30 |
TravT | "_index": "searchlight" | 22:30 |
TravT | } | 22:30 |
TravT | for the NS records | 22:30 |
TravT | i do | 22:30 |
TravT | "records": [ | 22:30 |
TravT | { | 22:30 |
TravT | "data": "ns1.devstack.org." | 22:30 |
TravT | } | 22:30 |
TravT | i just ran index sync on it | 22:30 |
TravT | and now there is data | 22:31 |
ekarlso | lemme take a lookj | 22:31 |
TravT | "records": [ | 22:31 |
TravT | { | 22:31 |
TravT | "data": "123.123.123.123" | 22:31 |
TravT | } | 22:31 |
ekarlso | haha :p | 22:32 |
ekarlso | TravT: 1 error on 1 line | 22:32 |
TravT | FWIW, my rbac queries now seem to work. | 22:35 |
ekarlso | yap I know ;) | 22:35 |
ekarlso | testing each time myself now :P | 22:35 |
TravT | so getting record sets back for the right project. | 22:35 |
TravT | :) | 22:35 |
openstackgerrit | Endre Karlson proposed openstack/searchlight: Add Designate plugin. https://review.openstack.org/199099 | 22:35 |
TravT | ok, i'll pull and try again | 22:35 |
ekarlso | gogogo :P | 22:35 |
TravT | ok, all of a sudden git review -d is hanging | 22:37 |
ekarlso | :P | 22:37 |
sjmc7 | call comcast ;) | 22:37 |
TravT | spent nearly 2 hours on the phone with them today.. | 22:38 |
TravT | it pulled. trying again... | 22:38 |
ekarlso | :P | 22:38 |
ekarlso | dont ya have fiber ? | 22:39 |
TravT | i have crappy co-ax coming to my house | 22:39 |
TravT | and its been flaky ever since my neighbor cut the line while landscaping | 22:39 |
ekarlso | http://imgur.com/03ZuKMc < TravT v2 horizon | 22:39 |
ekarlso | off sl topic but | 22:40 |
TravT | cool! | 22:40 |
TravT | back to checking plugin again | 22:41 |
ekarlso | :P | 22:41 |
ekarlso | tomorrow you'll get a new plugin too to review :P | 22:41 |
ekarlso | Cue | 22:41 |
TravT | ok, i swear i updated... but still empty record data checking to see if I messed up | 22:43 |
ekarlso | I dont | 22:45 |
ekarlso | I checked and it works ! :D | 22:45 |
TravT | :S | 22:46 |
ekarlso | anyways, if not leave me a pitch here and i'll check | 22:48 |
ekarlso | tmrw | 22:48 |
ekarlso | bed time now! | 22:48 |
TravT | ok, | 22:48 |
TravT | will run through it... | 22:48 |
TravT | question | 22:49 |
TravT | no notification for delete of record? | 22:50 |
TravT | is that the same designate but as ACTIVE status | 22:50 |
TravT | ? | 22:50 |
sjmc7 | "dns.recordset.delete" | 22:50 |
sjmc7 | ? | 22:50 |
sjmc7 | oh, no notification from designate? that's sad | 22:51 |
sjmc7 | maybe a designate bug? | 22:51 |
TravT | yeah, if i delete a recordset via horizon, no notification received. | 22:51 |
TravT | i do get it when deleting the whole domain / zone | 22:51 |
TravT | he was saying there is a designate bug on recordset notifications with a fix proposed to designate | 22:53 |
TravT | maybe same thing here | 22:53 |
sjmc7 | yep, sounds like it | 22:53 |
TravT | i'm still not getting recordset data though under devstack... guess i will fire up the debugger | 22:54 |
TravT | for notifications on addr recrods | 22:54 |
sjmc7 | hmm. i was | 22:54 |
TravT | i get the recordsets | 22:54 |
TravT | but on notification, there isn't a data property under the records | 22:55 |
TravT | so | 23:13 |
TravT | the record data isn't in the notification payload | 23:14 |
TravT | so, i don't think this is a bug in our plugin | 23:14 |
*** david-lyle has quit IRC | 23:20 | |
*** TravT has quit IRC | 23:27 | |
*** TravT has joined #openstack-searchlight | 23:41 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!