*** TravT_ has joined #openstack-searchlight | 01:29 | |
*** TravT has quit IRC | 01:31 | |
*** sballe has quit IRC | 01:46 | |
*** david-ly_ has joined #openstack-searchlight | 01:51 | |
*** david-lyle has quit IRC | 01:54 | |
*** TravT_ has quit IRC | 02:03 | |
*** TravT has joined #openstack-searchlight | 02:04 | |
*** lakshmiS has joined #openstack-searchlight | 05:01 | |
*** lakshmiS has quit IRC | 08:27 | |
*** lakshmiS has joined #openstack-searchlight | 08:30 | |
*** GB21 has joined #openstack-searchlight | 10:32 | |
*** GB21 has quit IRC | 11:37 | |
*** sigmavirus24_awa is now known as sigmavirus24 | 14:11 | |
*** sballe has joined #openstack-searchlight | 14:40 | |
lakshmiS | nikhil_k_: added a patch in glance for generating notifications for image members. Hopefully it can get in the release somewhere :) | 14:41 |
---|---|---|
lakshmiS | https://bugs.launchpad.net/glance/+bug/1441453 | 14:41 |
openstack | Launchpad bug 1441453 in Glance "Image members CRUD doesn't generate notifications which impacts searchlight RBAC." [Undecided,In progress] - Assigned to Lakshmi N Sampath (lakshmi-sampath) | 14:41 |
nikhil_k_ | lakshmiS: thanks! will look | 14:46 |
*** david-ly_ is now known as david-lyle | 14:53 | |
TravT | awesome, thanks, lakshmiS | 15:06 |
nikhil_k_ | lakshmiS: patch looks good at first glance, I think we should be able to make it to rc1 (sept 25) | 15:08 |
*** TravT is now known as TravT_away | 15:08 | |
lakshmiS | nikhil_k: thanks | 15:09 |
*** GB21_ has joined #openstack-searchlight | 15:57 | |
*** GB21 has joined #openstack-searchlight | 15:57 | |
*** GB21 has quit IRC | 15:58 | |
*** GB21_ is now known as GB21 | 15:58 | |
*** lakshmiS has quit IRC | 16:01 | |
*** TravT_away is now known as TravT | 16:42 | |
*** GB21 has quit IRC | 17:01 | |
*** TravT_ has joined #openstack-searchlight | 17:21 | |
*** TravT has quit IRC | 17:21 | |
*** openstackgerrit has quit IRC | 18:31 | |
*** openstackgerrit has joined #openstack-searchlight | 18:32 | |
openstackgerrit | Steve McLellan proposed openstack/searchlight: Allow indexing of single type or index https://review.openstack.org/210759 | 20:37 |
*** ekarlso has quit IRC | 20:38 | |
*** asahlin_ has quit IRC | 20:47 | |
*** ekarlso has joined #openstack-searchlight | 20:57 | |
*** ekarlso has quit IRC | 20:57 | |
*** ekarlso has joined #openstack-searchlight | 20:57 | |
ekarlso | oii | 20:57 |
openstackgerrit | Endre Karlson proposed openstack/searchlight: Add Designate plugin. https://review.openstack.org/199099 | 20:58 |
ekarlso | TravT_: u got time to give ^ a go ? | 20:58 |
TravT_ | hey ekarlso | 20:59 |
TravT_ | sure, i'll look it over | 20:59 |
TravT_ | let me stash away something first. | 20:59 |
*** TravT_ is now known as TravT | 20:59 | |
ekarlso | TravT: am I not 1st priority in your life ?!?!? :P | 21:15 |
sjmc7_ | :D | 21:19 |
TravT | okay, stashed away work. | 21:22 |
TravT | :D | 21:22 |
TravT | will look now | 21:22 |
TravT | ekarlso: i got the notification of zone create | 21:34 |
ekarlso | worked ? | 21:34 |
TravT | but doesn't look like i got anything for recordset create | 21:34 |
ekarlso | uh | 21:34 |
ekarlso | it should! | 21:34 |
TravT | what do i need in my config | 21:34 |
TravT | ? | 21:34 |
TravT | for designate? | 21:34 |
ekarlso | nothing more then the usual | 21:34 |
ekarlso | as in what's in the changeset doc | 21:35 |
TravT | well, here's the behavior i'm seeing | 21:35 |
TravT | i created a domain from horizon ui | 21:36 |
TravT | in elasticsearch I see a zone record | 21:36 |
TravT | but no recordset records | 21:36 |
ekarlso | lemme see in my env. | 21:36 |
TravT | so, i did a complete searchlight-manage index sync | 21:36 |
TravT | and then i see recordsets | 21:36 |
ekarlso | hmmmms | 21:36 |
TravT | i didn't actually put a breakpoint or anything. | 21:36 |
ekarlso | ok, i'll take a look | 21:36 |
TravT | just running under /opt/stack/searchlight | 21:37 |
TravT | i did have a question on the relationship between recordsets and zones in my last set of comments | 21:37 |
ekarlso | TravT: a zone is the parent resource of a rs | 21:37 |
TravT | are recordsets only related to a single zone? | 21:37 |
ekarlso | TravT: yes | 21:38 |
ekarlso | TravT: recordsets internally actually has a subresource called records.. | 21:38 |
TravT | well, i'm wondering if you would want to index recordset as a nested type in a zone then | 21:39 |
TravT | and not have a distinct type in ES | 21:39 |
TravT | right now, you have the zone_id in recordset | 21:39 |
TravT | which is good for relational mapping | 21:39 |
ekarlso | TravT: well a zone can have thousands of recordsets.. | 21:39 |
TravT | hmm... | 21:40 |
ekarlso | and each recordset again can hold tens of records :p | 21:40 |
TravT | sjmc7^ | 21:40 |
ekarlso | since it's really in dns: zone travis.name. > A "recordset" travis.name > ["10.0.0.1", "10.0.0.2"] | 21:40 |
ekarlso | ish | 21:40 |
ekarlso | but the API exposes just zones / sets | 21:40 |
ekarlso | or if you go the MX route | 21:40 |
TravT | so, will people be looking to find a particular zone by the recordset? | 21:41 |
TravT | or looking for particular recordset? | 21:41 |
ekarlso | TravT: might be | 21:41 |
ekarlso | TravT: find me any rs that's type A with address 10.0.0.1 | 21:41 |
ekarlso | f ex | 21:41 |
TravT | ok. | 21:42 |
ekarlso | totally valid usecase | 21:42 |
ekarlso | i checked it here TravT, listener is getting notifications on recordset create.. | 21:43 |
TravT | ok. let me mess with it a bit. | 21:44 |
ekarlso | TravT: try this: openstack recordset create <domain name> www --type A --records 10.0.0.1 | 21:44 |
TravT | any chance you don't receive it when creating a new zone? | 21:44 |
ekarlso | that should give u a notification | 21:44 |
ekarlso | TravT: receiving a notification or ? | 21:44 |
TravT | yeah, so i just created a new zone | 21:44 |
TravT | it showed up with two records | 21:44 |
TravT | i didn't make a discrete call | 21:44 |
ekarlso | there's a dns.domain.create thing going out when creating a zone | 21:44 |
TravT | let me add a record to an already existing zone | 21:45 |
ekarlso | btw TravT we have v2 stuff underway for angular / horizon | 21:46 |
ekarlso | :P | 21:46 |
TravT | cool | 21:46 |
TravT | so, i get a notification when adding a record to a zone | 21:47 |
ekarlso | yap :p | 21:47 |
ekarlso | expected! | 21:47 |
ekarlso | i'd be *sadface* if not | 21:47 |
TravT | but when i create a new zone, the records in it aren't indexed it seems | 21:47 |
TravT | via the notification route | 21:47 |
ekarlso | TravT: well it might be that we woud need to add a call there to fetch the existing records.. | 21:48 |
ekarlso | but it's the NS records to unsure how "necassary" it is | 21:48 |
TravT | but if i do index from CLI to update everything they get indexed | 21:48 |
TravT | looks like a NS and SOA record | 21:48 |
TravT | let me poke at the RBAC for a few minutes. I saw that now you are remapping tenant_id to project_id. | 21:50 |
ekarlso | TravT: i'll put up a new PS tomorrow *again* to fetch the recordsets upon the initial indexation of the zone | 21:51 |
TravT | This is pretty awesome to see it this close | 21:51 |
ekarlso | :p | 21:52 |
ekarlso | TravT: I was starting yesterday to poke at Cue stuff for ya as well | 21:52 |
TravT | sweet! | 21:52 |
TravT | I'll run through everything again and see if there's anything else for you. | 21:52 |
TravT | i want to poke at a few queries on the recordsets | 21:52 |
TravT | but it seems that we are getting very close to saying it can go in and deal with bugs as they come. | 21:53 |
ekarlso | TravT: you are a horizon dev no ? | 21:58 |
TravT | i spend a fair amount of time there | 22:01 |
ekarlso | TravT: so, in v2 of designate we have PRIMARY / SECONDARY zones, on the creation wizard I wanna do a conditional thing with the inputs | 22:08 |
ekarlso | based on the selected type, any clues ? | 22:08 |
ekarlso | I figured a radio button thing with pirmary / secondary but sjmc7_ said it wasn't that widely used in horizon | 22:08 |
TravT | is this in angular or in the python based way? | 22:09 |
ekarlso | TravT: angular | 22:09 |
ekarlso | v2 is gonna be angular I think ; | 22:09 |
TravT | well, the classic launch instance used a drop down for conditional inputs | 22:10 |
TravT | e.g. source type of volume gave additional fields. | 22:10 |
sjmc7_ | i don't recall seeing any radio buttons in horizon | 22:10 |
ekarlso | ok :) | 22:10 |
ekarlso | dropdown it is! | 22:10 |
TravT | in the angular one, we used a radio button but styled more like mobile | 22:11 |
ekarlso | btw, you should really bundle the html files in a .js file.. | 22:11 |
ekarlso | instead of loading 1 by 1 | 22:11 |
TravT | optimizing horizon loading is something to be discussed further i think... | 22:13 |
TravT | but let's not get into that here. | 22:13 |
TravT | ekarlso: | 22:13 |
ekarlso | TravT: ;P | 22:13 |
TravT | you might want to ask Piet to put up a mock for you | 22:13 |
TravT | it will cost you a few days of time... | 22:14 |
TravT | but it can also be helpful | 22:15 |
*** sigmavirus24 is now known as sigmavirus24_awa | 22:30 | |
*** TravT has quit IRC | 23:36 | |
*** TravT has joined #openstack-searchlight | 23:39 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!