Monday, 2014-12-01

« Sunday, 2014-11-30 Index Tuesday, 2014-12-02 »
*** ryanpetrello has quit IRC00:13
*** GonZo2K has quit IRC00:15
*** timsim has quit IRC00:32
*** jmcbride has joined #openstack-dns00:35
*** jmcbride has quit IRC01:21
*** nosnos has joined #openstack-dns01:44
*** jmcbride has joined #openstack-dns01:49
*** ryanpetrello has joined #openstack-dns02:17
*** jmcbride has quit IRC02:36
*** GonZo2K has joined #openstack-dns03:05
*** rjrjr has quit IRC03:08
*** nosnos has quit IRC03:52
*** GonZo2K has quit IRC04:00
*** ryanpetrello has quit IRC04:28
*** nosnos has joined #openstack-dns04:44
*** mwagner_lap has quit IRC05:41
*** alokj has joined #openstack-dns05:41
*** mwagner_lap has joined #openstack-dns05:54
*** nosnos has quit IRC05:58
*** nosnos has joined #openstack-dns05:58
*** nosnos_ has joined #openstack-dns06:29
*** nosnos has quit IRC06:29
*** k4n0 has joined #openstack-dns07:24
*** nihilifer has joined #openstack-dns07:34
*** yfujioka has joined #openstack-dns08:00
*** alokj has quit IRC08:31
*** alokj has joined #openstack-dns08:43
*** alokj is now known as Guest5594308:44
*** ekarlso- has quit IRC08:52
*** jordanP has joined #openstack-dns08:59
*** Guest55943 has quit IRC09:05
*** ekarlso- has joined #openstack-dns09:08
*** Guest55943 has joined #openstack-dns09:19
*** ToMiles has quit IRC09:27
*** ToMiles has joined #openstack-dns09:29
*** nosnos_ has quit IRC09:31
*** nosnos has joined #openstack-dns09:31
*** Guest55943 is now known as alokj_09:32
*** openstackgerrit has quit IRC09:50
*** openstackgerrit has joined #openstack-dns09:50
*** ChanServ sets mode: +v openstackgerrit09:50
*** nosnos has quit IRC09:52
*** nosnos has joined #openstack-dns09:58
*** nkinder has quit IRC10:21
*** ToMiles has quit IRC10:36
*** yfujioka has quit IRC10:51
*** alokj_ has quit IRC11:35
*** rediskin has joined #openstack-dns11:40
*** alokj_ has joined #openstack-dns11:49
openstackgerritRon Rickard proposed openstack/designate: Pool Manager  https://review.openstack.org/13165612:01
*** rjrjr has joined #openstack-dns12:02
*** nosnos has quit IRC12:09
*** nosnos has joined #openstack-dns12:11
*** alokj_ has quit IRC12:13
*** kiall_ has joined #openstack-dns12:13
*** alokj_ has joined #openstack-dns12:24
*** alokj_ has quit IRC12:28
*** nosnos has quit IRC12:29
kiall_rjrjr: nice, passes gate :)12:30
rjrjryeah.  sudoers issue.12:31
rjrjri'm looking at the rally issue right now.12:31
kiall_I saw, rally issue isn't you (unless you change has made it worse, haven't checked those logs)12:32
rjrjri'm thinking rally is running without pool manager (no logs) and the issue might be related to the synchronization code that was introduced.12:32
kiall_Yea, rally is different all right.. I'd be 99% certain it won't be picking up the pool manager change.. AND, its failing anyway.12:33
rjrjrokay, if you want to have someone else look at the rally issue, i want to look at some other issues i'm seeing with pool manager.12:34
rjrjri would love for the current code to get committed if nobody sees anything wrong with it.12:34
kiall_Yep, I'll check out rally.. Also we'll want to undo your gate_hook s12:34
kiall_Changes once its ready to merge. I have another change to gate hook that will left both powerdns and bind run12:35
rjrjrone issue i'm seeing in pool manager, we add the domain to storage and the backend, then we add the NS and SOA records.12:35
kiall_But need the infra folks to approve the second job :(12:35
kiall_Okay, so bind is failing to axfr due to the missing ns/soa?12:35
rjrjrcorrect.  there is some work to do in central still.12:36
rjrjri did some of the needed work, but didn't want to start tearing it apart before the pool manager code was committed.12:36
rjrjrwe'll need to test with and without pool manager once we start this next step.12:37
*** nosnos has joined #openstack-dns12:37
kiall_Yea, so, I'm wanting to hit merge on this soon noe that it works in gate etc, then start ripping out to old stuff so we can have an easier time fixing stuff up12:38
rjrjrthe central code can be streamlined tremendously.12:38
rjrjri think we are in agreement.12:38
rjrjri'll hold off committing any changes, but i'm starting that next step now.  i'll start looking at fixing central so it will fit better with pool manager.12:39
kiall_Okay, can you remove the gate hook change from the latest patchset? I'll give a final review today and will ask mugsie to do the same.. Unless we find snny blockers I'd like to see it merged :)12:40
*** ryanpetrello has joined #openstack-dns12:40
rjrjrcool!12:40
rjrjrremoving the gated change now.12:40
kiall_I'll also ping the infra folks to try and get the second job merged, that way powerdns and bind will be gated :)12:41
kiall_(I'll submit a patch to move pdns to a pool backend)12:41
rjrjrkiall, i'm sure you noticed i took some of your changes (besides the gated change) to my change.12:41
rjrjri didn't take the change you made to the zone view however.  it broke the unit testing.12:42
rjrjr(adding 'action' to the view.)12:42
kiall_Yea, that was just me debugging ;)12:42
rjrjri didn't want to get sidetracked from the goal of getting gated working. 8^)12:42
openstackgerritRon Rickard proposed openstack/designate: Pool Manager  https://review.openstack.org/13165612:44
rjrjrokay, i reverted gated.12:45
*** nosnos has quit IRC12:45
*** mwagner_lap has quit IRC14:00
*** nihilifer has quit IRC14:15
*** nihilifer has joined #openstack-dns14:15
openstackgerritMerged openstack/designate: Update DevStack PORT vars for naming consistency  https://review.openstack.org/13784314:19
*** betsy has quit IRC14:29
*** rickerc has joined #openstack-dns14:35
*** richm has joined #openstack-dns14:39
*** ryanpetrello has quit IRC15:02
*** jmcbride has joined #openstack-dns15:07
*** timsim has joined #openstack-dns15:14
*** ryanpetrello has joined #openstack-dns15:15
*** k4n0 has quit IRC15:16
*** EricGonczer_ has joined #openstack-dns15:18
*** LinStatSDR has quit IRC15:22
*** EricGonczer_ has quit IRC15:25
*** jmcbride has quit IRC15:33
*** Kiall has quit IRC15:37
*** kiall_ is now known as Kiall15:37
*** Kiall has quit IRC15:37
*** Kiall has joined #openstack-dns15:37
*** nihilifer has quit IRC15:39
timsimKiall: You didn't happen to find the mockup for attaching counts to List objects in storage, did you? I was about to take a stab at it.15:43
*** EricGonczer_ has joined #openstack-dns15:44
KiallI did.. let me dig it out again15:45
Kialltimsim: http://paste.openstack.org/show/85814/  and logs of conversation are http://eavesdrop.openstack.org/irclogs/%23openstack-dns/%23openstack-dns.2014-07-09.log15:47
KiallTiny example, but shows how the extra info might be attached to the Object and passed back out as part of 1 call15:47
timsimThat makes sense, cool. Thanks!15:48
*** betsy has joined #openstack-dns15:50
*** paul_glass has joined #openstack-dns15:51
*** vinod1 has joined #openstack-dns15:52
*** jmcbride has joined #openstack-dns16:01
*** jmcbride has quit IRC16:01
*** jmcbride has joined #openstack-dns16:02
*** rmoe has quit IRC16:05
*** rmoe has joined #openstack-dns16:23
Kiallvinod1 / betsy - if either of you are around, It would be great to merge this before the matching -infra change to add the second job lands :) https://review.openstack.org/#/c/137505/16:26
betsyI’m here. I’ll take a look16:27
*** jmcbride has quit IRC16:27
KiallThanks :)16:27
betsykiall: question: It looks like you’re enabling designate-pool-manager only if it’s bind9, and not if it’s powerdns16:30
vinod1With the pool manager shouldn't the backend driver be pool_manager_proxy?16:30
KiallCorrect, when powerdns i pool based we'll make pool-manager always be enabled16:30
Kiallvinod1: that's handled inside devstack itself16:31
betsyokay16:31
betsylooks good to me16:31
betsyvinod: any other questions?16:31
KiallPS18 of rjrjr's change shows the options that were needed to make it pass gate with the default changed to bind9 - https://review.openstack.org/#/c/131656/18/contrib/devstack/gate_hook.sh16:32
Kiall(The RNDC port option turned out to be unnecessary though)16:32
vinod1So DESIGNATE_BACKEND_DRIVER is actually ignored when pool manager is enabled?16:32
*** jmcbride has joined #openstack-dns16:33
KiallKinda, it's used in some places but not others.. Once pools is "the only way" it'll be used in all the places again16:34
Kiall(It's a bit backwards right now due to supporting pool  vs not-pool based backends - But the "API" is correct, so things will fall back into place as we remove the non-pool route)16:35
*** EricGonczer_ has quit IRC16:35
vinod1Thanks Kiall. Betsy - no more questions from me16:35
Kiall:)16:35
betsyapproved16:35
KiallI was talking with Ron earlier, his latest pools patch is working nicely and passing the basic exercises checks etc.. I'd like to merge it, migrate PowerDNS to pools based, and start removing support for non-pool backends16:36
betsykiall: sounds good16:37
*** ryanpetrello_ has joined #openstack-dns16:41
*** ryanpetrello has quit IRC16:43
*** ryanpetrello_ is now known as ryanpetrello16:43
openstackgerritMerged openstack/designate: Update gate_hook to support multiple drivers  https://review.openstack.org/13750516:47
*** nihilifer has joined #openstack-dns16:55
*** nihilifer has quit IRC16:55
*** nihilifer has joined #openstack-dns16:55
*** EricGonczer_ has joined #openstack-dns16:56
rjrjrmy manager is wondering if we can change the mid-cycle summit location from PHX to SJC.  since i'm the only one in PHX, he is thinking it would be easier to hold it in SJC.17:04
rjrjrthoughts?17:04
mugsierjrjr: for us, its a transatlantic flight17:05
mugsiebut, it might suit kiall better17:05
mugsie(his meeting after is in CA anyway)17:05
rjrjrvinod1: how about rackspace?  will SJC (San Jose) work?17:09
vinod1I'll check with the others and let you know17:09
rjrjrvinod1: thanks.  sorry for the change.  darshan and pk want to attend and it makes more sense for me to travel to them instead of vice versa.17:10
*** EricGonczer_ has quit IRC17:11
rjrjrkiall: will San Jose work for you for the mid-cycle meetup?17:11
mugsierjrjr: he is on a call right now, but I will get him to check in a few mins17:12
ekarlso-is there any api to control pools atm ?17:19
mugsieekarlso-: nope, not yet17:22
*** rjrjr_ has joined #openstack-dns17:22
ekarlso-mugsie: I thought there was a api for it in the v2 folder ? pools.py ? :p17:22
mugsiethere is no real "pools" api anyway - most is done via the config files17:22
rjrjr_mugsie: was the Horizon GUI component added to Designate?17:28
mugsieyea, there is a review up wiht the code17:28
rjrjr_okay.17:28
mugsiehttps://review.openstack.org/#/c/137480/17:28
rjrjr_thanks for doing that so quickly.17:28
mugsienp17:29
*** jmcbride has quit IRC17:30
vinod1ekarlso: By pools api if you mean create/update/delete pools - yes there is an API now17:31
vinod1The pool manager and central however do not make use of any of the created pools yet17:31
vinod1The pool manager just uses the default pool now17:32
rjrjr_yes and no.17:32
rjrjr_i haven't tested it, but the pool manager can use another pool besides the default pool.17:33
rjrjr_at least, it is coded to do so.17:33
rjrjr_you still need to specify the pool_id in the designate.conf file though.17:33
*** jmcbride has joined #openstack-dns17:34
rjrjr_did want to ask how we ensure a pool manager is not taking requests off the queue that are not managed by that particular instance?17:35
*** EricGonczer_ has joined #openstack-dns17:35
rjrjr_i know we do this in sink.17:35
vinod1kiall might be able to answer that17:36
rjrjr_i looked at sink and it wasn't obvious to me how it does that though.17:36
*** jmcbride has quit IRC17:37
*** jmcbride has joined #openstack-dns17:38
mugsievinod1: afaik its the topic of the rmq message17:40
*** jmcbride has quit IRC17:44
*** jmcbride has joined #openstack-dns17:44
vinod1rjrjr_: With your new patchset, when I try to create a new domain, I get the error - rndc: 'addzone' failed: already exists. Is this a known issue?17:57
timsimThat means that bind9 already has a zone with that name.17:57
vinod1The domain does not exist previously17:58
*** jordanP has quit IRC17:58
vinod1In fact when I try to delete the domain, I get the error message rndc: 'delzone' failed: not found17:58
timsimlol17:58
openstackgerritTim Simmons proposed openstack/designate: Add metadata, totalcount fields to zones/recordsets collections  https://review.openstack.org/10885917:59
rjrjr_vinod: try starting with a fresh database.18:00
*** nihilifer has quit IRC18:00
vinod1will do18:01
rjrjr_also, make sure the *.nzf file and any slave files are cleaned up.18:01
rjrjr_it sounds to me like some things exist in BIND9 and not the database.  (so cleaning up the database might not be necessary.)18:02
rjrjr_one reason you might have seen that problem is with a slight change made to the bind9 pool driver this weekend.18:12
rjrjr_before, the zone stanza read 'zone example.com. {...' (note the trailing dot) - now the zone stanza reads 'zone example.com {...' (no trailing dot)18:13
rjrjr_rich and i diagnosed this problem about 6 months ago and kiall added the fix this weekend.  with BIND9 9.9.5 (and maybe slightly older), that trailing dot causes problems.  with BIND9 9.8, it doesn't seem to matter.18:14
rjrjr_so, i'm running on ubuntu 12.X (BIND9 9.8.X) and the Jenkins servers are running ubuntu 14.X (BIND9 9.9.X).  i don't have a problem with or without the trailing dot, but the Jenkins servers were.18:17
rjrjr_end result, the addzone and delzone are not using a zone with a trailing dot, so if you had an existing zone with the trailing dot, i can see this being a problem.18:19
*** jmcbride has quit IRC18:19
*** mwagner_lap has joined #openstack-dns18:19
*** EricGonczer_ has quit IRC18:23
*** betsy has quit IRC18:34
*** paul_glass has quit IRC18:42
*** betsy has joined #openstack-dns18:42
*** jmcbride has joined #openstack-dns18:48
*** EricGonczer_ has joined #openstack-dns18:48
*** EricGonczer_ has quit IRC18:55
*** jmcbride has quit IRC19:05
*** jmcbride has joined #openstack-dns19:05
vinod1rjrjr_: Regarding the midsummit, San Jose should not be a problem19:08
vinod1That said, however we have not yet got the travel approved19:09
vinod1Regarding the issue of rndc failing - the problem is that the rndc call was executed twice19:09
vinod1Looking closer I find that there are 2 bind_pool backends loaded. Trying to figure out how that was possible19:10
rjrjr_do you have multiple designate.conf files in your environment?19:11
*** paul_glass has joined #openstack-dns19:11
vinod1not that i know if19:11
rjrjr_kiall had this problem on Friday.19:11
rjrjr_or Wednesday. 8^)19:11
Kiall:)19:11
vinod1The pool manager log reports only one Found configuration at path result19:14
*** GonZo2K has joined #openstack-dns19:17
*** EricGonczer_ has joined #openstack-dns19:21
vinod1Got it - [backend:bind9_pool:*] was specified twice in the config file19:22
vinod1I am now able to see the status go from PENDING to ACTIVE for a created_domain with some changes19:25
vinod1Here are the changes that I had to do - http://paste.openstack.org/show/rMSR9ocGQVpaaVsKPXM3/19:28
rjrjr_you and i are working on the same code.  i have that fix already.19:29
rjrjr_here are the bugs I am fixing:19:29
rjrjr_1) creating domain does not have NS/SOA records ready (you see this one)19:31
rjrjr_2) updating recordset does not change status of records19:31
rjrjr_but, i'm hoping the code i have for review can be +2'd.  i didn't want to keep adding new code to what is already there.19:32
vinod1i am in the process of reviewing your code - will have it done before eod today19:32
rjrjr_cool.19:33
timsim(-1, 27 comments) incoming for you rjrjr_ ;)19:33
timsimfrom vinod1 ^19:33
vinod1One other thing - unrelated to your current change19:34
vinod1https://github.com/openstack/designate/blob/master/designate/storage/impl_sqlalchemy/tables.py#L105 specifies the default action as CREATE19:34
vinod1But actions does not have CREATE - https://github.com/openstack/designate/blob/master/designate/storage/impl_sqlalchemy/tables.py#L3619:35
vinod1instead it specifies ADD19:35
rjrjr_that would be a bug.  it should be CREATE.19:36
rjrjrtimsim: cannot wait...19:39
*** EricGonczer_ has quit IRC19:50
*** nihilifer has joined #openstack-dns19:50
*** EricGonczer_ has joined #openstack-dns19:56
rjrjrvinod1: i have the CREATE fixed too.  do you want me to push the fix for the 2 bugs I know about when i fix up the 27 comments you are sending over? 8^)19:57
rjrjri was trying to avoid having my fixes "grow" in such a way that this never gets through the review process. 8^(19:58
vinod1If you can do that in a different changeset, then that would be helpful - if not can you can wait till the review for this changeset is done.20:00
rjrjri'll wait.  still not entirely sure how kiall added a patch that depended on my patch.20:00
Kiallrjrjr: lol .. Just add a second commit on top of your existing one, and gerrit will see the dependency20:01
vinod1I notice that when I add a recordset, it creates a record with serial 1 - is this a known issue?20:01
vinod1soa and ns records though have the correct serial number20:01
rjrjryes.  recordsets (create, delete, update) haven't been integrated with pool manager.20:01
rjrjri have a fix for this though.  (see 2 above.)20:02
rjrjr2 should read "updating recordset does not change status, action, or serial of records"20:03
rjrjrkiall: okay.  i know just enough git to be useless. 8^)20:04
openstackgerritTim Simmons proposed openstack/designate: Add metadata, totalcount fields to zones/recordsets collections  https://review.openstack.org/10885920:04
timsimThanks for the review on that Kiall ^20:05
*** amcrn has joined #openstack-dns20:25
*** betsy has quit IRC20:37
*** betsy has joined #openstack-dns20:46
*** betsy has quit IRC20:55
*** alexiz has joined #openstack-dns21:21
*** kiall_ has joined #openstack-dns21:23
*** harmw_ has joined #openstack-dns21:24
*** amcrn_ has joined #openstack-dns21:26
*** nihilifer_ has joined #openstack-dns21:27
*** amcrn has quit IRC21:35
*** nihilifer has quit IRC21:35
*** Kiall has quit IRC21:35
*** shakayumi has quit IRC21:35
*** russellb has quit IRC21:35
*** harmw has quit IRC21:35
openstackgerritTim Simmons proposed openstack/designate: Add metadata, totalcount fields to zones/recordsets collections  https://review.openstack.org/10885921:37
*** ToMiles has joined #openstack-dns21:43
rjrjr_vinod1: thanks for the comments.  I responded.21:45
rjrjr_good questions by the way.21:48
timsimrally passed on my latest patchset: https://review.openstack.org/#/c/108859/ wat21:51
openstackgerritJoe McBride proposed openstack/designate: Add docs for MX, SSHFP, SPF and SRV record set types  https://review.openstack.org/13371221:53
*** EricGonczer_ has quit IRC22:02
*** vinod1 has quit IRC22:06
*** ryanpetrello has quit IRC22:36
*** ryanpetrello has joined #openstack-dns22:36
*** ryanpetrello has quit IRC22:42
timsimekarlso-: about?22:43
*** paul_glass has quit IRC22:48
timsimekarlso-: left a comment on your review, be back online tomorrow ;)22:52
*** timsim has quit IRC22:56
*** ryanpetrello has joined #openstack-dns22:57
*** ryanpetrello_ has joined #openstack-dns23:02
*** ryanpetrello has quit IRC23:04
*** ryanpetrello_ is now known as ryanpetrello23:04
openstackgerritJoe McBride proposed openstack/designate: Add docs for MX, SSHFP, SPF and SRV record set types  https://review.openstack.org/13371223:13
*** jmcbride has quit IRC23:23
*** betsy has joined #openstack-dns23:36
*** EricGonczer_ has joined #openstack-dns23:50
*** rediskin has left #openstack-dns23:54
« Sunday, 2014-11-30 Index Tuesday, 2014-12-02 »

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