Tuesday, 2014-02-11

*** krow has quit IRC00:00
*** krow has joined #openstack-dns00:01
*** jorgem has quit IRC00:02
*** krow1 has joined #openstack-dns00:04
*** krow has quit IRC00:06
*** mwagner_lap has quit IRC00:15
*** jmcbride has quit IRC00:15
*** mwagner_lap has joined #openstack-dns00:15
*** mwagner_lap has quit IRC00:16
*** vinod has joined #openstack-dns00:23
*** CaptTofu has joined #openstack-dns00:39
*** richm has quit IRC00:52
*** vinod has quit IRC01:07
*** CaptTofu has quit IRC01:16
*** CaptTofu has joined #openstack-dns01:19
*** vinod has joined #openstack-dns01:19
*** vinod has quit IRC01:21
*** jmcbride has joined #openstack-dns01:24
*** ctracey is now known as ctracey|away01:37
*** ctracey|away has joined #openstack-dns01:43
*** vinod has joined #openstack-dns01:49
*** elemecca has quit IRC01:50
*** vinod has quit IRC01:50
*** krow1 has quit IRC01:53
*** jorgem has joined #openstack-dns01:55
*** jorgem1 has joined #openstack-dns01:55
*** jorgem has quit IRC01:55
*** jmcbride has quit IRC01:56
*** krow has joined #openstack-dns02:02
*** jorgem1 has quit IRC02:02
*** sn6i23a has quit IRC02:02
*** jmcbride has joined #openstack-dns02:14
*** krow has quit IRC02:17
*** krow has joined #openstack-dns02:20
*** jmcbride has quit IRC02:22
*** krow has quit IRC02:25
*** krow has joined #openstack-dns02:26
*** krow has quit IRC02:30
*** krow has joined #openstack-dns02:31
*** ctracey|away is now known as ctracey02:34
*** krow has quit IRC02:36
*** krow has joined #openstack-dns02:37
*** krow has quit IRC02:41
*** krow has joined #openstack-dns02:43
*** krow has quit IRC02:47
*** krow has joined #openstack-dns02:48
*** krow1 has joined #openstack-dns02:50
*** jmcbride has joined #openstack-dns02:53
*** krow has quit IRC02:53
*** krow1 has quit IRC02:55
*** krow has joined #openstack-dns02:55
*** mikal has quit IRC03:02
*** mikal has joined #openstack-dns03:03
*** CaptTofu has quit IRC03:06
*** vinod has joined #openstack-dns03:12
*** rossk has quit IRC03:27
*** vinod has quit IRC03:47
*** vinod has joined #openstack-dns03:48
*** krow has quit IRC03:58
*** krow has joined #openstack-dns04:01
*** krow has quit IRC04:02
*** krow has joined #openstack-dns04:03
*** krow has quit IRC04:34
*** jmcbride has quit IRC04:36
*** krow has joined #openstack-dns04:53
*** krow has quit IRC04:58
*** CaptTofu has joined #openstack-dns05:07
*** CaptTofu has quit IRC05:11
*** jmcbride has joined #openstack-dns05:29
*** jmcbride has quit IRC05:33
*** ctracey is now known as ctracey|away05:56
*** vinod has quit IRC06:09
*** CaptTofu has joined #openstack-dns07:07
*** CaptTofu has quit IRC07:12
*** zigo has joined #openstack-dns07:47
*** zoresvit has joined #openstack-dns08:21
*** zoresvit has quit IRC08:23
*** zoresvit has joined #openstack-dns08:23
*** CaptTofu has joined #openstack-dns09:08
*** CaptTofu has quit IRC09:13
harmwok vinod, lets see if I can doe something to get this working on havana then09:39
openstackgerritA change was merged to stackforge/designate: General Plugin Cleanups  https://review.openstack.org/7228409:44
*** krow has joined #openstack-dns10:10
*** zoresvit has quit IRC10:27
*** krow has quit IRC10:40
*** CaptTofu has joined #openstack-dns10:57
*** CaptTofu has quit IRC11:02
*** artom has quit IRC11:13
*** artom has joined #openstack-dns11:15
mugsieharmw: that requirement is part of the global openstack requirements - https://git.openstack.org/cgit/openstack/requirements/tree/global-requirements.txt12:18
*** jmcbride has joined #openstack-dns12:34
*** jmcbride has quit IRC12:35
harmwmugsie: thanks, but how would this fit into a havana deployment?12:52
harmwor should I just upgrade to icehouse (which isn't gold yet, right?)12:53
*** jmcbride has joined #openstack-dns12:56
mugsieharmw: that was the requirement for Openstack in Havana13:03
mugsie(all Openstack projects HAVE to keep the same requirements)13:04
mugsiei just checked the nova / havana branch - it is 2.4.8 there as well13:04
harmwhm, weird13:16
harmwlet me check what my setup says about that :)13:17
harmwah, perhaps I've built/compiled my designate rpm against the openstack version from epel (instead of rdo)13:20
harmwanyway, thanks for the pointers mugsie13:20
mugsieno problem13:26
*** artom has quit IRC13:26
*** CaptTofu has joined #openstack-dns13:44
*** jmcbride has quit IRC13:45
*** jmcbride has joined #openstack-dns13:45
*** eankutse has joined #openstack-dns14:03
*** jmcbride has quit IRC14:52
*** CaptTofu has quit IRC15:00
*** jmcbride has joined #openstack-dns15:02
*** jorgem has joined #openstack-dns15:07
*** richm has joined #openstack-dns15:11
*** eankutse has quit IRC15:14
*** jmcbride has quit IRC15:36
*** jmcbride has joined #openstack-dns15:39
*** vinod has joined #openstack-dns15:43
*** nkinder has joined #openstack-dns15:45
*** vinod has quit IRC15:47
harmwhm, im running openstack-nova-2013.2.1-1.el6.noarch but that uses python-kombu-1.1.3-1.el6.noarch15:56
*** eankutse has joined #openstack-dns16:08
*** eankutse has quit IRC16:08
*** eankutse has joined #openstack-dns16:08
mugsieharmw: epel packages may tweak the requirements - I am not massivly familiar with CentOS i am afraid16:09
*** eankutse1 has joined #openstack-dns16:15
*** CaptTofu has joined #openstack-dns16:16
*** eankutse has quit IRC16:16
harmwnp, Ill happily dive into this myself :)16:21
*** jorgem has quit IRC16:30
*** jorgem1 has joined #openstack-dns16:30
*** eankutse1 has quit IRC16:32
*** eankutse has joined #openstack-dns16:32
*** vinod has joined #openstack-dns16:32
vinodkiall: are you around?16:34
*** ctracey|away is now known as ctracey16:51
betsymugsie: you around?16:58
mugsiebetsy: hey whats up?16:59
betsyI think my blacklist patch is ready to be merged. Do you want to approve it or should I?17:00
mugsiedid i not?17:00
mugsiesorry, I meant to - will do it now17:00
betsyYou +2'd it, but didn't approve, so it's not merged17:01
betsythanks! :)17:01
mugsieah - my mistake17:01
mugsiethat is heading in now17:02
*** rossk has joined #openstack-dns17:08
*** tsimmons has joined #openstack-dns17:08
*** tsimmons has quit IRC17:09
*** tsimmons has joined #openstack-dns17:09
openstackgerritA change was merged to stackforge/designate: Create API calls to Manage Blacklisted Domains  https://review.openstack.org/6535917:10
*** jmcbride1 has joined #openstack-dns17:13
*** jmcbride has quit IRC17:14
*** CaptTofu has quit IRC17:19
*** rjrjr has joined #openstack-dns17:26
tsimmonskiall:  mugsie: Are you about?17:31
mugsiehey17:31
tsimmonshey, so I was looking at the devstack plugin and the Jenkins jobs. I recently did some bind9 stuff, and I was thinking I could work on integrating bind9 into the devstack plugin/ jenkins setup17:32
kialltsimmons: heya - Yea, getting BIND9 into the devstack plugin would be great!17:35
kiallI recently moved the devstack plugin from it's own repo to https://github.com/stackforge/designate/tree/master/contrib/devstack17:36
kiallMakes MUCH more sense there17:36
tsimmonsRight. So that's everything right? I was looking over it and didn't see much about MySQL or Postgres which is what that Jenkins server is running.17:37
kiallAh - So, that's a plugin to devstack itself: https://github.com/openstack-dev/devstack/17:38
kiallwhich provides mysql/postgres17:38
kiallhttps://github.com/stackforge/designate/blob/master/contrib/devstack/lib/designate17:39
kiallhas (some) powerdns stuff in it17:39
kialle.g. https://github.com/stackforge/designate/blob/master/contrib/devstack/lib/designate#L176 - Installing PowerDNS17:40
kiallGetting the DB setup: https://github.com/stackforge/designate/blob/master/contrib/devstack/lib/designate#L16117:40
tsimmonsOk, that's what I thought. So bind9 would kind of go along beside that. Then perhaps after that we could set up Jenkins jobs to use them both to test commits on, or do those already use powerdns?17:40
kialland, telling Designate to use it:17:40
kiallhttps://github.com/stackforge/designate/blob/master/contrib/devstack/lib/designate#L7517:40
kiallthe missing piece is pointing PowerDNS at the right database though ;)17:40
kialltoday, they just use the fake backend.. Regarding the Jenkins jobs - I talked with the infra guys, we should be able to make use of the standard openstack jenkins, without the 24->48 hour delay to get results...17:41
*** eankutse1 has joined #openstack-dns17:42
kiallSo - Once we have powerDNS/bind in there and working right.. We should probably just switch over to that.. But it's tricky getting it working right :)17:42
*** CaptTofu has joined #openstack-dns17:43
tsimmonsAh, ok. So basically I'll modify those current scripts to allow it to install bind9, as an option, and test it by actually running devstack + the designate plugin with that option and making sure it works.17:43
kiallYep - Once we have that, it's figuring out how to convince the OpenStack jenkins to run with the plugin+our options17:44
kiallWhich seems to be harder that it should be ;)17:44
kiallWorst case - We can add more jobs to our Jenkins in the meantime, if we can't figure ^ out soon17:44
*** eankutse has quit IRC17:44
tsimmonsRight, well we can't directly control that it seems. :P17:44
tsimmonsYeah that works too.17:45
kiallWe kinda can ;) It's all in Git, but it's a maze to try and understand what's actually happening and what calls what ;)17:45
tsimmonsHm. Well that can come later. i'll start working on this bit….after lunch.17:46
*** mwagner_lap has joined #openstack-dns17:48
vinodkiall: I had a few questions on your comments in https://review.openstack.org/#/c/72017/1/designate/manage/tlds.py17:49
vinodCan we get the context or do we have to use self.context?17:50
kiallIf we're just looking for "any old context" like we are there, then we should use self.context rather than building a new one..17:51
kiallPart of the reasoning is that the request_id on self.context is forced to be "designate-manage", so it's traceable in the logs17:51
*** elemecca has joined #openstack-dns17:51
vinodcan we get a context with the actual user running desiginate-manage?17:52
kiallSince designate-manage doesn't make API calls - there is no user (at least, no Keystone user)17:52
vinodwhich brings me to the other comment that you had about redundant code - does it make sense for designate-manage to do the api calls rather than call central?17:54
kiallThe other comment was more about something we should fix in the future, and less about your review :)17:56
kialli.e. some of our validations should be moved inwards, towards central.17:56
kiallThat way - the API, designate-manage, MiniDNS (if/when it learns to handle write queries) can share it17:57
kialloh - tsimmons BTW - that review still a total experiment, trying to understand how things need to be updated to adapt.. It most likely won't merge as is.. Once I have a better idea of how it might tie in, I'll file a BP detailing it18:00
vinodWith the import tlds in designate-manage, we cannot track who ran the command - correct - only somebody with admin access.18:01
mugsievinod: yup18:02
kiallYea, someone who had access to SSH to the server (and could read the designate config file..)18:02
*** msisk has joined #openstack-dns18:06
vinodthis looks to be a problem for us18:08
vinodwe would like to track who added the entries even if it as an admin18:08
vinodand preferably without ssh access18:09
vinodEmmanuel had another suggestion - adding multiple tld entries to the api18:09
vinodeither at the same endpoint for create or a different end point18:09
vinodthoughts?18:09
kiallYea - Once we figure out how we do "bulk" API actions, that should be extendable to all the API resources - including TLD's18:10
eankutse1Kiall18:13
eankutse1 I was thinking that just as we create records18:13
mugsievinod: i think your proposal could be useful for most operators, just not massive ones18:13
eankutse1and specify more than one record (I am assuming we are able to specify more than one record)18:13
mugsieso I say we get yours in, and then look at the API bulk actions later on18:13
eankutse1we can specify more than on TLD entry in the request and get it all added in one request18:14
kialleankutse1: today, we don't supporting adding multiple records in the API.. One of the "bulk  .." blueprints talks about adding it (can't remember which)18:15
kiallLaunchpad seems to be down .. So can check :/18:15
kiallcan't*18:15
eankutse1ok. thx18:15
vinodmugsie: Are you saying that you want the current code review to be submitted (changes to designate-manage?)18:15
mugsieyeah, with the few modifications that Kiall suggested18:16
kiallmugsie / vinod .. I think it depends.. If we're going to get bulk actions done soon, then it makes sense to leave out the designate-manage piece18:16
vinodkiall / mugsie:  Currently the only way to add TLDs is calling the API.  How about providing the designate-manage piece as an interim solution till we get the bulk action done.  Once that is done we can remove this18:22
kiallvinod: Sounds good to me18:22
mugsievinod: +118:22
vinodThanks kiall and mugsie.  I will make the changes that kiall suggested and submit it shortly18:23
*** vinod has quit IRC18:24
*** elemecca1 has joined #openstack-dns18:35
*** elemecca has quit IRC18:37
tsimmonskiall: Totally cool. I was having a bit of trouble understanding it without seeing it in context, but that makes sense.18:41
kiallYea - It's not actually used anywhere yet, still trying to figure out how it might hook in :)18:42
*** vinod has joined #openstack-dns18:52
*** elemecca has joined #openstack-dns19:10
*** elemecca1 has quit IRC19:13
*** artom has joined #openstack-dns19:31
*** artom has quit IRC19:43
*** artom has joined #openstack-dns19:54
*** eankutse1 has quit IRC19:57
*** eankutse has joined #openstack-dns19:58
mugsiemy abstract is up - https://docs.google.com/document/d/1D6FcXMiWMvFTIggWiRu3VajVTnerCFDErTdafcSyxl0/edit?usp=sharing20:12
*** eankutse has quit IRC20:12
*** eankutse has joined #openstack-dns20:13
*** eankutse1 has joined #openstack-dns20:13
*** eankutse has quit IRC20:13
tsimmonsAnd mine:20:14
tsimmonshttps://docs.google.com/document/d/1xIprT3xEzujFPWJhVGnIKRdsB7gkmpgTgo0DfG3Ml4s/edit?usp=sharing20:14
*** krow has joined #openstack-dns20:49
openstackgerritVinod Mangalpally proposed a change to stackforge/designate: Import TLDs to Designate  https://review.openstack.org/7201721:00
*** vinod has quit IRC21:02
*** eankutse1 has quit IRC21:03
*** rjrjr has quit IRC21:03
*** eankutse has joined #openstack-dns21:03
*** vinod has joined #openstack-dns21:06
*** krow has quit IRC21:15
*** krow has joined #openstack-dns21:19
artomHopefully I didn't embarrass myself with my first review...21:29
kiallartom: consider yourself embarrassed21:29
kiall(Kidding - haven't read it)21:30
artomAww, I started blushing and everything.21:30
*** ytwu has joined #openstack-dns21:30
kiallIs it at all possible do to the validation using the same code as the API? [1] If we're going to validate in different places, we should at least validate in the same way ;) <-- I left a similar comment on the last revision - It's something we need to cleanup later21:30
artomI thought yours was about *where* we do validation.21:31
artomAccepting that for now we're doing in two different places, can we use the same schema validator in both?21:31
*** ytwu1 has quit IRC21:31
kiallWe do :)21:31
kiallfrom designate.schema import format <-- same thing the API uses21:32
kiall(kinda)21:32
artomBut for example the description length is checked to be < 160 "manually"21:32
kiallYea, I don't think it's possible to actually just apply the JSON schema here ..21:33
kiall(I think)21:33
*** krow has quit IRC21:33
kiallSo - We're left re-using the validators like he us21:33
artomHrmm, it might indeed be more work to convert CSV -> JSON to reuse the schema.21:34
artomAs you said, consider myself embarassed.21:34
kiallDon't be - I picked that up too, just the fix is more complex than would be nice :(21:35
artomlets moev all validashun to central lul21:36
kiallKinda ;)21:37
kiallI'm thinking the central rpcapi.py file is a good place, still runs "client" side (i.e. in the api, or minidns, or..) but is shared21:38
kiallthe trick is the error "formats" are different between v1 and v221:38
artomFormatting isn't handled by middleware?21:39
*** krow has joined #openstack-dns21:42
kiallNot that stuff, no..21:44
kiallIt's handled currently in the APIv2 views21:44
kiallwhich might extend to include validation errors or something21:44
vinodartom: Thanks for doing the review.  The more eyes the better21:50
vinodBut as kiall mentioned rather than convert to json and do the same validation as api - i did the equivalent21:51
artomvinod, fair enough.21:51
vinodRegarding your other comment about being Pythonic, being new to Python, I am not sure what the standard is21:53
vinodI just went by (1) No complaints from tox (2) Easy to read/understand21:53
artomBah, 'if errors:' reads easier than 'if errors > 0:'21:53
vinodPersonally I like explicit validations :-)21:54
kiallpython -c 'import this' <-- vinod definition of "Pythonic" is basically printed ;)21:54
vinodBut I can go ahead and make the change21:54
artomAs I said, microscopic nit :)21:55
vinodExplicit is better than implicit. - 2nd line in there21:55
*** jorgem1 has quit IRC21:56
artomAre they in descending order?21:56
artomBecause 'Readability counts' is in there ;)21:56
vinodI assume so - since you have21:56
vinodSimple is better than complex.21:57
vinodComplex is better than complicated.21:57
artomAnyways, let me write up a long rant to start a flamewar. Then, when no one's talking to each other, someone who's stayed out of it can +2 and your patch will discretely merge ;)21:58
artomIsn't that how style issues are normally solved?21:58
kiallartom: ++ :P21:58
harmwbtw guys, I'm seeing some quantum* stuff in /etc/designate/designate.conf.. you might want to change those to neutron22:00
*** krow has quit IRC22:01
kiallharmw: I think we have in the current master branch..22:01
kiall(I think)22:01
harmwah ok22:01
harmwim looking at 2013.2, so...22:01
harmw:)22:01
harmw2014-02-11 23:00:55    ERROR [designate.openstack.common.threadgroup] 'Connection' object has no attribute 'connection_errors'22:03
harmwanyone seen that before btw?22:03
*** krow has joined #openstack-dns22:05
*** artom has quit IRC22:07
kiallharmw: Humm - Yes, I think I have at some point..22:10
*** vinod has quit IRC22:10
kiallFrom memory - it's an old dependency somewhere22:10
kiallHow did you install designate?22:11
harmwfrom rpm, though a missing dep could very well be the case here22:11
harmw(I'm rolling my own rpms)22:11
kiallAh.. Okay. Then I'd be pretty sure it's a dep :)22:12
kiallharmw: everything we use is a subset of what all the other services use..22:13
kiallSo - http://ubuntu-cloud.archive.canonical.com/ubuntu/dists/precise-updates/havana/main/binary-amd64/Packages has the versions we depend on22:13
kiallBut it's a bugger list than we need!22:14
harmwthanks :)22:14
kiallhttps://github.com/stackforge/designate/blob/stable/havana/requirements.txt22:14
kiallis the specific ones we use22:14
kiallbut there not all version pinned22:14
harmwI'm looking at the .txt right now22:14
kiallThat last link was the havana .txt, master will be different22:14
harmwsomething else I'm wondering about, shouldn't I be able to configure my rabbit account somewhere in designate.conf?22:15
harmwyea, I'm using havana so thats ok :)22:15
kiallYea - we just dont have all the options in there22:15
harmwah ok, so its just the usual (rabbit_userid/rabbit_password/rabbit_host)22:16
kiallYep - It's the same options as the other services ..22:16
harmwexcellent22:16
kiall(That code is actually shared among most of the projects)22:16
harmwhttp://designate.readthedocs.org/en/latest/configuration.html lists them aswell, nice22:17
kiallBTW - I assume RDO has all the right deps for Havana?22:17
harmwyea, I'm running havana already - just trying to get designate to roll with it :) its just some more work since RDO doesn't supply the designate package, leaving me to do the hard work of going through deps and all22:18
harmw(and I'm not installing it on my controller yet, so nothing is installed out of necessity by any of the other packages)22:19
kiallharmw: BTW - If you get a working rpc spec file, and know how to contribute stuff through OpenStacks review process, we'll happily get that into the repo .. People keep asking for RPM's22:22
*** jmcbride1 has quit IRC22:31
harmwbingo, python-amqplib was missing22:38
harmwok kiall, thanks22:38
*** sn6i23a has joined #openstack-dns22:47
*** eankutse has quit IRC22:51
*** mwagner_lap has quit IRC22:52
*** krow has quit IRC22:53
*** CaptTofu has quit IRC22:53
*** krow has joined #openstack-dns22:59
*** jmcbride has joined #openstack-dns23:02
*** krow has quit IRC23:03
*** tsimmons has quit IRC23:03
*** krow has joined #openstack-dns23:04
*** jmcbride has quit IRC23:09
*** vinod has joined #openstack-dns23:10
*** ytwu has quit IRC23:16
*** ytwu has joined #openstack-dns23:17
*** jmcbride has joined #openstack-dns23:20
*** vinod has quit IRC23:35
*** jmcbride has quit IRC23:36
*** vinod has joined #openstack-dns23:44
*** ctracey is now known as ctracey|away23:47

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