Tuesday, 2013-03-05

*** jcru has quit IRC00:02
*** kagan has joined #openstack-meeting-alt00:27
*** kagan_ has joined #openstack-meeting-alt00:29
*** vipul|away is now known as vipul00:29
*** kagan has quit IRC00:32
*** kagan_ is now known as kagan00:32
*** grapex has quit IRC00:54
*** jrodom has quit IRC01:30
*** vipul is now known as vipul|away02:00
*** vipul|away is now known as vipul02:05
*** ChadL has joined #openstack-meeting-alt02:37
*** kagan has quit IRC02:42
*** sacharya has quit IRC03:56
*** kagan has joined #openstack-meeting-alt04:55
*** vipul is now known as vipul|away05:59
*** kagan has quit IRC07:23
*** vipul|away is now known as vipul08:03
*** dhellmann-afk is now known as dhellmann13:17
*** grapex has joined #openstack-meeting-alt14:34
*** imsplitbit has joined #openstack-meeting-alt14:54
*** djohnstone has joined #openstack-meeting-alt15:15
*** jcru has joined #openstack-meeting-alt15:17
*** amyt has joined #openstack-meeting-alt15:30
*** amyt has quit IRC15:32
*** amyt_ has joined #openstack-meeting-alt15:32
*** amyt has joined #openstack-meeting-alt15:33
*** cp16net is now known as cp16net|away15:44
*** cp16net|away is now known as cp16net15:44
*** rnirmal has joined #openstack-meeting-alt15:44
*** jcru is now known as jcru|away15:45
*** rnirmal_ has joined #openstack-meeting-alt15:46
*** rnirmal has quit IRC15:49
*** jcru|away is now known as jcru15:51
*** sacharya has joined #openstack-meeting-alt16:11
*** cp16net is now known as cp16net|away16:13
*** cp16net|away is now known as cp16net16:13
*** rnirmal has joined #openstack-meeting-alt16:14
*** bdpayne has joined #openstack-meeting-alt16:19
*** joe5081 has joined #openstack-meeting-alt16:35
*** joe5081 has quit IRC16:35
*** joe5081 has joined #openstack-meeting-alt16:37
*** jcru has quit IRC16:39
*** joe5081 has quit IRC16:47
*** djohnstone1 has joined #openstack-meeting-alt17:02
*** djohnstone1 has joined #openstack-meeting-alt17:03
*** imsplitbit has quit IRC17:04
*** djohnstone has quit IRC17:05
*** grapex has quit IRC17:07
*** grapex has joined #openstack-meeting-alt17:08
*** heckj has joined #openstack-meeting-alt17:59
*** heckj has joined #openstack-meeting-alt18:00
*** cp16net is now known as cp16net|away18:04
*** cp16net|away is now known as cp16net18:04
*** kagan has joined #openstack-meeting-alt18:09
*** esp1 has joined #openstack-meeting-alt18:11
*** esp1 has left #openstack-meeting-alt18:12
*** esp1 has joined #openstack-meeting-alt18:34
*** dhellmann is now known as dhellmann-afk18:55
*** cp16net is now known as cp16net|away18:57
*** jcru has joined #openstack-meeting-alt19:02
*** cp16net|away is now known as cp16net19:02
*** imsplitbit has joined #openstack-meeting-alt19:14
*** cp16net is now known as cp16net|away19:17
*** esp1 has left #openstack-meeting-alt19:19
*** cp16net|away is now known as cp16net19:19
*** imsplitbit has quit IRC19:21
*** imsplitbit has joined #openstack-meeting-alt19:26
*** amyt has quit IRC19:41
*** rnirmal has quit IRC20:11
*** dhellmann-afk is now known as dhellmann20:16
*** vipul is now known as vipul|away20:16
*** vipul|away is now known as vipul20:16
*** rnirmal has joined #openstack-meeting-alt20:33
*** rnirmal has quit IRC20:33
*** vipul is now known as vipul|away20:52
*** vipul|away is now known as vipul20:56
*** cp16net is now known as cp16net|away20:57
*** cp16net|away is now known as cp16net20:58
*** bcwaldon has quit IRC20:59
*** grapex has quit IRC21:20
*** cp16net is now known as cp16net|away21:25
*** cp16net|away is now known as cp16net21:25
*** jcru is now known as jcru|away21:33
*** jcru|away is now known as jcru21:36
*** hub_cap has joined #openstack-meeting-alt21:38
*** grapex has joined #openstack-meeting-alt21:43
imsplitbitbam21:48
imsplitbitha, hub_cap hai21:48
*** grapex has quit IRC21:52
*** grapex has joined #openstack-meeting-alt21:52
*** datsun180b has joined #openstack-meeting-alt21:56
hub_caphai21:57
cp16nethi21:57
*** heckj has quit IRC21:58
datsun180bhello21:58
djohnstone1hi21:58
*** djohnstone1 is now known as djohnstone21:58
grapexSalutations21:58
*** esp1 has joined #openstack-meeting-alt21:59
hub_cap#startmeeting reddwarf21:59
openstackMeeting started Tue Mar  5 21:59:53 2013 UTC.  The chair is hub_cap. Information about MeetBot at http://wiki.debian.org/MeetBot.21:59
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.21:59
*** openstack changes topic to " (Meeting topic: reddwarf)"21:59
hub_capcome one come all21:59
openstackThe meeting name has been set to 'reddwarf'21:59
esp1hello22:00
*** SlickNik has joined #openstack-meeting-alt22:00
hub_cap#link https://wiki.openstack.org/wiki/Meetings/RedDwarfMeeting22:00
hub_cap#link http://eavesdrop.openstack.org/meetings/reddwarf/2013/reddwarf.2013-02-26-21.59.html22:01
*** dkehn has joined #openstack-meeting-alt22:01
SlickNikwoot22:01
* hub_cap starts a timer for 60s22:02
SlickNikGive people a minute or so to show up?22:02
SlickNiksounds good.22:02
hub_cap:)22:02
hub_capimport time22:03
vipulhere22:03
vipulsorry late22:03
hub_capnp we havent started yet22:03
SlickNikno problem, still waiting maybe 15 seconds.22:03
steveleonhello22:03
hub_captimers done22:04
hub_capvipul: yer up w/ the update snapshots wiki22:04
vipulcrap, nope.. was going to do it today but stuck in meetings22:04
hub_caplol so did we22:04
hub_capits been meeting hell today22:05
vipul#action vipul to update snapshot wiki w/snapshots deleted in swift22:05
hub_capSlickNik: secgroups extension22:05
SlickNikStill working on it.22:05
*** steveleon is now known as esmute22:05
SlickNikGot the object model done, working on the controllers.22:05
hub_capcool. had a chance to do any work w/ the API?22:05
hub_capas in the api docs22:05
hub_cap(im covering your other action item)22:06
SlickNikYes, I've uploaded a review to gerrit.22:06
hub_capOH NOEZ22:06
hub_cap#topic action items22:06
*** openstack changes topic to "action items (Meeting topic: reddwarf)"22:06
SlickNikwhoops.22:06
SlickNiklol22:06
kaganbtw, quick Q, when is the time to discuss out of band issues ?22:06
hub_capkagan: the end of meeting durin open discussion22:06
kaganok22:06
SlickNikusually at the end of the meeting, kagan.22:06
SlickNik#link https://review.openstack.org/#/c/23618/22:06
hub_capeveryone go in and update thier watchd projects!!22:06
datsun180b#link https://review.openstack.org/#/q/is:watched+status:open,n,z22:07
esp1datsun180b: thx!22:07
SlickNikI've marked that as work in progress since I still need to put in the request-response snippets.22:07
hub_capdatsun180b: assuming database-api is in your watched projects list ;)22:07
hub_capSlickNik:  kk22:07
datsun180bconfiguration detail22:07
SlickNikBut info on the entities/API calls are up there already.22:07
hub_capstill nice to see whats going on22:07
datsun180bit also assumes you're logged in22:07
hub_caplol datsun180b22:08
hub_capSlickNik: one thing to note, plz make the lines < 80 char22:08
hub_capcheck out the begin (lines 4~7)22:08
hub_capvs line 47522:08
SlickNikAh, okay will snip em...22:08
hub_cap<322:08
hub_capgreat work22:08
SlickNiknp.22:09
vipulwho was going to add snapshots api there/22:09
hub_capso vipul kagan, i barely touched base w/ u wrt the group stuff in reddwarf + percona22:09
kaganwhat does that has to do with percona ?22:09
hub_capis someone working on that vipul? the snaps feature?22:09
SlickNik#action SlickNik still working on Security-Groups. Hope to have patch up for review by beginning next week.22:09
vipul#action vipul to update api-specs wiht Snapshot API22:10
hub_capas per last wk kagan22:10
hub_capvipul hub_cap and kagan to look into making the service_type code work properly w/ percona image22:10
juicehub_cap: kmansel and I are working on some prep stuff but I don't know how has the meat of the work22:10
vipulhub_cap: i think we covered that?22:10
hub_capcool. looks like vipul is gonna take a stab at the api juice22:10
vipulthe percoan service type piece22:10
hub_capwe covered it, id like to get a finalization for the group22:11
kaganif i understand you guys correctly, i think i've done that already22:11
hub_capim not saying u havent22:11
hub_capjsut want to cover the action item from last wk :)22:11
vipulSo kagan added code to support 'percoan' and 'mysql' as types in the API request, defaulting to 'mysql'22:11
vipulthis is a conf value in reddwarf.conf22:11
hub_capperfect vipul thx!22:11
hub_capwas it a ton of work?22:11
vipulwhen the image is updated by redstack, it's added to service_images as percona or mysql22:11
* hub_cap hopes it was not a lot of work22:12
vipulonly kagan would know :)22:12
kaganno it wasn't22:12
hub_capsweet22:12
hub_capcp16net: go home22:12
hub_capdid u do that last wk?22:12
SlickNikI sure hope so :)22:12
cp16netyup22:13
cp16neti'm at home now22:13
cp16netsuccess!22:13
hub_capperfect!!22:13
hub_capok done w/ teh action items22:13
esp1I wish I was home22:13
hub_cap#topic Quotas / Limits Updates22:14
*** openstack changes topic to "Quotas / Limits Updates (Meeting topic: reddwarf)"22:14
esp1lookes like Quotas got merged :)22:14
grapexLimits looks good to me22:14
datsun180bsounds done to me22:14
hub_capyuuuuup22:14
SlickNikhellz yea!22:14
grapexOnly question I have is mockito22:15
vipulwoohoo22:15
esp1I've started planning out the next few steps to bring quota's into the API call and update the python reddwarf cli22:15
hub_capgrapex: thas not a question :)22:15
esp1grapex: good eye :)22:15
grapexI'm ok with us adding that, but won't it not tie into the standards or something?22:15
hub_capyet another mocking lib?22:16
grapexI thought the powers that be were wanting us to use mock22:16
esp1I will let juice speak to it.22:16
*** sacharya1 has joined #openstack-meeting-alt22:16
hub_capjuice: speak to it22:16
juiceit's well worth it22:16
esp1I have idea22:16
grapexOk. I'm going to +2 it anyway, just wanted to hear the reasoning. :)22:16
juiceI have struggled with the mocking libraries22:16
juiceused this extensively in java22:16
juiceworks wonders in both languagues22:16
esp1I mean I have no idea.  but it does look cool22:16
juicethe best part is that it is easy to use22:16
juiceand you can actually make sense of the test code22:17
juice…most of the times anyways :)22:17
grapexAnother tangent22:17
grapex"@test(enabled=RD_CLIENT_OK)"22:17
grapexThat's kind of cool22:17
grapexI was wondering if we could build the version number into the client22:17
grapexso those tests would get enabled automatically22:17
esp1well at first that was the plan22:18
esp1but I think I will just remove it once reddwarf cli merges.22:18
grapexAnother idea is we could invent a decorator that would check the client version, and run SkipTest if the client wasn't up to date. The reason is, skips end up being far more visible than disabling tests.22:18
grapexOk22:18
*** sacharya has quit IRC22:18
datsun180bWell just having the client version would be great, I don't want to have to look at pip freezes after tests fail22:18
grapexIf no one objects, I'll create a blueprint for that.22:18
esp1I thought about that too.  I think that would be a good idea.22:18
esp1+122:19
grapexCool22:19
SlickNikI'm in favor of creating a blueprint for that, think it would be useful.22:19
hub_cap+1 to the BP, action item it grapex22:20
grapexCool, I'm on it22:20
hub_capi mean actually type it in the channel grapex!!22:21
grapexWas going to do a client BP, doing a reddwarf one instead22:22
grapexWe may be able to find a way to do this that doesn't involve changing the client. Let's talk about it after the meeting.22:22
hub_capgrapex: !!!22:23
hub_cap#action teach grapex to use #action22:23
grapex#action make a RD blueprint to enable tests depending on RD client version22:23
hub_cap:)22:23
grapexlol... sorry hub_cap!22:23
hub_caphaha that was hilarious22:23
SlickNik:)22:23
hub_capok we good on quotas/limits? im STOKED they are in/merging22:23
vipulgood job guys22:24
hub_capWOOOOOOT!! Big +1 to the HP guys22:24
hub_cap#topic Percona Image Updates22:24
*** openstack changes topic to "Percona Image Updates (Meeting topic: reddwarf)"22:24
kagani've posted another version today22:25
kagani think it should be good22:25
grapexYeah, that code is looking really good.22:25
kaganneed you guys to take a look22:25
grapexThanks guys22:25
grapexkagan: I'm sorry I've been unable to look at that code yet, I've been in a lot of meetings this week22:25
grapexI'll try to catch up soon.22:25
kagani've added the package name as a config with default to mysql22:25
kagansure22:25
hub_capsweet! we will be looking @ it today/tomorrow kagan for sure22:25
kaganso, one thing still missing:22:26
hub_capi like that kagan! (the config / pkg default)22:26
kaganif the image is clean, i.e. no mysql installed22:26
kaganthen percona will not be installed22:26
kaganwe still don't support that22:26
kaganit shouldn't break, since the only time the pkg name is set to percona's is when you actually build a percona image22:27
kagani think we need to discuss whether we even want to support such a use case, and if so, find a decent way to handle it.22:27
hub_capAhh so yer saying the image _has_ to be seeded w/ percona mysql to test percona mysql22:27
kaganinstalling percona has some more steps to it.22:27
hub_capif its no mysql, itll fail for percona22:27
hub_capthat makes sense22:27
kaganwell, it won't fail, cause it won't think it's percona ...22:27
hub_capah okey22:27
hub_capitll just install stock mysql22:28
kaganit's set to look for percona pkg only at time of building a percona image22:28
kaganright22:28
hub_capthat makes sense22:28
kaganif provided with a clean image.22:28
vipulwe could support it, but probbly need ot refactor that install code a bit22:28
hub_capok, well if yall want to add that to the guest it would make sense...22:28
kaganso, we ended up dropping a lot of what i initially put it to make the tests work22:28
hub_capbut if not, putting in image def works :)22:28
kagani think we have a pretty solid version checked in now.22:28
hub_capw00t22:28
kaganany questions?22:29
SlickNikI think in the future, we may want to do something like install percona (like mysql), but for now this seems like a good plan…22:29
hub_capkagan: safe to assume uve run it w/ stock mysql recently? ill pull it down tomorrow and run it w/ stock too22:29
kaganyes, i did22:29
hub_capsweet22:29
kaganjust had some issues this morning with tox22:29
kaganwould pass on my machine, but failed jenkins ...22:29
kaganindeed there were errors, funny fox didn't find it ...22:30
kagannow all should be good, as far as i can tell ...22:30
kaganif you wanna test it, make sure you also pull latest changes to reddwarf-integration22:30
kaganor have they merged already ...22:30
hub_capokey thx kagan22:30
kagansure22:31
hub_capnow our PM's will be asking when we are going to put maria in22:31
hub_cap:P22:31
kaganit's easy, 3 days tops …. :)22:31
imsplitbithub_cap, they already are22:31
vipulheh22:31
SlickNikWhy wait for the PM's?22:31
grapexNice.22:31
hub_caphahahahaha22:31
SlickNikhub_cap, when are we going to put Maria in? :P22:31
SlickNikj/k22:32
vipulwho's maria22:32
esp1uhm, kagan is going on vacation for like 1 month..22:32
hub_capi think vipul told me SlickNik was going to do it22:32
vipul:D22:32
hub_capvipul: lol22:32
SlickNiklol.22:32
hub_capwoah kagan when?22:32
imsplitbitvipul, it's hub_cap's gf22:32
*** kmansel has joined #openstack-meeting-alt22:32
vipulheh22:32
kaganin 2 weeks.22:32
hub_caplets get his code reviewed/merged quick22:32
hub_capok we should have more than enough time :)22:32
kagani'll try to be working most of the time. going to Israel, so 10 hours ahead ..22:32
kagan\22:32
hub_capcool!22:32
kaganand little overlap with all the weekend shift ....22:32
kagangonna be tricky22:33
datsun180b10 hours ahead of seattle?22:33
kaganyep22:33
hub_capvipul: do we need to cover snapshots? they are still on the agenda22:33
kaganit's easier to take two hours back and flip :)22:33
vipulNo, not really, besides we're started work on it22:33
grapexHey that's only a mere 8 hours ahead of us.22:33
datsun180bexactly22:33
hub_capany progress to be chatted about vipul?22:33
vipulwe're enabling Swift and bringing in swiftclient in reddwarf22:33
hub_capahh cool22:33
hub_capintegration work first22:34
juicekevin has the swift client enabled by default.22:34
SlickNikjuice and kmansel working on that… (swift / swiftclient)22:34
vipulwell some of that is kinda necessary22:34
hub_capimma change the meeting to say Snapshots Update22:34
hub_capdef vipul22:34
juiceI am writing the wrapper code to get the client and looking into a way of getting away from Fakes22:34
hub_capit was blueprint feedback....22:34
juicefor use with testing with Swift in a non-integrated env22:34
kmanselyeah i'm working on that integration with redstack22:34
juiceshould have something to commit by tmw22:34
hub_capsweet!22:35
vipulcool, annashen is also looking at the models piece22:35
vipulgoing to submitting the migration soon i believe22:35
grapexjuice: Do you mean you're changing the test doubles to some other kind of test double, or that you're going to try to call a real instance of swift?22:35
juicei was thinking of not using a test double but instead of using mocks22:36
juiceI certainly do not think we should have Fake code in the production code a la Client22:36
grapexFor the integration tests?22:37
vipulwasn't there a bug for that issue?22:37
juicesure22:37
vipultha we mix fakes with real clients22:37
hub_capfor removing fakemode?22:37
vipulI think juice is referring to the 'remotes.py'22:37
juicegrapes - is there a reason not to use mocks in fake mode?22:38
grapexNo, it would make sense in some cases.22:38
hub_caplol grapes22:38
juice:) I just caught that22:38
grapexThe only exception is if you stand up Reddwarf by itself to run in fake mode with eventlet turned on22:38
hub_capid prefer to see a way to make fakemode configuration driven, just fyi22:38
grapexIt would just be hard to get the mocks to behave persistently22:38
juiceYes, I would not suggest using mocks to keep track of state22:39
grapexSo the only reason is it would be hard. :) If you can find a way to do it where fake mode can still chug along then that's great22:39
juicethat's where the fakes would be a better choice22:39
grapexPossibly having two configurations could help as well, or just using mocks from the unit tests.22:39
juiceI am just getting into it so this is somewhat speculative22:40
grapexI think the "fakes" are best when you're writing integration tests for the happy path, and any unhappy paths that are liable to be hit frequently22:40
hub_caplike looking for a oil well juice?22:40
grapexIt's easier to hit the edge cases with unit tests and mocks22:40
grapexbut if you can find a way around it feel free, as long we as we don't lose functionality it could be really cool.22:41
juiceI will post findings in the regular channel over the next day or two22:41
hub_capcool22:42
juiceI really just want to provide some light weight tools developers can get in there to change code and test their changes.  So that is the goal.  As a newcomer to the code base I don't feel like it's there yet22:42
hub_capso do we really have a CI update? i konw datsun180b had some issues w/ CI due to the reddwarf->database change.22:43
hub_cap#topic Redwarf CI Update22:43
*** openstack changes topic to "Redwarf CI Update (Meeting topic: reddwarf)"22:43
hub_capdatsun180b: care to elaborate?22:43
datsun180bI think I've resolved everything I can within SF22:43
vipulstackforge?22:43
datsun180baye22:43
datsun180bso as far as i can tell for now we're okay22:43
datsun180bwell, there's a couple things that i put in my modify-user fix requests22:44
datsun180bat this point i think it's better that i just submit that on its own22:44
vipulCI update is that we are working on a solution with our own Jenkins that will listen to Gerrit ... the idea is that we run the VM-gate here .. Rax could do the same fi we need ot have multiple set ups22:44
grapexvipul: So ultimately, either HP or Rax will run Jenkins and not StackForge?22:45
vipulgrapex: For the time being, yes22:45
vipulthe concernt openstack-ci has is that they can't support all stackforge projects since vm-gate is a pretty heavy thing22:45
vipuland htey have no way to set priorities on projects22:45
datsun180bdid you try to tell them that we're #1?22:45
vipuluntil they get multi-master jenkins setup in openstack CI22:46
vipulwe're gong to set up our own..22:46
vipulwhen they get it set up, i beleive we'll be 'officially supported' again22:46
hub_capvipul: is yours open?22:46
grapexOk. As long as we just have it running22:46
SlickNikyes, Openstack CI is having issues with the load on their Jenkins master, so they don't want to take on  integration testing for stackforge projects...22:46
hub_capas in will we be able to hit it?22:46
vipulhub_cap: No, that's what we're trying to figure out22:46
grapexMakes sense.22:46
vipulhow to get you guys access22:46
hub_capkk vipul22:46
vipulworst case, ew'll push logs up?22:46
hub_capwe could probably spin up a cloud server...22:46
hub_capwe have the power22:47
vipulwe're having trouble with the gerrit hook at this point22:47
vipulso this may/may not go anywhere.. and a Plan C may need to be developed22:47
hub_capya i think gerrit needs to push to the jenkins infra right?22:47
grapexWorst case scenario22:47
grapexwe just run it periodically without listening to hooks22:47
hub_capok if we need to pop a cloud server22:47
hub_capwe can do that22:47
SlickNikhub_cap it's a pull model...22:47
vipulYea, honor system?22:47
grapexAt least then we'd see it and could be aware when it broke22:47
hub_capvipul: BOOOOOOOO22:47
vipulyea don't like it either22:48
hub_capSlickNik: k22:48
SlickNikin openstack_ci zuul monitors the gerrit stream and triggers jenkins...22:48
hub_capright so gerrit needs to be able to talk to jenkins22:48
SlickNikgerrit event stream*22:48
hub_capthat what i was gettin at22:48
hub_capif jenkins is private....22:48
hub_capthen gerrit no knowie22:48
vipulso HP or Rax jenkins can be given voting privileges22:48
*** yidclare has joined #openstack-meeting-alt22:48
grapexYeah, it needs to be on a public Nova server.22:48
vipulthat's already something they support.. you might be right, not behind VPN22:49
hub_capmaybe we should put it in the public and use openid/launchpad integration for ssh/jenkins access22:49
vipul#link http://ci.openstack.org/third_party.html#the-jenkins-gerrit-trigger-plugin-way22:49
hub_capvipul: im pretty sure it wont work unless u punch a hole in yer firewall22:49
hub_capwhich im _sure_ your netsec guys are totally cool with22:49
vipulwell don't tell anyone but we have a jenkins in nova today :D22:50
hub_capso why dont we spin up a cloud servers jenkins?22:50
hub_capoh ic so it _is_ public?22:50
hub_capand i wont tell anynoe whos not in the room or has ears vipul22:50
vipulsort of.. ports are blocked22:50
hub_cappromise22:50
vipulsecuriryt gorup rules22:50
hub_capah ic22:50
vipulugh22:50
hub_capok lets table this i think. we can come up w/ a solution22:51
vipulanyway, so still some work to be done before we have that.. but look at the link ^^22:51
hub_capand it sounds like yer already workin on it22:51
vipulif you guys want to try to set something up on your end22:51
esp1hub_cap: who do you know that doesn't have ears?  I know a dude who's missing a thumb.22:51
*** cp16net is now known as cp16net|away22:52
hub_capwhales?22:52
hub_caplets go w/ visible ears esp122:52
hub_cap#topic Open Discussion22:52
esp1sorry for the interruption...22:52
*** openstack changes topic to "Open Discussion (Meeting topic: reddwarf)"22:52
kagannow time for issues?22:52
kagani have 2 of those ...22:53
datsun180bissues you say22:53
hub_capyar22:53
SlickNikyes kagan.22:53
SlickNikgo for it.22:53
kaganfirst one, code style/pep822:53
kaganis there a tool we use for that?22:53
grapextox -epep822:53
datsun180byes, the tool is called pep822:53
kaganthat's to check22:53
esp1yeah that's what I run too..22:53
grapexYou mean from an IDE?22:53
kagani mean to apply it ...22:53
vipulautomagically fix?22:53
esp1IDE is okay.. but seems slow (PyCharm)22:53
kaganso, for eclips/pydev there is some thing22:54
vipuli think juice had some tricks22:54
kaganbut it will apply to a full folder and not a single file.22:54
hub_capif thre is that'd rock22:54
kagani tried it, and all the files where modified, so naturally you're not all using that … ;)22:54
juicepycharm will warn you and offer to fix it22:54
hub_capbut i mod them manually before i chek in22:54
kaganalso, there is "autopep8 -i my_file.py"22:54
kaganeclipse/pydev will also warn.22:54
hub_capim slightly scared of that, personally22:54
hub_capbut i say go fer it22:55
kagani just thing it would be good if we could use a tool, and preferably the same one22:55
juicethe benefit to pycharm is that it finds it sort of real time22:55
hub_capkagan: i agree22:55
hub_capwe had nice formatters in java22:55
kagansame with pydev, juice22:55
hub_capthat woudl do it for us22:55
kaganwhat ide are you guys using?22:55
datsun180bvim22:55
kagannice ...22:55
vipullol22:55
djohnstonehahaha22:55
datsun180byou think i'm kidding22:55
vipulwrong crowd22:56
kagan"réal monkeys, ehh… programers! use vi!!"22:56
hub_caphahaah ide :P22:56
SlickNikemacs here, but let's not get started on that again :)22:56
hub_capi use textmate22:56
kaganok, i see22:56
hub_caplets all agree to disagree22:56
imsplitbityeah that fight has gone on 30 years22:56
grapexSublime. PyCharm is pretty good but Sublime is faster.22:56
vipulbtw, openstack has a free PyCharm license22:56
vipulif you are contributing to Openstack that is22:56
vipuljust an FYI22:56
kaganso it either has to be some stand alone tool, like autopep8 or we find that it actually does the same as an integrated extension22:56
imsplitbitpycharm on any modern computer works fine22:56
datsun180bso i did want to bring up an issue briefly22:57
kaganok, so do you agree we might want to use some formatting tool?22:57
grapexkagan: I think we should let people use the tools they want and trust tox pep8 to check it.22:57
kaganand we don't want that every time we use it, it'll modify it differently than the other guy?22:57
imsplitbitgrapex, agreed22:57
vipulkagan: if you find one.. and it works with tox -epep8, please share22:57
hub_capya will using a tool or not using a tool not provide the same output?22:57
grapexI guess I miss the point- pep8 should keep us from having a different format from each other22:58
datsun180boh i can show you ways that even pep8 doesn't enforce the spirit of PEP822:58
kaganthe thing is, if i want to use such a tool, it'll introduce too many changes every time i touch a file, unless we all use the same tool ...22:58
imsplitbit:-)22:58
datsun180bi trust humans over robots in this case22:58
hub_cap+1 to humans22:58
grapexkagan: If that's the case, maybe that tool needs to be tweaked to match pep8. The other OS projects are using it so we may have trouble changing the format. :(22:58
kaganyou can split a line in many places to have both halves at the proper length. just an example ...22:59
datsun180bthere's also the consideration that we'd have to include the tool in our AUTHORS file22:59
kaganit complies with pep8 of course !!!22:59
grapexdatsun180b: lol!22:59
grapexkagan: The community might be open to a superset of pep822:59
kaganwe just need to agree to use the same formatter22:59
hub_capkagan: i dont think thats gonna happen honestly22:59
juicesame format of pep8 compliant?22:59
kaganagain, i just don't want too many changes every time i apply a formatting tool to a file22:59
hub_capwe can mandate pep822:59
hub_capbut telling eveyrone who ever codes on the proejct to use tool X instead of just being compliant is not gonna go over well23:00
kaganso, how do you guys make it pep8? run tox and fix manually as you see fit?23:00
hub_capyup23:00
hub_capi run tox before i git review23:00
vipulyea that's how most people do it23:00
hub_capor a few times during a cycle if its big23:00
kaganwell, wouldn't you prefer a tool to do that for you?23:00
juicehave you seen a discrepancy between pydev/charm pep 8 and fox?23:00
hub_cap_prefer_ is a strong word23:00
kaganif it's a stand alone tool, you can still edit wherever you like ...23:01
vipullol23:01
hub_capi dont prefer a tool to do it at all23:01
SlickNik_tool_ is a stronger word. :)23:01
annashenyou can run pep8 individually23:01
hub_capyes of which i am one most of the time SlickNik23:01
vipulhehe23:01
hub_capbut yall know that23:01
grapexkagan: So I've noticed running different versions of pep8 gives different results23:01
grapexfor awhile datsun180b was running pep8 locally23:01
vipulyea that's probably why it looks different23:01
datsun180bthat is true, the newer ones are a lower limbo bar23:01
grapexthen we introduced tox and he found out he'd formatted it differently23:01
datsun180bi AM pep823:01
grapexmaybe its a simple as making sure your IDE is using the exact same version as tox?23:02
kagani don't understand the objection to use a formatter tool … please explain23:02
hub_capsimple23:02
hub_capnot everyone will use it23:02
vipulit's like a religious war.. not that simple..23:02
hub_capunless we hook it in to a post commit hook23:02
datsun180buse the formatter if you want, it'll still have to pass under pep8's scrutiny23:02
hub_capsomeone will not use it23:02
vipulyep23:02
hub_capand then when u format it it will change their code23:02
hub_capand we will have this convesation again23:02
datsun180bI'll say it now, I'm not going to outsource my style to a robot23:02
hub_capand it will recycle again and again23:02
hub_cap+1 datsun180b23:02
kaganyou guys are crazy … ;)23:03
juiceI also don't like getting a bunch of changes that are all whitespace23:03
vipulkagan: if you find one that basically keeps the 'good' files the same23:03
juicemakes merges messy23:03
hub_capkagan: go ask nova to do it23:03
hub_capif u can get them to comply23:03
hub_capwe will gladly comply23:03
grapexdatsun180b was much like you kagan, until he too was forced to drink the kool-aid... join us... join us... :)23:03
kagani hate to format manually, that all ...23:03
hub_capand ill give u 100 bucks23:03
kagan:)23:03
datsun180bmore like you guys didn't understand pep823:03
kaganwhat do you mean?23:03
datsun180bi had to beat the \ off the end of your broken lines23:03
hub_capthis coming from a guy who hates pep823:03
kaganis \ not a pep8 ?23:04
hub_capkagan: datsun180b referiing to the rest of the rackers23:04
datsun180bi hate pep8, the checking tool. I embrace PEP8, the guideline document23:04
esp1lol23:04
hub_capgo hug your pep8 datsun180b23:04
datsun180b>>> import this23:04
SlickNiklolol23:04
kaganbut we all have to comply with pep8, right?23:04
hub_capdef23:04
datsun180bkagan: absolutely23:04
hub_capbut we dont have to comply w/ some robots version of it :)23:04
kaganso why not use something like autopep8 to do it for us?23:05
kaganwhat do you care ??23:05
hub_capcuz we cannot guarantee everyone will do that23:05
kaganwe just need to try ...23:05
hub_caphow can u make everyone from now to the end of this project guarantee to use it?23:05
datsun180bbecause we're better at parsing pep8's message than any fsm-based machine23:05
vipulyea the moment one person doesn't...23:05
hub_capBOOM23:05
kaganif you all *dont* want to use it, then no one can ...23:05
vipulyou could.. on individual files23:05
hub_capnot true... just annoying for that person ;)23:05
vipulthat you actually modified23:05
esp1fyi no one has beaten dror in a fillibuster...23:05
kaganwell, if only one occasionally doesn't it's not that bad, but i think i'm a minority here ...23:06
hub_caplol... we can try23:06
imsplitbithub_cap, http://tech.myemma.com/python-pep8-git-hooks/23:06
datsun180bfull reveal, if you check our commit history i do have commits with the comment "pep8 fixes"23:06
hub_capimsplitbit: im sure it _can_ be done :)23:06
imsplitbitI have pre commit hooks to run pep8 on my python23:06
imsplitbitcode that is23:06
vipullol23:06
hub_capsnazzy23:06
kaganso, i guess no agreement here … ok, i tired.23:07
kagansecond issue:23:07
esp1kagan: you should give PyCharm a try.  you might find that it does a decent job.23:07
kagandebugging unittest code.23:07
kagando you guys debug the nose tests?23:07
kagancause when we tried, they failed just by running in debug mode.23:07
hub_capas in, via pdb?23:07
kaganwe then found 2 places in the code where if i put sleep (no matter how small the sleep time is) it will fail the test even when not in debug mode23:08
datsun180bi did want to bring up an issue about testing23:08
datsun180bso you're probably leading to the point i wanted to make23:08
kaganwas wondering if you ever manage to debug those, as in step by step23:08
vipulwell it's not related to debugging23:08
vipulit's if you put a sleep in a certain spot23:08
vipulthe tests fail23:08
vipulno matter how small the sleep is23:09
datsun180bdevil's advocate: sleep(0) ?23:09
kagani tried 0.00000000000123:09
kagannot sure about 0 though ...23:09
kaganwould guess it'd fail as well ...23:09
vipulwondering there is something in the proboscic framework that causes this?23:09
grapexIs it when you call time.sleep?23:09
kaganyes23:09
grapexOr just when you literally spend some time with the program paused?23:09
kaganno23:10
kaganwe tried spending time with a long loop23:10
grapexThat's because time.sleep is monkey patched out so the tests will run quickly23:10
kaganit worked23:10
kaganright!23:10
kaganwe saw that ...23:10
vipulhah..23:10
grapexSo it runs the event simulator23:10
datsun180boh, flashbacks to poll_until23:10
grapexbasically running things that would have run as different threads right away23:10
kaganwe didn't understand how come the whole test suite ran faster than a single sleep, when we know the sleep had to happen 12 times !!!23:10
vipulbut why does that cuase it to fail?23:11
kaganbut why would it fail the tests?23:11
vipulsince it should be ignored?23:11
grapexIf you'd like to debug the tests, it's possible if we change it not to simulate events23:11
grapexIn run_tests.py, change line 4323:12
grapexBecause, calling sleep tells it to run any pending events.23:12
kaganchange to what?23:12
grapexI'm not sure why it fails. We never had luck running PyCharm in debug mode for anything other than simple unit tests.23:12
grapexkagan: In the reddwarf code23:13
kaganand is that the same reason if fails while in debug mode?23:13
kaganchange the line to be what?23:13
*** dhellmann is now known as dhellmann-afk23:13
grapexLine 43 - just comment it out23:13
kaganok23:13
kagani'll try23:13
grapexDitto to line 8923:13
kaganline 89 is only if i want to use sleep, right?23:14
kaganwe used sleep just to prove a point. our main concern was running in debug mode23:14
grapexLine 89 changes sleep to run functions which would have run in a seperate thread23:14
hub_capid prefer to comment all lines %7 ==0 out23:14
vipuland just see what happens :)23:15
kaganhub_cap, what do you mean ??23:15
datsun180bhub_cap: oh there's a sed oneliner for that23:15
juicethere was an option in intellij that would pause all threads in debug mode when stepping through to prevent timer threads from timing out while you did you're mental investigation23:15
kaganjoke, right ??!23:15
hub_capLOL kagan grapex23:15
hub_capof course :P23:15
datsun180bi can't remember how to select just lines %7==0 as a movement in vim23:16
kagande vencci code running in front of me …23:16
vipuldatsun180b you had an 'issue'?23:16
SlickNikyou were mentioning something about test timing?23:16
hub_capha kagan23:17
datsun180byeah, i lost some time trying to get my work done until i got https://review.openstack.org/#/c/23018/ done23:17
datsun180bspinning up a vm with clean pulls from stackforge before that change, i simply couldn't build and spin up an instance23:17
grapexdatsun180b: Were the public real mode tests working before that commit?23:18
vipuloh yea that was my bad.. i should have fixed the issue23:18
vipulso we have two paths to deploying now23:18
vipuli've been using the local.sh approach23:18
vipulwhich doesn't suffer from this23:18
datsun180bvipul: it's not an issue of blame, but one of being able to prevent that that i care about23:18
grapexvipul: Is that in devstack?23:18
vipulno, that's in redstack..23:18
vipulso if build your environment the 'old' way23:19
vipulredstack install, redstack build, redstack kick-start.. etc23:19
vipulthat change is required for things to work23:19
vipulif you copy 'local.sh' into devstack/23:19
vipuland just run stack.sh23:19
vipulthen you don't see the issue23:19
vipulwhat it boils down to is... we need to not have two paths23:19
datsun180bwould probably help not to just be keeping that in your head23:19
SlickNikI've been meaning to unify the two approaches so that redstack uses the hooks in devstack to run, but I haven't had much time to work on it. :(23:20
vipulyea.. i noticed after some others saw it23:20
SlickNikI should really do that since it would help not have some of these issues creep up.23:20
*** grapex has left #openstack-meeting-alt23:20
vipulsorry, should have put something in the reddwarf channel23:20
*** grapex has joined #openstack-meeting-alt23:20
hub_caphai grapex23:20
grapexGood ole OSX key bindings...23:20
esp1yeah I lost 5hrs one day.  SlickNik saved me.23:21
hub_capLOL23:21
grapexThat broke us at Rax for awhile23:21
vipulSo do we want to trim down redstack? so we don't have to maintain 2 ways of set up?23:21
datsun180beven now there's one more change to make to our internal fork that i haven't nailed down yet, something about keystone23:21
grapexWe need to be careful when we make changes like this to make sure we keep track of changes that will they may mandate in Reddwarf Integration23:21
vipulright.. this is where the vm-gate would have caught this23:21
hub_capi would love to trip redstack if we can23:22
hub_cap*trim23:22
vipulchanges to multiple projects are hard to coordinate23:22
grapexvipul: Agreed. What if we just changed redstack to call local.sh?23:22
vipulyea, or rather copy local.sh to devstack and invoke stack.sh23:22
SlickNikgrapex: that was what my comment above was about.23:22
grapexSlickNik:Ok, I probably missed it when I ducked out. :)23:22
grapexvipul: I think copying it to devstack makes the most sense23:22
grapextrying to think if doing it that way precludes us from anything23:23
vipul#link https://wiki.openstack.org/wiki/Reddwarf-installation23:23
vipulFYI ^^ is the 'preferred' way of set up23:23
vipuluntil we trim Redstack23:23
kaganso i took line 43 out, (and all the mod7 ones as well) but now the tests takes much more time to run and still fail ...23:23
kaganwhy would this line break the tests if ran in debug mode?23:23
grapexkagan: Those are integration tests, it's going to be very tricky to debug them.23:24
hub_caplol kagan to mod723:24
kaganno, i'm not talking about those ...23:24
SlickNik#action SlickNik look into trimming down redstack to install devstack using the local.sh way.23:24
grapexIf you need to run a debugger, run reddwarf server in fake mode and set break points, then run the tests from another process23:24
kaganjust the unit test ones23:24
datsun180b:g/^/if !(line('.')%7)|s/^/#/|endif23:24
datsun180bby the way23:24
kagani have no live environment ...23:24
grapexkagan: there's got to be some way to run the testr tests by themselves without the run_test.py script23:24
SlickNikhahah, nicely done datsun180b...23:24
grapexkagan: You don't need a live environment thankfully. You can run reddwarf in fake mode locally23:25
kagantester tests run nicely under debug mode23:25
* hub_cap shakes head at datsun180b23:25
kagantestr23:25
vipulthanks SlickNik for the actino23:25
grapexkagan: Look at reddwarf/bin/reddwarf-server23:25
* datsun180b wonders how to run pycharm in debug mode from vim23:25
hub_capso should we call this meeting?23:25
kaganbut if i run run_test.py it only runs well in run mode, not debug mode ...23:25
grapextestr is running unit tests which are simplier.23:25
hub_capand move this to #reddwarf?23:25
datsun180bin think we've overstayed our reservation, or nearly so23:26
kagani'm ok with this23:26
hub_cap<323:26
hub_cap#endmeeting23:26
SlickNiknp vipul, it's been at the back of my mind. I think I've even bugged it. Just haven't had a chance to do it yet.23:26
*** openstack changes topic to "OpenStack meetings (alternate) || Development in #openstack-dev || Help in #openstack"23:26
openstackMeeting ended Tue Mar  5 23:26:31 2013 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)23:26
openstackMinutes:        http://eavesdrop.openstack.org/meetings/reddwarf/2013/reddwarf.2013-03-05-21.59.html23:26
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/reddwarf/2013/reddwarf.2013-03-05-21.59.txt23:26
openstackLog:            http://eavesdrop.openstack.org/meetings/reddwarf/2013/reddwarf.2013-03-05-21.59.log.html23:26
clarkbreally quick. I think you should get the proper version of python-reddwarfclient now23:26
clarkbthe mirror only has version 1.2.0 at the moment23:26
datsun180bclarkb: thanks a lot.23:27
grapexclarkb: Cool23:27
SlickNiksweet.23:27
*** SlickNik has left #openstack-meeting-alt23:28
*** imsplitbit has quit IRC23:28
*** djohnstone has quit IRC23:29
*** esp1 has left #openstack-meeting-alt23:29
*** jcru has quit IRC23:35
*** datsun180b has quit IRC23:52
*** jrodom has joined #openstack-meeting-alt23:58

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