*** nkinder has quit IRC | 00:05 | |
*** rjrjr has joined #openstack-dns | 00:06 | |
*** penick has joined #openstack-dns | 00:06 | |
*** lpmulligan has quit IRC | 00:07 | |
*** EricGonczer_ has joined #openstack-dns | 00:22 | |
*** EricGonczer_ has quit IRC | 00:46 | |
*** penick has quit IRC | 01:07 | |
*** GonZo2K has quit IRC | 01:07 | |
*** rmoe has quit IRC | 01:10 | |
*** rmoe has joined #openstack-dns | 01:25 | |
*** johnbelamaric has quit IRC | 01:33 | |
*** nkinder has joined #openstack-dns | 01:55 | |
*** EricGonczer_ has joined #openstack-dns | 01:59 | |
*** EricGonczer_ has quit IRC | 02:39 | |
*** EricGonczer_ has joined #openstack-dns | 02:43 | |
*** EricGonczer_ has quit IRC | 02:48 | |
*** EricGonczer_ has joined #openstack-dns | 03:23 | |
*** EricGonc_ has joined #openstack-dns | 03:25 | |
*** EricGonczer_ has quit IRC | 03:28 | |
*** EricGonc_ has quit IRC | 03:31 | |
*** EricGonczer_ has joined #openstack-dns | 03:43 | |
*** EricGonczer_ has quit IRC | 03:47 | |
*** andrewbogott is now known as andrewbogott_afk | 03:59 | |
*** richm has quit IRC | 04:28 | |
*** csoukup has joined #openstack-dns | 04:32 | |
rjrjr | quick question: is mdns integral to Juno? i see we call notify_zone_changed in update_domain and update_recordset which is why i am asking. | 04:54 |
---|---|---|
*** c_soukup has joined #openstack-dns | 05:09 | |
*** csoukup_ has joined #openstack-dns | 05:10 | |
*** csoukup has quit IRC | 05:13 | |
*** c_soukup has quit IRC | 05:14 | |
*** csoukup_ has quit IRC | 05:32 | |
*** andrewbogott_afk is now known as andrewbogott | 06:35 | |
*** chlong has quit IRC | 07:34 | |
*** jordanP has joined #openstack-dns | 09:00 | |
*** andrewbogott is now known as andrewbogott_afk | 09:01 | |
*** openstackgerrit has quit IRC | 09:36 | |
*** openstackgerrit has joined #openstack-dns | 09:36 | |
*** ChanServ sets mode: +v openstackgerrit | 09:36 | |
*** untriaged-bot has joined #openstack-dns | 11:02 | |
untriaged-bot | Untriaged bugs so far: | 11:02 |
untriaged-bot | https://bugs.launchpad.net/designate/+bug/1416337 | 11:02 |
openstack | Launchpad bug 1416337 in Designate "Designate server create with concurrent request is not listing all servers even after successful creation." [Undecided,Incomplete] | 11:02 |
*** untriaged-bot has quit IRC | 11:02 | |
openstackgerrit | Kiall Mac Innes proposed openstack/designate: Retry transactions on database deadlocks https://review.openstack.org/134524 | 11:14 |
*** chlong has joined #openstack-dns | 11:57 | |
openstackgerrit | Kiall Mac Innes proposed openstack/designate: Retry transactions on database deadlocks https://review.openstack.org/134524 | 11:57 |
Kiall | rjrjr: no, it's not.. if mdns isn't running RMQ will simply drop that message and we move on with life.. There probably should have been a config option to totally disable it though | 11:58 |
Kiall | elarson: nope, I didn't have anything specific in mind when I added that to the docs etherpad, mostly just "we should have a Designate REST API 101" page ;) | 11:59 |
openstackgerrit | Kiall Mac Innes proposed openstack/designate: Retry transactions on database deadlocks https://review.openstack.org/134524 | 12:01 |
openstackgerrit | Kiall Mac Innes proposed openstack/designate: Set PYTHONDONTWRITEBYTECODE=1 in tox.ini https://review.openstack.org/158669 | 12:36 |
openstackgerrit | Kiall Mac Innes proposed openstack/designate: Correct PoolAttribute module name https://review.openstack.org/158670 | 12:36 |
*** chlong has quit IRC | 12:44 | |
*** mwagner_ has quit IRC | 12:58 | |
*** mwagner_lap has quit IRC | 12:59 | |
*** lpmulligan has joined #openstack-dns | 13:14 | |
openstackgerrit | Kiall Mac Innes proposed openstack/designate: Retry transactions on database deadlocks https://review.openstack.org/134524 | 13:19 |
openstackgerrit | Graham Hayes proposed openstack/designate: Deprecate designate-sink non managed mode. https://review.openstack.org/158690 | 13:40 |
openstackgerrit | Graham Hayes proposed openstack/designate: Deprecate designate-sink non managed mode https://review.openstack.org/158690 | 13:41 |
*** mwagner_lap has joined #openstack-dns | 13:47 | |
*** mwagner_ has joined #openstack-dns | 13:47 | |
*** csoukup has joined #openstack-dns | 13:50 | |
openstackgerrit | Graham Hayes proposed openstack/designate: Fixed _is_superdomain check in central https://review.openstack.org/158693 | 13:55 |
*** lpmulligan has quit IRC | 13:59 | |
*** mwagner_ has quit IRC | 14:00 | |
*** richm has joined #openstack-dns | 14:00 | |
*** richm1 has joined #openstack-dns | 14:06 | |
*** lpmulligan has joined #openstack-dns | 14:08 | |
*** richm has quit IRC | 14:08 | |
*** nkinder has quit IRC | 14:18 | |
elarson | Kiall: thanks for the clarification. I added a small bit to the beginning of the REST docs I'll submit a patch for and we can see how far off I am ;) | 14:23 |
Kiall | :) | 14:23 |
openstackgerrit | Eric Larson proposed openstack/designate: A small intro to the REST API https://review.openstack.org/158702 | 14:23 |
Kiall | rjrjr: about? Could you try this for me, worried about the performance hit (if any?) and I know your dev VM is among the slowest test times :) | 14:24 |
Kiall | https://review.openstack.org/#/c/158669/ | 14:24 |
Kiall | can you time `tox -epy27` for the commit before that, and that commit after with this before each: | 14:24 |
Kiall | find designate -name *.pyc -exec rm {} \; | 14:24 |
Kiall | I don't notice a difference on my PC, but the tests take seconds for me! | 14:25 |
elarson | do you all have a trick to getting to all the designate reviews in gerrit other than keeping a bookmark of the search page? | 14:31 |
elarson | ie https://review.openstack.org/#/q/status:open+project:openstack/designate,n,z | 14:32 |
Kiall | IRC is probably going to kill these link, but I use two bookmarks: | 14:34 |
Kiall | https://review.openstack. | 14:34 |
Kiall | org/#/dashboard/?foreach=%28project%3Aopenstack%2Fdesignate+OR+project%3Aopenstack%2Fpython%2Ddesignateclient+OR+project%3Aopenstack%2Fdesignate%2Dspecs%29+status%3Aopen+NOT+owner%3Aself+NOT+label%3AWorkflow%3C%3D%2D1+label%3AVerified%3E%3D1%252cjenkins+NOT+label%3ACode%2DReview%3E%3D%2D2%252cself+branch%3Amaster&title=Designate+Review+Inbox+%28master+branch+only%29&Needs+Feedback+%28Changes+older+than+5+days+that+have+not+been+reviewed+by+any | 14:34 |
Kiall | one%29=NOT+label%3ACode%2DReview%3C%3D2+age%3A5d&Your+are+a+reviewer%2C+but+haven%27t+voted+in+the+current+revision=reviewer%3Aself&Needs+final+%2B2=label%3ACode%2DReview%3E%3D2+limit%3A50&Passed+Jenkins%2C+No+Negative+Feedback=NOT+label%3ACode%2DReview%3E%3D2+NOT+label%3ACode%2DReview%3C%3D%2D1+limit%3A50&Wayward+Changes+%28Changes+with+no+code+review+in+the+last+2days%29=NOT+label%3ACode%2DReview%3C%3D2+age%3A2d&Specs+in+need+of+review=proje | 14:34 |
Kiall | ct%3Aopenstack%2Fdesignate%2Dspecs | 14:34 |
Kiall | and | 14:34 |
Kiall | https://review.openstack.org/#/q/status:open+(project:openstack/designate+OR+project:openstack/python-designateclient+OR+project:openstack/designate-specs),n,z | 14:34 |
ekarlso | elarson: wiki.openstack.org/designate | 14:34 |
ekarlso | there's a link there :) | 14:34 |
elarson | ekarlso: awesome, thanks! | 14:34 |
Kiall | Ah - That link causes a memory leak in Chrome (long story!) let me update it | 14:34 |
ekarlso | :p | 14:35 |
* elarson is sure emacs will be OK with it ;) | 14:35 | |
Kiall | Oh, looks like the link on the wiki has already been updated ;) | 14:36 |
Kiall | mugsie: do you remember if you updated that link before or after the chrome-crashing fix landed? ;) | 14:36 |
Kiall | elarson: theres also https://github.com/stackforge/gertty | 14:37 |
Kiall | CLI frontend to Gerrit | 14:37 |
mugsie | Kiall: not sure | 14:37 |
elarson | Kiall: thanks! | 14:37 |
rjrjr | kiall: thanks for the response. | 14:37 |
Kiall | np... | 14:38 |
rjrjr | give me a second to time the tox runs. | 14:52 |
*** nkinder has joined #openstack-dns | 14:52 | |
*** vinod1 has joined #openstack-dns | 14:55 | |
ekarlso | vinod1: how's your split going ? | 15:00 |
mugsie | normal - tox -e py27 142.45s user 7.57s system 352% cpu 42.588 total | 15:00 |
mugsie | with the PYTHONDONTWRITEBYTECODE set - tox -e py27 142.27s user 7.19s system 353% cpu 42.295 total | 15:00 |
vinod1 | I will upload another patchset today. After that the last one should be remaining - hope to get it done today | 15:00 |
*** paul_glass has joined #openstack-dns | 15:09 | |
rjrjr | @kiall: master - real 3m7.856s user 2m8.764s sys 1m10.516s | 15:14 |
rjrjr | patch - real 2m48.002s user 2m9.608s sys 1m7.112s | 15:14 |
Kiall | Wait - Faster? | 15:15 |
Kiall | -_- | 15:15 |
openstackgerrit | Endre Karlson proposed openstack/designate: Support secondary zones https://review.openstack.org/133682 | 15:15 |
rjrjr | one sample. i'd have to run it several times to be sure. | 15:15 |
Kiall | lol .. Maybe it's faster because it's saving all the disk I/O of writing out all the .pyc files? | 15:15 |
openstackgerrit | Endre Karlson proposed openstack/designate: MDNS part of Secondary zones https://review.openstack.org/133683 | 15:15 |
Kiall | Either way - It doesn't appear to have added any time, which was what I was hoping for :) | 15:16 |
rjrjr | agreed. | 15:17 |
openstackgerrit | Kiall Mac Innes proposed openstack/designate: Implement an Object.from_dict() method https://review.openstack.org/158725 | 15:18 |
Kiall | I have a meeting to run to now, but some thoughts on ^ would be good.. Aint done yet though. | 15:18 |
Kiall | Hopefully ^ will make the pools config API easier :) | 15:18 |
openstackgerrit | Tim Simmons proposed openstack/designate: Agent configuration and BIND9 improvements https://review.openstack.org/158462 | 15:41 |
*** csoukup has quit IRC | 15:43 | |
openstackgerrit | Merged openstack/designate: Set PYTHONDONTWRITEBYTECODE=1 in tox.ini https://review.openstack.org/158669 | 16:00 |
openstackgerrit | Merged openstack/designate: Correct PoolAttribute module name https://review.openstack.org/158670 | 16:00 |
*** csoukup has joined #openstack-dns | 16:11 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/designate: Updated from global requirements https://review.openstack.org/158747 | 16:13 |
openstackgerrit | Vinod Mangalpally proposed openstack/designate: Modifications for Resilient Pool Manager Cache https://review.openstack.org/158754 | 16:16 |
openstackgerrit | Merged openstack/designate: Deprecate designate-sink non managed mode https://review.openstack.org/158690 | 16:18 |
*** richm1 is now known as richm | 16:30 | |
*** jroll has joined #openstack-dns | 16:36 | |
jroll | hey y'all, I'm trying to find out where designate.api.v1.app:setup_app is called from (e.g. how the pecan config gets passed in there) | 16:37 |
jroll | is that a wsgi thing or? | 16:37 |
timsim | jroll: You probably mean v2.app right? | 16:41 |
jroll | timsim: yes, sorry | 16:41 |
timsim | Good question. | 16:46 |
* elarson suspects the app is returned and run with whatever wsgi server we use for the API but that isn't very obvious | 16:48 | |
mugsie | its an olso.wsgi server | 16:48 |
mugsie | which is depricated | 16:48 |
mugsie | but we are stil using it | 16:48 |
elarson | mugsie: ah ok, thanks! | 16:49 |
timsim | https://github.com/openstack/designate/blob/master/etc/designate/api-paste.ini#L24 | 16:49 |
elarson | mugsie: out of curiosity, what would you use instead? | 16:50 |
timsim | Which probably makes something happen in here: https://github.com/openstack/designate/blob/master/designate/api/v2/__init__.py best guess lol | 16:51 |
* elarson has found gunicorn and uwsgi to a little painful when trying to include a server with a single executable script | 16:51 | |
mugsie | elarson: not sure, but not a depricated non updated one ;) | 16:53 |
elarson | mugsie: sound advice! | 16:54 |
openstackgerrit | Tim Simmons proposed openstack/designate: Agent configuration and BIND9 improvements https://review.openstack.org/158462 | 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/1416337 | 17:02 |
openstack | Launchpad bug 1416337 in Designate "Designate server create with concurrent request is not listing all servers even after successful creation." [Undecided,Incomplete] | 17:02 |
untriaged-bot | https://bugs.launchpad.net/designate/+bug/1425117 | 17:02 |
openstack | Launchpad bug 1425117 in Designate "Designate does not work with postgres" [High,New] | 17:02 |
*** untriaged-bot has quit IRC | 17:02 | |
*** rmoe has quit IRC | 17:03 | |
jroll | wow, I just saw all of this chatter, thanks :) | 17:06 |
*** rmoe has joined #openstack-dns | 17:21 | |
*** penick has joined #openstack-dns | 17:26 | |
*** penick has quit IRC | 17:27 | |
elarson | can anyone describe what the `attributes` are for a pool? | 17:35 |
timsim | elarson: In the future. A zone added to Designate could have a choice of which pool it goes to. That might be based on some killer features like Geo-IP, or maybe someone wants a Pool that only has US-based Nameserver presence. The attributes are meant to be consumed by a scheduler (yet to be written) that when a new zone comes in with some attributes attached to it, there can be a decision on which pool to put it in. | 17:36 |
mugsie | what timsim said | 17:37 |
elarson | great! | 17:37 |
elarson | thanks! | 17:37 |
timsim | There's some examples of what it might look like here: https://github.com/openstack/designate-specs/blob/master/specs/kilo/server-pools-api.rst | 17:38 |
mugsie | I was in the middle of typing it out when he beat me to it | 17:38 |
* elarson was using it for some pool docs, but will probably just stick to adding a nameserver (aka ns_record post Kiall's patch) | 17:39 | |
*** GonZo2K has joined #openstack-dns | 18:00 | |
*** GonZo2K has joined #openstack-dns | 18:00 | |
openstackgerrit | Kiall Mac Innes proposed openstack/designate: Implement an Object.from_dict() method https://review.openstack.org/158725 | 18:02 |
Kiall | You know that feeling when your code works, but you have no idea why -_- | 18:03 |
*** andrewbogott_afk is now known as andrewbogott | 18:06 | |
*** EricGonczer_ has joined #openstack-dns | 18:07 | |
*** EricGonczer_ has quit IRC | 18:25 | |
*** lpmulligan has quit IRC | 18:25 | |
*** jordanP has quit IRC | 18:29 | |
*** penick has joined #openstack-dns | 18:35 | |
*** lpmulligan has joined #openstack-dns | 18:36 | |
openstackgerrit | Kiall Mac Innes proposed openstack/designate: Implement an Object.from_dict() method https://review.openstack.org/158725 | 18:46 |
*** vinod1 has quit IRC | 18:53 | |
*** penick has quit IRC | 18:53 | |
*** penick_ has joined #openstack-dns | 18:53 | |
*** EricGonczer_ has joined #openstack-dns | 19:04 | |
*** EricGonc_ has joined #openstack-dns | 19:08 | |
openstackgerrit | Kiall Mac Innes proposed openstack/designate-specs: WIP: Add validation-cleanup spec https://review.openstack.org/130265 | 19:10 |
*** csoukup has quit IRC | 19:11 | |
*** EricGonczer_ has quit IRC | 19:11 | |
*** crc32 has joined #openstack-dns | 19:13 | |
openstackgerrit | Merged openstack/designate: Updated from global requirements https://review.openstack.org/158747 | 19:16 |
*** EricGonc_ has quit IRC | 19:20 | |
*** EricGonczer_ has joined #openstack-dns | 19:22 | |
*** EricGonc_ has joined #openstack-dns | 19:23 | |
*** penick_ has quit IRC | 19:24 | |
openstackgerrit | Kiall Mac Innes proposed openstack/designate: Implement an Object.from_dict() method https://review.openstack.org/158725 | 19:27 |
*** EricGonczer_ has quit IRC | 19:27 | |
*** EricGonc_ has quit IRC | 19:52 | |
*** EricGonczer_ has joined #openstack-dns | 19:57 | |
openstackgerrit | Eric Larson proposed openstack/designate: A small intro to the REST API https://review.openstack.org/158702 | 19:59 |
*** csoukup has joined #openstack-dns | 20:11 | |
openstackgerrit | Kiall Mac Innes proposed openstack/designate: Update tests to use Object.from_dict() method https://review.openstack.org/158850 | 20:11 |
*** vinod1 has joined #openstack-dns | 20:11 | |
Kiall | timsim / vinod1 - BTW, re https://review.openstack.org/#/c/158725/ and https://review.openstack.org/#/c/158850/ - these are about making the tests (and API) for the Pools DB Config stuff easier .. being able to take a JSON blob and build out the correct set of objects etc.. In case you're wondering where that review came from ;') | 20:21 |
timsim | Whenver you get into those objects I'm all confused. | 20:22 |
Kiall | lol | 20:23 |
vinod1 | Kiall: Are those still wip or are they ready for review? | 20:23 |
Kiall | they should be good to go now - tests all passing locally and pep8 issues fixed ;) | 20:23 |
Kiall | but - the second is going to conflict with the Retry on Deadlock timsim just approved .. oh well ;) | 20:24 |
Kiall | Been working through the first part of the Pools->DB stuff, and not having something like those made me want to tear my hair out .. lol | 20:25 |
timsim | So, once it passes schema we'll build out an Object that basically reflects the dictionary exactly right? And the way that you make sure that the nested things work is by having relation? | 20:25 |
openstackgerrit | Merged openstack/designate: Retry transactions on database deadlocks https://review.openstack.org/134524 | 20:26 |
Kiall | timsim: exactly :) We have some relation stuff in already which Endre did for .. something? I can't remember exactly what.. This builds on that to know "Oh, I should create another object at this spot" during the new from_dict() method.. | 20:27 |
Kiall | The goal was to kill code that looked like the left hand L452 of https://review.openstack.org/#/c/158850/1/designate/tests/__init__.py | 20:28 |
Kiall | which is pretty common in both the tests and the APIs | 20:28 |
timsim | Why is L179 if relation is false here: https://review.openstack.org/#/c/158725/4/designate/objects/base.py ? | 20:28 |
Kiall | instance.FIELDS[field].get('relation', False) <-- defaults to False if not set.. so we skip over stuff that relation is both undefined or explicitly set False | 20:29 |
timsim | Oh durr ok. | 20:30 |
Kiall | (that code also made me .. unhappy.. | 20:30 |
Kiall | but, it's all internal in objects so it didn't make me unhappy enough to fix today ;)) | 20:30 |
*** andrewbogott is now known as andrewbogott_afk | 20:36 | |
openstackgerrit | Eric Larson proposed openstack/designate: A small intro to the REST API https://review.openstack.org/158702 | 21:11 |
*** lpmulligan has quit IRC | 21:15 | |
*** EricGonczer_ has quit IRC | 21:16 | |
*** EricGonczer_ has joined #openstack-dns | 21:21 | |
*** EricGonczer_ has quit IRC | 21:22 | |
*** lpmulligan has joined #openstack-dns | 21:35 | |
*** andrewbogott_afk is now known as andrewbogott | 21:39 | |
*** EricGonczer_ has joined #openstack-dns | 21:50 | |
*** mwagner_lap has quit IRC | 22:02 | |
*** c_soukup has joined #openstack-dns | 22:02 | |
*** csoukup has quit IRC | 22:04 | |
elarson | ekarlso: I just added some comments and a +0 on https://review.openstack.org/#/c/133675/ it is my first review of others code, so please feel free to give me any suggestions on what I could do better! | 22:11 |
ekarlso | elarson: :P | 22:12 |
*** GonZoPT has joined #openstack-dns | 22:16 | |
*** GonZo2K has quit IRC | 22:17 | |
openstackgerrit | Merged openstack/designate: A small intro to the REST API https://review.openstack.org/158702 | 22:28 |
*** EricGonczer_ has quit IRC | 22:32 | |
*** openstackgerrit has quit IRC | 22:37 | |
*** openstackgerrit has joined #openstack-dns | 22:37 | |
*** ChanServ sets mode: +v openstackgerrit | 22:37 | |
openstackgerrit | Eric Larson proposed openstack/designate: Add howto for creating PTR records https://review.openstack.org/157995 | 22:37 |
*** nkinder has quit IRC | 22:39 | |
*** c_soukup has quit IRC | 22:55 | |
*** paul_glass has quit IRC | 23:02 | |
*** andrewbogott is now known as andrewbogott_afk | 23:14 | |
*** chlong has joined #openstack-dns | 23:19 | |
*** lpmulligan has quit IRC | 23:20 | |
*** lpmulligan has joined #openstack-dns | 23:20 | |
*** andrewbogott_afk is now known as andrewbogott | 23:43 | |
*** chlong_ has joined #openstack-dns | 23:50 | |
*** lpmulligan has quit IRC | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!