Friday, 2015-03-13

*** mwagner_lap has quit IRC00:53
*** tundrajimmy has quit IRC00:57
*** bandarji has quit IRC01:09
*** MentalRay has joined #openstack-dns01:12
*** stanzgy has joined #openstack-dns01:30
*** stanzgy_ has joined #openstack-dns01:31
*** rmoe has quit IRC01:34
*** rmoe has joined #openstack-dns01:51
*** EricGonczer_ has joined #openstack-dns02:41
*** EricGonczer_ has quit IRC02:42
*** richm has quit IRC03:04
*** MentalRay has quit IRC03:19
*** EricGonczer_ has joined #openstack-dns03:33
*** EricGonczer_ has quit IRC03:34
*** EricGonczer_ has joined #openstack-dns03:34
*** EricGonczer_ has quit IRC03:48
openstackgerritOpenStack Proposal Bot proposed openstack/designate: Imported Translations from Transifex  https://review.openstack.org/16408406:09
*** chlong has quit IRC08:09
*** openstackgerrit has quit IRC08:21
*** openstackgerrit has joined #openstack-dns08:21
*** ChanServ sets mode: +v openstackgerrit08:21
*** jordanP has joined #openstack-dns09:16
*** stanzgy has quit IRC10:31
*** stanzgy_ has quit IRC10:31
*** untriaged-bot has joined #openstack-dns11:02
untriaged-botUntriaged bugs so far:11:02
untriaged-bothttps://bugs.launchpad.net/designate/+bug/142566811:02
openstackLaunchpad bug 1425668 in Designate "Poor error message when using same database for designate and the pool manager cache" [Undecided,New]11:02
untriaged-bothttps://bugs.launchpad.net/designate/+bug/142511711:02
openstackLaunchpad bug 1425117 in Designate "Designate does not work with postgres" [High,New]11:02
untriaged-bothttps://bugs.launchpad.net/designate/+bug/143097611:02
openstackLaunchpad bug 1430976 in Designate "Pool Manager creates/deletes can go into an infinite loop" [Undecided,In progress] - Assigned to Tim Simmons (tim-simmons-t)11:02
*** untriaged-bot has quit IRC11:02
*** puck has quit IRC11:12
*** puck has joined #openstack-dns11:15
*** EricGonczer_ has joined #openstack-dns11:21
openstackgerritEndre Karlson proposed openstack/designate: MDNS part of Secondary zones  https://review.openstack.org/13368311:38
openstackgerritEndre Karlson proposed openstack/designate: Support secondary zones  https://review.openstack.org/13368211:38
openstackgerritEndre Karlson proposed openstack/designate: MDNS part of Secondary zones  https://review.openstack.org/13368311:45
ekarlsoelarson: you around ? p11:45
openstackgerritEndre Karlson proposed openstack/designate: Support secondary zones  https://review.openstack.org/13368211:45
*** mwagner_lap has joined #openstack-dns11:57
openstackgerritEndre Karlson proposed openstack/designate: MDNS part of Secondary zones  https://review.openstack.org/13368312:10
*** chlong has joined #openstack-dns12:25
openstackgerritKiall Mac Innes proposed openstack/designate: Implement TSIG Support in mDNS  https://review.openstack.org/16054312:42
Kiallelarson: while looking at your oslo.policy review, I spotted the "HttpCheck" .. I guess that's similar to what you suggested yesteray?12:52
openstackgerritMerged openstack/designate: Update localrc docs sample  https://review.openstack.org/16349912:57
openstackgerritKiall Mac Innes proposed openstack/designate: Enable Tempest Functional Tests  https://review.openstack.org/16314212:58
elarsonKiall: it is a little similar. I looked a bit more closely though and it seems the policy checker really is meant to be limited to verifying some auth / role has access. at least that is what the API seems intended for. what I needed to do is call some arbitrary code, more like a hook point.13:03
ekarlsoelarson: not sure this is above your needs, but checked out congress ?13:04
KiallYep, but the set of check classes are "dynamic" letting you hook in new checks into the policy engine on-the-fly13:04
Kiallre what there meant to do.. There generally meant to be simple "Can $user perform $task on $resource", the actual implementations of how you get a Yes/No answer to that is IMO pretty free-form13:05
KiallAlso - I -1'd your oslo.policy change..  sorry :P13:06
elarsonKiall: right, the thing that wasn't as dynamic were the args passed to the policy check. the signature was a little too static. that said, I'm still working on what could work, if my latest idea crashes and burns, I might give it another look13:06
elarsonekarlso: I haven't heard of congress. what is it?13:06
* elarson googles13:07
ekarlsohttp://github.com/stackforge/congress13:07
KiallYea, the signature is fixed.. but there's lots of scope for us to extend what we pass into the policy check as the target.. we mostly just pass the ID's in today13:07
elarsonekarlso: congress looks pretty interesting, but I doubt it is what I'm looking for in this case.13:09
elarsonfwiw, the reason I looked into oslo.policy was that 1) designate already uses it and 2) it seemed easy to include other plugins. the actual task is really to find ways to integrate with a legacy system, which could go beyond simply checking if a user can perform some action.13:10
KiallSo - Q - What is the legacy systems purpose?13:12
elarsonKiall: it is the legacy dns management system that talks to the same bind instances as designate13:13
KiallWould you guys ever fix your your apt-mirrors :P Pools API change failed with:13:13
Kiall2015-03-13 12:46:00.974 | Err http://mirror.rackspace.com/ubuntu/ trusty-security/main libxenstore3.0 amd64 4.4.1-0ubuntu0.14.04.413:13
Kiall2015-03-13 12:46:00.974 |   404  Not Found13:13
Kiall;)13:13
KiallSo, RAX's existing Cloud DNS system? I guess the Q should have been, what's the intent of integrating it into Designate?13:14
* elarson wishes he could fix that13:14
elarsonKiall: from what I can tell, it is necessary for migration from the system. so the goal is not to integrate as much as offering a hook to call some org specific code installed in another package.13:16
KiallOkay, there are other possible integration points.. You could have a proxy/shim storage/backend driver that calls out.. But it depends what the code is and if those points are suitable..13:17
KiallThe other possibility.. Fork.. I know we have an internal fork with some HP weirdness that simply doesn't belong upstream, this could be the same...13:18
elarsonalso, it doesn't have to be limited to calling legacy code. one example was it could also be used to post events to something like statsd13:18
elarsonKiall: my goal is to make something that doesn't add anything org specific and hopefully could help avoid having to keep internal forks13:20
elarsonbut we'll see!13:20
KiallYea, I guess without knowing exactly what your trying to do it's hard to make better suggestions :D13:20
elarsonKiall: and I'm still working out what have to do, which doesn't help me ask good questions ;)13:21
Kiall:D13:21
elarsonKiall: thanks for rechecking the Pool API docs13:22
*** richm has joined #openstack-dns13:41
*** richm1 has joined #openstack-dns13:48
*** mikehn_ has quit IRC13:48
*** richm has quit IRC13:49
*** mikehn has joined #openstack-dns13:49
*** richm1 is now known as richm13:49
openstackgerritGraham Hayes proposed openstack/designate: Added Schemas to DesignateObjects  https://review.openstack.org/16313113:58
*** vinod1 has joined #openstack-dns14:04
*** paul_glass has joined #openstack-dns14:05
*** EricGonczer_ has quit IRC14:12
*** EricGonczer_ has joined #openstack-dns14:14
*** csoukup has joined #openstack-dns14:16
*** MentalRay has joined #openstack-dns14:33
elarsonis this an error the gate will complain about? ./designate/policy.py:18:31: H301  one import per line14:35
vinod1you can run tox locally and the same errors that you see locally should be the ones that you see at the gate14:37
paul_glasselarson: tox -e pep8, i think14:38
vinod1elarson: This line shows which errors are ignored - https://github.com/openstack/designate/blob/master/tox.ini#L8114:38
Kiall`tox -epep8` will be identical to the gate pep8 job, unless you haven't comitted yet and end tyhe first line in a period.. heh14:38
elarson:)14:39
elarsonvinod1: thanks, that is what I was thinking of14:39
* elarson adds that to his local .flake814:40
openstackgerritTim Simmons proposed openstack/designate: Smarter Create/Delete in BIND9/Agent  https://review.openstack.org/16390114:44
Kialltimsim: lol - I'd hang you out to dry over that if there was a better way, but there isn't ;)14:46
timsimI agonized for like an hour on whether I would do that or not.14:46
KiallI have much worse code ;) http://paste.openstack.org/show/TybRabhPHA9k1yyXbWLt/14:49
timsimoof14:49
KiallYea... Out of necessity sadly..14:50
openstackgerritEric Larson proposed openstack/designate: Use the oslo.policy package  https://review.openstack.org/16313914:50
timsimI was hoping rndc would do something cool like throw a different response code, but nah. I requested that ISC add something like "rndc zonestatus" to tell you if a zone exists or not, and the serial, so maybe that'll happen in a few years.14:51
openstackgerritKiall Mac Innes proposed openstack/designate: Enable Tempest Functional Tests  https://review.openstack.org/16314214:51
*** andrewbogott_afk is now known as andrewbogott14:51
Kialltimsim: lol - yea, oh well!14:51
jbrattontimsim: I have a phone call with them on Monday to discuss, I'll let you know14:54
jbrattonit seems like a simple enough addition14:54
Kialljbratton: interesting.. Maybe they can give a definitive "this is how to check with 100% certainty"14:54
jbrattonyeah, I'll definitely ask14:55
timsimThanks jbratton, oh ubiquitous one.14:55
jbrattonhaha14:55
timsimmugsie: Are https://review.openstack.org/#/c/163131/2 and https://review.openstack.org/#/c/163461/2 the only ones of yours out there that need reviewing right now?15:01
mugsietimsim: yeah, they are the 2 main onhes15:02
mugsieones*15:02
timsimCool, it looked like the other two are pending the API change to remove { "zone" etc15:02
*** einarr has quit IRC15:08
openstackgerritEric Larson proposed openstack/designate: Use the oslo.policy package  https://review.openstack.org/16313915:08
mugsietimsim: yeah. not sure how I am going to order these so they keep working between each patch ;)15:09
Kiallelarson: lol.. I just spotted 1 more thing in your oslo.policy patch.. sorry ;)15:10
Kiallmodule=policy should be removed from openstack-common.conf15:10
elarsonno worries!15:10
* elarson fixes now15:11
KiallAlso.. elarson.. 1 sec ;)15:11
*** einarr has joined #openstack-dns15:11
elarsontake your time :)15:11
KiallCan you add `Implments: blueprint oslo-policy-migration` as a trailer on the commit message? ;)15:12
elarsonhappy to15:13
openstackgerritGraham Hayes proposed openstack/designate: Add Admin API and add extensions  https://review.openstack.org/16421315:14
openstackgerritEric Larson proposed openstack/designate: Use the oslo.policy package  https://review.openstack.org/16313915:14
mugsienot 100% important, but moved it out of a git stash on my machine15:15
mugsiewanted to get it in before we call /v2 stable15:15
elarsonspeaking of which...15:16
openstackgerritGraham Hayes proposed openstack/designate: Add Admin API and add extensions  https://review.openstack.org/16421315:16
elarsonKiall: did your v2 changes get merged?15:16
* elarson finds the review15:16
KiallNo, my weekend is currently blocked off to finally finish them15:16
elarsonKiall: cool, I'll keep an eye out an update the pool docs (unless you end up doing that as well).15:18
Kialllol - elarson someone in RAX is pulling network cables out from under your pools api docs change15:18
elarsondid the gate fail again?15:19
KiallYep :)15:19
elarson.blame timsim15:19
KiallI see you're a quick learner - Already picked on the fact that everything bad is always timsim's fault! Including all wars, ever.15:21
Kiallpicked up on*15:21
timsimMy bad homie15:22
elarsonmy hope is that timsim will be inspired to implement the snarky commands that don't exist yet in the irc bot :)15:23
timsimLol. I'm going to set that up so that you can do it elarson15:23
elarsonthat would be fanatically fantastic15:23
openstackgerritPaul Glass proposed openstack/designate: Enable Tempest Functional Tests  https://review.openstack.org/16314215:25
openstackgerritPaul Glass proposed openstack/designate: Add Tempest zone tests  https://review.openstack.org/16421815:25
paul_glassKiall: oops, I think..15:26
paul_glasswhat's the proper way to do a dependent commit?15:26
elarsonpaul_glass: you can check out the dev branch locally and then create a new branch based on the dev branch and git review will do the right thing15:27
* elarson can show you in emacs if that would help ;)15:27
paul_glassoh, I think I'm just dependent on an abandoned commit.15:27
* elarson will leave paul_glass to it then!15:27
paul_glassoh no. I'm just confused.15:28
paul_glasseverything's okay.15:28
openstackgerritMerged openstack/designate: Imported Translations from Transifex  https://review.openstack.org/16408415:34
*** rmoe has quit IRC15:35
openstackgerritMerged openstack/designate: Send the correct domain object to update_domain  https://review.openstack.org/16395215:36
Kiallpaul_glass: nope, you stole my review and uploaded an old version ;) Theif!15:38
Kiall(No worries, it happens.. I push the latest one back up)15:38
paul_glasssorry, it's been a while. I'm rereading the gerrit workflow thing.15:39
KiallIt's actually pretty hard to avoid, you were based on on old revision of my patchset, you would have needed to A) know there was a new one.. B) Fetch it, C) Rebase your change ontop of it..15:40
paul_glassso as long as I'm rebased with your latest patch-set, it does the right thing?15:41
KiallYep15:41
paul_glassokay, cool. I'll not be a theif next time...15:43
Kiall:D15:43
KiallAnnoying.. It won't let me put the latest one back up.. lol15:50
openstackgerritKiall Mac Innes proposed openstack/designate: Enable Tempest Functional Tests  https://review.openstack.org/16314215:50
Kiallrebased on the last changes that merged and it took ;)15:51
Kiallpaul_glass: BTW - I know your with rackspace, but not on their DNS team.. What team are you on?15:52
*** rmoe has joined #openstack-dns15:52
*** tundrajimmy has joined #openstack-dns15:52
Kiallpaul_glass: another trick to avoid the issue is to use `git review -R`, preventing it from auto rebasing, which prevents any changes to your dependant patches, which in turns means no stealing.. forgot about that flag ;)15:53
*** logan2 has joined #openstack-dns15:54
paul_glassKiall: I am on their dns team, but I'm a quality engineer so I sometimes work on other projects15:54
KiallAh, okay.. I thought I knew all the folks on their DNS team ;)15:54
elarsonall the quality engineers, like paul_glass, are off doing real work while the so-so guys hang out on IRC ;)15:56
elarson.zing15:56
*** vinod1 has quit IRC15:58
mugsieelarson: look what you did - you made vinod leave15:59
mugsie;)15:59
*** vinod1 has joined #openstack-dns15:59
elarson:)15:59
KiallEither that, or vinod and the RAX ubuntu mirrors are running over the same cable timsimkeeps unplugging16:01
Kialltimsim keeps unplugging*16:02
* elarson wonders if that is the wire under his desk he keeps kicking...16:03
KiallEasy way to check - cut it and see who comes running, if it's your NOC - thats it.. If it's building IT, it's not.16:04
openstackgerritPaul Glass proposed openstack/designate: Add Tempest zone tests  https://review.openstack.org/16421816:04
paul_glassthere we go.16:05
Kiall:)16:05
paul_glassI'm open to doing those clients/models for the tempest tests in a totally different style. I just wrote what came to mind.16:07
Kiallpaul_glass: I saw them and had no opnion ;) I'll poke around what other projects do before passing remark on them...16:09
paul_glassyeah, eventually we could insert validation in those models if we want.16:10
KiallYea, and port the other tempest tests at some point too :)16:21
openstackgerritGraham Hayes proposed openstack/designate: Added ValidationErrorMiddleware  https://review.openstack.org/16424116:29
openstackgerritGraham Hayes proposed openstack/designate: Added ValidationErrorMiddleware  https://review.openstack.org/16424116:32
*** vinod1 has quit IRC16:40
*** untriaged-bot has joined #openstack-dns17:02
untriaged-botUntriaged bugs so far:17:02
untriaged-bothttps://bugs.launchpad.net/designate/+bug/142566817:02
openstackLaunchpad bug 1425668 in Designate "Poor error message when using same database for designate and the pool manager cache" [Undecided,New]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-bothttps://bugs.launchpad.net/designate/+bug/143097617:02
openstackLaunchpad bug 1430976 in Designate "Pool Manager creates/deletes can go into an infinite loop" [Undecided,In progress] - Assigned to Tim Simmons (tim-simmons-t)17:02
*** untriaged-bot has quit IRC17:02
*** jordanP has quit IRC17:18
*** MentalRay has quit IRC17:55
*** bandarji has joined #openstack-dns18:00
*** EricGonczer_ has quit IRC18:14
*** EricGonczer_ has joined #openstack-dns18:14
openstackgerritMerged openstack/designate: Use the oslo.policy package  https://review.openstack.org/16313918:45
*** jordanP has joined #openstack-dns19:03
*** jordanP has quit IRC19:03
openstackgerritMerged openstack/designate: Initial Pool API docs  https://review.openstack.org/16001919:17
* ekarlso sees all other changes going in but his own :|19:18
timsimekarlso: one of them is still failing gate isn't it?19:19
elarsonekarlso: I thought you already made @property.setter in an earlier change here? https://review.openstack.org/#/c/133682/40/designate/objects/domain.py#5619:25
Kiallelarson: that didn't interact nicely with some other code :(19:26
elarsonKiall: ah ok19:26
elarsonKiall: btw, I might have roped myself into getting oslo.cache started which will end up deprecating the memcache stuff with a dogpile.cache driver instead.19:28
elarsonthat is what I get for looking at updating dependencies ;)19:28
KiallLOL.. Doug will be happy to get more help on oslo :)19:28
*** MentalRay has joined #openstack-dns19:47
openstackgerritEndre Karlson proposed openstack/designate: MDNS part of Secondary zones  https://review.openstack.org/13368319:50
openstackgerritGraham Hayes proposed openstack/designate: Move the DesignateAdapter to return non-wrapped objects  https://review.openstack.org/16432619:52
openstackgerritGraham Hayes proposed openstack/designate: Add API_v2 Adapter Objects  https://review.openstack.org/16432719:52
openstackgerritGraham Hayes proposed openstack/designate: Add API_v1 Adapter Objects  https://review.openstack.org/16432819:52
paul_glassokay, I'm seeing this on latest master (c6c72b1): Create a zone. Abandon that zone. Recreate that same zone. The zone eventually goes to ERROR and never goes to ACTIVE.19:53
paul_glassand this is with threshold_percentage = 30% and 3 nameservers.19:54
timsimWith Bind9, that's not going to work until you add my patch. It's going to rndc addzone something that's already there and fail every time.19:55
ekarlsook, can someone pleeease look at secondaries now ?19:55
ekarlsoor i'll cry to sunday :(19:55
ekarlsos/sunday/monday19:55
timsimekarlso: I think vinod was going to look at it today19:55
ekarlsotimsim: probably wont be merged today anyways :P19:56
timsimpaul_glass ^^^19:59
*** vinod1 has joined #openstack-dns20:00
ekarlsovinod1: you looking at secondary zones ? :D20:01
vinod1i am20:02
ekarlsovinod1: finding anything ?20:09
openstackgerritMerged openstack/designate: Smarter Create/Delete in BIND9/Agent  https://review.openstack.org/16390120:19
*** EricGonc_ has joined #openstack-dns20:21
*** EricGonczer_ has quit IRC20:22
vinod1ekarlso: In _create_domain_in_storage, you remove setting domain.serial. How is it set for PRIMARY zones?20:29
vinod1This is in central/service.py20:29
vinod1https://review.openstack.org/#/c/133682/40/designate/central/service.py20:29
ekarlsovinod1: tables.py     Column('serial', Integer, default=timeutils.utcnow_ts, nullable=False),20:37
vinod1ok - i see20:39
*** andrewbogott is now known as andrewbogott_afk20:40
*** MentalRay has quit IRC20:53
*** ByPasS has joined #openstack-dns20:56
openstackgerritPaul Glass proposed openstack/designate: Add Tempest zone tests  https://review.openstack.org/16421821:13
*** csoukup has quit IRC21:15
*** vinod1 has quit IRC21:16
*** MentalRay has joined #openstack-dns21:17
*** paul_glass has quit IRC21:39
*** andrewbogott_afk is now known as andrewbogott21:53
*** GonZo2000 has joined #openstack-dns21:55
*** GonZo2000 has joined #openstack-dns21:55
*** andrewbogott is now known as andrewbogott_afk22:04
*** GonZo2K has joined #openstack-dns22:04
*** GonZo2K has joined #openstack-dns22:04
*** andrewbogott_afk is now known as andrewbogott22:04
*** GonZo2K has quit IRC22:04
*** GonZo2000 has quit IRC22:06
*** GonZo2000 has joined #openstack-dns22:09
*** andrewbogott is now known as andrewbogott_afk22:10
*** andrewbogott_afk is now known as andrewbogott22:11
*** csoukup has joined #openstack-dns22:15
*** tundrajimmy has left #openstack-dns22:22
*** dvorak is now known as clayton22:28
*** clayton is now known as dvorak22:28
*** ByPasS has quit IRC22:45
*** MentalRay has quit IRC22:52
*** andrewbogott is now known as andrewbogott_afk23:02
*** bandarji has quit IRC23:08
*** EricGonc_ has quit IRC23:17
*** csoukup has quit IRC23:41

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