/srv/irclogs.ubuntu.com/2015/01/29/#juju-dev.txt

katcodoes juju have a wrapper around exec.Command?00:26
perrito666katco: iirc there is one in utils00:30
perrito666sadly there is more than one around00:30
katcoperrito666: ty sir00:30
perrito666it does little to wrap it :)00:31
perrito666yw00:31
* perrito666 wonders why is he even reading this channel at thos hour00:31
jw4perrito666: because you love it here - admit it00:32
katco:)00:32
perrito666jw4: well have no life :p00:32
jw4:)00:32
katcocode is life!00:32
katco(of course there are more important things, but coding is a lot of fun :))00:32
* jw4 whispers "don't tell katco any different"00:32
katcolol00:33
jw4:)00:33
katcowell right now my daughter is asleep by 6:3000:33
katco18:3000:33
jw4wow - how old is she?00:33
katcoi'm sure when that changes i'll be a little more scarce00:33
katcoshe is 8 months00:33
jw4awww00:33
jw4does she sleep through the night though?00:34
katcomost of the time00:34
jw4nice!00:34
katcoshe's a good little baby :)00:34
jw4yeah; our first slept through the night at 8 weeks, but our son (second born) took much much longer00:34
katcohow many do you have john?00:35
jw4two00:35
jw4daughter 13 and son 900:35
katcowow, how nice :)00:35
katcothe teenage years haha00:35
jw4don't envy the daipers and sleepless nights even though some monster occasionally replaces my daugter00:35
katcowell, i'm having fun so far :)00:36
jw4good00:36
jw4I get nostalgic until I think about some of the practicalities of infants00:36
katcohehe00:37
jw4perrito666: I forget - you don't have kids right?00:37
jw4he took himself seriously00:38
katcohaha00:38
menn0katco, jw4: we make baaad sleepers00:41
menn0katco, jw4: my son is 19 months and he's only /just/ started sleeping through00:41
katcooh man00:42
katcomenn0: i feel lucky for sure00:42
jw4menn0: yeah? That's not unusual I think... but it's a long time to have disruptive sleep00:42
jw4menn0: for the parents I mean00:43
perrito666jw4: nope, it shows right?00:46
jw4perrito666: you don't look weary? ;)00:47
perrito666well if when I have kids they are anything like me the will begin to be good sleepers around 2100:47
perrito666years that is00:48
jw4perrito666: lol00:48
thumpermenn0: are you close to merging your env worker branch  ?00:52
perrito666wallyworld: you are a hard client to please :p00:53
wallyworld:-(00:53
wallyworlddid what i say make sense though?00:54
perrito666wallyworld: not really but I havent read the last review just the email00:54
wallyworldthe review is just a note or two00:54
wallyworldwe have existing code to set machine/unit status00:55
wallyworldso we need to use that instead of duplicating new code00:55
perrito666lets go priv so logs dont suck later00:55
wallyworldwe add the agent status setter and the apiserver layer thunks accordingly00:55
wallyworldok00:55
katcoaxw: wallyworld: boom! 2015-01-29 02:10:34 INFO juju.storage.provisioner provisioner.go:128 block devices created: [{1 1771eb70-b0ca-49e3-84d4-7b73b1401d18 /dev/loop102:11
katco    1  false}]02:11
axwkatco: sweet :)02:11
wallyworldwhoot02:11
katcogoing to sanity check the image rq, but we might be good02:11
wallyworldseems so02:11
axwkatco: are you generating a UUID for the volume ID now?02:12
katcoaxw: currently, yes. with todos to use the algorithm you suggested02:12
katcoaxw: i just didn't have a machine id to hang off of02:12
axwUUID would probably be fine TBH02:12
axwassuming you're creating the image file with that too02:13
katcoyes02:14
katcoaxw: i think we'll still have to put some thought into how to name these things so persisting across reboots isn't a pain02:14
katcoaxw: but that is trivial i think02:14
katco/dev/loop1: [0801]:51515891 (/var/lib/juju/storage/block/loop/var/lib/juju/storage/block/loop/1771eb70-b0ca-49e3-84d4-7b73b1401d18)02:14
katco 02:14
axwcool02:15
katcocharm is hello worlding along just fine, i'm going to push these changes and have dinner02:15
katcoi'll check back in after to see if there are any issues02:15
katcoaxw: https://github.com/wallyworld/juju/pull/402:18
wallyworldlooking02:18
katcoaxw: fair warning, the lxc tests are not passing (stupid trivial stuff to fix)02:18
katcowallyworld: ^^02:18
katcook i'm eating some dinner. bbiab02:19
wallyworldty02:21
rick_h_mattyw: let me know when the data is loaded and will check it out. It needs rating info because that's the api we hit02:30
rick_h_mattyw: so it needs the sub, the plan, and a starting rating data so that the my accout page in the UI had some records to it02:30
mattywrick_h_, stop02:31
rick_h_mattyw: party02:31
mattywthumper, ping?02:33
thumpermattyw: hey02:33
thumpermattyw: wwitzel3 reviewed that branch :)02:34
mattywthumper, hey there, ok cool - I didn't get around to much reviews yesterday so I'm going to try to get to some today02:34
mattywthumper, quick question for you02:34
thumpershoot02:34
mattywthumper, recently I've been wanting to find the envuuid from juju via the command line - I don't think we have one at the moment, and it feels like it might be something that would be useful to have in juju status02:35
mattywthumper, wanted to know what thoughts/ plans you had for that kind of thing02:35
thumpermattyw: there is one02:35
* thumper bootstraps to check02:35
thumper$ juju api-info environ-uuid02:36
thumper607aa9ab-dd2e-4943-86c3-3fc65b80045002:36
thumpermattyw: ^^02:36
thumperalthough, in status?02:36
thumpermaybe02:36
mattywthumper, awesome!02:37
mattywthumper, that certainly makes me happy for now02:37
mattywthumper, I didn't know about the api-info command - feels like Christmas :)02:37
mattywthumper, thanks for the help :)02:37
thumpernp02:37
mattywthumper, as along as it's available somewhere I'm happy for now02:38
thumperkk02:39
wallyworldaxw: another oarsum hack https://github.com/wallyworld/juju/pull/1602:46
axwwallyworld: looking02:46
axwwallyworld: reviewed02:49
wallyworldty02:49
katcowallyworld: i've addressed your concerns. ok if i merge?03:01
wallyworldkatco: would be awesome03:01
katcowallyworld: i think you might have to do it since i don't have write access03:02
wallyworldkatco: oh ffs i keep forgetting03:02
wallyworldkatco: merged \o/ tyvm03:02
wallyworldi'll try it out now03:03
wallyworldafter meeting03:03
katcowallyworld: i would feel a lot better if you or axw could poke at things and make sure this is working (multiple block devices etc.)03:03
wallyworldwill do03:03
katcowallyworld: i'm not familiar enough yet to know what edges there are03:03
axwkatco: sure, I'll have a play with it03:03
katcoaxw: tyvm03:03
katcook, i'm going to take care of a few things... please ping me if you find something!03:04
axwwallyworld: FYI, the loop devices won't get cleaned up if you destroy the machine03:53
axwwallyworld: we should be marking these as "persistent" later, and preventing env destruction till they're gone03:54
wallyworldaxw: i just noticed that03:54
wallyworldi ate my disk with a large loop device03:54
thumperaxw,wallyworld: either of you remember where is the cloud-init.log file is on cloud machines?04:01
axwthumper: I thought it was just /var/log/cloud-init-output.log ?04:01
thumperaxw: ta04:02
wallyworldwhat he said04:02
wallyworldaxw: sorta works, but something is not quite wired up - attached hooks complaining storage foo/0 is not provisioned, just digging into that now04:13
wallyworldthis is for loop on local04:13
axwwallyworld: yeah, I'm having some issues too but I've modified the provisioner04:13
wallyworldso close04:14
anastasiamacaxw: wallyworld: there r separate discussion going on..04:33
anastasiamacaxw: wallyworld: is it pool manager responsibility to vaidate provider type supplied04:34
anastasiamacaxw: wallyworld:?04:34
anastasiamacaxw: wallyworld: on pool create that is...04:34
wallyworlddepends if we think pool manager should know about envieons04:35
axwI guess not04:35
axwit should validate the storage provider type, but probably not whether it is valid for the current environment04:35
wallyworldthat is my initial thought, but i'm happy to be wrong04:35
axwit does make me a little nervous that something else will come along and use the pool manager without validating, that's all04:36
wallyworldmay need another component - EnvironPoolManager04:36
wallyworldor maybe that's overkill04:37
axwthat might make it clearer, but you could still go through the PoolManager directly (unless that gets hidden)04:37
wallyworldyes04:37
wallyworldnot too fussed either way04:37
wallyworldwas just trying to reduce external dependencies of pool manager04:38
wallyworldwould i think prefer an EnvironPoolManager to live in /environ/storage04:38
wallyworldand the api calls that04:38
axwI vote we defer this decision, and do validation in the apiserver code for now04:39
wallyworldyep, sounds good04:39
wallyworldwas about to say that :-)04:39
wallyworldam annoyed we have so much business logic in apiserver layer - need to fix that sometime04:40
thumperheh04:44
thumperwallyworld: I'm sure it is because we don't have a business layer04:45
thumperwe have apiserver and state04:45
thumperand people didn't want it in state04:45
thumperso guess where it goes04:45
wallyworldthumper: i know too well the reason - been annoyed about it for ages as you know04:45
thumperyeah.... I know04:45
* thumper has the munchies04:45
wallyworldthumper: if i just install django, is there a default page like for apache or tomcat etc?04:49
thumperkinda04:50
thumperyou mean locally or the charm?04:50
axwwallyworld: sorry, I think your SetProvisionedBlockDevices method is broken. it's only calling state.DemoSetProvisionedBlockDeviceInfo if err != nil above04:50
wallyworldthumper: i just want to expose the service and point a browser to it to see it worked04:51
thumperwallyworld: it might work, but not entirely sure04:51
wallyworldkk04:51
wallyworldaxw: ffs i'm stupid04:51
axwwallyworld: I'm changing the storage provisioner to periodically check for unprovisioned storage instances04:52
axwwallyworld: not using a watcher because the entities are in the wrong places (not associated with machine, but units)04:53
wallyworldcan you fix that stuff up while you're there?04:53
axwyes04:53
wallyworldta04:53
axwjust testing now04:53
wallyworlddontcha love hacks for demos04:53
menn0wallyworld: you too huh? :)04:58
wallyworldnoooooo, we aren't using *any* dirty hacks04:58
wallyworldwe're far better than that04:59
menn0wallyworld: of course not. we aren't either. ;-/05:00
wallyworldglad to hear it05:00
=== kadams54 is now known as kadams54-away
axwwallyworld: I think we should be running those LXC changes by someone, could be opening up security holes05:38
axwwallyworld: in particular, looks like we're dropping all AA protection05:39
wallyworldi think we can tighten up the rules05:39
wallyworldafter the demo :-)05:39
axwof coursed05:39
=== anthonyf is now known as Guest44978
wallyworldaxw: nice branch. we should also be able to register the new providers for ec2 as well as local i would expect?08:45
axwwallyworld: yep08:45
wallyworldwanna add that real quick?08:45
axwwallyworld: sure, just testing a tmpfs provider.. I think it needs more lxc config hacks to work08:46
axwlikewise with formatting/mounting block devices in local, I think08:46
wallyworldah bollocks08:46
axwwallyworld: actually I think I've just mucked up the mount syntax, but could still be broken... will come back to it08:47
wallyworldrighto08:47
axwwallyworld: testing rootfs on ec2, will take a few mins08:50
wallyworldsure, np08:50
wallyworldmy connection has been shit today, so ec2 testing is painful08:50
axwwallyworld: updated09:02
wallyworldty looking09:03
wallyworldthanks axw, i'll merge09:06
axwcool09:06
axwwallyworld: I was worried about nothing, tmpfs works on local09:13
wallyworldfarking oarsum09:13
axwwallyworld: https://github.com/wallyworld/juju/pull/1909:19
wallyworldlooking09:19
wallyworldsigh, ec2 is slooooow right now09:20
anastasiamacaxw:wallyworld: running tests after rebasing, m getting09:20
anastasiamaccmd/juju/storage/listformatters.go:11:2: cannot find package "github.com/dustin/go-humanize"09:20
anastasiamacaxw: ?09:20
axwanastasiamac: godeps -u dependencies.tsv09:20
wallyworldaxw: i did have a way of registering common providers but removed it due to import loops09:20
wallyworldi plan on adding it back later09:21
axwwallyworld: cool09:21
* anastasiamac rolles eyes09:21
anastasiamacaxw: of course :D09:21
axwwallyworld: unless you think there's anything else we need for next week, I'm going to take a step back from code and go back to the model rework09:22
wallyworldaxw: should storage-dir in tmpfs default to <libdir>/storage or similar?09:27
axwlibdir?09:27
wallyworldthe juju lib dir setting09:27
wallyworldcan't remember the name09:27
wallyworldcomes from agentconfig09:27
axwwallyworld: data-dir?09:28
axwwallyworld: as in /var/lib/juju/storage?09:28
wallyworldyeah09:28
axwisn't that where I put them?09:28
axw/var/lib/juju/storage/fs09:28
wallyworldah, haven't seen that in the diff yet09:29
wallyworldah there it is09:30
axwwallyworld: I really think we need to change the way that works, since (as I noted in a TODO) the data-dir isn't necessarily the same on all machines09:30
wallyworldat the bottom09:30
wallyworldit should be decided late09:30
axwso it can't be a property of the pool, but of the provider as instantiated in the agent09:30
wallyworldyep09:30
wallyworldwill do for now :-)09:31
axwyep09:31
wallyworldaxw: i'll fix that failing test anastasia mentioned. on my machine the cmd/juju tests *always* hang and time out :-(09:32
axwokey dokey09:33
axwthanks09:33
axwwallyworld: actually before I go off to do model things, one more thing... we should be able to use the storage subordinate now09:36
wallyworldaxw: only web app i can find that works with postgres is discourse and it fails to install - apt issues, fixed, but now sig verification failure. just seems too out of date09:36
axw:(09:37
wallyworldoh storage subordinate09:37
wallyworldhmmm09:37
wallyworldif there were a bundle with that with a web app charm, might be good09:37
wallyworldi'd still like to find a web apps just just uses postgres like wordpress uses mysql09:38
wallyworldi might see about making mysql chamr use storage09:38
wallyworldbut first food09:39
axwwallyworld: https://jujucharms.com/rails-example-single/6 ?09:40
axwI'll test it out..09:40
wallyworldnice, how'd you find that?09:40
wallyworldmy charm store search foo must be bad09:40
axwwallyworld: jujucharms.com, bundles09:40
axwlooked for one with an elephant :)09:41
wallyworld:-)09:41
wallyworldright, i need to eat first then charms09:41
voidspaceTheMue: https://github.com/voidspace/juju/compare/dimitern:wip-ec2-addressable-containers...wip-preparecontainerinterfaceinfo10:14
voidspaceTheMue: there's a comment in there "// TODO: (mfoord) we should add the machine ID to the IP"10:16
voidspaceTheMue: that's out of date and can  be removed (the call to addr.AllocateTo does this)10:16
wallyworld_axw: data=rootfs,10M   <- forces the use of the 10M or else it errors. we shouldn't need that for filesystemmounts10:24
axwwallyworld_: that's not always true. Ceph will need it...10:25
wallyworld_rightio, so it needs to be optional10:25
axwwallyworld_: yep. it should default to 1G anyway, which will then be ignored10:26
wallyworld_yep10:26
wallyworld_axw: loop file systems, as accessed from inside container, are owned by root. we will need to make the ownership ubuntu.ubuntu10:33
axwwallyworld_: do we? the charms run as root10:33
wallyworld_unity also notified me of a new device and showed the disk icon in the launcher. and i can see what files i create from inside the container10:33
axwnice :)10:33
wallyworld_ah yeah, fair enough10:33
axwwallyworld_: that fails app fails when I try to install it...10:34
wallyworld_joy10:34
axwwallyworld_: apparently you can use django, but I have nfi about it10:34
wallyworld_me either10:34
wallyworld_i was hoping django had a welcome page10:34
wallyworld_like apache, tomcat etc10:34
wallyworld_but i didn't see one10:35
wallyworld_axw: i think juju storage list should be sorted by owner. thoughts? i had a few units and would prefer to see storages for the units listed together10:39
menn0anyone able to look at this one line fix: http://reviews.vapour.ws/r/820/10:39
axwwallyworld_: yeah, probably10:39
axwwallyworld_: in the tabular format anyway10:39
wallyworld_menn0: oh, you ask all the hard questions10:39
wallyworld_axw: yep10:40
menn0wallyworld_: sorry :)10:40
menn0wallyworld_, axw: thanks guys10:40
wallyworld_anastasiamac: can you please sort the tabular storage list by owner? when you are free10:42
perrito666morning10:52
voidspaceperrito666: morning11:21
wallyworld__axw: this fixes some tests (yours and mine :-) and also makes default size 1G and default count 1 always https://github.com/wallyworld/juju/pull/2011:26
menn0wallyworld__, axw: pushing a branch now that fixes the panic that's currently being frequently seen during merge attempts11:27
wallyworld__which test?11:28
menn0wallyworld__: the leadership feature tests are failing frequently11:29
menn0the problem isn't actually with those tests but with my recent change to how the machine agent workers are started11:30
menn0subtle race11:30
wallyworld__ok, i hadn't noticed as ee've been landing to a feature branch11:30
wallyworld__great that you caught it11:30
menn0wallyworld__: http://reviews.vapour.ws/r/821/11:30
menn0wallyworld__: funnily enough, this related to a review comment you made :)11:31
wallyworld__oh, i hate to ask, which one?11:31
menn0wallyworld__: where we were discussing whether the envWorkerManager should clean up the returned runner even when an error was returned11:32
wallyworld__ah right11:32
menn0wallyworld__: I was using an unusual pattern, which you picked up on11:32
wallyworld__ancient history, > 1 week ago :-)11:32
menn0wallyworld__: this fix in the same area11:32
wallyworld__menn0: done, ty for fix11:34
menn0wallyworld__: thanks. good idea re the comment11:35
wallyworld__menn0: i can't recall, maybe even a comment next to where singular runner is used if it's releavnt to do so11:35
menn0wallyworld__: I actually have a better idea which is clearer and reduces the assumptions about the singular runner's behaviour11:36
wallyworld__ok11:36
menn0wallyworld__: pls take a look again. this is much clearer IMO. http://reviews.vapour.ws/r/821/diff/11:43
wallyworld__ok11:43
wallyworld__menn0: can we remove "&& singularRunner != nil" now?11:44
wallyworld__looks much nicer11:45
menn0wallyworld__: probably but I've been bitten by not have a guard like that before11:45
menn0wallyworld__: let me try it11:45
wallyworld__menn0: i can't see that it's needed as we exit above with an err before the defer11:45
menn0wallyworld__: yeah I know, but I've thought that before and had problems11:46
menn0wallyworld__: that was in a slightly different case though11:46
wallyworld__menn0: runner and singular runner usage looks the same and runner has no guard11:46
wallyworld__oh wait11:47
wallyworld__sorry11:47
wallyworld__can't read11:47
menn0wallyworld__: it does for the defer that cleans11:47
wallyworld__LGTM11:47
menn0wallyworld__: i'm going to remove them and re-run the tests and my little tester that was triggering the panics11:47
menn0wallyworld__: the guards irk me too11:47
wallyworld__ok11:47
menn0wallyworld__: yep, doesn't work. panics due to nil pointer dereference11:49
wallyworld__doh11:49
menn0wallyworld__: I don't get why... something special about defers perhaps and how they get executed?11:49
wallyworld__nfi right now11:49
menn0wallyworld__: I don't care right now. bed awaits and I still have some other (non-work) jobs to do11:50
wallyworld__yep, land tha sucker11:50
menn0wallyworld__: that's in. thanks for the late night review.12:15
wallyworld__np, catch you later12:15
wallyworld__really late for you12:15
menn0yeah :)12:15
jamwallyworld__: anything to go over before I see you in Capetown ?12:18
jamfwereade: ^^12:19
wallyworld__jam: nothing urgent - just need go over storage, health etc prior to seeing mark12:19
wallyworld__jam: there's a few things re: storage we can disuss over there12:20
=== kadams54-away is now known as kadams54
katcowallyworld__: axw: hey, just waking up. how are things with storage?13:23
wallyworld__katco: good :-)13:23
katcowallyworld__: woohoo!13:23
wallyworld__loop provider works13:23
wallyworld__was tweaked a bit13:23
katcowallyworld__: cool13:23
wallyworld__axw added tmpfs and rootfs13:24
katcowallyworld__: anything i should know going into today?13:24
katcovery cool13:24
wallyworld__katco: we do need to 1. evaluate the permissions, especially the aa ones on the container (but that can wait); 2. loop at mounting an external block device13:25
wallyworld__look not loop13:25
axwand bind mounting into local provider13:26
katcowallyworld__: understood on both accounts. 1 is part of a lot of cleanup we'll have to do13:26
wallyworld__yep13:26
katcowallyworld__: axw: i was planning on looking at 2 today13:26
wallyworld__doesn't matter for demo of course13:26
wallyworld__great :-)13:26
axwcool13:26
katcowallyworld__: axw: ok i'm off to get ready. you guys rock!13:27
katcoanastasiamac: and you as well if you're around13:27
axwkatco: later, hope you have a good day13:27
wallyworld__axw: not sure if you saw before, i fixed some tests and added default size handling https://github.com/wallyworld/juju/pull/2013:28
axwwallyworld__: I didn't. looking now13:28
wallyworld__i made count default to 1 regardless of size13:28
wallyworld__if count wasn't specified13:28
axwwallyworld__: reviewed13:37
wallyworld__ty13:37
wallyworld__axw: changes pushed13:44
axwwallyworld__: LGTM, thanks13:45
wallyworld__merged13:45
wallyworld__katco: also, if you're bored, there's also investigating how to provision block devices on openstack (as we do with EBS volumes on AWS)13:48
dimiternjam, hey, as OCR can you have a look at this please? http://reviews.vapour.ws/r/822/14:04
dimiternwallyworld__, axw, ^^14:16
dimiternwell not as OCRs but if you have some time :)14:16
=== kadams54 is now known as kadams54-away
=== kadams54-away is now known as kadams54
alexisbwallyworld__, axw you guys must be robots that don't need sleep15:08
jamestunnicliffedimitern: Could you review http://reviews.vapour.ws/r/823/ please?15:17
dimiternjamestunnicliffe, sure, looking15:17
mgz....snow15:18
=== kadams54 is now known as kadams54-away
=== kadams54-away is now known as kadams54
ericsnowperrito666: are we having standup? (no natefinch, no wwitzel3)16:05
perrito666ericsnow: we can wave it16:05
perrito666ericsnow: yet Ill priv you a question16:05
ericsnowperrito666: k16:05
TheMuevoidspace: why do you think so?16:10
voidspaceTheMue: no reason...16:10
jamestunnicliffedimitern: http://reviews.vapour.ws/r/824/ should work. Deleted the old merge and review request16:16
dimiternjamestunnicliffe, reviewed16:25
=== kadams54 is now known as kadams54-away
dimiternmgz, ping17:14
mgzdimitern: hey17:15
dimiternmgz, hey! so a quick question about the merge bot17:16
mgzhit me17:16
dimiternmgz, jamestunnicliffe - our new team mate wants to land a PR - https://github.com/juju/juju/pull/1502 - and he is in juju Hackers group on GH, has access to RB and all17:16
dimiternmgz, but it seems the bot is not picking up his $$merge$$ comment17:16
mgzjamestunnicliffe: have you set your membership of the juju group to public?17:17
dimiternmgz, ah! that might be it :)17:17
dimiternmgz, he went for some tea - will ask him in a moment :)17:17
mgzcool, poke if that doesn't help17:18
dimiternmgz, cheers! I can see his membership is private, that should be it17:18
dimiternvoidspace, http://reviews.vapour.ws/r/822/diff/17:28
=== anthonyf is now known as Guest89771
=== mwenning is now known as mwenning-rr5
perrito666wallyworld__:  you are going to hate me when you are back19:34
=== ChanServ changed the topic of #juju-dev to: https://juju.ubuntu.com | On-call reviewer: see calendar | Open critical bugs: 1416006
=== jam1 is now known as Guest8948
wwitzel3ericsnow: I'm back btw, just trying to finish up this final once over I promised for the cloudsigma provider19:53
ericsnowcool19:54
ericsnowwwitzel3: how'd it go?19:54
wwitzel3ericsnow: how is the services stuff going? also we should probably start poking people to have another look at GCE now that the first set of review fixes are up.19:54
alexisbwwitzel3, so are you officially moved in?19:54
ericsnowwwitzel3: I'm wrapping things up (hope to have an initial patch up soon)19:54
ericsnowwwitzel3: the systemd stuff on top of that will be almost trivial :)19:55
wwitzel3ericsnow, alexisb: went great, keys in hand, we are going to take some stuff over tonight and tomorrow evening and stay the night tomorrow :)19:55
ericsnow\o/19:55
wwitzel3ericsnow: yeah, I was thinking I would do that, no better test of an abstraction than having someone who didn't write it, us it :)19:55
wwitzel3s/us/use19:55
ericsnowwwitzel3: perfect19:55
alexisbwwitzel3, sweet, first nights are always fun19:56
wwitzel3alexisb: yeah, pizza on the floor as been my tradition since my first house ~12 years ago, so probably do that again19:56
=== liam_ is now known as Guest91492
menn0thumper, waigani: that CI blocker details are a little off but there is a problem20:25
menn0thumper, waigani: I suspect this is unrelated to the issue I already fixed20:25
waiganiyay20:25
menn0thumper, waigani: for whatever reason it seems much more likely on i386 than other archs20:25
menn0thumper, waigani: in fact it hasn't happened yet on amd6420:26
menn0waigani: you haven't committed any presence changes yet have you?20:26
waiganimenn0: nop20:26
menn0waigani: cause it's now the presence pinger that's dying20:26
waiganioh really?20:26
menn0waigani: I have a theory that the presence pinger is still active when state is shut down20:26
waiganimenn0: could it be one of our multi-env tests - sets up two states, and the pinger for each will currently listen to everything from all envs?20:28
menn0thumper, waigani : even though from a quick look at the code that doesn't seem possible20:28
menn0thumper, waigani : I'll keep digging post-kids swimming. I have to go now.20:28
waiganimenn0: cool.20:28
sinzuimenn0: Right, you branch fixed some of the panics CI saw, but not all :(20:40
=== kadams54 is now known as kadams54-away
=== kadams54-away is now known as kadams54
=== kadams54 is now known as kadams54-away
=== kadams54-away is now known as kadams54
=== kadams54 is now known as kadams54-away
=== arosales_ is now known as arosales
* thumper sad-face21:09
wallyworld__perrito666: that maas issue is easy to fix - i can do it. the time to wait is more than sufficient on vmaas but clearly not on hardware so i'll increase it21:12
perrito666wallyworld__: isnt there a better way?21:25
wallyworld__sadly no21:25
wallyworld__the deployment process involves a reboot21:25
wallyworld__and all juju can do is poll21:25
wallyworld__i discussed the approach with the maas guys21:26
thumperwaigani, menn0, cherylj, davecheney: http://paste.ubuntu.com/9943662/21:47
wallyworld__trivial review for 1.22 maas fix http://reviews.vapour.ws/r/825/21:47
thumperhuh...21:48
thumperwhy am I surprised that this is working?21:48
waiganithumper: is that juju status of another env? Nice!21:48
thumperwaigani: yep21:48
thumperand I just did 'juju ssh 0'21:48
thumperand got in to it21:48
waiganisweet21:49
waiganithumper: next test would be to expose a service, i.e. wordpress/mysql and visit it21:50
thumperthere are problems with the rsyslog worker21:50
thumperhmm...21:51
thumperinside the machine, the logdir is /var/log/juju21:51
thumperhowever on the host it is /var/log/juju-tim-another21:51
thumperand the rsyslog worker is looking for the host log dir not the machine log dir21:51
waiganithumper: extend RsyslogConfig to hold the dir path?21:54
thumperI'll be looking into it next21:55
thumperhuh21:56
thumperI now have ubuntu deployed in both environments21:56
thumpertime for some wordpress/mysql action21:56
waigani=D21:57
thumper:(22:17
thumpernow I have machines for the secondary environment, we need waigani's destroy branch22:17
* thumper taps fingers22:17
* thumper does things manually22:18
menn0thumper: that's awesome that it's working22:30
menn0jw4: ping22:30
menn0thumper: can you push what you have to your repo soon so I can grab it?22:31
sinzuiwallyworld_: I am fighting google22:32
jw4menn0: yep22:37
thumpermenn0: pushing it up now22:42
thumpermenn0: https://github.com/howbazaar/juju/tree/juju-environment-create22:46
menn0thumper: cheers22:48
menn0davecheney: do you know if "go test" run tests in parallel by default?23:08
waiganimenn0: I asked davecheney the same question a while ago: answer was no23:14
menn0waigani: interesting. one of the stacktraces i'm looking had has teardowns running from different test suites in different packages (in separate goroutines)23:15
waiganimenn0: with pingInfo, do we need to add a docID and leave Slot as an int64? i.e. the same as you suggested for beingInfo?23:15
waiganioh really?23:16
davecheneymenn0: the answer is yes23:16
davecheneyand no23:16
davecheneytests inside a package are run in sequence23:16
menn0waigani: re pingInfo, yeah that sounds right (so change the underlying field name for Slot)23:16
davecheneymultiple packages are tested in parallel, just like they are built in parallel23:16
menn0davecheney: ok, that explains that then. thanks.23:17
waiganiah right23:17
menn0davecheney: I hadn't noticed that before so it wierded me out23:17
davecheneymenn0: you've not noticed that running the tests uses all your cores ?23:21
davecheneysorry, that sounded passive agressive23:23
davecheneywhat i meant to say is23:23
davecheneygo test .../juju/... would take over an hour if packages were not tested in parallel23:23
menn0davecheney: I wasn't offended :)23:25
menn0davecheney: I run tests for the package i'm working on most of the time which just uses one core23:26
menn0davecheney: for full runs I often start it and walk away so don't really notice core usage23:26

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