*** vinod1 has joined #openstack-dns | 00:03 | |
Kiall | darshan104: excellent :) | 00:06 |
---|---|---|
Kiall | vinod: If you're still around, I had to rebase https://review.openstack.org/#/c/107972/ (the oslo.i18n patch).. Needs it's +A again :( | 00:07 |
vinod1 | Just got to Seattle - I will take a look at it shortly | 00:07 |
vinod1 | kiall: Code-Review+2 Workflow+1 | 00:19 |
openstackgerrit | Darshan Sanghani proposed a change to openstack/designate: Empty RecordSets can be re-provisioned https://review.openstack.org/108840 | 00:21 |
*** nkinder has joined #openstack-dns | 00:24 | |
openstackgerrit | A change was merged to openstack/designate: Switchover to oslo.i18n package https://review.openstack.org/107972 | 00:35 |
*** rossk has quit IRC | 00:43 | |
*** darshan104 has quit IRC | 00:44 | |
*** rossk has joined #openstack-dns | 00:45 | |
*** vinod1 has quit IRC | 01:31 | |
*** nosnos has joined #openstack-dns | 01:38 | |
*** rossk has quit IRC | 01:41 | |
*** darshan104 has joined #openstack-dns | 01:45 | |
*** vinod1 has joined #openstack-dns | 01:49 | |
*** darshan104 has quit IRC | 01:49 | |
*** darshan104 has joined #openstack-dns | 01:53 | |
*** darshan104 has quit IRC | 02:02 | |
*** vinod1 has quit IRC | 02:21 | |
*** vinod1 has joined #openstack-dns | 02:26 | |
*** vinod1 has quit IRC | 02:28 | |
*** vinod1 has joined #openstack-dns | 02:46 | |
*** vinod1 has quit IRC | 02:50 | |
*** nosnos has quit IRC | 03:55 | |
*** betsy has joined #openstack-dns | 04:27 | |
*** darshan104 has joined #openstack-dns | 04:28 | |
*** nosnos has joined #openstack-dns | 04:51 | |
*** bandarji has joined #openstack-dns | 05:11 | |
*** bandarji has quit IRC | 05:33 | |
*** k4n0 has joined #openstack-dns | 05:56 | |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/designate: Imported Translations from Transifex https://review.openstack.org/109500 | 06:08 |
openstackgerrit | Endre Karlson proposed a change to openstack/designate: Add proxy support to dynect client https://review.openstack.org/109515 | 07:44 |
openstackgerrit | Endre Karlson proposed a change to openstack/designate: Add proxy support to dynect client https://review.openstack.org/109516 | 07:49 |
*** darshan104 has quit IRC | 09:23 | |
*** nosnos has quit IRC | 10:15 | |
*** k4n0 has quit IRC | 11:38 | |
*** vinod1 has joined #openstack-dns | 13:15 | |
*** vinod1 has quit IRC | 13:24 | |
*** jaycaz has joined #openstack-dns | 13:28 | |
*** eankutse has joined #openstack-dns | 13:45 | |
*** timsim has joined #openstack-dns | 13:56 | |
*** jasondotstar has joined #openstack-dns | 14:02 | |
*** msisk has joined #openstack-dns | 14:02 | |
openstackgerrit | Jordan Cazamias proposed a change to openstack/designate: Add instructions on installing Designate with MySQL backend https://review.openstack.org/109604 | 14:19 |
*** mwagner_lap has quit IRC | 14:25 | |
openstackgerrit | Jordan Cazamias proposed a change to openstack/designate: Add quotas documentation https://review.openstack.org/109142 | 14:31 |
openstackgerrit | Jordan Cazamias proposed a change to openstack/designate: Add quotas documentation https://review.openstack.org/109142 | 14:59 |
*** bandarji has joined #openstack-dns | 15:14 | |
*** jasondotstar has quit IRC | 15:30 | |
*** bandarji has quit IRC | 16:26 | |
*** richm has joined #openstack-dns | 16:42 | |
*** rustlebee has joined #openstack-dns | 16:56 | |
rustlebee | Kiall: o/ since i wasn't able to automate juno-2 tagging/tarball generation, would you like to manually make a juno-2 tarball? then i can upload it to launchpad | 16:57 |
*** vinod1 has joined #openstack-dns | 17:11 | |
*** crc32 has joined #openstack-dns | 17:13 | |
Kiall | rustlebee: sure | 17:43 |
Kiall | Let me do that now | 17:43 |
Kiall | rustlebee: pushed + building | 17:47 |
Kiall | rustlebee: completed - http://tarballs.openstack.org/designate/designate-2014.2.b2.tar.gz | 17:53 |
mugsie | vinod1: / rjrjr / betsy / richm : we are runnign a few mins late - I will start the hangout about 11:05 | 17:58 |
vinod1 | ok | 17:58 |
rjrjr | np | 17:59 |
mugsie | rjrjr: we have started | 18:08 |
Kiall | https://wiki.openstack.org/wiki/Designate/Paris/Presentations | 18:09 |
*** darshan104 has joined #openstack-dns | 18:12 | |
timsim | http://en.wikipedia.org/wiki/Comparison_of_DNS_server_software | 18:20 |
openstackgerrit | Jordan Cazamias proposed a change to openstack/designate: WIP: Add instructions on installing Designate with MySQL backend https://review.openstack.org/109604 | 18:20 |
*** darshan104 has quit IRC | 18:25 | |
*** jaycaz has left #openstack-dns | 18:26 | |
mugsie | https://wiki.openstack.org/wiki/Designate/Atlanta/Workshop_1 | 18:31 |
Kiall | https://etherpad.openstack.org/p/DesignateAtalantaWorkshop | 18:31 |
*** darshan104 has joined #openstack-dns | 18:34 | |
*** darshan104 has quit IRC | 18:47 | |
*** darshan104 has joined #openstack-dns | 18:48 | |
rjrjr | what are the default quotas for a tenant? is the best way to overcome this limit to enable quotas and change the default per tenant? | 19:07 |
timsim | rjrjr: I believe it's configurable. Just a sec... | 19:22 |
timsim | https://github.com/openstack/designate/blob/master/designate/quota/__init__.py | 19:23 |
timsim | You can set those options in the config file. If you don't,those are the defaults. | 19:23 |
timsim | You can also change them using the /quotas endpoint on a per-tenant basis | 19:23 |
*** darshan104 has quit IRC | 19:28 | |
rjrjr | timsim: thanks. i'm reading through the code now. | 19:35 |
rjrjr | it's not very clear which section of designate.conf needs to have the defaults set. | 19:36 |
timsim | I've put them in the first section before: https://github.com/openstack/designate/blob/master/etc/designate/designate.conf.sample#L1 | 19:38 |
rjrjr | timsim: i don't see it. i've tried service:api and service:central. everytime i list the quotas for a tenant, i keep getting the original, out of the box values, not the values i set. | 19:44 |
rjrjr | never mind, i got it. first section = [DEFAULT]. duh! | 19:47 |
*** vinod1 has quit IRC | 20:01 | |
*** timsim has quit IRC | 20:14 | |
*** eankutse has quit IRC | 20:23 | |
*** vinod1 has joined #openstack-dns | 20:25 | |
*** darshan104 has joined #openstack-dns | 20:29 | |
*** darshan104 has quit IRC | 20:33 | |
*** darshan104 has joined #openstack-dns | 20:38 | |
*** darshan104 has quit IRC | 20:44 | |
*** darshan104 has joined #openstack-dns | 20:44 | |
betsy | Kiall: You around? | 20:57 |
betsy | If you get a chance could you check out mine and Vinod’s comments on https://review.openstack.org/#/c/108797/ | 20:58 |
openstackgerrit | Vinod Mangalpally proposed a change to openstack/designate: Add support for AXFR requests https://review.openstack.org/109731 | 21:08 |
Kiall | betsy: I am, but about to start a deploy | 21:09 |
vinod1 | kiall/betsy: An AXFR response needs to have the SOA record at the beginning and end. The records in between do not contain the SOA record. So I am wondering if it would make sense to handle the SOA record in a special manner | 21:14 |
vinod1 | Currently in Mdns I use storage.find_recordsets to get the recordsets. If this returns SOA too, then I might need to filter that out | 21:15 |
vinod1 | Please see the review at https://review.openstack.org/109731 for more details | 21:16 |
betsy | kiall: No problem. Just whenever you get a chance. No hurry | 21:18 |
betsy | vinod1: I was wondering if there might need to be a special call for the soa record | 21:18 |
betsy | Could you use find_recordset and filter by zone id and type? | 21:19 |
vinod1 | I wonder if I could filter by type != SOA, which would get all the recordsets other than SOA | 21:19 |
*** eankutse has joined #openstack-dns | 21:20 | |
betsy | Hmm. Maybe? | 21:20 |
*** eankutse has quit IRC | 21:21 | |
*** eankutse has joined #openstack-dns | 21:21 | |
*** rmoe has joined #openstack-dns | 21:22 | |
betsy | I thought in the API call of GET recordsets, we did want to return NS and SOA records, but I see that you need to exclude them | 21:22 |
openstackgerrit | Kiall Mac Innes proposed a change to openstack/designate: Set python hash seed to 0 in tox.ini https://review.openstack.org/109740 | 21:32 |
Kiall | darshan104: that tox >= 1.7.0 issue you noted yesterday should be fixed with ^ .. The openstack-infra folks are looking into it.. | 21:33 |
Kiall | rjrjr: The gate change has been approved, fingers crossed once it merges and goes live your patch works :D | 21:34 |
Kiall | mugsie / betsy / vinod1: https://review.openstack.org/109740 and https://review.openstack.org/#/c/109744/ will need to merge ASAP - Before infra update the slaves to 1.7.2 :) | 21:45 |
vinod1 | looking at them now | 21:45 |
Kiall | BG info @ http://lists.openstack.org/pipermail/openstack-dev/2014-July/041283.html | 21:46 |
betsy | I was already looking at the first one | 21:46 |
Kiall | (Not that I really understand the issue ;)) | 21:46 |
darshan104 | Kiall: oh, great! | 21:46 |
openstackgerrit | Kiall Mac Innes proposed a change to openstack/designate: devstack exercise for deletion, dns checking https://review.openstack.org/97628 | 21:57 |
openstackgerrit | A change was merged to openstack/designate: Set python hash seed to 0 in tox.ini https://review.openstack.org/109740 | 22:00 |
rjrjr | kiall: which patch? | 22:01 |
openstackgerrit | Vinod Mangalpally proposed a change to openstack/designate: Add support for AXFR requests https://review.openstack.org/109731 | 22:01 |
*** msisk has quit IRC | 22:03 | |
Kiall | rjrjr: the DNS devstack chceks | 22:07 |
Kiall | Ah.. that's richm's ;) | 22:07 |
Kiall | autocomplete fail -_- | 22:07 |
openstackgerrit | Kiall Mac Innes proposed a change to openstack/designate: devstack exercise for deletion, dns checking https://review.openstack.org/97628 | 22:19 |
Kiall | richm: WOO - It passed :) | 22:19 |
Kiall | (Just did some tiny cleanup there) | 22:19 |
*** eankutse1 has joined #openstack-dns | 22:33 | |
*** eankutse1 has quit IRC | 22:33 | |
*** eankutse has quit IRC | 22:37 | |
Kiall | betsy / vinod1 - time to get richm's patch merged while it still passes :P https://review.openstack.org/#/c/97628 | 22:42 |
betsy | Ok. I’ll take a look at it. :) | 22:43 |
vinod1 | ditto | 22:43 |
Kiall | Cool - it's taken wayyy too long to get it passing :) | 22:43 |
Kiall | and - betsy: you were looking for earlier? | 22:44 |
betsy | Vinod and I both made comments on https://review.openstack.org/#/c/108797/ | 22:45 |
betsy | Was wondering if you could comment | 22:45 |
Kiall | So - yea, I think I agree with your comment betsy - really, priority being it's own column should be either removed entirely, or hidden in the storage layer IMO | 22:48 |
Kiall | (goes back to the whole "why do we treat priority specially and not X, Y or Z" :)) | 22:48 |
betsy | Yeah. That’s what I was thinking. Cool | 22:48 |
Kiall | IMO - The best course of action is to drop that column entirely, and merge it into the data field like all the other rdata fields.. but it may be easier to start by just hiding the extra col, I've not looked at how engrained it is :) | 22:49 |
openstackgerrit | A change was merged to openstack/designate: Add quotas documentation https://review.openstack.org/109142 | 22:50 |
betsy | Yeah. I think there’s a lot of database re-work that could be done | 22:57 |
betsy | But not sure how high that falls on the priority list right now | 22:58 |
betsy | Oh, and it is still separate in v1 | 22:58 |
Kiall | betsy: agreed, your idea to split the types out into there own table is the long term way to go.. For now, I'll settle for consistency :) | 22:58 |
Kiall | Yea, V1 will need that "get_priority" etc method you wrote :) | 22:59 |
betsy | So, probably best to leave it as a separate column for now | 22:59 |
Kiall | I'm not sure really, if we hide it so outside of storage can't see it, then if it's a col or not shouldn't matter, as everything is dealing with the combined format | 23:01 |
Kiall | e.g. kill 2 birds with 1 stone, fix the V2 API by fixing the general data model, and add the split into separate fields into the V1 layer | 23:02 |
betsy | The way I have it coded now, is v1 still gets it directly from the db | 23:02 |
betsy | So, change the db table or just the model? | 23:03 |
Kiall | I guess it depends on how much of a pain that is :) I've not looked at what would be involved, but if the level of work to fix the DB all the way out to the API is roughly the same as combining in the storage layer, and reading/splitting the combined in the API, then it makes sense to get the DB fixed while we're at it | 23:05 |
betsy | Well, I guess I was thinking about waiting until the db tables get split out. Makes more sense to me to do it then | 23:05 |
betsy | The way I coded it now, i didn’t touch the storage layer | 23:06 |
betsy | I just combine it in the object after it’s pulled from storage for v2 and leave it alone for v1 | 23:07 |
Kiall | Well, the object should be "global" the same object and fields on the object should be used by both APIs | 23:08 |
Kiall | But - I think I see what issue your hitting (reading the code again ;))) | 23:08 |
Kiall | Right now, the base Record object has a "priority" field, which really shouldn't be there.. and we're not really yet using the RType specific objects | 23:10 |
betsy | Right. And I couldn’t change it for v1 | 23:11 |
Kiall | and https://github.com/openstack/designate/blob/master/designate/api/v2/views/recordsets.py#L46 | 23:11 |
Kiall | "records": [r.data for r in recordset['records']], | 23:11 |
Kiall | should really be | 23:11 |
betsy | So, I made the change in v2 as close as possible to storage | 23:11 |
Kiall | "records": [r.to_text() for r in recordset['records']], | 23:11 |
Kiall | i.e. maybe to do it right, we need to finish out https://review.openstack.org/#/c/102307/ | 23:12 |
Kiall | (It got blocked by a bunch of other objects reviews, and forgotten about) | 23:12 |
betsy | Ah. Yes. I forgot that was part of your change | 23:13 |
betsy | I can change that and resubmit | 23:13 |
Kiall | Looking at that patchset again, I think it's something where all the objects blockers have been fixed.. | 23:14 |
Kiall | But it's totally out of date and likely needs to be started fresh.. I might do that tomorrow | 23:14 |
betsy | ok | 23:14 |
Kiall | Almost all the code in that review needs to be tossed out as it will conflict like mad with current master ;) | 23:14 |
openstackgerrit | Kiall Mac Innes proposed a change to openstack/designate: Install only the necessary files during a DevStack gate run https://review.openstack.org/109771 | 23:15 |
betsy | Definitely! :( | 23:15 |
betsy | I wasn’t sure if you were abandoning those changes or not | 23:15 |
betsy | So does my patchset need to be changed? | 23:16 |
Kiall | Let me look at your patch again now that we've looked at that :) | 23:17 |
betsy | ok | 23:17 |
*** shuffleb1t has joined #openstack-dns | 23:24 | |
*** retr0h_ has joined #openstack-dns | 23:25 | |
*** Kiall_ has joined #openstack-dns | 23:25 | |
Kiall_ | Kiall_ <-- ehh, who's that? lol | 23:26 |
Kiall_ | wtf.. weird | 23:27 |
Kiall_ | my nick changed, and I saw myself join | 23:27 |
betsy | :D | 23:27 |
*** puck` has joined #openstack-dns | 23:27 | |
*** puck` is now known as Guest17434 | 23:28 | |
*** puck has quit IRC | 23:29 | |
*** Kiall has quit IRC | 23:29 | |
*** ekarlso has quit IRC | 23:29 | |
*** retr0h has quit IRC | 23:29 | |
*** shufflebot has quit IRC | 23:29 | |
*** ekarlso has joined #openstack-dns | 23:31 | |
*** rmoe has quit IRC | 23:34 | |
*** Kiall_ is now known as Kiall | 23:43 | |
*** rmoe has joined #openstack-dns | 23:43 | |
openstackgerrit | A change was merged to openstack/designate: devstack exercise for deletion, dns checking https://review.openstack.org/97628 | 23:45 |
*** darshan104 has quit IRC | 23:57 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!