*** ryanpetrello has joined #openstack-dns | 00:43 | |
*** tylerdurden has quit IRC | 00:54 | |
*** shakamunyi has joined #openstack-dns | 00:55 | |
*** ryanpetrello has quit IRC | 00:59 | |
*** ryanpetrello has joined #openstack-dns | 01:01 | |
*** ryanpetrello has quit IRC | 01:12 | |
*** boris-42 has quit IRC | 01:13 | |
*** thetrav has joined #openstack-dns | 01:55 | |
*** Stanley00 has joined #openstack-dns | 01:57 | |
*** thetrav has quit IRC | 02:16 | |
*** GonZo2K has quit IRC | 02:48 | |
*** thetrav has joined #openstack-dns | 03:23 | |
thetrav | so I'm looking to pull the keystone project/tenant name out out on the nova/neutron events and use it in the dns entry | 03:41 |
---|---|---|
thetrav | anyone else done this? | 03:41 |
thetrav | wondering if I can make keystone api calls from the existing code base or if I need to pull in the keystone client | 03:41 |
*** harmw has quit IRC | 04:09 | |
*** harmw has joined #openstack-dns | 04:09 | |
*** boris-42 has joined #openstack-dns | 05:01 | |
*** thetrav has quit IRC | 06:18 | |
*** simonmcc has quit IRC | 06:26 | |
*** simonmcc_ has joined #openstack-dns | 06:27 | |
*** nihilifer has joined #openstack-dns | 06:44 | |
*** yfujioka has quit IRC | 08:07 | |
*** k4n0 has joined #openstack-dns | 08:25 | |
openstackgerrit | Endre Karlson proposed openstack/designate: DynECT tests using requests-mock. https://review.openstack.org/141664 | 09:03 |
*** nihilifer has quit IRC | 09:24 | |
*** nihilifer has joined #openstack-dns | 09:26 | |
*** jordanP has joined #openstack-dns | 09:28 | |
*** ttrumm has joined #openstack-dns | 09:29 | |
openstackgerrit | Endre Karlson proposed openstack/designate: DynECT: Handle errors correctly when creating domain https://review.openstack.org/141291 | 09:54 |
openstackgerrit | Endre Karlson proposed openstack/designate: DynECT tests using requests-mock. https://review.openstack.org/141664 | 09:54 |
*** Stanley00 has quit IRC | 10:05 | |
openstackgerrit | Endre Karlson proposed openstack/designate: DynECT tests using requests-mock https://review.openstack.org/141664 | 10:11 |
*** GonZo2K has joined #openstack-dns | 10:39 | |
*** nihilifer has quit IRC | 10:46 | |
*** nihilifer has joined #openstack-dns | 10:50 | |
*** untriaged-bot has joined #openstack-dns | 11:02 | |
untriaged-bot | Untriaged bugs so far: | 11:02 |
untriaged-bot | https://bugs.launchpad.net/designate/+bug/1390357 | 11:02 |
untriaged-bot | https://bugs.launchpad.net/designate/+bug/1289444 | 11:02 |
uvirtbot | Launchpad bug 1390357 in designate "Designate sync creates duplicated records in powerdns backend" [Medium,New] | 11:02 |
*** untriaged-bot has quit IRC | 11:02 | |
uvirtbot | Launchpad bug 1289444 in designate "Designate with postgres backend is having issues" [Undecided,New] | 11:02 |
*** mikehn has joined #openstack-dns | 11:07 | |
*** GonZo2K has quit IRC | 11:21 | |
*** nihilifer has quit IRC | 12:39 | |
openstackgerrit | Endre Karlson proposed openstack/designate: DynECT: Fix error handling when creating domain https://review.openstack.org/141291 | 12:44 |
openstackgerrit | Merged openstack/designate: Adds "domain-servers-list" to test_domains.py https://review.openstack.org/140935 | 12:46 |
openstackgerrit | Merged openstack/designate: Cleanup validation regex's https://review.openstack.org/141056 | 12:48 |
*** nihilifer has joined #openstack-dns | 12:56 | |
betsy | kiall/mugsie: Could we go ahead and merge https://review.openstack.org/#/c/133549/? I ended up reworking it in the next dependent patchset https://review.openstack.org/#/c/136440/ instead of in the parent like I should have. | 13:51 |
Kiall | dooh :) K - Let me look over the two in a few mins | 13:52 |
betsy | I’ve still got another patchset to submit for the second one | 13:53 |
betsy | It’s almost ready but not quite | 13:54 |
mugsie_ | cool, will lookm now | 13:54 |
*** mugsie_ is now known as mugsie | 13:55 | |
betsy | I’m deleting most of the pool_attributes in the child patchset | 13:57 |
Kiall | Maybe it's best to just merge the two patchsets if the second undoes lots of the first? | 13:58 |
Kiall | (bearing in mind, I haven't read them yet!) | 13:59 |
*** richm1 has joined #openstack-dns | 14:10 | |
ekarlso- | hhttps://review.openstack.org/#/c/141291/ anyone wanna signoff ? | 14:14 |
*** nihilifer has quit IRC | 14:25 | |
openstackgerrit | Endre Karlson proposed openstack/designate: MDNS part of Secondary zones https://review.openstack.org/133683 | 14:25 |
openstackgerrit | Endre Karlson proposed openstack/designate: Support secondary zones https://review.openstack.org/133682 | 14:25 |
openstackgerrit | Kiall Mac Innes proposed openstack/designate: Pools Config Changes https://review.openstack.org/141166 | 14:30 |
openstackgerrit | Kiall Mac Innes proposed openstack/designate: Convert PowerDNS to a Pools Backend https://review.openstack.org/141167 | 14:30 |
openstackgerrit | Kiall Mac Innes proposed openstack/designate: Remove the non-pools bind9 backend https://review.openstack.org/141368 | 14:30 |
*** betsy has quit IRC | 14:34 | |
*** timsim has joined #openstack-dns | 14:53 | |
*** ttrumm has quit IRC | 15:01 | |
openstackgerrit | Kiall Mac Innes proposed openstack/designate: Remove the agent service https://review.openstack.org/141819 | 15:12 |
*** vinod has joined #openstack-dns | 15:28 | |
openstackgerrit | Kiall Mac Innes proposed openstack/designate: Pool Manager Integration with Central https://review.openstack.org/138406 | 15:36 |
openstackgerrit | Kiall Mac Innes proposed openstack/designate: Remove the agent service and old-style backend tests https://review.openstack.org/141819 | 15:36 |
*** nkinder_away has joined #openstack-dns | 15:36 | |
vinod | Kiall: for Pool Manager Integration with Central https://review.openstack.org/138406 | 15:41 |
vinod | did you just do a rebase for patchset 6 or were there more changes? | 15:41 |
openstackgerrit | Merged openstack/designate: Add Werkzeug>=0.7 to requirements.txt https://review.openstack.org/140394 | 15:44 |
*** richm1 has quit IRC | 15:50 | |
Kiall | vinod: mostly a rebase on top of the Config+PowerDNS change, also updated it to match: https://github.com/openstack/designate/commit/22db5ad7395c061dbc55bb74775d39586e12eadf | 15:51 |
Kiall | oh - and "Removed transitional pool_manager_proxy backend" since it wasn't used anymore... | 15:52 |
Kiall | Looks like they all are about to pass gate now, fingers crossed ;) | 15:52 |
*** betsy has joined #openstack-dns | 15:52 | |
vinod | And after that will it be ready to merge? | 15:53 |
Kiall | ah crap - forgot to update the pdns devstack to not use the proxy backend | 15:53 |
Kiall | vinod: in theory, it'll add goof to merge now bar ^ .. re-reviews of the lot should probably be done anyway | 15:54 |
Kiall | Yea, I have some DS changes still needed .. grr. | 15:55 |
*** nkinder_away is now known as nkinder | 15:55 | |
*** paul_glass has joined #openstack-dns | 16:00 | |
*** jmcbride has joined #openstack-dns | 16:03 | |
*** jmcbride1 has joined #openstack-dns | 16:04 | |
*** jmcbride has quit IRC | 16:04 | |
*** jmcbride1 has quit IRC | 16:04 | |
*** jmcbride1 has joined #openstack-dns | 16:05 | |
Kiall | vinod: https://review.openstack.org/#/c/141166/ - https://review.openstack.org/#/c/141167/ - https://review.openstack.org/#/c/141368/ should be good, first three in the chain .. | 16:07 |
vinod | will look at them now | 16:08 |
openstackgerrit | Kiall Mac Innes proposed openstack/designate: Remove the agent service and old-style backend tests https://review.openstack.org/141819 | 16:09 |
*** paul_glass has quit IRC | 16:13 | |
*** richm1 has joined #openstack-dns | 16:14 | |
openstackgerrit | Kiall Mac Innes proposed openstack/designate: Pool Manager Integration with Central https://review.openstack.org/138406 | 16:17 |
Kiall | Still not sure why pDNS failed on the last patchset of ^ .. Building a clean devstack VM to try reproduce :'( | 16:17 |
Kiall | https://review.openstack.org/#/c/141819/ should be good now too.. found+removed the last traces of the agent... | 16:18 |
Kiall | Humm.. Or not. Where did the DS part of that change go -_- | 16:21 |
*** jmcbride1 has quit IRC | 16:26 | |
*** jmcbride has joined #openstack-dns | 16:31 | |
*** tylerdurden has joined #openstack-dns | 16:33 | |
*** tylerdurden has quit IRC | 16:33 | |
*** tylerdurden has joined #openstack-dns | 16:34 | |
*** tylerdurden has quit IRC | 16:34 | |
*** mugsie has quit IRC | 16:43 | |
*** vinod has quit IRC | 16:45 | |
*** mugsie has joined #openstack-dns | 16:50 | |
*** k4n0 has quit IRC | 16:59 | |
*** barra204_ has quit IRC | 16:59 | |
*** untriaged-bot has joined #openstack-dns | 17:02 | |
untriaged-bot | Untriaged bugs so far: | 17:02 |
untriaged-bot | https://bugs.launchpad.net/designate/+bug/1390357 | 17:02 |
uvirtbot | Launchpad bug 1390357 in designate "Designate sync creates duplicated records in powerdns backend" [Medium,New] | 17:02 |
untriaged-bot | https://bugs.launchpad.net/designate/+bug/1289444 | 17:02 |
uvirtbot | Launchpad bug 1289444 in designate "Designate with postgres backend is having issues" [Undecided,New] | 17:02 |
*** untriaged-bot has quit IRC | 17:02 | |
*** shakamunyi has quit IRC | 17:03 | |
*** rjrjr has joined #openstack-dns | 17:14 | |
*** rmoe has quit IRC | 17:24 | |
Kiall | rjrjr: heya | 17:29 |
Kiall | about? | 17:29 |
Kiall | Guessing you'll see this in a min or two ;) I updated your patchset on top of the pDNS related ones, and it all works locally, but still failing in the gate -_-.. | 17:32 |
Kiall | viod asked what I changed earlier.. so easier to Copy+Paste that: <Kiall> vinod: mostly a rebase on top of the Config+PowerDNS change, also updated it to match: https://github.com/openstack/designate/commit/22db5ad7395c061dbc55bb74775d39586e12eadf | 17:33 |
Kiall | <Kiall> oh - and "Removed transitional pool_manager_proxy backend" since it wasn't used anymore... | 17:33 |
*** rjrjr has quit IRC | 17:33 | |
*** betsy has quit IRC | 17:39 | |
*** rmoe has joined #openstack-dns | 17:41 | |
*** jordanP has quit IRC | 18:06 | |
*** rustlebee is now known as russellb | 18:09 | |
openstackgerrit | Kiall Mac Innes proposed openstack/designate: TEST: Some quick PDNS+gate debugging https://review.openstack.org/141865 | 18:09 |
openstackgerrit | Tim Simmons proposed openstack/designate: Agent - Add AXFR Capability https://review.openstack.org/140817 | 18:21 |
openstackgerrit | Tim Simmons proposed openstack/designate: Agent - Add basic NOTIFY support https://review.openstack.org/140816 | 18:21 |
openstackgerrit | Tim Simmons proposed openstack/designate: Agent - Add Backend Capabilities https://review.openstack.org/140819 | 18:21 |
openstackgerrit | Tim Simmons proposed openstack/designate: Agent - Add Support for Receiving Private CLASS/RRDATA Messages https://review.openstack.org/140818 | 18:21 |
timsim | Sorry ^ :P Just making sure they pass gate before some changes. | 18:21 |
*** pk__ has joined #openstack-dns | 18:38 | |
mugsie | timsim: not good enough ;) | 18:39 |
pk__ | Hi... Good Morning. | 18:40 |
pk__ | I am looking for search functionality... for ex, given a partial domain, get all the matching domains... | 18:41 |
pk__ | I looked https://designate.readthedocs.org/en/latest/rest.html I am not seeing | 18:41 |
pk__ | an api for that. | 18:42 |
Kiall | pk__: if your an admin, you can do a GET /v1/domains?all_tenants=1&name='*foo.com' etc | 18:42 |
pk__ | Do Designate has a search feature? | 18:42 |
*** vinod has joined #openstack-dns | 18:42 | |
Kiall | otherwise drop the all_tenants param and you'll get it for just your current tenant | 18:42 |
pk__ | ok cool. | 18:42 |
pk__ | Ok. Then only the doc is missing. | 18:43 |
pk__ | Thank Kiall. I will play around and come back. | 18:43 |
Kiall | Well - kinda.. we have it in the v2 API docs here: http://docs.openstack.org/developer/designate/rest/v2/collections.html#filtering | 18:43 |
Kiall | (It might not be complete though..) | 18:43 |
pk__ | ok... Thanks for the link. | 18:44 |
openstackgerrit | Kiall Mac Innes proposed openstack/designate: TEST: PDNS Gate Fix.. Force a NOTIFY upon zone create https://review.openstack.org/141876 | 18:47 |
ekarlso- | who's pk__ ? :D | 18:49 |
openstackgerrit | Kiall Mac Innes proposed openstack/designate: Pool Manager Integration with Central https://review.openstack.org/138406 | 18:49 |
Kiall | pk__: works with Ron in ebay from memory | 18:49 |
ekarlso- | oh ok ^ | 18:53 |
*** jmcbride has quit IRC | 18:56 | |
openstackgerrit | Graham Hayes proposed openstack/designate: Fixed the syntax of obj_reset_changes in SQLA https://review.openstack.org/141879 | 18:59 |
openstackgerrit | Graham Hayes proposed openstack/designate: Fixed the syntax of obj_reset_changes in SQLA https://review.openstack.org/141879 | 19:03 |
*** jmcbride has joined #openstack-dns | 19:04 | |
*** vinod has quit IRC | 19:12 | |
*** vinod has joined #openstack-dns | 19:12 | |
Kiall | Woo - Finally passing gate, needed to send a NOTIFY after creating the zone for PowerDNS to reliably AXFR the newly created... | 19:17 |
Kiall | newly created zone*] | 19:17 |
Kiall | rjrjr / vinod / betsy / mugsie : the entire chain of changes should be good now... With k1 release on Thursday I'd love to get them merged ASAP so any new changes don't break / delay us from being able to merge em | 19:19 |
vinod | on it reviewing them now | 19:19 |
Kiall | Cool, thanks :) | 19:20 |
*** paul_glass has joined #openstack-dns | 19:32 | |
*** betsy has joined #openstack-dns | 19:32 | |
*** pk__ has quit IRC | 19:41 | |
*** GonZo2K has joined #openstack-dns | 19:45 | |
*** betsy has quit IRC | 19:47 | |
openstackgerrit | Graham Hayes proposed openstack/designate: Added smart update of Records Objects during a v2 RRSet PUT operation https://review.openstack.org/141887 | 19:49 |
openstackgerrit | Graham Hayes proposed openstack/designate: Added smart update of Records Objects in v2 https://review.openstack.org/141887 | 19:51 |
openstackgerrit | Merged openstack/designate: Pools Config Changes https://review.openstack.org/141166 | 19:57 |
openstackgerrit | Graham Hayes proposed openstack/designate: Changed oslo.messaging's default log level https://review.openstack.org/141889 | 19:58 |
openstackgerrit | Tim Simmons proposed openstack/designate: Agent - Add Backend Capabilities https://review.openstack.org/140819 | 20:04 |
*** rjrjr has joined #openstack-dns | 20:10 | |
rjrjr | Kiall: i am here. | 20:10 |
Kiall | heya - see my messages earlier? | 20:10 |
*** jmcbride1 has joined #openstack-dns | 20:11 | |
Kiall | The gist of it was, I put your patchset ontop of the PDNS -> Pools patchsets, with some very minor changes, and it's all passing gate.. Vinod/Graham are working their way through the chain merging stuff as they go :) | 20:13 |
*** jmcbride has quit IRC | 20:14 | |
vinod | Kiall: Had a question on https://review.openstack.org/#/c/141819/. Left it as a comment. | 20:16 |
Kiall | vinod: replied :) | 20:17 |
vinod | Thanks Kiall. timsim too had a comment in there | 20:20 |
*** timfreund has quit IRC | 20:30 | |
*** gohko has quit IRC | 20:30 | |
*** timfreund has joined #openstack-dns | 20:31 | |
*** gohko has joined #openstack-dns | 20:31 | |
Kiall | Will sortout timsim's comment in another patchset since it's +A'd anyway, and the gate is taking AGES :'( | 20:32 |
timsim | Yeah, it looks like the next patch gets most of it, like vinod said. | 20:32 |
Kiall | I actually had made those changes earlier.. and somehow lost them.. heh | 20:32 |
vinod | While all of you are in the reviewing mode - could you also look at https://review.openstack.org/#/c/139748/ | 20:34 |
Kiall | vinod: some very quick comments inline, would need to pull it down and test before +2'ing etc | 20:39 |
rjrjr | okay, just caught up on the chat. do you need me to work on anything to get these changes in? also, glad to see everything is working together. 8^) | 20:40 |
*** amcrn has joined #openstack-dns | 20:41 | |
Kiall | rjrjr: nope, I have noticed one weirdness which I'm a little unsure about though, would would be a post-merge cleanup IMO.. | 20:41 |
Kiall | in the PM service, update_domain is fully async, we allow the update_status method to move the domain from pending -> complete... | 20:41 |
Kiall | But - The create_domain method is synchronous, updating the status to complete straight away.. | 20:42 |
mugsie | vinod: looking now, but will have to wait for functional testing (have 2 installs currently running ;) ) | 20:42 |
Kiall | That works nicely for BIND, where `rndc addzone` does a AXFR before returning, but other backends dont.. | 20:42 |
Kiall | I had to add an explicit NOTIFY trigger there to make the pDNS+Pools backend reliable, and I figure we should really make that aync too | 20:43 |
Kiall | BUT - I didn't want to change too much in your patchset, I only wanted to bare essentials to rebase + make it work with pDNS etc :@) | 20:43 |
rjrjr | okay. i can fix that. i'll open that bug when i open the others i owe the team. been off working on other in-house things this weekend but will be back on this this evening. | 20:44 |
Kiall | Yep, no rush.. I'd consider it a cleanup thing rather than a blocker etc etc | 20:44 |
rjrjr | cool. still, expect code to start flowing from me again by tomorrow. | 20:45 |
Kiall | Yep - Now that the bulk is in, we can start doing cleanup in smaller more easily reviewable patchsets :) | 20:46 |
rjrjr | or rather, expect some patches tomorrow for this and other things i promised to work on. 8^) | 20:46 |
vinod | mugsie: Regarding your comment in https://review.openstack.org/#/c/139748/3/designate/api/v2/controllers/root.py | 20:49 |
vinod | I personally think for things like ^ where people will be writing code against it, we can't change without an API increase. | 20:50 |
vinod | I don't fully understand what you mean. Could you elaborate? | 20:50 |
mugsie | vinod: for example with this change people could be writing Icinga / Nagios checks against it, so changing it out from under people would not be good | 20:55 |
mugsie | i personally think an API version is a contract, and once it is released, we can't break it | 20:55 |
vinod | even for extensions? | 20:56 |
mugsie | yup, extentions that we release | 20:56 |
mugsie | if it is under the "/v2" path, its part of the contract | 20:56 |
mugsie | users don't care if the functionality is part of an extention, they just care that the URL they used yesterday doesnt work now | 20:57 |
Kiall | mugsie suggested (across the desk) putting it under a "/admin" - where we would provide no guarantees cross-release... | 20:57 |
vinod | should it then be /admin/status or /admin/pools/status? | 20:58 |
mugsie | i think /admin/status/pools | 20:59 |
mugsie | as we could have /admin/status/api | 20:59 |
mugsie | or /admin/status/central | 20:59 |
mugsie | etc | 20:59 |
Kiall | I'd have gone with /admin/pools/status.. grouping pools stuff toether ;) | 20:59 |
mugsie | i would group status togehter, as you could get /admin/status and get an overview back | 20:59 |
mugsie | and then drill down if the central showed issues (for example) | 21:00 |
timsim | I like that ^ | 21:00 |
Kiall | Yea.. Maybe, either works! | 21:01 |
vinod | Okay I will make it /admin/status/pools. I will upload the next patchset once the current chain of pool manager changes goes through. | 21:01 |
vinod | Kiall/mugsie: still at office? | 21:02 |
*** vipul has quit IRC | 21:02 | |
*** ekarlso- has quit IRC | 21:02 | |
mugsie | vinod: yup | 21:02 |
*** betsy has joined #openstack-dns | 21:02 | |
Kiall | time to get outta here though ;) | 21:03 |
timsim | Does that mean the other extensions should go behind that /admin endpoint as well? | 21:04 |
vinod | one other thing before you head out - so would it be an extension at /admin/status/pools or a regular api at /admin/status/pools? | 21:04 |
Kiall | timsim: no, I guess /admin would be for admin-only stuff, while there can be extensions that end users can use | 21:06 |
Kiall | vinod: I think it's OK to leave it always-on if it's under a /admin | 21:07 |
Kiall | (and /admin has a enable_admin_api setting, like we have a enable_v{1,2}_api | 21:07 |
timsim | Fair enough. I guess I'm not sure if some of the stuff in extensions is usable (or should be) by everyone, but that's another conversation. | 21:08 |
*** vipul has joined #openstack-dns | 21:10 | |
*** amcrn has quit IRC | 21:13 | |
*** ekarlso- has joined #openstack-dns | 21:15 | |
*** jmcbride1 has quit IRC | 21:16 | |
*** betsy has quit IRC | 21:21 | |
openstackgerrit | Graham Hayes proposed openstack/designate: Added smart update of Records Objects in v2 https://review.openstack.org/141887 | 21:31 |
*** jmcbride has joined #openstack-dns | 21:36 | |
*** jmcbride has quit IRC | 21:37 | |
*** jmcbride has joined #openstack-dns | 21:37 | |
*** ryanpetrello has joined #openstack-dns | 21:48 | |
*** openstackgerrit has quit IRC | 21:50 | |
*** openstackgerrit has joined #openstack-dns | 21:50 | |
*** ChanServ sets mode: +v openstackgerrit | 21:50 | |
openstackgerrit | Merged openstack/designate: Convert PowerDNS to a Pools Backend https://review.openstack.org/141167 | 21:59 |
openstackgerrit | Merged openstack/designate: Changed oslo.messaging's default log level https://review.openstack.org/141889 | 21:59 |
*** ryanpetrello has quit IRC | 22:15 | |
*** ryanpetrello_ has joined #openstack-dns | 22:16 | |
*** ryanpetrello_ is now known as ryanpetrello | 22:16 | |
*** jmcbride has quit IRC | 22:16 | |
*** jmcbride has joined #openstack-dns | 22:32 | |
*** vinod has quit IRC | 22:41 | |
*** timsim has quit IRC | 22:56 | |
*** GonZo2K has quit IRC | 23:04 | |
*** GonZo2K has joined #openstack-dns | 23:05 | |
*** betsy has joined #openstack-dns | 23:12 | |
*** thetrav has joined #openstack-dns | 23:18 | |
*** paul_glass has quit IRC | 23:20 | |
*** jmcbride has quit IRC | 23:20 | |
openstackgerrit | Merged openstack/designate: Remove the non-pools bind9 backend https://review.openstack.org/141368 | 23:33 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!