Wednesday, 2014-11-19

*** yfujioka has joined #openstack-dns00:07
*** EricGonczer_ has joined #openstack-dns00:10
*** tryggvil has joined #openstack-dns00:11
*** vinod has joined #openstack-dns00:12
*** vinod has quit IRC00:12
*** EricGonczer_ has quit IRC00:26
*** betsy has quit IRC00:27
*** vinod has joined #openstack-dns00:32
*** nkinder has joined #openstack-dns01:09
*** rmoe has quit IRC01:31
*** mwagner_lap has joined #openstack-dns01:36
*** rmoe has joined #openstack-dns01:48
*** vinod has quit IRC01:49
*** nosnos has joined #openstack-dns02:02
*** vinod has joined #openstack-dns02:02
*** amcrn has quit IRC02:08
*** EricGonczer_ has joined #openstack-dns02:11
*** vinod has quit IRC02:23
*** vinod has joined #openstack-dns02:30
*** ryanpetrello has quit IRC02:32
*** ryanpetrello has joined #openstack-dns02:34
*** EricGonczer_ has quit IRC02:52
*** EricGonczer_ has joined #openstack-dns03:11
*** EricGonczer_ has quit IRC03:12
*** jmcbride has joined #openstack-dns03:12
*** GonZo2K has quit IRC03:14
*** ryanpetrello has quit IRC03:20
*** nosnos has quit IRC03:20
*** ryanpetrello has joined #openstack-dns03:22
*** jmcbride has quit IRC03:43
*** vinod has quit IRC03:44
*** rektide has quit IRC04:02
*** rektide has joined #openstack-dns04:10
*** nosnos has joined #openstack-dns04:12
*** ryanpetrello has quit IRC04:54
*** tryggvil has quit IRC05:28
*** jmcbride has joined #openstack-dns05:29
*** jmcbride has quit IRC05:29
*** boris-42 has joined #openstack-dns05:39
*** dtalton has joined #openstack-dns05:50
*** k4n0 has joined #openstack-dns05:52
*** k4n0 has quit IRC06:39
*** dtalton has quit IRC06:42
*** MasterPiece has quit IRC06:53
*** k4n0 has joined #openstack-dns06:54
*** nihilifer has joined #openstack-dns07:16
*** k4n0 has quit IRC07:31
*** k4n0 has joined #openstack-dns07:45
*** boris-42 has quit IRC07:47
*** GonZo2K has joined #openstack-dns07:51
*** yfujioka has quit IRC08:12
*** jordanP has joined #openstack-dns08:49
*** nosnos has quit IRC09:07
*** nosnos has joined #openstack-dns09:08
*** tryggvil has joined #openstack-dns09:11
*** ToMiles has quit IRC09:17
*** vipul has quit IRC09:27
*** GonZo2K has quit IRC09:39
*** vipul has joined #openstack-dns09:59
*** boris-42 has joined #openstack-dns10:40
*** xianghui_ is now known as xianghui10:58
*** ekarlso- has quit IRC11:44
*** ekarlso- has joined #openstack-dns11:49
*** mwagner_lap has quit IRC12:08
*** vinod has joined #openstack-dns12:19
*** rediskin has quit IRC12:24
*** vinod has quit IRC12:35
*** jmcbride has joined #openstack-dns12:41
*** ekarlso- has quit IRC12:50
*** ekarlso- has joined #openstack-dns12:50
*** tryggvil has quit IRC13:21
*** richm has joined #openstack-dns13:22
*** tryggvil has joined #openstack-dns13:23
ekarlso-anyone up for a review ? :)13:28
ekarlso-jmcbride: ? :D13:28
openstackgerritEndre Karlson proposed openstack/designate: MDNS part of Secondary zones  https://review.openstack.org/13368313:28
*** richm has quit IRC13:44
*** ryanpetrello has joined #openstack-dns13:48
openstackgerritEndre Karlson proposed openstack/designate-specs: Add secondary zones  https://review.openstack.org/13075813:54
*** betsy has joined #openstack-dns14:00
ekarlso-hey betsy :)14:05
betsyekarlso: good morning14:05
ekarlso-heh, allmost evening today :D14:05
ekarlso-s/today/here14:05
betsyoh, yeah. :)14:05
ekarlso-btw, i'm adding zone transfers to the v2 bindings / cli14:06
betsycool14:06
ekarlso-betsy: any other stuff u would want in there ?14:07
betsyidk - I haven’t looked at the cli code in awhile14:08
ekarlso-wonder why >https://review.openstack.org/#/c/134711/ hasn't merged yet14:09
betsylet me look later and I’ll let you know14:09
*** jmcbride has quit IRC14:09
ekarlso-betsy: kewl : )14:09
betsyhmm - good question. I didn’t notice that14:09
betsyShould have merged yesterday14:09
ekarlso-for those of you that wants to see, here's a blog about the secondary zone stuff: https://cloudistic.net/blog/drafts/designate-secondary-zones.html14:10
ekarlso-feedback would be appreciated before i remove the draft status ;)14:10
betsynice. I’ll take a look14:10
*** mwagner_lap has joined #openstack-dns14:11
ekarlso-has full examples of getting a devstack vm going with nsd as the master14:11
ekarlso-btw can people stop using WIP in the commit names and do a workflow -1 instead ? :)14:12
ekarlso-I see the WIP everywhere :/14:12
*** richm has joined #openstack-dns14:14
*** richm has left #openstack-dns14:14
*** richm has joined #openstack-dns14:14
betsyWell, you can also see a big red X in some of them14:16
betsyI used workflow -1 for mine. I guess it’s about half and half now. :D14:16
betsyekarlso: Nice right up btw14:16
betsyI’ll have to try the demo later14:17
ekarlso-ahhh14:18
ekarlso-I see why the rally stuff hasn't merged, kiall has a change that it depends on that's not merged :p14:18
betsyAh14:20
betsyOops. And I actually gave that one a -114:21
openstackgerritKiall Mac Innes proposed openstack/designate: Move Central notifications to a decorator  https://review.openstack.org/13470714:21
ekarlso-:p14:22
betsyhe’s got perfect timing14:22
ekarlso-:p14:23
ekarlso-always :)14:23
*** Kiall has joined #openstack-dns14:28
*** openstackgerrit has quit IRC14:33
*** openstackgerrit has joined #openstack-dns14:33
*** ChanServ sets mode: +v openstackgerrit14:33
*** tryggvil has quit IRC14:38
*** tryggvil has joined #openstack-dns14:40
*** jbratton has quit IRC14:44
*** jmcbride has joined #openstack-dns14:45
*** openstackgerrit has quit IRC14:49
*** openstackgerrit has joined #openstack-dns14:49
*** ChanServ sets mode: +v openstackgerrit14:49
*** paul_glass has joined #openstack-dns14:50
*** nosnos has quit IRC14:52
*** tryggvil has quit IRC14:55
*** timsim has joined #openstack-dns14:59
*** nihilifer has quit IRC15:07
*** betsy has quit IRC15:20
Kialleandersson: BTW - You said you were going to test those patches for your use case, did they resolve your immediate issue?15:27
*** vinod has joined #openstack-dns15:28
boris-42ekarlso-: Kiall yo!15:34
Kiallboris-42: heya15:34
boris-42Kiall: great news for you guys15:34
* Kiall hides ;)15:34
boris-42Kiall: now in rally report (that generates job)15:34
boris-42Kiall:  http://logs.openstack.org/11/134711/4/check/gate-rally-dsvm-designate-designate/ab34394/rally-plot/results.html.gz#/DesignateBasic.create_and_delete_records#failures15:34
boris-42Kiall: you can see failures with traces*15:34
KiallReally? I can't see any there! (I'm probably just blind though ;))15:35
*** rediskin has joined #openstack-dns15:36
KiallAhh15:36
*** rediskin has left #openstack-dns15:36
KiallLink didn't seem to work for me :)15:36
KiallWe should probably fix the client to show a real error for timeouts ;)15:36
Kiallboris-42: also.. I'm starting to wonder if gate VMs are so much slower than my devstack VM's, the gate shows rally fails when I can't get fails locally ;)15:37
boris-42Kiall: hm?)15:37
Kiall(Still a designate issue.. Just painful to find+fix!)15:37
boris-42Kiall: so why link doesn't work for you lol?15:38
KiallI'm betting it's because # is used twice15:39
boris-42Kiall: that should work15:39
KiallIt brings me to the standard "landing page" rather than the specific section15:39
KiallNot in Chrome ;)15:39
boris-42Kiall: in FF works as well..15:39
boris-42Kiall: just tested15:39
KiallWorks in FF for me, but not Chrome v39...15:40
KiallOh... Now it works15:40
KiallWTF15:40
boris-42Kiall: lol15:40
boris-42Kiall: okay so now you can see exeptions from clients/scenarios15:40
KiallAh, yes.. I see it15:40
boris-42Kiall: just in report*15:40
Kiallhttp://logs.openstack.org/11/134711/4/check/gate-rally-dsvm-designate-designate/ab34394/rally-plot/results.html.gz#/DesignateBasic.create_and_delete_records%23failures15:40
Kiallthe second # is getting urlencoded to %2315:41
boris-42lol15:41
Kiall(Since it's against the spec to have a second # that's not urlencoded...)15:41
*** tryggvil has joined #openstack-dns15:42
*** EricGonczer_ has joined #openstack-dns15:43
boris-42Kiall: we will change second # to /15:43
Kiall:)15:44
boris-42Kiall: so it will work more stable=)15:44
boris-42Kiall: as well now you can see SLA http://logs.openstack.org/11/134711/4/check/gate-rally-dsvm-designate-designate/ab34394/rally-plot/results.html.gz#/DesignateBasic.create_and_delete_records#overview15:44
boris-42Kiall: so it's a bit more user friendly report now+)15:44
Kiallboris-42: so, 1 thing which would be really interesting to add there is the request-id for failed requests.. Most APIs return it in the "x-openstack-request-id" header.. How you would go about getting that in a generic cross-project way, I'm not sure..15:46
boris-42Kiall: yep such stuff like osprofiler traces and requsts id are in our roadmap15:47
boris-42Kiall: SO15:47
boris-42Kiall: somehow we will get that =)15:47
Kiall:)15:51
openstackgerritMerged openstack/designate: Move Central notifications to a decorator  https://review.openstack.org/13470715:58
*** EricGonczer_ has quit IRC15:58
*** EricGonczer_ has joined #openstack-dns15:59
openstackgerritKiall Mac Innes proposed openstack/designate: Add Rally to Vagrant VM  https://review.openstack.org/13471115:59
Kiallbetsy / mugsie - had to rebase https://review.openstack.org/#/c/134711/ - could one of you +A it again :)16:00
*** jmcbride has quit IRC16:01
*** jmcbride has joined #openstack-dns16:01
boris-42Kiall: ^ i think you can do it yourself=)16:15
Kiallre the +A? Nope, I can't +A my own change ;)16:15
Kiallvinod / betsy / mugsie can though ;)16:15
mugsieKiall: done16:16
boris-42Kiall: seems like bureaucracy =)16:17
boris-42Kiall: it already has 2x+2 and it was just rebased=)16:17
Kiallbureaucracy is openstack's specialty ;)16:17
*** jmcbride has quit IRC16:17
boris-42Kiall:  )16:19
KiallBaaS .. Bureaucracy-as-a-Service ;)16:20
vinodmugsie: Had a question, regarding one of your comments in ekarlso-'s Secondary zones spec/16:21
mugsievinod: shoot16:22
vinodwould pool manager send the create's, updates and deletes of secondary zones to the pool servers?16:22
mugsieyup, the same way it would for a normal zone16:22
mugsiecentral -> pool manager -> pool name servers16:23
KiallYea, secondary zones look identical to the pool manager.. The difference is on the "far side" of central, in that we have the REST API and inbound-AXFR as two methods to manage zone contents16:24
vinodso would the pool servers have minidns as the master for the secondary zones?16:24
mugsieyup16:24
KiallYes, they all look the same on that side :)16:24
*** betsy has joined #openstack-dns16:24
vinodso how would secondary zones be used? I am assuming this is something you need at HP.16:26
mugsiea customer just wants to use desingate like they would akamai16:28
mugsieor Dyn16:28
mugsie(they want the global servers that these providers have)16:28
mugsieor, you could have federated designates16:29
mugsiea private internal one, and with a pool that has a "designate" backend, for external records16:29
vinodinteresting scenarios16:31
Kiallvinod: re something we need at HP - Yep, it is.. it would allow our private cloud customers to transparently use our public cloud for the actual DNS servers..16:31
vinodekarlso-: Could you update the spec with mugsie's comments and it should be good to go.16:33
*** EricGonczer_ has quit IRC16:36
*** EricGonczer_ has joined #openstack-dns16:36
openstackgerritMerged openstack/designate: Add Rally to Vagrant VM  https://review.openstack.org/13471116:38
*** EricGonc_ has joined #openstack-dns16:42
*** EricGonczer_ has quit IRC16:42
ekarlso-vinod: what's missing there ?16:48
*** EricGonc_ has quit IRC16:48
*** EricGonczer_ has joined #openstack-dns16:49
vinodthe fact that secondary zones are similar to primary zones wrt to the pool manager - as regards create/delete and update16:50
vinodsimilarly for the status updates16:50
ekarlso-where would you want notes for that to go ?16:51
vinodmugsie had responded to some of my comments on your review - i wanted that information reflected in the spec16:51
vinodit could probably go at the beginning16:52
vinodMay be under Proposed change16:54
openstackgerritEndre Karlson proposed openstack/designate-specs: Add secondary zones  https://review.openstack.org/13075816:54
ekarlso-vinod:  ^16:54
openstackgerritMerged openstack/designate-specs: Refactor Zone import functionality  https://review.openstack.org/13085216:55
*** k4n0 has quit IRC16:56
ekarlso-this is also ready: https://review.openstack.org/#/c/133208/16:57
ekarlso-in fact it was already +2'd before by Kiall and mugsie16:57
*** betsy has quit IRC17:08
*** betsy has joined #openstack-dns17:08
betsyI think we’re back17:08
betsyI tried to say...17:08
betsyI’m working on the server to pool_attributes table migration. And changing the v1 api to CRUD pool_attributes instead of servers while still looking like it’s doing servers. And then changing ALL the code to not use servers17:08
Kiallwrong channel :D17:09
*** jmcbride has joined #openstack-dns17:12
*** tryggvil has quit IRC17:19
*** rjrjr has joined #openstack-dns17:39
rjrjrvinod: when you have a minute, let me know what mdns calls i'm making incorrectly.17:39
vinodpulling up the code17:40
*** dtalton has joined #openstack-dns17:41
vinodFor poll_for_serial_number mdns, expects context, domain, destination, timeout,17:42
vinod                               retry_interval, max_retries17:42
vinodtrying to find the place where pool manager calls it17:44
vinodAh okay - it is not checked in yet17:46
vinodhttps://review.openstack.org/#/c/131656/2/designate/pool_manager/service.py17:46
vinodpoll_for_serial_number calls pool_manager_api.update_status17:48
vinodMdns sends the status as 0 or 1. I think pool manager expects a string - 'SUCCCESS' or 'ERROR'17:49
vinodI am fine with either one17:49
vinodWe need to change one of the 2 so that both of them match up17:50
*** rjrjr has quit IRC17:50
vinodAlso I notice that the central's update status takes domain.id instead of domain17:51
vinodFor consistency it would be good it central also takes domain instead of just domain.id17:51
*** EricGonczer_ has quit IRC17:55
*** rjrjr has joined #openstack-dns17:56
*** EricGonczer_ has joined #openstack-dns17:56
rjrjrthe update_domain method?17:56
vinodupdate_status method17:56
*** rmoe has quit IRC17:56
*** jordanP has quit IRC17:56
rjrjri don't have that method implemented yet.17:57
rjrjrwhat is the problem you see?17:57
vinodPool manager uses strings everywhere for success /error.17:58
vinodWe should probably change minidns to use that too17:59
rjrjragreed.  or an enum.17:59
vinodMinidns currently sends a 0/1 when it calls pool manager's update_status17:59
vinodenum would probably be good17:59
rjrjri see.  very C like.17:59
vinodpool manager's update_status takes a domain but central's update_status takes a domain id18:00
vinodwould be good it both take domain18:00
*** paul_glass has quit IRC18:00
rjrjrit would save a call to the database.18:01
vinodthe domain object would have domain id, so you would get to the id by just accessing domain.id18:01
rjrjri'll fix up minidns.  let me take care of that.18:03
vinodcool - thanks rjrjr18:04
vinodOne other thing I noticed - in contrib/devstack/lib/designate_plugins/backend-bind9_pool we have18:05
vinod# Enable with:18:05
vinod# DESIGNATE_BACKEND_DRIVER=bind9-pool18:05
vinodshould it be bind9_pool instead of bind9-pool?18:05
rjrjrbind9_pool18:08
rjrjri'll fix that too.18:08
*** dtalton has quit IRC18:10
*** rmoe has joined #openstack-dns18:12
eanderssonKiall: The initial fix you submitted. I back-ported that one to 2014.2. I see that you are working on a new solution, but is the old one still viable?18:19
eanderssonI tested the first one out, and it seems to work at least. :p18:20
*** GonZo2K has joined #openstack-dns18:22
rjrjrvinod: looking at the update_status method in central.18:25
rjrjri think passing a domain_id makes more sense.18:25
rjrjrwhy do central's and pool_manager's update_status methods have to take the same arguments?18:26
*** EricGonczer_ has quit IRC18:26
vinodsince we pass around objects - this method too can pass around objects rather than just the id18:27
rjrjrsure, but i'm still not going to use the object and just the ID to get the object from the database.18:28
rjrjri want the most current domain object from the database, not the domain object that has been passed around pool_manager and mdns.18:29
rjrjrand we do pass IDs in several methods in central.18:30
vinodthe object should always be up to date and it would be tracking the changes that are different from the one in the database18:32
rjrjrwe are passing that object from central to pool_manager to mdns, back to pool_manager and back to central.  it can be out of date.18:36
rjrjrwhile the domain object is in pool_manager, for example, someone can come along and add a record to it.  now it has a different serial number.18:37
vinodthe domain object that we pass along between the pool manager and mdns tracks the expected serial number18:38
vinodthe domain object could have a different serial number18:39
vinodbut for the call we know the expected serial through domain.serial18:39
rjrjrvinod, i need 2 things from a domain, id and serial.  why would i want to pass the entire object for those 2 things?  especially since i'm just going to turn around and get the latest domain object from the database using the ID in that method?18:46
rjrjrit seems wasteful.18:47
rjrjrif i wasn't going to call the database to get a domain object, i can see passing the domain object, but in this case, we are going to call the database and get the object anyway.18:48
vinodlet me look through the central's code18:48
vinodi'll be back in a little while18:48
rjrjrsure.18:48
*** vinod has quit IRC18:49
rjrjrand yes, we definitely need to be synchronizing (locking?) code in central.18:49
rjrjrwe don't want the domain changing while we are setting the status.18:50
rjrjri'm aware of it and will have it in the code.18:54
*** paul_glass has joined #openstack-dns19:06
*** ryanpetrello_ has joined #openstack-dns19:15
*** ryanpetrello has quit IRC19:17
*** ryanpetrello_ is now known as ryanpetrello19:17
*** jordanP has joined #openstack-dns19:20
*** paul_glass has quit IRC19:40
*** tryggvil has joined #openstack-dns19:42
*** paul_glass has joined #openstack-dns19:43
*** jmcbride has quit IRC19:46
*** jmcbride has joined #openstack-dns19:47
*** MasterPiece has joined #openstack-dns19:48
*** jmcbride has quit IRC20:32
*** jmcbride has joined #openstack-dns20:36
*** jmcbride1 has joined #openstack-dns20:37
*** jmcbride1 has quit IRC20:37
*** jmcbride1 has joined #openstack-dns20:37
*** jmcbride has quit IRC20:38
*** amcrn has joined #openstack-dns20:40
Kialleandersson: there's a couple of parts to the issue, each fix addresses 1 part :)20:46
KiallYou're only hitting 1 part if just the first patch fixed it :)20:47
*** tryggvil has quit IRC20:47
*** MasterPiece has quit IRC20:48
*** MasterPiece has joined #openstack-dns20:50
openstackgerritTim Simmons proposed openstack/designate: Add metadata, totalcount fields to zones/recordsets collections  https://review.openstack.org/10885920:56
betsykiall/mugsie: Y’all still around?21:04
*** ToMiles has joined #openstack-dns21:06
*** shakamunyi_ has joined #openstack-dns21:12
*** MasterPiece has quit IRC21:18
*** MasterPiece has joined #openstack-dns21:18
*** paul_glass has quit IRC21:24
*** paul_glass has joined #openstack-dns21:36
*** DNS`jbratton has joined #openstack-dns21:42
*** tryggvil has joined #openstack-dns21:48
*** paul_glass has quit IRC21:58
*** shakamunyi_ has quit IRC21:59
mugsiebetsy: hey22:01
mugsiesorry - was away for a biot22:01
mugsiebit*22:01
betsyOh, hey22:04
betsyJust wanted to ask a question about the tests22:05
betsyWhen testing the api, and it does a response = self.get(‘servers’)...22:05
betsyThat’s actually calling the api layer, right?22:05
mugsieyeah22:05
mugsiei think22:06
betsyI think it does too, but it goes to … def get(self): return test_notifer.NOTIFICATIONS22:07
mugsieyup, just checked22:07
betsyOkay. Cool22:07
openstackgerritMerged openstack/designate-specs: Add secondary zones  https://review.openstack.org/13075822:08
*** openstackgerrit has quit IRC22:10
*** openstackgerrit has joined #openstack-dns22:10
*** ChanServ sets mode: +v openstackgerrit22:10
*** mwagner_lap has quit IRC22:10
*** jmcbride1 has quit IRC22:17
ekarlso-awesome :P22:19
*** jmcbride has joined #openstack-dns22:28
*** jmcbride has quit IRC22:28
*** jmcbride has joined #openstack-dns22:28
eanderssonKiall: Ah, makes sense. I'll review and implement all your fixes.22:29
eanderssonDo you guys have any plans to back port them to 2014.2?22:29
*** ryanpetrello has quit IRC22:39
*** jmcbride has quit IRC22:55
*** betsy has quit IRC22:56
ekarlso-eandersson: not running trunk ? :D22:57
*** GonZoPT has joined #openstack-dns22:57
*** GonZo2K has quit IRC22:58
*** mwagner_lap has joined #openstack-dns23:02
*** timsim has quit IRC23:03
*** jmcbride has joined #openstack-dns23:17
*** jordanP has quit IRC23:17
*** jmcbride has quit IRC23:20
*** EricGonczer_ has joined #openstack-dns23:26
*** EricGonczer_ has quit IRC23:30
*** richm has quit IRC23:50

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