/srv/irclogs.ubuntu.com/2016/04/08/#juju-dev.txt

mupBug #1567598 changed: storage-get causes state server to 'error: bad request' <storage> <juju-core:Invalid> <https://launchpad.net/bugs/1567598>00:06
mwhudsonperrito666: ping?00:08
thumperdamn you schema.ForceInt00:09
thumperschema.Int -> int6400:09
thumperscheam.ForceInt -> int00:09
thumperfor no good reason00:09
thumperbah00:09
alexisbmwhudson, it is very much past his eod, if you can open a bug with the info you have that owuld be most helpful00:10
mwhudsonalexisb: ok00:11
mwhudsonalexisb: was actually typing up a bug report now, as it happens :-)00:11
alexisbperfect :)00:11
mwhudsonalexisb: https://bugs.launchpad.net/juju-core/+bug/156770800:14
mupBug #1567708: unit tests fail with mongodb 3.2 <juju-core:New> <https://launchpad.net/bugs/1567708>00:15
alexisbmwhudson, thank you!00:15
mwhudsonnp00:15
mwhudsonsome of the errors are clearly quite silly00:15
mwhudson(like the error message mongo gives when reporting an auth failure has changed)00:16
mwhudsoni don't know if any are deep00:16
mupBug #1567708 opened: unit tests fail with mongodb 3.2 <juju-core:New> <https://launchpad.net/bugs/1567708>00:21
natefinchevening all00:31
mupBug #1567708 changed: unit tests fail with mongodb 3.2 <juju-core:New> <https://launchpad.net/bugs/1567708>00:33
rediraxw: yt?00:36
mupBug #1567708 opened: unit tests fail with mongodb 3.2 <juju-core:New> <https://launchpad.net/bugs/1567708>00:36
mupBug #1567712 opened: [juju create-model] Specifying a cloud when referring to a credential is redundant <docteam> <juju-core:New> <https://launchpad.net/bugs/1567712>00:51
axwredir_afk: "yt"?00:51
davecheneycherylj: http://pad.lv/156630300:53
davecheneydid this pass CI testing ?00:53
davecheneyi have access to a windows machine now and can investigate if it's still failing00:54
redirI am00:55
rediraxw: yt == you there00:55
axwredir: oh, well, you have your answer ;)00:55
rediraxw: do you have a minute for a hangout?00:55
axwredir: sure, see you in the tanzanite hangout00:55
redirk00:55
mupBug #1567712 changed: [juju create-model] Specifying a cloud when referring to a credential is redundant <docteam> <juju-core:New> <https://launchpad.net/bugs/1567712>00:57
* thumper heads out to walk the dog01:03
mupBug #1567712 opened: [juju create-model] Specifying a cloud when referring to a credential is redundant <docteam> <juju-core:New> <https://launchpad.net/bugs/1567712>01:06
cheryljdavecheney: no, the windows test failed again:  http://reports.vapour.ws/releases/386301:15
davecheneycherylj: thanks, i'll try to get on the windows test machine now01:20
davecheneycherylj: i don't understand that page01:21
davecheneywhere is the windows failure ?01:21
cheryljrun-unit-tests-win2012-amd6401:21
davecheneyok i see it01:21
mupBug #1567458 changed: destroy-controller message and failure is not user friendly <juju-core:New> <https://launchpad.net/bugs/1567458>01:27
mupBug #1567458 opened: destroy-controller message and failure is not user friendly <juju-core:New> <https://launchpad.net/bugs/1567458>01:36
redircherylj: so you started landing those updates?01:37
redirhelp text updates.01:37
cheryljredir: I made an Executive Decision01:37
cheryljand pulled the trigger01:37
cherylj:)01:37
redirOK01:37
davecheneymenn0: https://bugs.launchpad.net/juju-core/+bug/147460701:38
mupBug #1474607: worker/uniter/relation: HookQueueSuite.TestAliveHookQueue failure <go1.5> <go1.6> <i386> <juju-core:Incomplete> <juju-core 1.25:Triaged> <https://launchpad.net/bugs/1474607>01:38
redirso I should fix up that other one and then resubmit or just merge?01:38
davecheneythis bug is fixed on master bug fires on 1.2501:38
redircherylj: ^^01:38
davecheneyfrom memory it was a _lot_ of work to fix on master01:38
cheryljredir: as long as you test that the help text looks right, then i'd say just merge01:38
redircherylj: OK. And have you settled on some formatting? indents all with line breaks etc?01:39
cheryljredir: basically, I copied what was in the bug for the most part, taking care to make sure lines were < 80 characters01:40
menn0davecheney: that bug doesn't ring any bells for me sorry01:40
redirOK. I'll do that, too, for the most part:)01:40
redircherylj: ^01:42
cheryljthanks, redir :)01:42
davecheneymenn0: i think this was fixed by william's manifold thinggy01:44
menn0davecheney: it won't be the manifold (dependency engine) stuff directly. that's got nothing to do with the uniter's internals.01:45
mupBug #1567458 changed: destroy-controller message and failure is not user friendly <juju-core:New> <https://launchpad.net/bugs/1567458>01:45
menn0davecheney: there have been many other improvements in the uniter recently though01:45
mupBug #1567719 opened: help text for juju list-shares needs improving <helpdocs> <juju-core:New> <https://launchpad.net/bugs/1567719>01:45
mupBug #1567721 opened: help text for juju set-default-credential needs improving <helpdocs> <juju-core:New> <https://launchpad.net/bugs/1567721>01:45
davecheneymenn0: that's what i'm worried about01:46
davecheneywe cannot backport them all01:46
mupBug #1567719 changed: help text for juju list-shares needs improving <helpdocs> <juju-core:New> <https://launchpad.net/bugs/1567719>01:48
mupBug #1567721 changed: help text for juju set-default-credential needs improving <helpdocs> <juju-core:New> <https://launchpad.net/bugs/1567721>01:48
mupBug #1567719 opened: help text for juju list-shares needs improving <helpdocs> <juju-core:New> <https://launchpad.net/bugs/1567719>01:54
mupBug #1567721 opened: help text for juju set-default-credential needs improving <helpdocs> <juju-core:New> <https://launchpad.net/bugs/1567721>01:54
mupBug #1567722 opened: help text for juju list-credentials needs improving <helpdocs> <juju-core:New> <https://launchpad.net/bugs/1567722>02:07
mupBug #1567724 opened: help text for juju add-cloud needs improving <helpdocs> <juju-core:New> <https://launchpad.net/bugs/1567724>02:07
mupBug #1567726 opened: help text for juju disable-user needs improving <helpdocs> <juju-core:New> <https://launchpad.net/bugs/1567726>02:07
mupBug #1567722 changed: help text for juju list-credentials needs improving <helpdocs> <juju-core:New> <https://launchpad.net/bugs/1567722>02:10
mupBug #1567724 changed: help text for juju add-cloud needs improving <helpdocs> <juju-core:New> <https://launchpad.net/bugs/1567724>02:10
mupBug #1567726 changed: help text for juju disable-user needs improving <helpdocs> <juju-core:New> <https://launchpad.net/bugs/1567726>02:10
menn0cherylj: I think I've figured out that local charm hash mismatch bug and I don't think it'll be hard to fix02:12
cheryljmenn0: excellent02:13
cheryljmenn0: there's plenty more for you once you're done with it ;)02:13
menn0cherylj: oh I know :)02:13
cheryljmenn0: I also still have that mgopurge problem from mario that I haven't done and could use help with :)02:14
menn0cherylj: ok, I can pick that up next02:15
mupBug # opened: 1567722, 1567724, 1567726, 1567728, 1567730, 1567732, 156773402:16
axwmenn0: I'm intending to add the names of top-level machines and services to modelDoc, so we can safely advance from Alive to Dead if a model is empty02:25
axwmenn0: just FYI, in case that would cause problems for model migration02:26
axwmenn0: also probably will change the hosted-model refcount to references to UUIDs, to avoid ABA problem at destroy-controller time02:26
menn0axw: that might have model migration implications. thumper wrote the import and export code.02:27
menn0thumper: thoughts?02:27
* thumper looks up02:27
mupBug # changed: 1567728, 1567730, 1567732, 156773402:28
thumperaxw: what names?02:28
menn0thumper, axw: it should be easy enough to account for, and tests should start failing when you add fields (but maybe only in the model-migration branch02:28
thumperif you add fields, tests will fail02:28
axwmenn0 thumper: i.e. when you add-machinem we'll $addToSet a "machines" field on the model doc02:28
axwand $pull when we remove the machine02:29
thumperwhy?02:29
axwthumper: https://bugs.launchpad.net/juju-core/+bug/156722802:29
mupBug #1567228: "juju destroy-controller" can leak hosted models <juju-core:In Progress by axwalk> <https://launchpad.net/bugs/1567228>02:29
axwthat's part of it02:30
axwthumper: https://bugs.launchpad.net/juju-core/+bug/1563615 that's the other part02:30
mupBug #1563615: destroy-controller blocks when you've not removed an empty default model <juju-release-support> <juju-core:In Progress by axwalk> <https://launchpad.net/bugs/1563615>02:30
axwthumper: basically we should short-circuit if the controller only has empty models, but we can't do that safely at the moment02:30
axwbecause the machines and services don't refcount or anything on the model02:31
thumperok02:31
thumperif you do this, you'll need to make sure the value is added when the model description is imported02:32
thumperthe export code probably won't need to change as the precondition is that the model is alive and all machines are alive02:32
thumperand no pending cleanups02:32
thumperso populating the field when importing should be "iterate through the machines"02:32
* axw nods02:33
axwthumper: where's the import code?02:34
thumperaxw: state/migration_import.go02:34
axwta02:34
menn0thumper or axw: I need a quick hangout to bounce some ideas for a bug fix off someone. either of you keen?02:34
thumpermenn0: sure02:34
axwmenn0 thumper: call me in if it's helpful, otherwise I want to get this sorted02:35
thumperkk02:35
thumpermenn0: 1:1 ?02:35
menn0thumper: yep02:35
mupBug # opened: 1567728, 1567730, 1567732, 156773402:37
davecheneybase_windows.go makes me sad02:48
davecheney1. because the registiry api is nuts02:48
davecheney2. because the way that code enables the api to continue to be nuts, is weird, codependant and unhealtht02:49
mupBug #1566339 changed: `juju run` needs a --all-machine, --all-service, --all-unit <juju> <machine> <run> <juju-core:Invalid> <https://launchpad.net/bugs/1566339>03:07
davecheneymenn0: cherylj03:12
davecheneyhttp://reviews.vapour.ws/r/4486/03:12
mwhudsonuh03:17
mwhudsonoh03:17
mwhudsonnm03:17
mwhudsoni should just upgrade to xenial03:17
natefinchmwhudson: if you like pain and suffering, that's a great idea.03:23
mwhudsonheh it shouldn't be so bad at this point03:23
natefinchI've just heard of a lot of problems with juju-specific things like lxd and having xenial listed as an LTS (though it may not be yet)03:24
mwhudsonoh yeah, lxd keeps moving things around03:25
natefinchI have OS updates that I'm not installing because there's lxd updates, and currently my LXD actually works.03:26
mwhudsonhuh um juju doesn't build with gccgo on xenial either03:26
natefinchand I don't have time to futz with network BS03:26
mwhudsonoh maybe only with the tip go tool03:27
mupBug #1567747 opened: "juju metadata generate-image" does not validate architectures <juju-core:Triaged> <https://launchpad.net/bugs/1567747>03:37
menn0davecheney: looking03:38
menn0davecheney: looks good03:39
natefinchdavecheney: do you happen to know why this function has an error return? https://golang.org/src/net/http/cookiejar/jar.go?s=2387:2421#L67  it's vexing my current code. I'd like to just ignore it to make my code cleaner, but it worries me that it might one day start returning an error.03:42
mupBug #1567747 changed: "juju metadata generate-image" does not validate architectures <juju-core:Triaged> <https://launchpad.net/bugs/1567747>03:49
mupBug #1567747 opened: "juju metadata generate-image" does not validate architectures <juju-core:Triaged> <https://launchpad.net/bugs/1567747>03:55
redirnight04:00
mupBug #1567763 opened: bootstrapping private openstack, with --metadata-source fails when instance-type constraint is specified <juju-core:Triaged> <https://launchpad.net/bugs/1567763>05:28
davecheneynatefinch ping ?05:35
davecheneyhmm, too late05:35
frobwaredimitern, voidspace, dooferlad: going to skip standup today.08:42
dooferladfrobware: ok08:42
frobwareis there anyway I can force an upgrade-tools? It works for a while, but subsequent upgrades result in "ERROR some agents have not upgraded to the current model version 2.0-beta4.4: machine-0"08:49
dimiternfrobware, ok08:50
dimiternfrobware, btw I'm about to finally propose a fix for bug 156439508:50
mupBug #1564395: newly created LXD container has zero network devices <bootstrap> <conjure> <network> <juju-core:In Progress by dimitern> <https://launchpad.net/bugs/1564395>08:50
babbageclunkvoidspace, frobware: I've got HR induction from 10-11 so I won't be in the meeting this morning (at least the first part - I'll jump in if it's still going when I get out).08:51
frobwarebabbageclunk, ack08:52
babbageclunkvoidspace: what I did is in the document from yesterday, though.08:52
* thumper is in the hangout08:58
thumperbabbageclunk: good luck08:58
* thumper wonders if he will be sitting alone....09:01
voidspacebabbageclunk: ok09:03
voidspacethumper: omw09:03
frobwaredimitern, happy to be a guinea pig for 156439509:07
frobwaredimitern, that's one less reboot, for there are many today. :/09:07
frobwaredimitern, did your change with my /e/n/i *cough* hack... ?09:07
frobwaredimitern, did your change work with my ...09:08
dimiternfrobware, testing always appreciated :) yes - it uses the /e/n/i hack09:13
frobwaredimitern, not sure my hack is working today; newer lxd since weds...09:14
dimiternfrobware, I did a dist-upgrade this morning, so I have rc9 I think09:15
dimiternyep09:15
frobwaredimitern, I'm trying to find the right place and time to delete the lxd supplied (and mandatory) eth0.cfg09:16
frobwaredimitern, and ensure any ifup brings up our definition of eth009:17
frobwaredimitern, I was gettting provisioning errors earlier where it failed to write to /var/lib/lxd/container/<container>/rootfs/...09:17
frobwaredimitern, the other thing is I'm only testing/trying this with trusty and xenial. I guess I should try precise and wily at some point.09:27
voidspacedimitern: babbageclunk: PR to merge maas2 onto master http://reviews.vapour.ws/r/4487/10:03
dimiternvoidspace, I'll swap it for this - http://reviews.vapour.ws/r/4488/ :)10:11
dimiternvoidspace, LGTM10:14
voidspacedimitern: thanks - looking10:14
dimiternvoidspace, ta!10:15
dimiternfwereade, hey, are you about?10:15
dimiternfrobware, ^^ pushed the fix10:16
frobwaredimitern, you rebased?10:17
dimiternyep10:17
frobwaredimitern, I was just looking at your changes since I pulled from earlier10:17
frobwaredimitern, ok10:17
dimiternfrobware, not removing /e/n/i.d/eth0.cfg doesn't seem to cause any harm - even rebooting the host/container is ok10:21
frobwaredimitern, I'm trying to determine whether that's just luck based on the 'source .../*.cfg' order10:21
frobwaredimitern, stgraber advised me to delete it10:21
dimiternfrobware, well, that's even better - a proper fix, but just saying what i've seen so far10:23
frobwaredimitern, me too. but have you rebooted a lot?10:24
dimiternfrobware, i did reboot a few times, nothing fancier though10:24
frobwaredimitern, I'm stuck with upgrade not working for me at the moment which make for slow progress10:24
dimiternfrobware, that upgrade-juju issue - that's on trusty or xenial?10:26
voidspacedimitern: I don't really like adding unused code10:30
voidspacedimitern: why not add the gate in the PR that uses it?10:30
frobwaredimitern, xenial10:31
dimiternvoidspace, well, it's used by discoverspaces, but not yet by others10:33
dimiternvoidspace, actually I did this because I thought I might need it in this PR10:34
dimiternvoidspace, but fair enough, I'll pull gate out of this PR10:34
voidspacedimitern: it's up to you - other than that LGTM10:36
frobwaredimitern, voidspace: seems like the right thing to do as it make reverting the PR saner - should it ever need to10:36
dimiternfrobware, it's pretty easy with gh web ui10:37
frobwaredimitern, what I mean is you would only be reverting stuff pertinent to the original fix10:38
frobwaredimitern, anyway, back to ENI - the interfaces do not always come up for me. :(10:38
dimiternfrobware, with my fix ?10:40
frobwaredimitern, nope, mine10:40
frobwaredimitern, the times I've bootstrapped and added a new machine without reboot seems to work fine with your fix \o/10:41
dimiternfrobware, I found out something else (probably related to what jamespage was seeing - if say eth0 is unconfigured, but has children which are not - it breaks10:42
dimiternfrobware, awesome!10:42
frobwaredimitern, removal of eth0.cfg in cloud-init and taking down / bringing up the 00-juju.cfg does not work reliably. or so it seems.10:42
frobwaredimitern, I raised a bug for that10:42
frobwaredimitern, https://bugs.launchpad.net/juju-core/+bug/156679110:43
mupBug #1566791: VLANs on an unconfigured parent device error with "cannot set link-layer device addresses of machine "0": invalid address <network> <juju-core:Triaged> <https://launchpad.net/bugs/1566791>10:43
dimiternfrobware, cheers, I'll be tackling this as well with the changes we discussed with thumper - adding 2 workers to do the NIC provisioning, devices creation, and updating provider network config after provisioning, as well as releasing container addresses as needed10:44
dimiternfrobware, in fact, I'm already thinking how we can do the job of the bridge script but in a worker instead, properly with a lot more useful context and less workarounds10:46
frobwaredimitern, but the script runs in cloud-init10:46
dimiternfrobware, it doesn't have to, does it?10:46
frobwaredimitern, well, as I'm finding out again, the combo of: ifdown, copy-in-new-file; ifup -a, is not guaranteed.10:47
dimiternfrobware, we can do it with proper retrying, etc. - bridge by bridge, based on what NICs we observe at run-time10:47
frobwaredimitern, I'm generally -1 on this atm, particularly if MAAS models bridges.10:48
frobwarein the near future10:48
dimiternfrobware, perhaps some if-[pre|post]-down script was lingering?10:48
frobwaredimitern, bbiab - 20 mins.10:48
dimiternfrobware, well, they will be at some point, but we need to think about other providers10:48
voidspacebabbageclunk: dimitern: frobware: well that took a long time! I now have a test that can successfully call StartInstance against MAAS 2 code.11:17
dimiternvoidspace, \\o// great!11:18
* dimitern should really install a 2.0 maas locally now :)11:19
* fwereade cheers at voidspace11:19
voidspacefwereade: :-)11:19
voidspacetesting it all will take a while to construct, it's not exactly a small code path - but I can call it now without it blowing up11:19
frobwarevoidspace, very NICE!11:20
bogdanteleagadimitern, https://bugs.launchpad.net/juju-core/+bug/156767611:37
mupBug #1567676: windows: networker tries to update invalid device and blocks machiner from working <juju-core:Triaged> <https://launchpad.net/bugs/1567676>11:37
bogdanteleagahelp? :D11:37
dimiternbogdanteleaga, hey - the networker is gone for a while now; which version of juju?11:38
bogdanteleagadimitern, latest, I wasn't sure what terminology to use11:39
dimiternbogdanteleaga, I see (might have helped to actually read the bug desc first :)11:39
dimiternbogdanteleaga, I know the issue11:39
dimiternbogdanteleaga, but that's already fixed in master IIRC - is that on a feature branch?11:40
bogdanteleagadimitern, it's from yesterday's master11:41
bogdanteleagadimitern, https://github.com/juju/juju/commit/b631875ff9ed67ec321f1d6fa90bc65d839b9e9311:41
dimiternbogdanteleaga, so the error in comment #1 is no longer happening11:43
dimiternbogdanteleaga, but the "<nil>" value of the address is troubling11:43
bogdanteleagadimitern, I'll try the latest master11:45
dimiternbogdanteleaga, it might be better to use --config logging_config='<root>=TRACE' to bootstrap to get more context (and paste in the bug please)11:47
* dimitern steps out for a couple of hours11:49
* perrito666 steps out for an hour too11:59
rogpeppean update to juju-core to fix the resources API used by charmrepo: http://reviews.vapour.ws/r/4490/12:36
rogpeppeshould be trivial and I've very much like to get it landed to day if poss...12:37
rogpeppeso reviews welcome :)12:37
axwfwereade: I'm EODing, but if you have any time up your sleep, this PR has some gnarly state changes that I'd appreciate a review of if you have some time today: http://reviews.vapour.ws/r/4491/12:58
mupBug #1567925 opened: help text for juju add-unit needs improving <helpdocs> <juju-core:New> <https://launchpad.net/bugs/1567925>13:08
voidspacebabbageclunk: frobware: dimitern: maas2 has landed on master - all further maas2 work to be against master13:09
mupBug #1567934 opened: uploadSuite.TestMockBuildTools relies on exact archive/zip output <juju-core:New> <https://launchpad.net/bugs/1567934>13:17
mupBug #1567938 opened: juju bootstrap requires network ID as config option on command line although it's specified in clouds.yaml <juju-core:New> <https://launchpad.net/bugs/1567938>13:17
alexisbmorning all13:27
voidspacealexisb: o/13:29
babbageclunkvoidspace: nice!13:30
katcoyaaaay life-giving internet13:30
alexisbkatco, hey13:30
voidspacekatco: morning13:30
katcoalexisb: hey13:30
alexisbheh13:30
katcovoidspace: morning from across the way13:30
alexisbis what I mean13:30
voidspacekatco: o/13:30
katcoalexisb: my neighborhood is 1 short street and they've been replacing the sidewalks 1 section at a time over the past month or so. we've had 3 outages now... o.013:32
alexisbkatco, sounds like it is time to move ;)13:33
* katco wry smile13:33
katcoperrito666: i have a patch to propose for the keystone issue... will you have a chance to tal?13:34
babbageclunkalexisb, katco: morning!13:36
katcobabbageclunk: o/13:36
katcobabbageclunk: good to hear from you again. how are you getting on with things?13:36
babbageclunkkatco: settling in ok, I think. Heaps and heaps to learn, obviously!13:38
katcobabbageclunk: that's what makes it fun :D13:38
babbageclunkkatco: :)13:39
mupBug #1567951 opened: help text for juju login needs improving <helpdocs> <juju-core:New> <https://launchpad.net/bugs/1567951>13:53
mupBug #1567952 opened: container/lxd: TestDetectSubnetLocal fails with link/none <juju-core:New> <https://launchpad.net/bugs/1567952>13:53
perrito666katco: sure, shoot13:54
katcoperrito666: http://reviews.vapour.ws/r/4492/13:57
katcoperrito666: ta13:58
natefinchericsnow: I think you're gonna like how the charmstore client code turned out... especially the amount of code needed to put into the csclientImpl wrapper type14:03
alexisbcmars, you around yet?14:03
natefinchericsnow: I still have to update the tests for that code, but shouldn't be too bad.14:04
perrito666katco: shit it with a doubt14:06
natefinchperrito666: lol14:07
perrito666meh14:07
perrito666sorry typo14:07
katcoperrito666: i think we only care about the major to resolve the client14:07
cheryljjam: you still around?14:07
katcoperrito666: i.e. if the major is 2, we'd settle on 2.x whatever that may be14:07
cheryljfrobware: ping?14:09
perrito666katco: I had miss interpreted your code, sorry14:10
katcoperrito666: ah no worries... so is it still OK?14:11
perrito666yes14:11
katcoperrito666: awesome... thanks for reviewing14:13
perrito666tx for that fix14:13
katcoperrito666: weird situation... i don't really understand why an openstack deployment would report a version that isn't working yet14:14
perrito666cherylj: speaking of reviews, could I get a rev on http://reviews.vapour.ws/r/4481/ ?14:15
cheryljperrito666: yes, I will take a look this morning  (about to have a call right now :/)14:15
katconatefinch: ericsnow: https://plus.google.com/hangouts/_/canonical.com/retrospective?authuser=114:33
katcoericsnow: ping?14:39
frankbancherylj: hey morning? any news on merging embedded-gui?14:40
katconatefinch: ericsnow: deferred half an hour; sorry, thought ericsnow was already on14:40
mupBug #1567963 opened: help text for juju remove-unit needs improving <helpdocs> <juju-core:New> <https://launchpad.net/bugs/1567963>14:41
cheryljfrankban: let me look at the CI reports, one sec14:41
frankbancherylj: sure ty14:42
perrito666https://www.mongodb.com/blog/post/mongodb-2-6-end-of-life?utm_content=buffere388e&utm_medium=social&utm_source=twitter.com&utm_campaign=buffer14:54
perrito666meh we could use a bot that says the title: "MongoDB 2.6 End of Life"14:55
natefinchperrito666: the url was pretty obvious ;)14:55
perrito666lol, I did not realize, I could have very well have stripped the get params14:55
* dimitern whew... almost 1h stuck in traffic :/14:56
dimiternvoidspace, frobware, babbageclunk, btw the sync call is not happening15:02
dimiternperhaps next week15:02
alexisbcherylj, fyi, I was able to deploy a few services yesterday off master15:16
dooferladfrobware: I have the switch in place and can confirm that 802.3ad works over juju-br0 once we put in pre-up/post-down steps to control the underlying bond.15:21
alexisbdooferlad, that is good news15:22
dooferladalexisb: :-)15:22
* alexisb changes locations15:59
frankbancherylj: can we stick a little trivial last change to the embedded-gui branch? is it too late?16:09
babbageclunkdimitern: sync call? I don't think I'm on it - can you please add me?16:09
evilnickveitchcherylj, we may have a problem with a lot of the updated help text16:09
frankbanhatch: you dropped16:09
cheryljfrankban: dont' put anything else into embedded gui16:09
cheryljwait for us to merge it and put it directly in master16:10
hatch:'(16:10
hatch:D16:10
frankbancherylj: sounds good16:10
cheryljevilnickveitch: what's going on with the help text?16:10
frankbancherylj: when is the deadline for putting changes into master?16:10
evilnickveitchcherylj, a lot of the user related commands reference 'share-model'16:10
evilnickveitch'sahre-model' seems to have disappeared16:10
evilnickveitchi mean, as a command16:11
cheryljfrankban: needs to be done today16:11
cheryljevilnickveitch: yeah, it was a late change to be juju grant16:11
evilnickveitchcherylj, :(16:12
=== redir_afk is now known as redir
evilnickveitchcherylj, well, be on the lookout for it in case we miss changing any - it appears in a lot of 'see also' text16:13
cheryljevilnickveitch: ok, thanks, I'll keep an eye out16:13
evilnickveitchcherylj, I will just go and rewrite all the user docs...16:13
dimiternbabbageclunk, sure - invite sent16:14
babbageclunkdimitern: thanks!16:15
frankbancherylj: actually I am lucky enough that the code I need to change is the GUI server already in master, will be just a trivial improvement16:18
cheryljfrankban: great!16:19
redircherylj: evilnickveitch there appears to be one unshare-model too16:20
cmarsi can review some things now. what should i look at first?16:32
cmarsalexisb, cherylj ^^ ?16:33
alexisbperrito666, are you still waiting for a review?16:34
alexisbcmars, this one is important: http://reviews.vapour.ws/r/4491/16:34
* cmars is reviewing16:36
alexisbcmars next inline would be this one: http://reviews.vapour.ws/r/4481/16:37
mupBug #1568028 opened: Juju fails to deploy with 2.0-beta3-xenial-arm64 <juju-core:New> <https://launchpad.net/bugs/1568028>16:38
alexisbfrobware, I am back online if you are still looking for me16:40
alexisbcmars, if you are done with the two reviews above, the next in priority are any from jam dealing with lxd and containers16:58
alexisblike: http://reviews.vapour.ws/r/4478/16:58
cmarsalexisb, still on the first one17:00
alexisbcmars, understood, thanks17:01
mupBug #1568028 changed: Juju fails to deploy with 2.0-beta3-xenial-arm64 <juju-core:New> <https://launchpad.net/bugs/1568028>17:02
cmars4491 is going to take me all afternoon to review, some of it i don't fully understand yet. deals with parts of state i'm unfamiliar with17:24
alexisbcmars, what about 4881>17:28
alexisb?17:28
cmarsalexisb, you mean 4481? that looks more manageable. i'll review over lunch17:29
alexisbyes that is what I meant sorry17:30
alexisband cool, thank you17:30
frobwarealexisb, just to say... great few days with christian, have some sort of fix (need testing) for bug #156680117:32
mupBug #1566801: LXD containers /etc/network/interfaces as generated by Juju  gets overwritten by LXD container start <network> <juju-core:Triaged by frobware> <https://launchpad.net/bugs/1566801>17:32
* frobware EOW 17:36
katcoericsnow: you have ship its17:51
ericsnowkatco: thanks17:51
natefinchkatco, ericsnow: down to 4 comments btw17:52
ericsnownatefinch: nice17:52
katconatefinch: yeah just looked... that is a ton of code lol17:52
natefinchkatco: yeah, there's a reason it took a while17:52
katconatefinch: ericsnow: thanks for pairing on that17:53
natefinchkatco: still need some new tests for some of the new code eric and I wrote yesterday17:53
mupBug #1568069 opened: Container networking cannot ssh <ci> <lxc> <network> <regression> <juju-core:Triaged> <https://launchpad.net/bugs/1568069>18:00
mupBug #1568069 changed: Container networking cannot ssh <ci> <lxc> <network> <regression> <juju-core:Triaged> <https://launchpad.net/bugs/1568069>18:06
alexisbkatco, I have a few minutes if you would like ot meet18:20
mupBug #1568069 opened: Container networking cannot ssh <ci> <lxc> <network> <regression> <juju-core:Triaged> <https://launchpad.net/bugs/1568069>18:21
mupBug #1568079 opened: github.com/juju/juju/apiserver/client unit tests fail if xenial is the LTS <xenial> <juju-core:Triaged> <https://launchpad.net/bugs/1568079>18:21
katcoalexisb: sure18:24
=== jog_ is now known as jog
alexisbkatco, 1x1 HO18:26
perrito666alexisb: sadly I am18:33
cmarsperrito666, reviewed 448118:37
perrito666cmars: tx18:37
perrito666cmars: tx a lot18:38
mupBug #1568090 opened: help text for juju sync-tools needs improving <helpdocs> <juju-core:New> <https://launchpad.net/bugs/1568090>18:45
natefinchericsnow, katco: and down to one comment, but it's probably the hardest one... fixing model migrations.  But for now I'm going to rebase, since github tells me I have conflicts. sigh.18:54
ericsnownatefinch: k18:54
ericsnownatefinch: let me know if I can help; as I mentioned, I did the same thing for channels18:54
natefinchdefinitely18:55
mupBug #1568092 opened: ConnectSuite.TestLocalConnectError: centos  cannot connect to local lxd servier <centos> <ci> <lxd> <regression> <test-failure> <unit-tests> <juju-core:Triaged> <https://launchpad.net/bugs/1568092>19:06
katconatefinch: sorry was otp. whoa, what does "fixing model migrations" mean? that doesn't sound fun =|19:06
natefinchkatco: adding a few lines to state.migration_import.go / _export.go  basically like what eric did with those files here: http://reviews.vapour.ws/r/4441/diff/#19:09
katconatefinch: oh b/c you're storing new info in state you have to support it when migrating models19:10
natefinchyep19:10
katconatefinch: gotcha... makes sense. sounded scary o/019:10
natefinchstill a little scary ;)19:10
katcoha19:10
natefinchuh hmm19:14
natefinchthere's nothing in there about charms19:14
natefinchericsnow: ^19:14
ericsnownatefinch: if we're not migrating charms then I'm guessing that Tim & co. haven't gotten to it yet19:15
ericsnownatefinch: however, I'd be surprised by that19:16
ericsnownatefinch: perhaps it's folded into the service representation? (and they will re-download the blobs)19:16
natefinchericsnow: I thought of that, but then they can't do that for local charms19:17
ericsnownatefinch: yuck19:17
ericsnownatefinch: so blobs have to be migrated too I suppose19:17
mupBug #1568101 opened: help text for juju deploy needs improving <helpdocs> <juju-core:New> <https://launchpad.net/bugs/1568101>19:18
natefinchericsnow: I would assume so....19:18
natefinchericsnow: hang on... I see some code related to charms in core/migration/migration.go .... reading19:19
ericsnownatefinch: ah, no MigrationSuite.TestCharmDocFields()19:19
natefinchericsnow: can you give the review another pass, and we can maybe try to get in touch with the migration guys?  I assume they're all still in their upside down beds, since it's saturday morning down there19:29
ericsnownatefinch: will do19:29
alexisbkatco, ericsnow: has this documentation made ericsnow's google doc obsolete?: https://github.com/juju/juju/wiki/Implementing-environment-providers19:31
katcoalexisb: axw mentioned he was going to incorporate ericsnow 's doc because he didn't know about it when he created that wiki page. i don't know that he ever did19:33
natefinchalexisb or katco: do you know if anyone familiar with model migrations is awake?  I added a macaroon field to charms stored in mongo, but I don't really see any place in the migration code where we migrate charm metadata...19:34
katconatefinch: i don19:35
katco't19:35
katconatefinch: menn0 is working on that primarily... thumper should still be able to answer questions as well19:35
natefinch..... do our tests create virtual ethernets on the machine they're running on?19:35
katconatefinch: omg... PLEASE tell me they don't19:36
natefinchsure looks like it19:36
natefinchI have like 4 virtual ethernet devices on my machine right now19:36
katcowhat. the. hell.19:36
natefinchI noticed because I saw a popup that say "connected to veth<somguid>19:36
alexisbnatefinch, fwereade sometimes is around late19:38
alexisbotherwise they are all on their weekend19:38
alexisbnatefinch, the MM work is not complete so it may be that it is not in master yet19:38
alexisbthough with menno's latest branch I was able to successfully migrate a basic mediawiki deploy successfully19:39
natefinchalexisb: ok19:40
natefinchkatco: http://i.imgur.com/d1ZKuPz.png19:43
ericsnowkatco: gah, I forgot to mention a patch I have up against charmrepo:  https://github.com/juju/charmrepo/pull/8419:44
katcoericsnow: k tal19:45
ericsnowkatco: ta19:45
natefinchthat feeling when you're pretty sure you're waiting for the 10 minute testing timeout :/19:46
cheryljI know that feeling19:46
redirnatefinch: everytime I run all the tests.19:47
natefinchor not: ok  github.com/juju/juju/state414.273s19:48
natefinchalmost 7 minutes just for state tests.  Huzzah.19:49
alexisbo the state tests...19:57
perrito666natefinch: the state tests are quite shroedinger cat-ish unthil they finish they have and have not failed in your mind19:58
katcoericsnow: lgtm... fairly straightforward changes :)19:59
ericsnowkatco: yep :)20:00
ericsnowkatco: thanks20:00
katcoericsnow: ty20:00
mupBug #1563364 changed: UniterSuite.TestUniterSteadyStateUpgradeForce fails <ci> <intermittent-failure> <test-failure> <juju-core:Invalid> <juju-core embedded-gui:Fix Released> <https://launchpad.net/bugs/1563364>20:09
mupBug #1568122 opened: help text for juju remove-machine needs improving <helpdocs> <juju-core:New> <https://launchpad.net/bugs/1568122>20:09
cmarsalexisb, should i try to continue reviewing 4491 or look at something else20:13
alexisbcmars, if they are easier you can do the lxd reviews20:21
alexisbhttp://reviews.vapour.ws/r/4478/20:21
cmarsalexisb, looking20:22
* perrito666 sees his refactoring interrupted by eye drops :p20:22
ericsnownatefinch: LGTM with a few small comments20:25
natefinchericsnow: ok20:26
natefinchericsnow: thanks20:26
ericsnownatefinch: np20:26
ericsnownatefinch: thanks for doing it :)20:26
ericsnownatefinch: how close are you to pulling the trigger on that patch?20:51
katcoericsnow: you were waiting on him to rebase weren't you?20:59
ericsnowkatco: was hoping he'd merge his patch before I land my last one21:00
katcoericsnow: might have to land yours first... your work needs to be in by EOD21:02
katcoericsnow: and then we can pair to help through the difficult merge21:03
ericsnowkatco: won't be hard, just tedious :)21:03
ericsnowkatco: we're changing a bunch of the same signatures21:03
katcoericsnow: he's probably gone to dinner, i'd say go ahead and begin merging21:05
perrito666aghh I broke the reboot test :(21:05
ericsnowkatco: sounds good21:06
natefinchericsnow: thanks, fotgot you were waiting on me21:19
ericsnownatefinch: np21:19
natefinchbbl21:19
mupBug #1568150 opened: xenial lxc containers not starting <juju-core:New> <https://launchpad.net/bugs/1568150>21:39
=== urulama is now known as urulama|___
alexisbperrito666, I am seeing reboot test failures too22:00
alexisbfrom master22:00
perrito666meh, It seems broken22:00
perrito666I believe its breaking mongod but it is not related from my changes22:00
perrito666this might prevent me from merging :(22:00
alexisbalrighty all I am out for a few hours will be back this evening22:01
alexisbperrito666, please open a bug with details22:01
perrito666ok22:01
alexisbthanks22:03
alexisbalexisb-afk22:03
perrito666alexisb: for when you come back, I cannot reproduce now but I feel we are leaking lxcs and mongos22:09
mupBug #1568160 opened: destroying a unit with an error gives no user feedback <juju-core:New> <https://launchpad.net/bugs/1568160>22:09
mupBug #1568161 opened: Pass whether service is primary or subordinate via relation <juju-core:New> <https://launchpad.net/bugs/1568161>22:09
mupBug #1568175 opened: destroy-controller failed: EOF <ci> <destroy-controller> <regression> <juju-core:Triaged> <https://launchpad.net/bugs/1568175>22:30
mupBug #1568176 opened: charm deployment requests invalid revision number <juju-core:New> <https://launchpad.net/bugs/1568176>22:30
mupBug #1568175 changed: destroy-controller failed: EOF <ci> <destroy-controller> <regression> <juju-core:Triaged> <https://launchpad.net/bugs/1568175>22:34
mupBug #1568176 changed: charm deployment requests invalid revision number <juju-core:New> <https://launchpad.net/bugs/1568176>22:34
mupBug # opened: 1568175, 1568176, 1568177, 156817922:40
mupBug #1568181 opened: clientRepoSuite.TestBlockRemoveServiceDeployPrincipal cannot put archive blob <ci> <intermittent-failure> <regression> <unit-tests> <juju-core:Incomplete> <juju-core 1.25:Triaged> <https://launchpad.net/bugs/1568181>23:01
mupBug #1568183 opened: GCE timeout pending new instance operation failed <ci> <gce-provider> <jujuqa> <reliability> <juju-core:Triaged> <https://launchpad.net/bugs/1568183>23:01
ericsnowkatco: third patch merging now :)23:01
mupBug #1568185 opened: statusHistoryPrunerSuite.TestWorker wrong number of logs <ci> <intermittent-failure> <unit-tests> <juju-core:Incomplete> <juju-core 1.25:Triaged> <https://launchpad.net/bugs/1568185>23:01
katcoericsnow: woot woot!23:01
mupBug #1568186 opened: TestUniterUpgradeGitConflicts never reached desired status <ci> <i386> <intermittent-failure> <unit-tests> <juju-core:Triaged> <juju-core 1.25:Triaged> <https://launchpad.net/bugs/1568186>23:31
mupBug #1568189 opened: RemoveRelationSuite.SetUpTest failed because of mongo <ci> <intermittent-failure> <mongodb> <juju-core:Triaged> <juju-core 1.25:Triaged> <https://launchpad.net/bugs/1568189>23:31
mupBug #1568190 opened: MachinerSuite.TestMachinerSetStatusStopped cannot set status <ci> <intermittent-failure> <unit-tests> <juju-core:Incomplete> <juju-core 1.25:Triaged> <https://launchpad.net/bugs/1568190>23:31

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