veebersmenn0: right this run failed with the same message too. What's the most useful way to present the logs? Also, which logs would be useful? (I'm seeing that error in unit-mongodb-0.log.gz).00:39
veebersmenn0: oh wait, I can more than likely see the same thing with the jenkins run, ignore me for now :-)00:39
menn0veebers: if the jenkins run has everything then that's perfect01:23
rick_h_menn0: can you parallelize make check in any way?01:31
menn0rick_h_: it is parallelised, but by package01:32
menn0rick_h_: so if you're running tests in just one package they get run serially01:32
rick_h_menn0: hmm, ok. I guess it might have been done but didn't give me my console back. I see xx passed, 1 skip, 4 failed.01:35
rick_h_menn0: but yea, a second run does show it running pretty parallel now that I start over. https://flic.kr/p/JQCkPg01:36
menn0rick_h_: yep... the way that all cores get pegged for most of the test run is also a good indication01:37
* rick_h_ is more patient for run #201:37
menn0rick_h_: is your /tmp on tmpfs? That makes a big difference for test run time.01:46
rick_h_menn0: no, ty for the tip. i need to spend some time getting comfy with go dev and tricks like that02:00
* rick_h_ needa an induction sprint :p02:00
menn0rick_h_: haha np :)02:00
menn0axw: big review done. phew!02:05
menn0axw: nice work.02:05
axwmenn0: TYVM02:12
axwwallyworld: and you too02:12
veebersmenn0: I'm just trying to get a run with that error. I'm also trying to confirm that it's not a red-herring. I believe that there are issues as the host machine is network confined02:43
menn0veebers: seems unlikely that it could be related. the lease manager is a fairly internal thing.02:48
wallyworldmenn0: i've had a look at a race in juju ci tests - looks like a loggo issue due to the recent changes. i can't repro, but i think i have convinced myself where the issue is https://github.com/juju/loggo/pull/1802:50
menn0wallyworld: ok, give me 2 mins02:50
wallyworldsure, np02:50
mupBug #1608952 changed: Deployer: KeyError: 'uuid' connecting to environment <blocker> <ci> <deployer> <regression> <juju-core:Invalid by wallyworld> <juju-deployer:New> <https://launchpad.net/bugs/1608952>03:21
menn0wallyworld: sorry, got distracted by other work03:33
wallyworldno problemo03:33
menn0wallyworld: should the mutex be on the the writer itself perhaps?03:33
wallyworldmenn0: the writer is an interface and my thought was that if e do that, we then require all concrete implementations to do the muxtex, whereas doing it in the context is safer?03:35
menn0wallyworld: yeah, fair enough03:35
menn0wallyworld: ship it... the PR description uses muxtex twice instead of mutex though :)03:36
wallyworldit is well know i can't type03:36
menn0wallyworld: I think you can blame the drugs this time03:37
wallyworldsigh, see ^^^^03:37
=== menn0 is now known as menn0-school-run
wallyworldaxw: since menno is away, want a 2 line review? http://reviews.vapour.ws/r/5359/03:43
axwwallyworld: looking03:51
axwwallyworld: LGTM03:51
veebersmenn0-school-run: I can't get the jenkins run of that test to fail in the same way. I'll let you know if I can get it to happen at some point04:38
=== menn0-school-run is now known as menn90
=== menn90 is now known as menn0
menn0veebers: ok bummer. I can take a look at the logs you gathered earlier if you like.04:38
veebersmenn0: I'm not sure if they'll be interesting. It could be a red-herring due to the network confinement etc. on that machine. I'll double check and get back to you tomorrow? (i.e. I may be doing something odd there)04:50
blahdeblahIs there a way to push juju tools when you manually provision a machine, or must it always pull from machine 0?05:03
blahdeblahOr perhaps give it a different pull location?05:04
=== frankban|afk is now known as frankban
menn0veebers: no problems05:46
rogpeppeaxw: ping10:28
mupBug #1609343 opened: [aws; beta13] Juju can pick subnets from the wrong VPC with spaces constraints specified <ec2-provider> <usability> <juju-core:In Progress by dimitern> <https://launchpad.net/bugs/1609343>11:00
perrito666anyone uses vim plus ctrlp to work on juju?12:17
* perrito666 looks at rick_h_ 12:17
rick_h_perrito666: well I do use vim and ctrlp but don't consider I've "worked on juju" atm12:18
perrito666rick_h_: ever had missing files? even tuning the max depth and limits I still miss files12:19
rick_h_perrito666: not tried it enough to say tbh12:19
lazyPowerperrito666 - yeah thats weird. ctrlp has worked well for me even on large code bases.. but i admit i haven't tried it on the juju codebase12:52
lazyPowermight some other bug causing it to drop files12:52
perrito666lazyPower: very likely, in my case the files are missing even from the buffers list12:56
=== redelmann_ is now known as redelmann
mupBug #1609405 opened: DeploySuite.SetUpTest connection was forcibly closed <ci> <intermittent-failure> <regression> <unit-tests> <juju-core:Triaged> <https://launchpad.net/bugs/1609405>13:31
mupBug #1609407 opened: remoteFunctionalSuite.TestUsingTCP no such network interface <ci> <intermittent-failure> <regression> <unit-tests> <juju-core:Triaged> <https://launchpad.net/bugs/1609407>13:31
mbruzekballoons: Are you there?13:31
dimiternkatco, babbageclunk: quite a tiny review fixing bug 1609343, when you have a moment please!13:31
mupBug #1609343: [aws; beta13] Juju can pick subnets from the wrong VPC with spaces constraints specified <ec2-provider> <usability> <juju-core:In Progress by dimitern> <https://launchpad.net/bugs/1609343>13:31
mbruzekmgz: I am having trouble installing juju today, juju-core gives me 1.25.6 and the juju package does not seem to include the juju binary.13:35
rick_h_fwereade_: do you have time to help dimitern by reviewing http://reviews.vapour.ws/r/5363/ please so it might make b14?13:36
mgzmbruzek: well, that's interesting13:36
fwereade_rick_h_, looking13:36
mgzmbruzek: using what ppa?13:36
rick_h_fwereade_: ty13:36
mbruzekmgz: but tab completion works like a charm!13:36
mgzmbruzek: I have reports to the opposite with other setup13:37
mgzso... things in general are not quite right13:37
dimiternrick_h_, fwereade_: thanks! :)13:37
mbruzekmgz: I have both devel and stable in my source.list.d directory:13:38
mbruzekdeb http://ppa.launchpad.net/juju/stable/ubuntu xenial main13:38
mbruzekdeb http://ppa.launchpad.net/juju/devel/ubuntu xenial main13:38
mgzmbruzek: so, the fix is likely you just need to install `juju-1.25` but I'm a little concerned the dep chain is wrong for the packages you hae installed13:38
rick_h_dimitern: care to trade for fwereade_'s branch http://reviews.vapour.ws/r/5362/ please?13:38
rick_h_dimitern: before jumping into the next pool13:38
dimiternrick_h_: sure - I'll have a look before digging in around bug 160810513:39
mupBug #1608105: LXD no longer activates all interfaces on initial deploy when using MAAS2rc3 and JUJU Beta13 <juju-core:Triaged> <https://launchpad.net/bugs/1608105>13:39
mgzugh, the ppa packaging is just all a mess still >_<13:39
mbruzekmgz: what if I wanted 2.0-beta13?13:39
mgzthat's what just 'juju' should get you if you have the devel package installed13:39
mgzbut I'm not sure that mixing devel and stable ppas actually works how we want atm13:40
mbruzekmgz: It was necessary to have both devel and stable at one point to get juju installed.13:40
mbruzekmgz: I did not do this to anger you, it was needed at one point13:40
mgzmbruzek: yeah, and it certainly should work13:41
mbruzekmgz: I could remove stable and try again if you think that will work13:41
mgzwe just have too many different sets of packaging and they don't all play nicely with each other13:41
mgzmbruzek: pastebin `dpkg -l "juju*"` first13:42
dimiternfwereade_: you've got a review ;)13:42
mbruzekmgz: http://paste.ubuntu.com/22028864/13:43
fwereade_dimitern, cheers :)13:43
* dimitern whew.. that's got to be the longest bug and PR descriptions for a fix with diff size less than 1/3 of the length of both those descs..13:44
rick_h_dimitern: :P13:45
mgzmbruzek: thanks, go ahead and futz now and see if you can get a working set?13:45
fwereade_dimitern, and so do you :)13:46
dimiternfwereade_: tyvm!13:46
dimiternrick_h_: so how's the process now - with +1 I need to ask e.g. mgz (hey there ;) for a second +1 and test the QA steps?13:47
mbruzekmgz: I removed the stable ppa, updated and got this message when trying to install juju-core:juju-core is already the newest version (1.25.6-0ubuntu1~16.04.1~juju1).13:48
rick_h_dimitern: no, you can just check the QA steps locally using the doc for any shortcuts if it's something that's pretty QA'able13:48
rick_h_dimitern: nothing there involves the QA team, it's all checking each other13:48
dimiternrick_h_: I see - but do I still need 2x LGTM or not?13:49
mgzit would be nice to actually have functional test coverage for subnet stuff on aws13:49
rick_h_dimitern: no, we're not doing 2x +1 atm13:49
rick_h_dimitern:  more +1 and QA OK13:49
dimiternrick_h_: ok then, thanks13:50
dimiternmgz: there are a couple in featuretests/13:50
rick_h_mgz: yes, we need to get some more of that for the AWS stuff since it does't require too much custom setup/etc like MAAS13:51
dimiternmgz: can you give http://reviews.vapour.ws/r/5363/ a QA OK using the steps there please?13:51
rick_h_dimitern: it's on our own team to check QA. Maybe natefinch for katco or fwereade_13:52
rick_h_dimitern: unless mgz is just curious :)13:52
mbruzekmgz: This is me futzing: http://paste.ubuntu.com/22029893/13:54
mbruzekmgz: Shall I open a bug or is this just my system messed up?13:54
* dimitern leaves it for standup13:54
mgzmbruzek: wanting to install juju-1.25 seems right14:00
rick_h_dimitern: fwereade_ natefinch ping for standup14:01
natefinchrick_h_: omw14:02
mbruzekmgz: right but how do I get 2.0 ?14:11
mgzmbruzek: a bug is probably a good idea regardless14:12
mgzwe need something to track sorting out the ppas14:12
mbruzekmgz: juju-core or is there a packaging project?14:13
mgzjust juju-core14:14
mbruzekmgz: https://bugs.launchpad.net/juju-core/+bug/160943714:23
mupBug #1609437: Unable to install juju 2.0 using apt <juju-core:New> <https://launchpad.net/bugs/1609437>14:23
mgz...that's a scarier bug title than it should be :)14:23
mbruzekmgz: sorry what would you like me to change it to?14:24
mgzmbruzek: don't worry about it, I cna edit14:25
mbruzekdo ash you wish. I am not feeling creative today14:25
katcorick_h_: so apparently my bug is no longer a blocker; same gameplan?14:25
rick_h_katco: /me looks14:27
katcorick_h_: you responded to that thread last night it looks like14:27
rick_h_balloons: oh, was #1603221 the one that axw noted was not implemented in lxd?14:27
mupBug #1603221: Charms utilizing storage fail on LXD <blocker> <jujuqa> <lxd-provider> <storage> <juju-core:In Progress by cox-katherine-e> <https://launchpad.net/bugs/1603221>14:27
katcorick_h_: yeah14:28
rick_h_katco: ok, then let's play normal OCR please14:28
rick_h_fwereade_: dimitern and if it's EOD please work with katco to help do QA with the shared info before you head out14:28
katcorick_h_: sounds good14:29
mgzfwereade_: mind if I go ahead and land your ErrMissing branch for trunk? we want a new change to test.14:30
natefinchreview for anyone interested, fairly simple code: http://reviews.vapour.ws/r/5364/14:30
fwereade_mgz, be my guest :)14:30
mupBug #1609437 opened: Unable to install juju 2.0 using apt <juju-core:New> <https://launchpad.net/bugs/1609437>14:31
katcopoor axw... did we ever figure out what was going on with his review? http://reviews.vapour.ws/r/5355/14:53
dimiternrick_h_: sure, will check with katco14:55
dimiternbabbageclunk: ping15:00
babbageclunkdimitern: hey15:00
dimiternbabbageclunk: hey :) I've seen your comment re not seeing the expected subnet15:00
dimiternbabbageclunk: can you give me more details about it?15:00
babbageclunkdimitern: Sure - I got an error running `juju add-space test subnet-096b3f72 subnet-0c9eaf65`15:01
babbageclunkdimitern: error: invalid arguments specified: "subnet-096b3f72" is not a valid CIDR15:02
dimiternbabbageclunk: hmm .. odd - I'm trying now to see if I'd made a typo15:03
babbageclunkdimitern: If I look in the AWS console under network interfaces (which might not be the right place) I can see subnet-0c9eaf6 but not 096b3f7215:05
dimiternbabbageclunk: have a look at the dropdown menu above on the left (Services v), pick VPC from it and then Subnets15:05
babbageclunkdimitern: Oh, ok - I can see it in there.15:06
babbageclunkdimitern: So I don't know why I was getting that error then.15:07
dimiternbabbageclunk: you're right - that error wasn't exactly expected :/15:08
dimiternbabbageclunk: it will work in 3 steps though (with update the QA steps shortly) - i.e. add-space test; add-subnet subnet-xxx test; add-subnet subnet-yyy test;15:09
babbageclunkdimitern: ok cool - happy to try doing it again in a bit.15:10
dimiternbabbageclunk: if you can - great! ;)15:15
dimiternbtw the shared aws account has these additional users, which never logged in: natefinch, alexisb; while wallyworld, and mfoord haven't logged since 2015 - please ping/mail me if you want account setup info (or password reset, etc. if you forgot)15:28
mupBug #1609437 changed: Unable to install juju 2.0 using apt <packaging> <juju-core:Won't Fix> <juju-release-tools:Triaged> <https://launchpad.net/bugs/1609437>15:37
mupBug #1609463 opened: upgrade-charm command is inconsistent with deploy when using local charms <juju-core:New> <https://launchpad.net/bugs/1609463>15:37
=== frankban is now known as frankban|afk
* rick_h_ goes for lunchables17:03
mupBug #1609494 opened: grant-revoke: disabled user still in admin read-write <ci> <grant> <regression> <user> <juju-core:Triaged> <https://launchpad.net/bugs/1609494>17:07
mupBug #1609494 changed: grant-revoke: disabled user still in admin read-write <ci> <grant> <regression> <user> <juju-ci-tools:Triaged> <juju-core:Triaged> <https://launchpad.net/bugs/1609494>17:16
mupBug #1609494 opened: grant-revoke: disabled user still in admin read-write <ci> <grant> <regression> <user> <juju-ci-tools:Triaged> <juju-core:Triaged> <https://launchpad.net/bugs/1609494>17:19
natefinchkatco: default is a keyword.  It unfortunately is a very useful variable name, and there aren't many good alternatives.  Suggestions more than welcome.  I agree dflt is terribad.18:28
natefinchkatco (or anyone else) could use a review on this (sorry no reviewboard link, not sure what happened) https://github.com/juju/schema/pull/1318:57
natefinchI wish hexchat had a hotkey to change font size19:00
katconatefinch: lol oops forgot about that19:13
katconatefinch: d3fault19:13
katconatefinch: dfault19:14
* katco shrugs19:14
natefinchdfault is pretty good19:15
natefinchsorta sounds like a DJ name19:15
natefinchgah, reflection is so horrible... once you start using it, you can't ever take it out, because you never know who is relying on it :/19:27
natefinchI'm pretty sure we're always passing a map[string]interface{} to coerce, but because we're using reflection, I can't know for sure... in theory it can take map[Stringer]interface{} too, but I'm pretty sure no one does that.19:30
* rick_h_ goes to get the boy from camp19:33
perrito666katco natefinch: byOmission sort of applies to default in a very pompous way19:42
mupBug #1609041 changed: Race in github.com/juju/juju/cmd/modelcmd <blocker> <ci> <race-condition> <regression> <unit-tests> <juju-core:Fix Released by wallyworld> <https://launchpad.net/bugs/1609041>19:43
mupBug #1609041 opened: Race in github.com/juju/juju/cmd/modelcmd <blocker> <ci> <race-condition> <regression> <unit-tests> <juju-core:Fix Released by wallyworld> <https://launchpad.net/bugs/1609041>19:46
katcoperrito666: hmmm... context for that comment?19:47
perrito666katco: you where discussing a replacement for the word default19:47
katcoperrito666: ah19:48
perrito666and I used my super spanish powers to find an old and unused one :p19:48
mupBug #1609041 changed: Race in github.com/juju/juju/cmd/modelcmd <blocker> <ci> <race-condition> <regression> <unit-tests> <juju-core:Fix Released by wallyworld> <https://launchpad.net/bugs/1609041>19:55
babbageclunkthumper: sorry! Here now if you want to do a quick chat?20:53
thumperbabbageclunk: ok20:54
alexisbwallyworld, ping21:22
alexisbyo dawg21:22
alexisbwallyworld, did you impliment the show-model command21:23
wallyworldwas multi model work i think21:23
alexisbthumper, was it you?21:23
perrito666wallyworld: how are you feeling?21:23
wallyworldmuch better today thanks21:23
wallyworldhad a raging headache and sore throat yesterday21:24
katcowallyworld: didn't realize you were under the weather :( glad you're feeling better21:24
perrito666wallyworld: cool, then I wont feel guilty saying I addressing your comments and you should take a second look :p21:24
wallyworldkatco: tis all good :-)21:24
katcoi don't know how, but even a remote globally distributed team still seems to demonstrate properties of infection clusters21:25
wallyworldkatco: visruses spread using computers you know21:28
katcowallyworld: i bet i transferred it from my compooper when i got my drink out of its cup-holder21:29
wallyworldseems plausible21:29
perrito666well we are clever enought to get together once every semester, when its winter for half of the team21:32
perrito666we are not that smart21:32
alexisbanastasiamac, running late21:33
katcoalexisb: not sure if you're there, but we lost you21:44
perrito666katco: that answers the question21:49
katcoperrito666: apparently :)21:52
mupBug #1558703 changed: PatchValue unsafe for SetUpSuite <testing> <unit-tests> <juju-core:Fix Released by jameinel> <https://launchpad.net/bugs/1558703>21:56
axwredir: that bug that's assigned to you about grant/revoke: I have a fix for it already. it's bundled up with some other changes, not sure if we want to gate on them22:01
axwkatco: nfi what's up with reviewboard, but my PR was reviewed on github :)22:02
katcoaxw: ah ok :) good to hear22:02
perrito666axw: point me to the redir issue (redir is on holidays iirc)22:02
katcoaxw: your code was so good RB knew it didn't need to be reviewed22:03
perrito666axw: I am giving some love to grant/revoke22:03
axwperrito666: https://bugs.launchpad.net/juju-core/+bug/160949422:03
mupBug #1609494: grant-revoke: reenabled users missing from list-users <blocker> <ci> <grant> <regression> <user> <juju-ci-tools:Invalid> <juju-core:Triaged by reedobrien> <https://launchpad.net/bugs/1609494>22:03
axwkatco: heh, probably more like it was +1500/-1600 lines and it fell over :p22:03
perrito666I think i would gladly give time of my life to change reviewboard for something else that we can think is better for a few months before starting hating that too22:05
katcoperrito666: i think all diffs and comments should be pasted here, in irc.22:06
perrito666katco: I think emacs got to your nervous system and its not you talking22:06
axwwallyworld: is the leads call no more?22:06
wallyworldaxw: yeah, seems so22:06
wallyworldi thought we were still going to have it but seems not22:07
axwcool, got up at 5:45 for no reason -_-22:07
wallyworldaxw: i only found out by chance when i saw the meeting invite come through yesterday and it was cancelled22:07
alexisbwallyworld, ping22:13
alexisbsorry lost network, back now22:13
wallyworldok, see you in 1:122:13
menn0thumper: I've just done a big card cleanup. All the cards on the onyx board that were still relevant have been moved to the A team board22:24
thumpermenn0: cool22:25
thumpermenn0: can we chat about the i/o timeout bug?22:25
menn0thumper: sure22:25
menn0thumper: i'm there already22:26
mupBug #1600257 changed: Broken bash completion with old ppa packages present <bash-completion> <packaging> <verification-done> <juju-core:Invalid> <juju-core (Ubuntu):Fix Released> <juju-core (Ubuntu Xenial):Fix Released> <https://launchpad.net/bugs/1600257>23:38
axwveebers: FYI this is the branch for juju-ci-tools: https://code.launchpad.net/~axwalk/juju-ci-tools/cli-model-owner23:40
axwveebers: ignore the one linked from the PR23:40
mupBug #1600257 opened: Broken bash completion with old ppa packages present <bash-completion> <packaging> <verification-done> <juju-core:Invalid> <juju-core (Ubuntu):Fix Released> <juju-core (Ubuntu Xenial):Fix Released> <https://launchpad.net/bugs/1600257>23:44
veebersaxw: axk23:44
veebers:-\ ack23:44
mupBug #1600257 changed: Broken bash completion with old ppa packages present <bash-completion> <packaging> <verification-done> <juju-core:Invalid> <juju-core (Ubuntu):Fix Released> <juju-core (Ubuntu Xenial):Fix Released> <https://launchpad.net/bugs/1600257>23:47

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