Tuesday, 2015-02-24

*** nkinder has quit IRC00:05
*** rjrjr has joined #openstack-dns00:06
*** penick has joined #openstack-dns00:06
*** lpmulligan has quit IRC00:07
*** EricGonczer_ has joined #openstack-dns00:22
*** EricGonczer_ has quit IRC00:46
*** penick has quit IRC01:07
*** GonZo2K has quit IRC01:07
*** rmoe has quit IRC01:10
*** rmoe has joined #openstack-dns01:25
*** johnbelamaric has quit IRC01:33
*** nkinder has joined #openstack-dns01:55
*** EricGonczer_ has joined #openstack-dns01:59
*** EricGonczer_ has quit IRC02:39
*** EricGonczer_ has joined #openstack-dns02:43
*** EricGonczer_ has quit IRC02:48
*** EricGonczer_ has joined #openstack-dns03:23
*** EricGonc_ has joined #openstack-dns03:25
*** EricGonczer_ has quit IRC03:28
*** EricGonc_ has quit IRC03:31
*** EricGonczer_ has joined #openstack-dns03:43
*** EricGonczer_ has quit IRC03:47
*** andrewbogott is now known as andrewbogott_afk03:59
*** richm has quit IRC04:28
*** csoukup has joined #openstack-dns04:32
rjrjrquick 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-dns05:09
*** csoukup_ has joined #openstack-dns05:10
*** csoukup has quit IRC05:13
*** c_soukup has quit IRC05:14
*** csoukup_ has quit IRC05:32
*** andrewbogott_afk is now known as andrewbogott06:35
*** chlong has quit IRC07:34
*** jordanP has joined #openstack-dns09:00
*** andrewbogott is now known as andrewbogott_afk09:01
*** openstackgerrit has quit IRC09:36
*** openstackgerrit has joined #openstack-dns09:36
*** ChanServ sets mode: +v openstackgerrit09:36
*** untriaged-bot has joined #openstack-dns11:02
untriaged-botUntriaged bugs so far:11:02
untriaged-bothttps://bugs.launchpad.net/designate/+bug/141633711:02
openstackLaunchpad 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 IRC11:02
openstackgerritKiall Mac Innes proposed openstack/designate: Retry transactions on database deadlocks  https://review.openstack.org/13452411:14
*** chlong has joined #openstack-dns11:57
openstackgerritKiall Mac Innes proposed openstack/designate: Retry transactions on database deadlocks  https://review.openstack.org/13452411:57
Kiallrjrjr: 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 though11:58
Kiallelarson: 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
openstackgerritKiall Mac Innes proposed openstack/designate: Retry transactions on database deadlocks  https://review.openstack.org/13452412:01
openstackgerritKiall Mac Innes proposed openstack/designate: Set PYTHONDONTWRITEBYTECODE=1 in tox.ini  https://review.openstack.org/15866912:36
openstackgerritKiall Mac Innes proposed openstack/designate: Correct PoolAttribute module name  https://review.openstack.org/15867012:36
*** chlong has quit IRC12:44
*** mwagner_ has quit IRC12:58
*** mwagner_lap has quit IRC12:59
*** lpmulligan has joined #openstack-dns13:14
openstackgerritKiall Mac Innes proposed openstack/designate: Retry transactions on database deadlocks  https://review.openstack.org/13452413:19
openstackgerritGraham Hayes proposed openstack/designate: Deprecate designate-sink non managed mode.  https://review.openstack.org/15869013:40
openstackgerritGraham Hayes proposed openstack/designate: Deprecate designate-sink non managed mode  https://review.openstack.org/15869013:41
*** mwagner_lap has joined #openstack-dns13:47
*** mwagner_ has joined #openstack-dns13:47
*** csoukup has joined #openstack-dns13:50
openstackgerritGraham Hayes proposed openstack/designate: Fixed _is_superdomain check in central  https://review.openstack.org/15869313:55
*** lpmulligan has quit IRC13:59
*** mwagner_ has quit IRC14:00
*** richm has joined #openstack-dns14:00
*** richm1 has joined #openstack-dns14:06
*** lpmulligan has joined #openstack-dns14:08
*** richm has quit IRC14:08
*** nkinder has quit IRC14:18
elarsonKiall: 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
openstackgerritEric Larson proposed openstack/designate: A small intro to the REST API  https://review.openstack.org/15870214:23
Kiallrjrjr: 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
Kiallhttps://review.openstack.org/#/c/158669/14:24
Kiallcan you time `tox -epy27` for the commit before that, and that commit after with this before each:14:24
Kiallfind designate -name *.pyc -exec rm {} \;14:24
KiallI don't notice a difference on my PC, but the tests take seconds for me!14:25
elarsondo 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
elarsonie https://review.openstack.org/#/q/status:open+project:openstack/designate,n,z14:32
KiallIRC is probably going to kill these link, but I use two bookmarks:14:34
Kiallhttps://review.openstack.14:34
Kiallorg/#/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+any14:34
Kiallone%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=proje14:34
Kiallct%3Aopenstack%2Fdesignate%2Dspecs14:34
Kialland14:34
Kiallhttps://review.openstack.org/#/q/status:open+(project:openstack/designate+OR+project:openstack/python-designateclient+OR+project:openstack/designate-specs),n,z14:34
ekarlsoelarson: wiki.openstack.org/designate14:34
ekarlsothere's a link there :)14:34
elarsonekarlso: awesome, thanks!14:34
KiallAh -  That link causes a memory leak in Chrome (long story!) let me update it14:34
ekarlso:p14:35
* elarson is sure emacs will be OK with it ;)14:35
KiallOh, looks like the link on the wiki has already been updated ;)14:36
Kiallmugsie: do you remember if you updated that link before or after the chrome-crashing fix landed? ;)14:36
Kiallelarson: theres also https://github.com/stackforge/gertty14:37
KiallCLI frontend to Gerrit14:37
mugsieKiall: not sure14:37
elarsonKiall: thanks!14:37
rjrjrkiall: thanks for the response.14:37
Kiallnp...14:38
rjrjrgive me a second to time the tox runs.14:52
*** nkinder has joined #openstack-dns14:52
*** vinod1 has joined #openstack-dns14:55
ekarlsovinod1: how's your split going ?15:00
mugsienormal - tox -e py27  142.45s user 7.57s system 352% cpu 42.588 total15:00
mugsiewith the PYTHONDONTWRITEBYTECODE set - tox -e py27  142.27s user 7.19s system 353% cpu 42.295 total15:00
vinod1I will upload another patchset today. After that the last one should be remaining - hope to get it done today15:00
*** paul_glass has joined #openstack-dns15:09
rjrjr@kiall: master -   real 3m7.856s   user 2m8.764s   sys 1m10.516s15:14
rjrjrpatch -    real 2m48.002s    user 2m9.608s   sys 1m7.112s15:14
KiallWait - Faster?15:15
Kiall-_-15:15
openstackgerritEndre Karlson proposed openstack/designate: Support secondary zones  https://review.openstack.org/13368215:15
rjrjrone sample.  i'd have to run it several times to be sure.15:15
Kialllol .. Maybe it's faster because it's saving all the disk I/O of writing out all the .pyc files?15:15
openstackgerritEndre Karlson proposed openstack/designate: MDNS part of Secondary zones  https://review.openstack.org/13368315:15
KiallEither way - It doesn't appear to have added any time, which was what I was hoping for :)15:16
rjrjragreed.15:17
openstackgerritKiall Mac Innes proposed openstack/designate: Implement an Object.from_dict() method  https://review.openstack.org/15872515:18
Kiall I have a meeting to run to now, but some thoughts on ^ would be good.. Aint done yet though.15:18
KiallHopefully ^ will make the pools config API easier :)15:18
openstackgerritTim Simmons proposed openstack/designate: Agent configuration and BIND9 improvements  https://review.openstack.org/15846215:41
*** csoukup has quit IRC15:43
openstackgerritMerged openstack/designate: Set PYTHONDONTWRITEBYTECODE=1 in tox.ini  https://review.openstack.org/15866916:00
openstackgerritMerged openstack/designate: Correct PoolAttribute module name  https://review.openstack.org/15867016:00
*** csoukup has joined #openstack-dns16:11
openstackgerritOpenStack Proposal Bot proposed openstack/designate: Updated from global requirements  https://review.openstack.org/15874716:13
openstackgerritVinod Mangalpally proposed openstack/designate: Modifications for Resilient Pool Manager Cache  https://review.openstack.org/15875416:16
openstackgerritMerged openstack/designate: Deprecate designate-sink non managed mode  https://review.openstack.org/15869016:18
*** richm1 is now known as richm16:30
*** jroll has joined #openstack-dns16:36
jrollhey 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
jrollis that a wsgi thing or?16:37
timsimjroll: You probably mean v2.app right?16:41
jrolltimsim: yes, sorry16:41
timsimGood 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 obvious16:48
mugsieits an olso.wsgi server16:48
mugsiewhich is depricated16:48
mugsiebut we are stil using it16:48
elarsonmugsie: ah ok, thanks!16:49
timsimhttps://github.com/openstack/designate/blob/master/etc/designate/api-paste.ini#L2416:49
elarsonmugsie: out of curiosity, what would you use instead?16:50
timsimWhich probably makes something happen in here: https://github.com/openstack/designate/blob/master/designate/api/v2/__init__.py best guess lol16:51
* elarson has found gunicorn and uwsgi to a little painful when trying to include a server with a single executable script16:51
mugsieelarson: not sure, but not a depricated non updated one ;)16:53
elarsonmugsie: sound advice!16:54
openstackgerritTim Simmons proposed openstack/designate: Agent configuration and BIND9 improvements  https://review.openstack.org/15846216:59
*** untriaged-bot has joined #openstack-dns17:02
untriaged-botUntriaged bugs so far:17:02
untriaged-bothttps://bugs.launchpad.net/designate/+bug/141633717:02
openstackLaunchpad bug 1416337 in Designate "Designate server create with concurrent request is not listing all servers even after successful creation." [Undecided,Incomplete]17:02
untriaged-bothttps://bugs.launchpad.net/designate/+bug/142511717:02
openstackLaunchpad bug 1425117 in Designate "Designate does not work with postgres" [High,New]17:02
*** untriaged-bot has quit IRC17:02
*** rmoe has quit IRC17:03
jrollwow, I just saw all of this chatter, thanks :)17:06
*** rmoe has joined #openstack-dns17:21
*** penick has joined #openstack-dns17:26
*** penick has quit IRC17:27
elarsoncan anyone describe what the `attributes` are for a pool?17:35
timsimelarson: 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
mugsiewhat timsim said17:37
elarsongreat!17:37
elarsonthanks!17:37
timsimThere's some examples of what it might look like here: https://github.com/openstack/designate-specs/blob/master/specs/kilo/server-pools-api.rst17:38
mugsieI was in the middle of typing it out when he beat me to it17: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-dns18:00
*** GonZo2K has joined #openstack-dns18:00
openstackgerritKiall Mac Innes proposed openstack/designate: Implement an Object.from_dict() method  https://review.openstack.org/15872518:02
KiallYou know that feeling when your code works, but you have no idea why -_-18:03
*** andrewbogott_afk is now known as andrewbogott18:06
*** EricGonczer_ has joined #openstack-dns18:07
*** EricGonczer_ has quit IRC18:25
*** lpmulligan has quit IRC18:25
*** jordanP has quit IRC18:29
*** penick has joined #openstack-dns18:35
*** lpmulligan has joined #openstack-dns18:36
openstackgerritKiall Mac Innes proposed openstack/designate: Implement an Object.from_dict() method  https://review.openstack.org/15872518:46
*** vinod1 has quit IRC18:53
*** penick has quit IRC18:53
*** penick_ has joined #openstack-dns18:53
*** EricGonczer_ has joined #openstack-dns19:04
*** EricGonc_ has joined #openstack-dns19:08
openstackgerritKiall Mac Innes proposed openstack/designate-specs: WIP: Add validation-cleanup spec  https://review.openstack.org/13026519:10
*** csoukup has quit IRC19:11
*** EricGonczer_ has quit IRC19:11
*** crc32 has joined #openstack-dns19:13
openstackgerritMerged openstack/designate: Updated from global requirements  https://review.openstack.org/15874719:16
*** EricGonc_ has quit IRC19:20
*** EricGonczer_ has joined #openstack-dns19:22
*** EricGonc_ has joined #openstack-dns19:23
*** penick_ has quit IRC19:24
openstackgerritKiall Mac Innes proposed openstack/designate: Implement an Object.from_dict() method  https://review.openstack.org/15872519:27
*** EricGonczer_ has quit IRC19:27
*** EricGonc_ has quit IRC19:52
*** EricGonczer_ has joined #openstack-dns19:57
openstackgerritEric Larson proposed openstack/designate: A small intro to the REST API  https://review.openstack.org/15870219:59
*** csoukup has joined #openstack-dns20:11
openstackgerritKiall Mac Innes proposed openstack/designate: Update tests to use Object.from_dict() method  https://review.openstack.org/15885020:11
*** vinod1 has joined #openstack-dns20:11
Kialltimsim / 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
timsimWhenver you get into those objects I'm all confused.20:22
Kialllol20:23
vinod1Kiall: Are those still wip or are they ready for review?20:23
Kiallthey should be good to go now - tests all passing locally and pep8 issues fixed ;)20:23
Kiallbut - the second is going to conflict with the Retry on Deadlock timsim just approved .. oh well ;)20:24
KiallBeen working through the first part of the Pools->DB stuff, and not having something like those made me want to tear my hair out .. lol20:25
timsimSo, 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
openstackgerritMerged openstack/designate: Retry transactions on database deadlocks  https://review.openstack.org/13452420:26
Kialltimsim: 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
KiallThe goal was to kill code that looked like the left hand L452 of https://review.openstack.org/#/c/158850/1/designate/tests/__init__.py20:28
Kiallwhich is pretty common in both the tests and the APIs20:28
timsimWhy is L179 if relation is false here: https://review.openstack.org/#/c/158725/4/designate/objects/base.py ?20:28
Kiallinstance.FIELDS[field].get('relation', False) <-- defaults to False if not set.. so we skip over stuff that relation is both undefined or explicitly set False20:29
timsimOh durr ok.20:30
Kiall(that code also made me .. unhappy..20:30
Kiallbut, 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_afk20:36
openstackgerritEric Larson proposed openstack/designate: A small intro to the REST API  https://review.openstack.org/15870221:11
*** lpmulligan has quit IRC21:15
*** EricGonczer_ has quit IRC21:16
*** EricGonczer_ has joined #openstack-dns21:21
*** EricGonczer_ has quit IRC21:22
*** lpmulligan has joined #openstack-dns21:35
*** andrewbogott_afk is now known as andrewbogott21:39
*** EricGonczer_ has joined #openstack-dns21:50
*** mwagner_lap has quit IRC22:02
*** c_soukup has joined #openstack-dns22:02
*** csoukup has quit IRC22:04
elarsonekarlso: 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
ekarlsoelarson: :P22:12
*** GonZoPT has joined #openstack-dns22:16
*** GonZo2K has quit IRC22:17
openstackgerritMerged openstack/designate: A small intro to the REST API  https://review.openstack.org/15870222:28
*** EricGonczer_ has quit IRC22:32
*** openstackgerrit has quit IRC22:37
*** openstackgerrit has joined #openstack-dns22:37
*** ChanServ sets mode: +v openstackgerrit22:37
openstackgerritEric Larson proposed openstack/designate: Add howto for creating PTR records  https://review.openstack.org/15799522:37
*** nkinder has quit IRC22:39
*** c_soukup has quit IRC22:55
*** paul_glass has quit IRC23:02
*** andrewbogott is now known as andrewbogott_afk23:14
*** chlong has joined #openstack-dns23:19
*** lpmulligan has quit IRC23:20
*** lpmulligan has joined #openstack-dns23:20
*** andrewbogott_afk is now known as andrewbogott23:43
*** chlong_ has joined #openstack-dns23:50
*** lpmulligan has quit IRC23:58

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