*** nkinder has joined #openstack-dns | 00:52 | |
*** rmoe has quit IRC | 01:00 | |
*** rmoe has joined #openstack-dns | 01:17 | |
*** nosnos has joined #openstack-dns | 02:14 | |
*** richm has quit IRC | 02:27 | |
*** vinod has joined #openstack-dns | 02:28 | |
*** vinod has quit IRC | 02:31 | |
*** vipul has quit IRC | 02:33 | |
*** vipul has joined #openstack-dns | 02:37 | |
*** richm has joined #openstack-dns | 02:44 | |
*** richm has quit IRC | 02:52 | |
*** nosnos has quit IRC | 03:01 | |
*** ryanpetrello has joined #openstack-dns | 03:18 | |
*** ryanpetrello has quit IRC | 03:41 | |
*** amcrn has quit IRC | 03:47 | |
*** ryanpetrello has joined #openstack-dns | 04:07 | |
*** nosnos has joined #openstack-dns | 04:16 | |
*** ryanpetrello has quit IRC | 04:23 | |
*** iraw_ has joined #openstack-dns | 04:40 | |
*** ryanpetrello has joined #openstack-dns | 04:49 | |
*** iraw_ has quit IRC | 05:13 | |
*** k4n0 has joined #openstack-dns | 06:03 | |
*** ryanpetrello has quit IRC | 06:05 | |
*** nihilifer has joined #openstack-dns | 07:05 | |
*** yingjun has joined #openstack-dns | 07:47 | |
yingjun | Hi, anyone there? i got an error when running devstack with designate, could anyone help? | 07:48 |
---|---|---|
yingjun | NoSuchMethod: Endpoint does not support RPC method create_server designate | 07:49 |
*** ryanpetrello has joined #openstack-dns | 07:51 | |
*** ryanpetrello has quit IRC | 07:56 | |
*** GonZo2K has joined #openstack-dns | 08:12 | |
*** yingjun has quit IRC | 08:41 | |
*** jordanP has joined #openstack-dns | 08:54 | |
*** yingjun has joined #openstack-dns | 09:08 | |
*** GonZo2K has quit IRC | 09:38 | |
*** rjrjr has joined #openstack-dns | 10:03 | |
rjrjr | is anyone else having problems running the 'tox -e py27 -r' with the latest code? | 10:04 |
rjrjr | i'm getting failures in the zone transfer code. | 10:04 |
rjrjr | Here is the output of the tox on a brand new 'git clone' - http://paste.openstack.org/show/138731/ | 10:10 |
*** jmcbride has joined #openstack-dns | 10:22 | |
*** rediskin has joined #openstack-dns | 10:31 | |
*** jmcbride has quit IRC | 10:33 | |
*** jmcbride has joined #openstack-dns | 10:33 | |
rjrjr | in addition to the unit tests failing, when i'm running central, i'm now getting errors concerning the zone transfer request - http://paste.openstack.org/show/138748/ | 10:45 |
ekarlso- | rjrjr: ? | 11:11 |
*** yingjun has joined #openstack-dns | 11:13 | |
yingjun | Help! i got an error when creating server with command: designate server-create --name ns1.devstack.org. Backend: Unknown backend failure: RemoteError(u'Remote error: NoSuchMethod Endpoint does not support RPC method create_server | 11:16 |
yingjun | anyone know what’s happened ? | 11:16 |
yingjun | ? | 11:16 |
rjrjr | ekarlso, i'm here. | 11:19 |
*** jordanP has quit IRC | 11:23 | |
rjrjr | i'm block right now on my code. if anyone can help me figure out why the transfer zone code is not working, it would be greatly appreciated. 8^) | 11:55 |
rjrjr | ignore me. not sure why, but tests are working now. | 12:29 |
*** jordanP has joined #openstack-dns | 13:06 | |
*** jordanP has quit IRC | 13:10 | |
*** jordanP has joined #openstack-dns | 13:18 | |
*** nosnos has quit IRC | 13:20 | |
openstackgerrit | Ron Rickard proposed openstack/designate: Pool Manager https://review.openstack.org/131656 | 13:26 |
*** ryanpetrello has joined #openstack-dns | 13:42 | |
*** yingjun has quit IRC | 13:43 | |
*** iraw_ has joined #openstack-dns | 14:03 | |
*** iraw_ has quit IRC | 14:03 | |
*** nkinder has quit IRC | 14:10 | |
*** RaginBajin has joined #openstack-dns | 14:11 | |
*** timsim has joined #openstack-dns | 14:12 | |
openstackgerrit | Endre Karlson proposed openstack/designate: Support secondary zones https://review.openstack.org/133682 | 14:22 |
openstackgerrit | Endre Karlson proposed openstack/designate: MDNS part of Secondary zones https://review.openstack.org/133683 | 14:22 |
*** vinod has joined #openstack-dns | 14:31 | |
*** mikeit has joined #openstack-dns | 14:42 | |
*** mikeit has quit IRC | 14:47 | |
*** mikeit has joined #openstack-dns | 14:48 | |
*** rediskin has left #openstack-dns | 14:54 | |
rjrjr | vinod: you around? | 14:54 |
ekarlso- | rjrjr: got your stuff going ? | 14:56 |
rjrjr | yes. | 14:57 |
rjrjr | ekarlso: do you know how i add items to today's meeting agenda? | 14:57 |
rjrjr | i always seem to update the wrong section on the meeting page. | 14:58 |
*** nkinder has joined #openstack-dns | 14:58 | |
rjrjr | do i add to the future/unscheduled agenda items? agenda? | 14:59 |
ekarlso- | rjrjr: not sure :| | 15:00 |
*** paul_glass has joined #openstack-dns | 15:00 | |
rjrjr | i've been in this place before. i'm updating both. 8^) | 15:00 |
*** vinod has quit IRC | 15:01 | |
*** jmcbride has quit IRC | 15:04 | |
*** jmcbride has joined #openstack-dns | 15:06 | |
*** betsy has quit IRC | 15:12 | |
*** nihilifer has quit IRC | 15:23 | |
*** vinod has joined #openstack-dns | 15:31 | |
vinod | rjrjr: I am here | 15:31 |
vinod | Looking at your changes now | 15:31 |
rjrjr | vinod: we have 2 issues and I'm looking for advice on how to fix one of them. | 15:34 |
rjrjr | we are calling notify_zone_changed followed by poll_for_serial_number. the problem, there is not a sufficient delay between the call. the result, the poll_for_serial_number fails to get the newest serial number. | 15:35 |
rjrjr | so, we obviously need a delay, just wanting to hear ideas on where that delay should be. | 15:36 |
rjrjr | since mdns has the other timing parameters (retry, interval, timeout) it makes sense to me to introduce a 'poll_delay' parameter there.. | 15:36 |
vinod | i was expecting to see this issue | 15:37 |
rjrjr | i would set this to 0 for notify_zone_changed and to the value in the config file (i'll add poll_delay to [service:pool_manager]. | 15:37 |
rjrjr | the config value for poll_for_serial_number. thoughts? | 15:38 |
rjrjr | i'll also set the default to 1 second. | 15:39 |
vinod | so would pool manager wait for the delay ( of 1 second) before issuing a poll_for_serial_number? | 15:42 |
rjrjr | no, i was thinking about adding this to mdns. | 15:42 |
rjrjr | just to be consistent since all the other timing parameters are being passed to mdns. | 15:43 |
rjrjr | it would be another parameter to poll_for_serial_number (and even notify_zone_changed). | 15:44 |
rjrjr | i would set it to 0 from pool manager for notify_zone_changed. i would set it to config file value for poll_for_serial_number. | 15:44 |
rjrjr | my thinking, it is just another timing parameter, so it should be passed to mdns like the others. | 15:45 |
vinod | currently poll_for_serial_number has a retry logic - i was wondering if there is a bug there | 15:46 |
rjrjr | bug in the retry logic? | 15:46 |
rjrjr | yeah, probably. | 15:46 |
rjrjr | the retry_interval isn't being honored for retries that are not timeouts. | 15:46 |
vinod | we should fix that so that it always sleeps for retry_interval between retries | 15:48 |
vinod | that would i think fix the problem | 15:49 |
*** GonZo2K has joined #openstack-dns | 15:49 | |
rjrjr | no, because the first attempt to get the serial number will always fail and trigger another attempt. | 15:49 |
rjrjr | i personally think that is a sloppy way of handling the issue. | 15:49 |
rjrjr | hence my wanting to introduce the delay. | 15:50 |
rjrjr | (also a 1 second delay is preferrable to a 2 second retry interval.) | 15:50 |
*** betsy has joined #openstack-dns | 15:50 | |
vinod | that is true | 15:53 |
rjrjr | okay, i'll introduce the delay parameter. if reviewers don't like it, they can speak up in the review stage. 8^) | 15:54 |
vinod | another alternative to delays is to communicate the status of notify_zone_changed or for mdns to call poll_for_serial_number after a successful call to notify_zone_changed | 15:54 |
rjrjr | true. but, the code is mostly working. this delay thing is one of a few items that needs to be hammered out. | 15:56 |
*** EricGonczer_ has joined #openstack-dns | 15:56 | |
*** paul_glass has quit IRC | 15:56 | |
*** GonZo2K has quit IRC | 15:57 | |
rjrjr | the only other bug I'm seeing has to do with NS and SOA records in the records table. right now, they have a serial number of 1 and the status/action is wrong initially. | 15:57 |
rjrjr | i know where i need to fix that. other than the delay and that issue, the code is working. | 15:57 |
*** paul_glass has joined #openstack-dns | 15:57 | |
rjrjr | i was going to fire up a few DNS servers today to do more testing. and i still have lots of unit tests to write. | 15:58 |
rjrjr | but, i'm hoping none of these will hold back getting this code reviewed. | 15:58 |
*** paul_glass has quit IRC | 15:58 | |
*** jmcbride has quit IRC | 15:58 | |
*** jmcbride has joined #openstack-dns | 15:58 | |
rjrjr | did find a bug with the ZoneTransferRequest object, but that is a separate issue entirely. 8^) | 15:59 |
rjrjr | (the object extends PersistentObjectMixin and also defines the id, created_at, update_at, and versions attributes.) | 16:00 |
*** EricGonczer_ has quit IRC | 16:01 | |
vinod | I saw some issue at startup with periodic-sync | 16:01 |
vinod | that should be a minor issue if at all | 16:02 |
rjrjr | hmmm... okay, i haven't seen any issue with that, but i'm pretty close to the code, so maybe i overlooked something. | 16:03 |
*** mikeit has quit IRC | 16:04 | |
vinod | so are you planning to fix mdns too for the retry_interval or do you want me to fix it? | 16:06 |
*** ryanpetrello has quit IRC | 16:06 | |
rjrjr | i'll take care of it. | 16:08 |
rjrjr | taking a break until meeting. i'll fix all this after the meeting. shouldn't take but an hour. | 16:09 |
rjrjr | also need to use passed parameters in notify_zone_changed. | 16:13 |
vinod | what do you mean? | 16:14 |
*** zigo has quit IRC | 16:16 | |
*** EricGonczer_ has joined #openstack-dns | 16:18 | |
*** EricGonczer_ has quit IRC | 16:23 | |
vinod | rjrjr: This is the error in the pool manager log related to periodic sync that I was talking about - http://paste.openstack.org/show/KXiLsL7j4WhfpvC0tJca/ | 16:24 |
rjrjr | i'm not seeing it. are you doing anything to induce the error? | 16:26 |
*** zigo has joined #openstack-dns | 16:27 | |
*** paul_glass has joined #openstack-dns | 16:27 | |
*** k4n0 has quit IRC | 16:30 | |
vinod | no - this is just after starting the pool manager service | 16:31 |
*** ryanpetrello has joined #openstack-dns | 16:31 | |
rjrjr | is all your code up to date (i.e. if you are using devstack, did you let it get the latest code including oslo?) | 16:35 |
rjrjr | i believe the olso library i'm pulling that from was updated a few weeks back. | 16:35 |
*** ryanpetrello_ has joined #openstack-dns | 16:45 | |
*** ryanpetrello has quit IRC | 16:48 | |
*** ryanpetrello_ is now known as ryanpetrello | 16:48 | |
*** rjrjr has quit IRC | 16:56 | |
*** rjrjr_ has joined #openstack-dns | 16:58 | |
*** ryanpetrello has quit IRC | 17:12 | |
*** rediskin has joined #openstack-dns | 17:27 | |
*** paul_glass has quit IRC | 17:35 | |
*** ryanpetrello has joined #openstack-dns | 17:35 | |
*** johnbelamaric has joined #openstack-dns | 17:36 | |
*** rmoe has quit IRC | 17:40 | |
*** rjrjr_ has quit IRC | 17:46 | |
*** rjrjr has joined #openstack-dns | 17:46 | |
*** rmoe has joined #openstack-dns | 18:00 | |
*** ryanpetrello_ has joined #openstack-dns | 18:01 | |
*** jordanP has quit IRC | 18:04 | |
*** ryanpetrello has quit IRC | 18:04 | |
*** ryanpetrello_ is now known as ryanpetrello | 18:04 | |
*** rjrjr has quit IRC | 18:11 | |
timsim | FYI: Most of the Americans are on vacation for the rest of the week :) Tomorrow is Thanksgiving, the one day a year where we give thanks and eat a lot with people who annoy us ;) | 18:28 |
mugsie | timsim: :) | 18:30 |
*** ryanpetrello_ has joined #openstack-dns | 18:30 | |
*** betsy has quit IRC | 18:31 | |
*** ryanpetrello has quit IRC | 18:33 | |
*** ryanpetrello_ is now known as ryanpetrello | 18:33 | |
Kiall | timsim: lol | 18:41 |
Kiall | It's also the one day of the year I have zero meetings :P | 18:41 |
ahu | congrats | 18:51 |
ekarlso- | any labrats for the v2 client ? | 19:11 |
*** timsim has quit IRC | 19:15 | |
*** johnbelamaric has left #openstack-dns | 19:22 | |
*** bitblt has joined #openstack-dns | 19:28 | |
*** bitblt has left #openstack-dns | 19:30 | |
*** GonZo2K has joined #openstack-dns | 19:56 | |
*** ToMiles has joined #openstack-dns | 19:56 | |
*** ryanpetrello has quit IRC | 20:00 | |
*** ryanpetrello has joined #openstack-dns | 20:02 | |
*** ryanpetrello has quit IRC | 20:10 | |
*** ryanpetrello has joined #openstack-dns | 20:20 | |
*** betsy has joined #openstack-dns | 20:25 | |
*** nkinder has quit IRC | 20:28 | |
*** ryanpetrello has quit IRC | 21:32 | |
*** rjrjr has joined #openstack-dns | 21:40 | |
rjrjr | kiall: you still around? | 21:41 |
rjrjr | actually, anyone who is familiar with the storage code around? | 21:42 |
Kiall | heya | 21:43 |
*** nkinder has joined #openstack-dns | 21:45 | |
Kiall | rjrjr: ^ | 21:46 |
rjrjr | i think i won't be able to do what i want via storage after all. i need to not include records that are about to be deleted (record.action = 'DELETE') from the AXFR. | 21:49 |
rjrjr | i think i'm going to have to do that in the AXFR code. | 21:50 |
rjrjr | so, ignore me. 8^) | 21:50 |
Kiall | Can we not do find_recordsets(domain_id, criterion={"action": "!DELETE"}) | 21:51 |
rjrjr | recordset doesn't have action. that is on a record. | 21:52 |
rjrjr | sorry, that is an attribute of a record, not recordset. | 21:52 |
Kiall | Oh, humm.. | 21:52 |
Kiall | We probably want to extent storage to support that, but maybe file a Launchpad BP for it and do it in code on the mDNS side for the moment? | 21:53 |
Kiall | extend* | 21:53 |
rjrjr | something like criterion={"record.action": "!DELETE"} would work. | 21:54 |
rjrjr | is that what you are thinking? | 21:54 |
rjrjr | about how to extend storage... | 21:55 |
Kiall | Yea, something along those lines | 21:59 |
Kiall | But.. We can skip it for right now, file a BP and log a Bug/BP for later.. | 21:59 |
Kiall | mDNS needs to iterate the list anyway for other reasons, so adding a if action = 'DELETE': continue is pretty harmless | 22:00 |
openstackgerrit | Kiall Mac Innes proposed openstack/python-designateclient: Refactor KS session creation and support CA certs https://review.openstack.org/137479 | 22:01 |
openstackgerrit | Kiall Mac Innes proposed openstack/python-designateclient: Refactor KS session creation and support CA certs https://review.openstack.org/137479 | 22:03 |
rjrjr | kiall: okay. | 22:11 |
rjrjr | kiall: just so i understand, right now, records are removed from the tables, but recordsets stick around. | 22:11 |
rjrjr | when we introduce IXFR, we'll need records to stick around (at least for a while.) | 22:12 |
openstackgerrit | Graham Hayes proposed openstack/designate: Added designate-dashboard code to contrib/designate-dashboard https://review.openstack.org/137480 | 22:12 |
Kiall | rjrjr: ^ | 22:12 |
rjrjr | cool. dashboard code. i'll let PK know. | 22:13 |
rjrjr | so, back to what i was saying. if i remove a record, even the last record in a recordset, the recordset is not removed. does that sound correct? | 22:13 |
mugsie | no... but it is known bug | 22:14 |
Kiall | I think recordsets sticking around was a bug, a bug I thought was fixed recently? | 22:14 |
mugsie | i dont think we ever hads code to fix it | 22:14 |
mugsie | but I could be wrong | 22:14 |
rjrjr | i might have re-introduced it with what i'm working on. | 22:15 |
rjrjr | i moved the removal of records to the update_status method. | 22:15 |
rjrjr | (from the table.) | 22:15 |
mugsie | ah, that could have done it | 22:15 |
rjrjr | besides, we'll want records and recordsets to stick around when we work on the IXFR code. | 22:15 |
Kiall | Humm, is that for IXFR support? | 22:16 |
rjrjr | no. | 22:16 |
mugsie | rjrjr: yeah, they will get the status column then | 22:16 |
mugsie | and the "deleted" col | 22:16 |
rjrjr | i have notes to remove the 'deleted' column since it is redundant with status='DELETED'. but, for now, it is not hurting anything. | 22:18 |
rjrjr | old code - in the delete_record method we removed the record from the table. new code - action is changed to 'DELETE' and status is 'PENDING'. update_status actually removes the record if a threshold of servers agree it is deleted. | 22:18 |
rjrjr | (i added a TODO about this to the code. and have notes to fix this in the IXFR code.) | 22:18 |
rjrjr | with IXFR, we are not going to want to remove the record from the table (also have this in a TODO and will fix this in the IXFR code.) | 22:19 |
rjrjr | i have to update the IXFR spec, so i'll make sure i capture all this in that. | 22:20 |
rjrjr | so, i thought i had things working well until i realized the records were not actually being removed from the zone. that's what started me down this whole path... | 22:21 |
Kiall | deleted column is actually quite important still :) | 22:24 |
Kiall | It allows for the unique constraints to work correctly | 22:25 |
rjrjr | good to know. | 22:25 |
Kiall | re "update_status actually removes the record if a threshold of servers agree it is deleted." I assume you mean updates status = COMPLETE and sets deleted = $id, or actually purges from the DB? | 22:25 |
rjrjr | the behavior is the same as before. when the update_status is called and status is 'SUCCESS' the record that has an action of 'DELETE' and a status of 'PENDING', i call the Storage delete_record method. i do the same for domains. | 22:28 |
rjrjr | the end result, i'm calling the same method that would have been called, just later in the process. | 22:29 |
rjrjr | mugsie: thanks for adding the synchronize domain code by the way. it was on my list of things to do, but now that it is already done. | 22:29 |
Kiall | K - I'm still wondering a little about the delete_record call being in the update_status method, I'm wondering how that might affect the V1 API | 22:30 |
rjrjr | kiall: if you look at what i checked in, i have an if statement testing which backend is called in the delete_record method. | 22:31 |
rjrjr | if we are using the pool_manager_proxy backend, we defer deleting the record until update_status. | 22:32 |
rjrjr | if we use any other backend, we follow the same logic it always had. | 22:32 |
rjrjr | it's easier to see this in code than for me to explain i think. | 22:33 |
rjrjr | that code is all there. | 22:33 |
Kiall | K - I'll test it out in the morning and have a better understanding :) | 22:34 |
rjrjr | the good news, the bulk of the code is complete. i'm just debugging. | 22:35 |
rjrjr | i added the delay that was needed between notify_zone_changed and poll_for_serial_number. | 22:35 |
Kiall | rjrjr: BTW - If your using the Vagrant VM to test our the Horizon bits etc, you'll need to bump the ram for the VM way up (say, 8GB) since all the openstack services need to be installed for Horizon not to explode | 22:35 |
rjrjr | i found this bug with the delete record not working (it doesn't remove the record from the zone in DNS.) | 22:36 |
Kiall | :'( | 22:36 |
rjrjr | and i also know about one more bug where the NS,SOA records do not have the correct status/action during the process and also have a serial number of 1. | 22:37 |
rjrjr | i am fixing all these now and will submit shortly. | 22:37 |
Kiall | timsim: http://paste.openstack.org/show/85814/ (vinod / betsy - if your in the office with him, can you pass it on :)) | 22:38 |
Kiall | rjrjr: cool | 22:38 |
Kiall | timsim: logs of conversation are http://eavesdrop.openstack.org/irclogs/%23openstack-dns/%23openstack-dns.2014-07-09.log | 22:38 |
rjrjr | ack concerning the VM ram. | 22:38 |
rjrjr | one final question, did we actually use domain or record status before? | 22:40 |
rjrjr | before this pool manager code? | 22:41 |
Kiall | mugsie added them as an initial step towards a pre-precursor to pools.. So, No, I don't believe so. | 22:41 |
rjrjr | i see something where we test for PENDING in the API, but I didn't see much else for status. | 22:41 |
rjrjr | cool. i was worried about breaking something else, but i could find what else i could break. | 22:42 |
rjrjr | well, have a good night/day. despite it being a 4 day weekend, i'll be working so we can get this pool manager code complete. so if you review the code, i'll be checking all weekend. | 22:43 |
Kiall | :) | 22:50 |
*** RaginBajin has quit IRC | 22:51 | |
vinod | Kiall: timsim will be at work on Monday - i'll pass on the code to him then | 22:53 |
Kiall | Pfft, taking the holidays off? Unacceptable ;) | 22:53 |
vinod | will let him know ;-) | 22:56 |
Kiall | vinod: actually! Any chance of a quick review of https://review.openstack.org/#/c/137479/ | 22:57 |
Kiall | It's .. umm.. blocking some HP work and I need to cut a release with the CA Cert support ;) | 22:57 |
Kiall | Ideally before everyone goes away on holidays :P | 22:57 |
vinod | looking at it now | 23:03 |
Kiall | Excellent, Thanks :) | 23:03 |
openstackgerrit | Kiall Mac Innes proposed openstack/python-designateclient: Refactor KS session creation and support CA certs https://review.openstack.org/137479 | 23:07 |
Kiall | Tiny mistake of course ;) | 23:08 |
*** ryanpetrello has joined #openstack-dns | 23:12 | |
openstackgerrit | Graham Hayes proposed openstack/designate: Added designate-dashboard code to contrib/designate-dashboard https://review.openstack.org/137480 | 23:18 |
mugsie | damn pep8 | 23:18 |
*** ryanpetrello has quit IRC | 23:19 | |
*** jmcbride1 has joined #openstack-dns | 23:21 | |
*** jmcbride has quit IRC | 23:24 | |
vinod | Kiall: you have a +2 from me assuming the gate checks pass | 23:26 |
vinod | hmm. close to midnight for you guys | 23:27 |
Kiall | FU**. | 23:27 |
Kiall | Missed the train home -_- | 23:27 |
mugsie | shit | 23:27 |
mugsie | damn you bad public transport in ireland | 23:28 |
vinod | so sorry to hear that - does not sound good at all | 23:30 |
vinod | most of us won't be around till Monday. if you need anything to be looked at before Monday - send me an email. | 23:33 |
Kiall | Yep, enjoy the time off :P | 23:34 |
*** vinod has quit IRC | 23:34 | |
*** GonZoPT has joined #openstack-dns | 23:47 | |
*** GonZo2K has quit IRC | 23:49 | |
*** rediskin has left #openstack-dns | 23:51 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!