/srv/irclogs.ubuntu.com/2015/10/02/#juju-dev.txt

rick_h_katco: pong00:25
axwlet the merge wars commence01:45
axwthumper wallyworld waigani anastasiamac: if you have things waiting, master is got unblocked01:46
wallyworld\o/01:46
axwnaturally I started mine merging before I told you01:46
waiganiaxw: lol, cheers01:46
mupBug #1501381 changed: panic: cannot pass empty version to VersionSeries() <blocker> <ci> <intermittent-failure> <juju-core:Fix Released by cherylj> <https://launchpad.net/bugs/1501381>01:53
mupBug #1501432 changed: BootstrapSuite tests fail on non-ubuntu platforms with no matching tools <blocker> <centos> <ci> <test-failure> <windows> <juju-core:Fix Released by cherylj> <https://launchpad.net/bugs/1501432>01:53
mupBug #1501786 changed: juju cannot provision precise instances: need a repository as argument <blocker> <ci> <precise> <regression> <juju-core:Fix Released by cherylj> <https://launchpad.net/bugs/1501786>01:53
=== Guest68817 is now known as anthonyjf
rick_h_perrito666: how big?02:18
perrito666rick_h_: something that can compile mongo in less than the time it takes for launchpad I am trying to fix the willy build of a mongo3 ppa and I dont have a willy nor intentions of breaking my laptop02:19
rick_h_perrito666: what's your LP id?02:20
perrito666rick_h_: hduran-802:20
rick_h_perrito666: oh bah, it's in use.02:23
perrito666rick_h_: ??02:23
rick_h_perrito666: well you can ssh to maas.jujugui.org and create an lxc container on that laptop and use it if you'd like, though tit's not that fast02:23
rick_h_perrito666: I thought we had an unused node in our guimaas the team uses02:24
perrito666rick_h_: I appreciate it, ill ponder on it if I dont fall asleep on the kb before02:24
rick_h_perrito666: but looks like the team put it to use02:24
rick_h_perrito666: ok, well let me know. We've got hardware if it'll help. However you should be able to create a wily lxc and do the builds in there and keep the laptop safe02:25
natefinchlol compiling mongo is super slow...  it took forever even on my quad core i7 with a beastly SSD.  That's C++ for you.02:26
rick_h_perrito666: worst case can ask urulama to take over the QA environment for a bit and get full machines02:26
natefinchI think it took like 45 minutes on my laptop02:27
rick_h_natefinch: have a 6 core 32gb machine if the instructions are easy for me to dupe02:27
natefinchrick_h_: I haven't done it in like a year, but IIRC, the directions were terrible02:29
rick_h_natefinch: :/ well if it would help happy to try to get something better working. shoot, maybe just get a giant ec2 machine for a bit. but understand if it's just a pita02:30
perrito666it is sort of a pita, especially the one I am trying to build, which is a debian package02:31
natefinchrick_h_: it's a good lesson for why perrito666 shouldn't buy such namby pamby laptops02:31
rick_h_lol02:32
rick_h_laptops are for portability, many core desktops with lots of ram for real work :)02:32
perrito666natefinch: my laptop kicks ass, I just dont want to install willy on it02:32
natefinchhaha02:32
rick_h_perrito666: but surely in an lxc it's safe?02:32
perrito666rick_h_: ill give it a shot, I hadn't thought of it02:32
natefinchperrito666: just giving you shit :)02:32
perrito666natefinch: so is mongo :p02:33
perrito666I really need to buy some heavy lifting machine I am just too lazy to import it02:33
perrito666and to cheap to pay local price for what I can get here02:33
natefinchperrito666: probably easier to just get us to stop using mongo02:33
natefinchperrito666: I saw our new competitor nomad uses boltdb... which blows mongo out of the water as far as compile time and ease of setup.  Of course, it doesn't have replication, so they must be doing that themselves somehow.. but still.. I love boltdb... pure Go, man.  That's the way to do it.02:35
cmarsI like boltdb, i've used it in some side-projects and it's always been great to work with. I've never really pushed it in terms of concurrent txns though.03:18
mupBug #1502016 opened: Juju-quickstart error message lacks detailed error info <canonical-bootstack> <juju-core:New> <https://launchpad.net/bugs/1502016>07:11
mupBug #1502016 changed: Juju-quickstart error message lacks detailed error info <canonical-bootstack> <juju-core:New> <https://launchpad.net/bugs/1502016>07:14
mupBug #1502016 opened: Juju-quickstart error message lacks detailed error info <canonical-bootstack> <juju-core:New> <https://launchpad.net/bugs/1502016>07:17
TheMuefwereade: Happy Birthday08:37
dimiternfwereade, oh wow, happy birthday indeed! :)08:48
anastasiamacfwereade: ¸¸♬·¯·♩¸¸♪·¯·♫¸¸Happy Birthday To You¸¸♬·¯·♩¸¸♪·¯·♫¸¸08:50
fwereadeTheMue, dimitern, anastasiamac: <308:58
* fwereade is tired and is taking a pre-travel swap day to relax08:59
voidspacefwereade: hippy bathday fella09:02
voidspacefwereade: enjoy the rest and seattle09:02
fwereadevoidspace, cheers :)09:02
=== Spads_ is now known as Spads
mfoordfrobware: dimitern: conflicts resolved and unit public address branch set to attempt to land on master (again...)09:21
dimiternmfoord, awesome!09:21
mfoordfrobware: dimitern: ec2 subnets branch also landing on master09:21
dimiternmfoord,09:21
dimiternmfoord, and it did land on 1.25 alread?09:22
mfoorddimitern: yup, already landed there09:22
* dimitern gaahh I seem to be eating letter today09:22
mfoord:-)09:22
rogpeppemfoord: fancy a review? http://reviews.vapour.ws/r/282209:34
mfoordrogpeppe: sure, coffee first09:36
rogpeppemfoord: ta!09:36
mfoordrogpeppe: the only wart is having assertErrorResponse return the failure for a single use case10:11
rogpeppemfoord: yeah, i had to think a bit about that10:12
rogpeppemfoord: if you've got a better suggestion... ?10:12
mfoordrogpeppe: haha, not sure I do :-)10:12
mfoordI'll have another look10:12
mfoordthe rest is straightforward10:12
rogpeppemfoord: thanks10:13
mfoordit's a common pattern in error testing (or testing in general) - to have some detail that only needs checking once10:13
mfoordrogpeppe: as doer is only used once, and it's a one line function, does it need to be a separate function?10:19
dimiternmfoord, is a Doer constructed by a Doerer ?:D10:22
mfoorddimitern: the doerer is the function that does the doer10:23
mfoordand what that does, well...10:23
dimiternis Do()es10:24
dimiternit even10:24
mfoordrogpeppe: LGTM with the minor proviso about (maybe) inlining doer10:25
rogpeppemfoord: thanks10:25
rogpeppemfoord: it's like that in the other tests10:25
mfoordrogpeppe: ah right, fair enough10:26
rogpeppemfoord: it could be inlined, but it's like it is to make it trivial to make other similar tests10:26
mfoordok10:26
mfoordfine, drop the issue then10:26
rogpeppemfoord: done :)10:49
rogpeppefwereade: ping10:50
TheMuerogpeppe: he's swapping due to Seattle10:57
rogpeppeTheMue: ah, np10:57
dimiternTheMue, reviewed - quite a lot of comments I'm afraid, ping me if anything is unclear11:06
TheMuedimitern: ok, and I appreciate any comments11:06
TheMuedimitern: jfi, I'll fix it in the 1.25 branch first but mark the issues in your review. later we can forward-port the 1.2511:16
dimiternTheMue, cheers, sounds good11:26
bogdanteleaganiemeyer: is -run deprecated in gocheck in favor of check.f?11:43
TheMuedimitern: great input, you speak juju networking fluently ;) changes are now visible in http://reviews.vapour.ws/r/2823/11:46
dimiternTheMue, looks good, thanks! I'll leave frobware / voidspace to proof-read as discussed11:50
TheMuedimitern: cheers11:51
frobwareTheMue, I have a few comments - just wondering whether you want to address dimiter's first before I add mine12:36
TheMuefrobware: I already have done and pushed it, see http://reviews.vapour.ws/r/2823/.12:37
TheMuefrobware: dimitern reviewed my master branch, this is the 1.25 one.12:37
frobwareTheMue, I've add some comments but will want to take another pass. Just adding and publishing now so you're not blocked. OK?12:44
TheMuefrobware: great, thanks12:44
TheMuefrobware: deploy.go:118, singular "space" or plural "spaces"? the latter sounds more natural to me.12:54
frobwareTheMue, You could drop this bit "but not of the cmd and the database space"12:57
TheMuefrobware: ok, thx12:58
frobwareTheMue, leaving it as "deploy 2 instances of haproxy on cloud instances inside subnets of the dmz space only"12:58
TheMuefrobware: short and clear, sounds even better ;)12:59
frobwareTheMue, also note I saw a reference to "carret" which should be "caret"13:00
TheMuefrobware: fixed13:01
niemeyerbogdanteleaga: Not deprecated per se.. -run still has its usual meaning, but it's orthogonal to gocheck.. The latter has check.f for its own use13:14
bogdanteleaganiemeyer: hmm, -run doesn't seem to work for me, while check.f does with the same simple regex13:15
mupBug #1502127 opened: TestNoUniterUpdateStatusHookInError never reached desired status <ci> <intermittent-failure> <unit-tests> <juju-core:Triaged> <https://launchpad.net/bugs/1502127>13:27
mupBug #1502130 opened: charm upload fails often on maas <charm> <ci> <maas-provider> <regression> <juju-core:Triaged> <https://launchpad.net/bugs/1502130>13:27
mupBug #1502139 opened: Panic in TestUpgradeJuju on wily and vivid <ci> <intermittent-failure> <panic> <unit-tests> <vivid> <wily> <juju-core:Triaged> <https://launchpad.net/bugs/1502139>13:30
mupBug #1502139 changed: Panic in TestUpgradeJuju on wily and vivid <ci> <intermittent-failure> <panic> <unit-tests> <vivid> <wily> <juju-core:Triaged> <https://launchpad.net/bugs/1502139>13:33
mupBug #1502139 opened: Panic in TestUpgradeJuju on wily and vivid <ci> <intermittent-failure> <panic> <unit-tests> <vivid> <wily> <juju-core:Triaged> <https://launchpad.net/bugs/1502139>13:36
mupBug #1502139 changed: Panic in TestUpgradeJuju on wily and vivid <ci> <intermittent-failure> <panic> <unit-tests> <vivid> <wily> <juju-core:Triaged> <https://launchpad.net/bugs/1502139>13:51
mgzwhat are you on mup...13:52
mupBug #1502139 opened: Panic in TestUpgradeJuju on wily and vivid <ci> <intermittent-failure> <panic> <unit-tests> <vivid> <wily> <juju-core:Triaged> <https://launchpad.net/bugs/1502139>13:54
niemeyerbogdanteleaga: Probably because you're expecting -run to do something it doesn't do13:55
bogdanteleaganiemeyer: the docs seem to say they do pretty much the same thing(-run and check.f), just that check.f takes suites as well13:57
mupBug # opened: 1502149, 1502153, 1502154, 150215813:57
frobwaredimitern, I was just getting back to the demo. Which region did you create the subnets in?14:00
dimiternfrobware, eu-central-114:00
TheMuefrobware: dimitern: done, now just swapping offices. if branch is ok I will immediately merge it into 1.25 and then port it to master14:00
dimiternTheMue, cheers, will have one last look14:01
mupBug #1502140 opened: panic in lastlogin upgrade step <blocker> <ci> <regression> <upgrade-juju> <juju-core:Incomplete> <juju-core 1.25:Triaged> <https://launchpad.net/bugs/1502140>14:07
frobwaredimitern, TheMue: taking another look14:09
niemeyerbogdanteleaga: Just that they are different, yes :-)14:09
mupBug #1502140 changed: panic in lastlogin upgrade step <blocker> <ci> <regression> <upgrade-juju> <juju-core:Incomplete> <juju-core 1.25:Triaged> <https://launchpad.net/bugs/1502140>14:10
frobwaredimitern, how do I get to eu-central? My choices are frankfurt or ireland.14:12
mgzthe 1.25 branch is blocked at present, see ^ that bug14:18
mupBug #1502140 opened: panic in lastlogin upgrade step <blocker> <ci> <regression> <upgrade-juju> <juju-core:Incomplete> <juju-core 1.25:Triaged> <https://launchpad.net/bugs/1502140>14:19
dimiternfrobware, eu-central-1 is in frankfurt :)14:27
frobwaredimitern, yeah. I noticed. <face palm>14:31
frobware:)14:31
natefinchfwereade: you around?14:38
natefinchericsnow: anything I need in the environments.yaml besides type: lxd?15:04
ericsnownatefinch: nope15:05
katcofwereade: hey can you give a high-level review of http://reviews.vapour.ws/r/2814 ? i'm doing a detailed review, but i want to make sure we're not doing anything wrong, conceptually15:08
katcofwereade: i.e. i think it will need a +2 from you15:08
katconatefinch: i see several places ericsnow's multi error patch would help :)15:09
katcoif only it were in the codebase.... le'sigh15:09
ericsnowkatco: what!!! someone isn't opposed!  ;)15:10
katcoericsnow: hey, i've always thought it was a good idea15:10
katcoericsnow: very common patterns15:10
ericsnowkatco: all providers use it (and not for necessarily API-related stuff)15:11
ericsnow(it = the pattern)15:11
natefinchdefinitely a common APIMultiError  would be good.... it's just proposed as being more general than it needs to be now15:12
ericsnowkatco: check out Environ.Instances in conjunction with ErrNoInstances and ErrPartialInstances15:12
ericsnownatefinch: I wrote it in the first place to meet a non-API use case15:13
katconatefinch: yeah not sure that's true... ericsnow ran into a place yesterday15:13
natefinchit just assumes you have an id for each error, which is not always the case... but it would fit many cases, most of which are bulk calls in the API15:14
ericsnownatefinch: the ID is neither required not necessarily unique15:15
ericsnow(not -> nor)15:15
perrito666bbl people15:16
mgznatefinch: I don't like that retry function much15:16
mgzmeans all those tests will have an extra 3s ceiling runtime?15:17
natefinchmgz: we have to wait for the worker... I don't know how to get around that15:17
natefinchmgz: and in theory it's up to 3 seconds... though in small scale tests I did find it took a couple seconds for the watcher to notify the worker and for the worker to do its job15:19
natefinchmgz: I don't know why the worker/watcher cycle is so slow15:20
natefinchmgz: I also don't know if that slowness is outside the norm.... are they usually instant?  I don't know the details of how the watchers and workers work.... but I know there's a lot of machinery between them.15:21
mgznatefinch: yeah, i don't have a good answer on overall speed, but why not usin LongAttempt15:22
natefinchmgz: I thought about using the attempt code, but it didn't really fit with the pattern I wanted, IIRC'15:23
natefinchmgz: I could certainly look at it again15:24
natefinchmgz: on second thought, I could make that work.. I missed the fact that I could call HasNext()15:25
mgznatefinch: that would make me happier, at least we're staying with the existing constants15:26
natefinchmgz: makes me happier, too.15:29
mupBug #1502202 opened: toolsversionchecker worker failing <juju-core:New> <https://launchpad.net/bugs/1502202>15:37
natefinchmgz: fixed :)15:37
mgznatefinch: thanks!15:38
natefinchmgz: thanks for getting me to look at the attempt code again.  I definitely prefer using something already written than handcrafting my own.15:38
aisraeljcastro: alexisb: lp:1502202 is the first I found. I'm trying to recreate the second.15:41
mupBug #1502202 changed: toolsversionchecker worker failing <juju-core:New> <https://launchpad.net/bugs/1502202>15:43
mupBug #1502202 opened: toolsversionchecker worker failing <juju-core:New> <https://launchpad.net/bugs/1502202>15:46
aisraeljcastro: alexisb: I can't recreate the second issue. I've been running 1.25 since alpha1 for daily work and it's been solid except for the previously mentioned issue. Kudos.15:55
katconatefinch: that is some hairy code. lgtm, but i think we really need someone like fwereade to look at it16:17
fwereadekatco, natefinch: link? (no promises today though)16:18
katcofwereade: http://reviews.vapour.ws/r/2814/16:18
fwereadekatco, cheers16:18
katcofwereade: a high-level overview should suffice. just need to make sure we're conceptually doing the right thing.16:18
katcofwereade: this is the combining txns you and natefinch have been discussing16:19
fwereadekatco, so I see -- awesome :)16:19
natefinchfwereade: I have a couple concerns... I'm using raw ids across the API... I think that's a no-no, but not sure if I should be creating a new tag just for my new mini collection16:23
fwereadenatefinch, for watcher results it's the least wrong thing to do actually16:24
fwereadenatefinch, might be inclined to go with unit ids though16:24
natefinchfwereade: that's certainly doable16:24
natefinchfwereade: the other one is that this is a general worker, but maybe it should be a singular worker?  We don't want multiples of these things doing assignments of the same things at the same time16:25
fwereadenatefinch, (translating id->tag at apiserver Next() time has complications -- but that's where it should be fixed, and for every watcher at once)16:25
fwereadenatefinch, I'd favour making it safe when racing with itself16:25
fwereadenatefinch, singular is a bit yucky16:26
fwereadenatefinch, but you're probably right16:26
fwereadenatefinch, yeah, make it singular, we just need a less layer--breaky singular implementation16:26
natefinchfwereade: yeah, I started to go down the singular route, but all the current examples seem to use state directly, which I know is a Bad Thing... and trying to get it working with the API was tricky16:26
fwereadenatefinch, yeah, it should just affect where you launch it in  the machine agent16:27
fwereadenatefinch, there's a singular wrappper func which at least keeps the evil out of its clients' hair16:27
fwereadenatefinch, gtg for now16:27
natefinchfwereade: ok, thanks for the input16:27
natefinchericsnow: gah, I hate having methods on a type outside the file where that type is defined... when we get some time, can we maybe not do that?18:03
ericsnownatefinch: yeah, we should split types like that up along the same line as the files (and compose them into the main type)18:05
ericsnownatefinch: otherwise we end up with enormous files18:05
natefinchericsnow: well.... if it's done well, not not just done to get the methods out of the same file, I think that's ok.  But given that the environ.go file is only 147 lines right now, I think we have a long way to go before we need to start breaking things up.18:07
alexisbfrobware, you potentially around?18:26
perrito666how do you people keep feature branches up to date?18:27
natefinchperrito666: merge from master once in a while18:27
natefinchperrito666: it's not fun18:27
perrito666natefinch: was thinking on the details :)18:27
perrito666pr?18:27
perrito666direct merge?18:27
natefinchperrito666: meh, just merge18:27
natefinchrebase if you're feeling lucky18:28
perrito666natefinch: and you push that to the feature branch directly?18:28
natefinchperrito666: oh uh... hmm..18:29
natefinchperrito666: I guess PR.. sorry, wasn't thinking18:29
natefinchperrito666: though it's generally not something that can really get reviewed18:29
perrito666mm, can I get permissions to push into a branch in our repo?18:35
mupBug #1502140 changed: panic in lastlogin upgrade step <blocker> <ci> <regression> <upgrade-juju> <juju-core:Incomplete> <juju-core 1.25:Triaged by cherylj> <https://launchpad.net/bugs/1502140>18:40
mupBug #1502140 opened: panic in lastlogin upgrade step <blocker> <ci> <regression> <upgrade-juju> <juju-core:Incomplete> <juju-core 1.25:Triaged by cherylj> <https://launchpad.net/bugs/1502140>18:43
mupBug #1502140 changed: panic in lastlogin upgrade step <blocker> <ci> <regression> <upgrade-juju> <juju-core:Incomplete> <juju-core 1.25:Triaged by cherylj> <https://launchpad.net/bugs/1502140>18:46
mupBug #1502306 opened: cannot find package gopkg.in/yaml.v2 <blocker> <ci> <regression> <juju-core:Incomplete> <juju-core lxd-provider:Triaged> <https://launchpad.net/bugs/1502306>19:44
mupBug #1502314 opened: juju-deployer never completes landscape deployment to vmLDS <kanban-cross-team> <landscape> <falkor:Triaged> <juju-core:New> <https://launchpad.net/bugs/1502314>20:29
mupBug #1502314 changed: juju-deployer never completes landscape deployment to vmLDS <kanban-cross-team> <landscape> <falkor:Triaged> <juju-core:New> <https://launchpad.net/bugs/1502314>20:32
mupBug #1502314 opened: juju-deployer never completes landscape deployment to vmLDS <kanban-cross-team> <landscape> <falkor:Triaged> <juju-core:New> <https://launchpad.net/bugs/1502314>20:35
alexisbaisrael, jcastro, I really appreciate the feedback on 1.25, that is great!20:41
mupBug #1502314 changed: juju-deployer never completes landscape deployment to vmLDS <kanban-cross-team> <landscape> <falkor:Triaged> <juju-core:New> <https://launchpad.net/bugs/1502314>20:44
mupBug #1502314 opened: juju-deployer never completes landscape deployment to vmLDS <kanban-cross-team> <landscape> <falkor:Triaged> <juju-core:New> <https://launchpad.net/bugs/1502314>20:47
beisneralexisb, do we have an eta on the fix for the being ppa-consumable somewhere? https://bugs.launchpad.net/juju-core/+bug/133588521:38
mupBug #1335885: destroy-environment reports WARNING cannot delete security group <amulet> <cloud-installer> <destroy-environment> <landscape> <openstack-provider> <security>21:38
mup<uosci> <juju-core:In Progress> <juju-core 1.24:Fix Committed by thumper> <juju-core 1.25:In Progress by thumper> <https://launchpad.net/bugs/1335885>21:38
alexisbheya beisner22:06
alexisbwe have a fix committed for that bug for 1.24, but we are currently working to get a blessed build for 1.24 on CI22:06
alexisbbeisner, eta for a 1.24.7 release is next week pending a blessed CI revision22:07
beisneralexisb, woot, thanks23:48

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