Monday, 2014-12-15

*** ryanpetrello has joined #openstack-dns00:43
*** tylerdurden has quit IRC00:54
*** shakamunyi has joined #openstack-dns00:55
*** ryanpetrello has quit IRC00:59
*** ryanpetrello has joined #openstack-dns01:01
*** ryanpetrello has quit IRC01:12
*** boris-42 has quit IRC01:13
*** thetrav has joined #openstack-dns01:55
*** Stanley00 has joined #openstack-dns01:57
*** thetrav has quit IRC02:16
*** GonZo2K has quit IRC02:48
*** thetrav has joined #openstack-dns03:23
thetravso I'm looking to pull the keystone project/tenant name out out on the nova/neutron events and use it in the dns entry03:41
thetravanyone else done this?03:41
thetravwondering if I can make keystone api calls from the existing code base or if I need to pull in the keystone client03:41
*** harmw has quit IRC04:09
*** harmw has joined #openstack-dns04:09
*** boris-42 has joined #openstack-dns05:01
*** thetrav has quit IRC06:18
*** simonmcc has quit IRC06:26
*** simonmcc_ has joined #openstack-dns06:27
*** nihilifer has joined #openstack-dns06:44
*** yfujioka has quit IRC08:07
*** k4n0 has joined #openstack-dns08:25
openstackgerritEndre Karlson proposed openstack/designate: DynECT tests using requests-mock.  https://review.openstack.org/14166409:03
*** nihilifer has quit IRC09:24
*** nihilifer has joined #openstack-dns09:26
*** jordanP has joined #openstack-dns09:28
*** ttrumm has joined #openstack-dns09:29
openstackgerritEndre Karlson proposed openstack/designate: DynECT: Handle errors correctly when creating domain  https://review.openstack.org/14129109:54
openstackgerritEndre Karlson proposed openstack/designate: DynECT tests using requests-mock.  https://review.openstack.org/14166409:54
*** Stanley00 has quit IRC10:05
openstackgerritEndre Karlson proposed openstack/designate: DynECT tests using requests-mock  https://review.openstack.org/14166410:11
*** GonZo2K has joined #openstack-dns10:39
*** nihilifer has quit IRC10:46
*** nihilifer has joined #openstack-dns10:50
*** untriaged-bot has joined #openstack-dns11:02
untriaged-botUntriaged bugs so far:11:02
untriaged-bothttps://bugs.launchpad.net/designate/+bug/139035711:02
untriaged-bothttps://bugs.launchpad.net/designate/+bug/128944411:02
uvirtbotLaunchpad bug 1390357 in designate "Designate sync creates duplicated records in powerdns backend" [Medium,New]11:02
*** untriaged-bot has quit IRC11:02
uvirtbotLaunchpad bug 1289444 in designate "Designate with postgres backend is having issues" [Undecided,New]11:02
*** mikehn has joined #openstack-dns11:07
*** GonZo2K has quit IRC11:21
*** nihilifer has quit IRC12:39
openstackgerritEndre Karlson proposed openstack/designate: DynECT: Fix error handling when creating domain  https://review.openstack.org/14129112:44
openstackgerritMerged openstack/designate: Adds "domain-servers-list" to test_domains.py  https://review.openstack.org/14093512:46
openstackgerritMerged openstack/designate: Cleanup validation regex's  https://review.openstack.org/14105612:48
*** nihilifer has joined #openstack-dns12:56
betsykiall/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
Kialldooh  :) K - Let me look over the two in a few mins13:52
betsyI’ve still got another patchset to submit for the second one13:53
betsyIt’s almost ready but not quite13:54
mugsie_cool, will lookm now13:54
*** mugsie_ is now known as mugsie13:55
betsyI’m deleting most of the pool_attributes in the child patchset13:57
KiallMaybe 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-dns14:10
ekarlso-hhttps://review.openstack.org/#/c/141291/ anyone wanna signoff ?14:14
*** nihilifer has quit IRC14:25
openstackgerritEndre Karlson proposed openstack/designate: MDNS part of Secondary zones  https://review.openstack.org/13368314:25
openstackgerritEndre Karlson proposed openstack/designate: Support secondary zones  https://review.openstack.org/13368214:25
openstackgerritKiall Mac Innes proposed openstack/designate: Pools Config Changes  https://review.openstack.org/14116614:30
openstackgerritKiall Mac Innes proposed openstack/designate: Convert PowerDNS to a Pools Backend  https://review.openstack.org/14116714:30
openstackgerritKiall Mac Innes proposed openstack/designate: Remove the non-pools bind9 backend  https://review.openstack.org/14136814:30
*** betsy has quit IRC14:34
*** timsim has joined #openstack-dns14:53
*** ttrumm has quit IRC15:01
openstackgerritKiall Mac Innes proposed openstack/designate: Remove the agent service  https://review.openstack.org/14181915:12
*** vinod has joined #openstack-dns15:28
openstackgerritKiall Mac Innes proposed openstack/designate: Pool Manager Integration with Central  https://review.openstack.org/13840615:36
openstackgerritKiall Mac Innes proposed openstack/designate: Remove the agent service and old-style backend tests  https://review.openstack.org/14181915:36
*** nkinder_away has joined #openstack-dns15:36
vinodKiall: for Pool Manager Integration with Central  https://review.openstack.org/13840615:41
vinoddid you just do a rebase for patchset 6 or were there more changes?15:41
openstackgerritMerged openstack/designate: Add Werkzeug>=0.7 to requirements.txt  https://review.openstack.org/14039415:44
*** richm1 has quit IRC15:50
Kiallvinod: mostly a rebase on top of the Config+PowerDNS change, also updated it to match: https://github.com/openstack/designate/commit/22db5ad7395c061dbc55bb74775d39586e12eadf15:51
Kialloh  - and "Removed transitional pool_manager_proxy backend" since it wasn't used anymore...15:52
KiallLooks like they all are about to pass gate now, fingers crossed ;)15:52
*** betsy has joined #openstack-dns15:52
vinodAnd after that will it be ready to merge?15:53
Kiallah crap - forgot to update the pdns devstack to not use the proxy backend15:53
Kiallvinod: in theory, it'll add goof to merge now bar ^ .. re-reviews of the lot should probably be done anyway15:54
KiallYea, I have some DS changes still needed .. grr.15:55
*** nkinder_away is now known as nkinder15:55
*** paul_glass has joined #openstack-dns16:00
*** jmcbride has joined #openstack-dns16:03
*** jmcbride1 has joined #openstack-dns16:04
*** jmcbride has quit IRC16:04
*** jmcbride1 has quit IRC16:04
*** jmcbride1 has joined #openstack-dns16:05
Kiallvinod: 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
vinodwill look at them now16:08
openstackgerritKiall Mac Innes proposed openstack/designate: Remove the agent service and old-style backend tests  https://review.openstack.org/14181916:09
*** paul_glass has quit IRC16:13
*** richm1 has joined #openstack-dns16:14
openstackgerritKiall Mac Innes proposed openstack/designate: Pool Manager Integration with Central  https://review.openstack.org/13840616:17
KiallStill not sure why pDNS failed on the last patchset of ^ .. Building a clean devstack VM to try reproduce :'(16:17
Kiallhttps://review.openstack.org/#/c/141819/ should be good now too.. found+removed the last traces of the agent...16:18
KiallHumm.. Or not. Where did the DS part of that change go -_-16:21
*** jmcbride1 has quit IRC16:26
*** jmcbride has joined #openstack-dns16:31
*** tylerdurden has joined #openstack-dns16:33
*** tylerdurden has quit IRC16:33
*** tylerdurden has joined #openstack-dns16:34
*** tylerdurden has quit IRC16:34
*** mugsie has quit IRC16:43
*** vinod has quit IRC16:45
*** mugsie has joined #openstack-dns16:50
*** k4n0 has quit IRC16:59
*** barra204_ has quit IRC16:59
*** untriaged-bot has joined #openstack-dns17:02
untriaged-botUntriaged bugs so far:17:02
untriaged-bothttps://bugs.launchpad.net/designate/+bug/139035717:02
uvirtbotLaunchpad bug 1390357 in designate "Designate sync creates duplicated records in powerdns backend" [Medium,New]17:02
untriaged-bothttps://bugs.launchpad.net/designate/+bug/128944417:02
uvirtbotLaunchpad bug 1289444 in designate "Designate with postgres backend is having issues" [Undecided,New]17:02
*** untriaged-bot has quit IRC17:02
*** shakamunyi has quit IRC17:03
*** rjrjr has joined #openstack-dns17:14
*** rmoe has quit IRC17:24
Kiallrjrjr: heya17:29
Kiallabout?17:29
KiallGuessing 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
Kiallviod 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/22db5ad7395c061dbc55bb74775d39586e12eadf17:33
Kiall<Kiall> oh  - and "Removed transitional pool_manager_proxy backend" since it wasn't used anymore...17:33
*** rjrjr has quit IRC17:33
*** betsy has quit IRC17:39
*** rmoe has joined #openstack-dns17:41
*** jordanP has quit IRC18:06
*** rustlebee is now known as russellb18:09
openstackgerritKiall Mac Innes proposed openstack/designate: TEST: Some quick PDNS+gate debugging  https://review.openstack.org/14186518:09
openstackgerritTim Simmons proposed openstack/designate: Agent - Add AXFR Capability  https://review.openstack.org/14081718:21
openstackgerritTim Simmons proposed openstack/designate: Agent - Add basic NOTIFY support  https://review.openstack.org/14081618:21
openstackgerritTim Simmons proposed openstack/designate: Agent - Add Backend Capabilities  https://review.openstack.org/14081918:21
openstackgerritTim Simmons proposed openstack/designate: Agent - Add Support for Receiving Private CLASS/RRDATA Messages  https://review.openstack.org/14081818:21
timsimSorry ^ :P Just making sure they pass gate before some changes.18:21
*** pk__ has joined #openstack-dns18:38
mugsietimsim: 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 seeing18:41
pk__an api for that.18:42
Kiallpk__: if your an admin, you can do a GET /v1/domains?all_tenants=1&name='*foo.com' etc18:42
pk__Do Designate has a search feature?18:42
*** vinod has joined #openstack-dns18:42
Kiallotherwise drop the all_tenants param and you'll get it for just your current tenant18: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
KiallWell - kinda.. we have it in the v2 API docs here: http://docs.openstack.org/developer/designate/rest/v2/collections.html#filtering18:43
Kiall(It might not be complete though..)18:43
pk__ok... Thanks for the link.18:44
openstackgerritKiall Mac Innes proposed openstack/designate: TEST: PDNS Gate Fix.. Force a NOTIFY upon zone create  https://review.openstack.org/14187618:47
ekarlso-who's pk__ ? :D18:49
openstackgerritKiall Mac Innes proposed openstack/designate: Pool Manager Integration with Central  https://review.openstack.org/13840618:49
Kiallpk__: works with Ron in ebay from memory18:49
ekarlso-oh ok ^18:53
*** jmcbride has quit IRC18:56
openstackgerritGraham Hayes proposed openstack/designate: Fixed the syntax of obj_reset_changes in SQLA  https://review.openstack.org/14187918:59
openstackgerritGraham Hayes proposed openstack/designate: Fixed the syntax of obj_reset_changes in SQLA  https://review.openstack.org/14187919:03
*** jmcbride has joined #openstack-dns19:04
*** vinod has quit IRC19:12
*** vinod has joined #openstack-dns19:12
KiallWoo - Finally passing gate, needed to send a NOTIFY after creating the zone for PowerDNS to reliably AXFR the newly created...19:17
Kiallnewly created zone*]19:17
Kiallrjrjr / 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 em19:19
vinodon it reviewing them now19:19
KiallCool, thanks :)19:20
*** paul_glass has joined #openstack-dns19:32
*** betsy has joined #openstack-dns19:32
*** pk__ has quit IRC19:41
*** GonZo2K has joined #openstack-dns19:45
*** betsy has quit IRC19:47
openstackgerritGraham Hayes proposed openstack/designate: Added smart update of Records Objects during a v2 RRSet PUT operation  https://review.openstack.org/14188719:49
openstackgerritGraham Hayes proposed openstack/designate: Added smart update of Records Objects in v2  https://review.openstack.org/14188719:51
openstackgerritMerged openstack/designate: Pools Config Changes  https://review.openstack.org/14116619:57
openstackgerritGraham Hayes proposed openstack/designate: Changed oslo.messaging's default log level  https://review.openstack.org/14188919:58
openstackgerritTim Simmons proposed openstack/designate: Agent - Add Backend Capabilities  https://review.openstack.org/14081920:04
*** rjrjr has joined #openstack-dns20:10
rjrjrKiall: i am here.20:10
Kiallheya - see my messages earlier?20:10
*** jmcbride1 has joined #openstack-dns20:11
KiallThe 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 IRC20:14
vinodKiall: Had a question on https://review.openstack.org/#/c/141819/. Left it as a comment.20:16
Kiallvinod:  replied :)20:17
vinodThanks Kiall. timsim too had a comment in there20:20
*** timfreund has quit IRC20:30
*** gohko has quit IRC20:30
*** timfreund has joined #openstack-dns20:31
*** gohko has joined #openstack-dns20:31
KiallWill sortout timsim's comment in another patchset since it's +A'd anyway, and the gate is taking AGES :'(20:32
timsimYeah, it looks like the next patch gets most of it, like vinod said.20:32
KiallI actually had made those changes earlier.. and somehow lost them.. heh20:32
vinodWhile all of you are in the reviewing mode - could you also look at https://review.openstack.org/#/c/139748/20:34
Kiallvinod: some very quick comments inline, would need to pull it down and test before +2'ing etc20:39
rjrjrokay, 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-dns20:41
Kiallrjrjr: nope, I have noticed one weirdness which I'm a little unsure about though, would would be a post-merge cleanup IMO..20:41
Kiallin the PM service, update_domain is fully async, we allow the update_status method to move the domain from pending -> complete...20:41
KiallBut - The create_domain method is synchronous, updating the status to complete straight away..20:42
mugsievinod: looking now, but will have to wait for functional testing (have 2 installs currently running ;) )20:42
KiallThat works nicely for BIND, where `rndc addzone` does a AXFR before returning, but other backends dont..20:42
KiallI had to add an explicit NOTIFY trigger there to make the pDNS+Pools backend reliable, and I figure we should really make that aync too20:43
KiallBUT - 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
rjrjrokay.  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
KiallYep, no rush.. I'd consider it a cleanup thing rather than a blocker etc etc20:44
rjrjrcool.  still, expect code to start flowing from me again by tomorrow.20:45
KiallYep - Now that the bulk is in, we can start doing cleanup in smaller more easily reviewable patchsets :)20:46
rjrjror rather, expect some patches tomorrow for this and other things i promised to work on. 8^)20:46
vinodmugsie: Regarding your comment in https://review.openstack.org/#/c/139748/3/designate/api/v2/controllers/root.py20:49
vinodI personally think for things like ^ where people will be writing code against it, we can't change without an API increase.20:50
vinodI don't fully understand what you mean. Could you elaborate?20:50
mugsievinod: for example with this change people could be writing Icinga / Nagios checks against it, so changing it out from under people would not be good20:55
mugsiei personally think an API version is a contract, and once it is released, we can't break it20:55
vinodeven for extensions?20:56
mugsieyup, extentions that we release20:56
mugsieif it is under the "/v2" path, its part of the contract20:56
mugsieusers don't care if the functionality is part of an extention, they just care that the URL they used yesterday doesnt work now20:57
Kiallmugsie suggested (across the desk) putting it under a "/admin" - where we would provide no guarantees cross-release...20:57
vinodshould it then be /admin/status or /admin/pools/status?20:58
mugsiei think /admin/status/pools20:59
mugsieas we could have /admin/status/api20:59
mugsieor /admin/status/central20:59
mugsieetc20:59
KiallI'd have gone with /admin/pools/status.. grouping pools stuff toether ;)20:59
mugsiei would group status togehter, as you could get /admin/status and get an overview back20:59
mugsieand then drill down if the central showed issues (for example)21:00
timsimI like that ^21:00
KiallYea.. Maybe, either works!21:01
vinodOkay 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
vinodKiall/mugsie: still at office?21:02
*** vipul has quit IRC21:02
*** ekarlso- has quit IRC21:02
mugsievinod: yup21:02
*** betsy has joined #openstack-dns21:02
Kialltime to get outta here though ;)21:03
timsimDoes that mean the other extensions should go behind that /admin endpoint as well?21:04
vinodone 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
Kialltimsim: no, I guess /admin would be for admin-only stuff, while there can be extensions that end users can use21:06
Kiallvinod: I think it's OK to leave it always-on if it's under a /admin21:07
Kiall(and /admin has a enable_admin_api setting, like we have a enable_v{1,2}_api21:07
timsimFair 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-dns21:10
*** amcrn has quit IRC21:13
*** ekarlso- has joined #openstack-dns21:15
*** jmcbride1 has quit IRC21:16
*** betsy has quit IRC21:21
openstackgerritGraham Hayes proposed openstack/designate: Added smart update of Records Objects in v2  https://review.openstack.org/14188721:31
*** jmcbride has joined #openstack-dns21:36
*** jmcbride has quit IRC21:37
*** jmcbride has joined #openstack-dns21:37
*** ryanpetrello has joined #openstack-dns21:48
*** openstackgerrit has quit IRC21:50
*** openstackgerrit has joined #openstack-dns21:50
*** ChanServ sets mode: +v openstackgerrit21:50
openstackgerritMerged openstack/designate: Convert PowerDNS to a Pools Backend  https://review.openstack.org/14116721:59
openstackgerritMerged openstack/designate: Changed oslo.messaging's default log level  https://review.openstack.org/14188921:59
*** ryanpetrello has quit IRC22:15
*** ryanpetrello_ has joined #openstack-dns22:16
*** ryanpetrello_ is now known as ryanpetrello22:16
*** jmcbride has quit IRC22:16
*** jmcbride has joined #openstack-dns22:32
*** vinod has quit IRC22:41
*** timsim has quit IRC22:56
*** GonZo2K has quit IRC23:04
*** GonZo2K has joined #openstack-dns23:05
*** betsy has joined #openstack-dns23:12
*** thetrav has joined #openstack-dns23:18
*** paul_glass has quit IRC23:20
*** jmcbride has quit IRC23:20
openstackgerritMerged openstack/designate: Remove the non-pools bind9 backend  https://review.openstack.org/14136823:33

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!