[00:06] <mup> Bug #1567598 changed: storage-get causes state server to 'error: bad request' <storage> <juju-core:Invalid> <https://launchpad.net/bugs/1567598>
[00:08] <mwhudson> perrito666: ping?
[00:09] <thumper> damn you schema.ForceInt
[00:09] <thumper> schema.Int -> int64
[00:09] <thumper> scheam.ForceInt -> int
[00:09] <thumper> for no good reason
[00:09] <thumper> bah
[00:10] <alexisb> mwhudson, it is very much past his eod, if you can open a bug with the info you have that owuld be most helpful
[00:11] <mwhudson> alexisb: ok
[00:11] <mwhudson> alexisb: was actually typing up a bug report now, as it happens :-)
[00:11] <alexisb> perfect :)
[00:14] <mwhudson> alexisb: https://bugs.launchpad.net/juju-core/+bug/1567708
[00:15] <mup> Bug #1567708: unit tests fail with mongodb 3.2 <juju-core:New> <https://launchpad.net/bugs/1567708>
[00:15] <alexisb> mwhudson, thank you!
[00:15] <mwhudson> np
[00:15] <mwhudson> some of the errors are clearly quite silly
[00:16] <mwhudson> (like the error message mongo gives when reporting an auth failure has changed)
[00:16] <mwhudson> i don't know if any are deep
[00:21] <mup> Bug #1567708 opened: unit tests fail with mongodb 3.2 <juju-core:New> <https://launchpad.net/bugs/1567708>
[00:31] <natefinch> evening all
[00:33] <mup> Bug #1567708 changed: unit tests fail with mongodb 3.2 <juju-core:New> <https://launchpad.net/bugs/1567708>
[00:36] <redir> axw: yt?
[00:36] <mup> Bug #1567708 opened: unit tests fail with mongodb 3.2 <juju-core:New> <https://launchpad.net/bugs/1567708>
[00:51] <mup> Bug #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] <axw> redir_afk: "yt"?
[00:53] <davecheney> cherylj: http://pad.lv/1566303
[00:53] <davecheney> did this pass CI testing ?
[00:54] <davecheney> i have access to a windows machine now and can investigate if it's still failing
[00:55] <redir> I am
[00:55] <redir> axw: yt == you there
[00:55] <axw> redir: oh, well, you have your answer ;)
[00:55] <redir> axw: do you have a minute for a hangout?
[00:55] <axw> redir: sure, see you in the tanzanite hangout
[00:55] <redir> k
[00:57] <mup> Bug #1567712 changed: [juju create-model] Specifying a cloud when referring to a credential is redundant <docteam> <juju-core:New> <https://launchpad.net/bugs/1567712>
[01:03]  * thumper heads out to walk the dog
[01:06] <mup> Bug #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:15] <cherylj> davecheney: no, the windows test failed again:  http://reports.vapour.ws/releases/3863
[01:20] <davecheney> cherylj: thanks, i'll try to get on the windows test machine now
[01:21] <davecheney> cherylj: i don't understand that page
[01:21] <davecheney> where is the windows failure ?
[01:21] <cherylj> run-unit-tests-win2012-amd64
[01:21] <davecheney> ok i see it
[01:27] <mup> Bug #1567458 changed: destroy-controller message and failure is not user friendly <juju-core:New> <https://launchpad.net/bugs/1567458>
[01:36] <mup> Bug #1567458 opened: destroy-controller message and failure is not user friendly <juju-core:New> <https://launchpad.net/bugs/1567458>
[01:37] <redir> cherylj: so you started landing those updates?
[01:37] <redir> help text updates.
[01:37] <cherylj> redir: I made an Executive Decision
[01:37] <cherylj> and pulled the trigger
[01:37] <cherylj> :)
[01:37] <redir> OK
[01:38] <davecheney> menn0: https://bugs.launchpad.net/juju-core/+bug/1474607
[01:38] <mup> Bug #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] <redir> so I should fix up that other one and then resubmit or just merge?
[01:38] <davecheney> this bug is fixed on master bug fires on 1.25
[01:38] <redir> cherylj: ^^
[01:38] <davecheney> from memory it was a _lot_ of work to fix on master
[01:38] <cherylj> redir: as long as you test that the help text looks right, then i'd say just merge
[01:39] <redir> cherylj: OK. And have you settled on some formatting? indents all with line breaks etc?
[01:40] <cherylj> redir: basically, I copied what was in the bug for the most part, taking care to make sure lines were < 80 characters
[01:40] <menn0> davecheney: that bug doesn't ring any bells for me sorry
[01:40] <redir> OK. I'll do that, too, for the most part:)
[01:42] <redir> cherylj: ^
[01:42] <cherylj> thanks, redir :)
[01:44] <davecheney> menn0: i think this was fixed by william's manifold thinggy
[01:45] <menn0> davecheney: it won't be the manifold (dependency engine) stuff directly. that's got nothing to do with the uniter's internals.
[01:45] <mup> Bug #1567458 changed: destroy-controller message and failure is not user friendly <juju-core:New> <https://launchpad.net/bugs/1567458>
[01:45] <menn0> davecheney: there have been many other improvements in the uniter recently though
[01:45] <mup> Bug #1567719 opened: help text for juju list-shares needs improving <helpdocs> <juju-core:New> <https://launchpad.net/bugs/1567719>
[01:45] <mup> Bug #1567721 opened: help text for juju set-default-credential needs improving <helpdocs> <juju-core:New> <https://launchpad.net/bugs/1567721>
[01:46] <davecheney> menn0: that's what i'm worried about
[01:46] <davecheney> we cannot backport them all
[01:48] <mup> Bug #1567719 changed: help text for juju list-shares needs improving <helpdocs> <juju-core:New> <https://launchpad.net/bugs/1567719>
[01:48] <mup> Bug #1567721 changed: help text for juju set-default-credential needs improving <helpdocs> <juju-core:New> <https://launchpad.net/bugs/1567721>
[01:54] <mup> Bug #1567719 opened: help text for juju list-shares needs improving <helpdocs> <juju-core:New> <https://launchpad.net/bugs/1567719>
[01:54] <mup> Bug #1567721 opened: help text for juju set-default-credential needs improving <helpdocs> <juju-core:New> <https://launchpad.net/bugs/1567721>
[02:07] <mup> Bug #1567722 opened: help text for juju list-credentials needs improving <helpdocs> <juju-core:New> <https://launchpad.net/bugs/1567722>
[02:07] <mup> Bug #1567724 opened: help text for juju add-cloud needs improving <helpdocs> <juju-core:New> <https://launchpad.net/bugs/1567724>
[02:07] <mup> Bug #1567726 opened: help text for juju disable-user needs improving <helpdocs> <juju-core:New> <https://launchpad.net/bugs/1567726>
[02:10] <mup> Bug #1567722 changed: help text for juju list-credentials needs improving <helpdocs> <juju-core:New> <https://launchpad.net/bugs/1567722>
[02:10] <mup> Bug #1567724 changed: help text for juju add-cloud needs improving <helpdocs> <juju-core:New> <https://launchpad.net/bugs/1567724>
[02:10] <mup> Bug #1567726 changed: help text for juju disable-user needs improving <helpdocs> <juju-core:New> <https://launchpad.net/bugs/1567726>
[02:12] <menn0> cherylj: I think I've figured out that local charm hash mismatch bug and I don't think it'll be hard to fix
[02:13] <cherylj> menn0: excellent
[02:13] <cherylj> menn0: there's plenty more for you once you're done with it ;)
[02:13] <menn0> cherylj: oh I know :)
[02:14] <cherylj> menn0: I also still have that mgopurge problem from mario that I haven't done and could use help with :)
[02:15] <menn0> cherylj: ok, I can pick that up next
[02:16] <mup> Bug # opened: 1567722, 1567724, 1567726, 1567728, 1567730, 1567732, 1567734
[02:25] <axw> menn0: 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 empty
[02:26] <axw> menn0: just FYI, in case that would cause problems for model migration
[02:26] <axw> menn0: also probably will change the hosted-model refcount to references to UUIDs, to avoid ABA problem at destroy-controller time
[02:27] <menn0> axw: that might have model migration implications. thumper wrote the import and export code.
[02:27] <menn0> thumper: thoughts?
[02:27]  * thumper looks up
[02:28] <mup> Bug # changed: 1567728, 1567730, 1567732, 1567734
[02:28] <thumper> axw: what names?
[02:28] <menn0> thumper, 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 branch
[02:28] <thumper> if you add fields, tests will fail
[02:28] <axw> menn0 thumper: i.e. when you add-machinem we'll $addToSet a "machines" field on the model doc
[02:29] <axw> and $pull when we remove the machine
[02:29] <thumper> why?
[02:29] <axw> thumper: https://bugs.launchpad.net/juju-core/+bug/1567228
[02:29] <mup> Bug #1567228: "juju destroy-controller" can leak hosted models <juju-core:In Progress by axwalk> <https://launchpad.net/bugs/1567228>
[02:30] <axw> that's part of it
[02:30] <axw> thumper: https://bugs.launchpad.net/juju-core/+bug/1563615 that's the other part
[02:30] <mup> Bug #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] <axw> thumper: basically we should short-circuit if the controller only has empty models, but we can't do that safely at the moment
[02:31] <axw> because the machines and services don't refcount or anything on the model
[02:31] <thumper> ok
[02:32] <thumper> if you do this, you'll need to make sure the value is added when the model description is imported
[02:32] <thumper> the export code probably won't need to change as the precondition is that the model is alive and all machines are alive
[02:32] <thumper> and no pending cleanups
[02:32] <thumper> so populating the field when importing should be "iterate through the machines"
[02:33]  * axw nods
[02:34] <axw> thumper: where's the import code?
[02:34] <thumper> axw: state/migration_import.go
[02:34] <axw> ta
[02:34] <menn0> thumper or axw: I need a quick hangout to bounce some ideas for a bug fix off someone. either of you keen?
[02:34] <thumper> menn0: sure
[02:35] <axw> menn0 thumper: call me in if it's helpful, otherwise I want to get this sorted
[02:35] <thumper> kk
[02:35] <thumper> menn0: 1:1 ?
[02:35] <menn0> thumper: yep
[02:37] <mup> Bug # opened: 1567728, 1567730, 1567732, 1567734
[02:48] <davecheney> base_windows.go makes me sad
[02:48] <davecheney> 1. because the registiry api is nuts
[02:49] <davecheney> 2. because the way that code enables the api to continue to be nuts, is weird, codependant and unhealtht
[03:07] <mup> Bug #1566339 changed: `juju run` needs a --all-machine, --all-service, --all-unit <juju> <machine> <run> <juju-core:Invalid> <https://launchpad.net/bugs/1566339>
[03:12] <davecheney> menn0: cherylj
[03:12] <davecheney> http://reviews.vapour.ws/r/4486/
[03:17] <mwhudson> uh
[03:17] <mwhudson> oh
[03:17] <mwhudson> nm
[03:17] <mwhudson> i should just upgrade to xenial
[03:23] <natefinch> mwhudson: if you like pain and suffering, that's a great idea.
[03:23] <mwhudson> heh it shouldn't be so bad at this point
[03:24] <natefinch> I'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:25] <mwhudson> oh yeah, lxd keeps moving things around
[03:26] <natefinch> I have OS updates that I'm not installing because there's lxd updates, and currently my LXD actually works.
[03:26] <mwhudson> huh um juju doesn't build with gccgo on xenial either
[03:26] <natefinch> and I don't have time to futz with network BS
[03:27] <mwhudson> oh maybe only with the tip go tool
[03:37] <mup> Bug #1567747 opened: "juju metadata generate-image" does not validate architectures <juju-core:Triaged> <https://launchpad.net/bugs/1567747>
[03:38] <menn0> davecheney: looking
[03:39] <menn0> davecheney: looks good
[03:42] <natefinch> davecheney: 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:49] <mup> Bug #1567747 changed: "juju metadata generate-image" does not validate architectures <juju-core:Triaged> <https://launchpad.net/bugs/1567747>
[03:55] <mup> Bug #1567747 opened: "juju metadata generate-image" does not validate architectures <juju-core:Triaged> <https://launchpad.net/bugs/1567747>
[04:00] <redir> night
[05:28] <mup> Bug #1567763 opened: bootstrapping private openstack, with --metadata-source fails when instance-type constraint is specified <juju-core:Triaged> <https://launchpad.net/bugs/1567763>
[05:35] <davecheney> natefinch ping ?
[05:35] <davecheney> hmm, too late
[08:42] <frobware> dimitern, voidspace, dooferlad: going to skip standup today.
[08:42] <dooferlad> frobware: ok
[08:49] <frobware> is 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:50] <dimitern> frobware, ok
[08:50] <dimitern> frobware, btw I'm about to finally propose a fix for bug 1564395
[08:50] <mup> Bug #1564395: newly created LXD container has zero network devices <bootstrap> <conjure> <network> <juju-core:In Progress by dimitern> <https://launchpad.net/bugs/1564395>
[08:51] <babbageclunk> voidspace, 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:52] <frobware> babbageclunk, ack
[08:52] <babbageclunk> voidspace: what I did is in the document from yesterday, though.
[08:58]  * thumper is in the hangout
[08:58] <thumper> babbageclunk: good luck
[09:01]  * thumper wonders if he will be sitting alone....
[09:03] <voidspace> babbageclunk: ok
[09:03] <voidspace> thumper: omw
[09:07] <frobware> dimitern, happy to be a guinea pig for 1564395
[09:07] <frobware> dimitern, that's one less reboot, for there are many today. :/
[09:07] <frobware> dimitern, did your change with my /e/n/i *cough* hack... ?
[09:08] <frobware> dimitern, did your change work with my ...
[09:13] <dimitern> frobware, testing always appreciated :) yes - it uses the /e/n/i hack
[09:14] <frobware> dimitern, not sure my hack is working today; newer lxd since weds...
[09:15] <dimitern> frobware, I did a dist-upgrade this morning, so I have rc9 I think
[09:15] <dimitern> yep
[09:16] <frobware> dimitern, I'm trying to find the right place and time to delete the lxd supplied (and mandatory) eth0.cfg
[09:17] <frobware> dimitern, and ensure any ifup brings up our definition of eth0
[09:17] <frobware> dimitern, I was gettting provisioning errors earlier where it failed to write to /var/lib/lxd/container/<container>/rootfs/...
[09:27] <frobware> dimitern, 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.
[10:03] <voidspace> dimitern: babbageclunk: PR to merge maas2 onto master http://reviews.vapour.ws/r/4487/
[10:11] <dimitern> voidspace, I'll swap it for this - http://reviews.vapour.ws/r/4488/ :)
[10:14] <dimitern> voidspace, LGTM
[10:14] <voidspace> dimitern: thanks - looking
[10:15] <dimitern> voidspace, ta!
[10:15] <dimitern> fwereade, hey, are you about?
[10:16] <dimitern> frobware, ^^ pushed the fix
[10:17] <frobware> dimitern, you rebased?
[10:17] <dimitern> yep
[10:17] <frobware> dimitern, I was just looking at your changes since I pulled from earlier
[10:17] <frobware> dimitern, ok
[10:21] <dimitern> frobware, not removing /e/n/i.d/eth0.cfg doesn't seem to cause any harm - even rebooting the host/container is ok
[10:21] <frobware> dimitern, I'm trying to determine whether that's just luck based on the 'source .../*.cfg' order
[10:21] <frobware> dimitern, stgraber advised me to delete it
[10:23] <dimitern> frobware, well, that's even better - a proper fix, but just saying what i've seen so far
[10:24] <frobware> dimitern, me too. but have you rebooted a lot?
[10:24] <dimitern> frobware, i did reboot a few times, nothing fancier though
[10:24] <frobware> dimitern, I'm stuck with upgrade not working for me at the moment which make for slow progress
[10:26] <dimitern> frobware, that upgrade-juju issue - that's on trusty or xenial?
[10:30] <voidspace> dimitern: I don't really like adding unused code
[10:30] <voidspace> dimitern: why not add the gate in the PR that uses it?
[10:31] <frobware> dimitern, xenial
[10:33] <dimitern> voidspace, well, it's used by discoverspaces, but not yet by others
[10:34] <dimitern> voidspace, actually I did this because I thought I might need it in this PR
[10:34] <dimitern> voidspace, but fair enough, I'll pull gate out of this PR
[10:36] <voidspace> dimitern: it's up to you - other than that LGTM
[10:36] <frobware> dimitern, voidspace: seems like the right thing to do as it make reverting the PR saner - should it ever need to
[10:37] <dimitern> frobware, it's pretty easy with gh web ui
[10:38] <frobware> dimitern, what I mean is you would only be reverting stuff pertinent to the original fix
[10:38] <frobware> dimitern, anyway, back to ENI - the interfaces do not always come up for me. :(
[10:40] <dimitern> frobware, with my fix ?
[10:40] <frobware> dimitern, nope, mine
[10:41] <frobware> dimitern, the times I've bootstrapped and added a new machine without reboot seems to work fine with your fix \o/
[10:42] <dimitern> frobware, I found out something else (probably related to what jamespage was seeing - if say eth0 is unconfigured, but has children which are not - it breaks
[10:42] <dimitern> frobware, awesome!
[10:42] <frobware> dimitern, 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] <frobware> dimitern, I raised a bug for that
[10:43] <frobware> dimitern, https://bugs.launchpad.net/juju-core/+bug/1566791
[10:43] <mup> Bug #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:44] <dimitern> frobware, 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 needed
[10:46] <dimitern> frobware, 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 workarounds
[10:46] <frobware> dimitern, but the script runs in cloud-init
[10:46] <dimitern> frobware, it doesn't have to, does it?
[10:47] <frobware> dimitern, well, as I'm finding out again, the combo of: ifdown, copy-in-new-file; ifup -a, is not guaranteed.
[10:47] <dimitern> frobware, we can do it with proper retrying, etc. - bridge by bridge, based on what NICs we observe at run-time
[10:48] <frobware> dimitern, I'm generally -1 on this atm, particularly if MAAS models bridges.
[10:48] <frobware> in the near future
[10:48] <dimitern> frobware, perhaps some if-[pre|post]-down script was lingering?
[10:48] <frobware> dimitern, bbiab - 20 mins.
[10:48] <dimitern> frobware, well, they will be at some point, but we need to think about other providers
[11:17] <voidspace> babbageclunk: dimitern: frobware: well that took a long time! I now have a test that can successfully call StartInstance against MAAS 2 code.
[11:18] <dimitern> voidspace, \\o// great!
[11:19]  * dimitern should really install a 2.0 maas locally now :)
[11:19]  * fwereade cheers at voidspace
[11:19] <voidspace> fwereade: :-)
[11:19] <voidspace> testing 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 up
[11:20] <frobware> voidspace, very NICE!
[11:37] <bogdanteleaga> dimitern, https://bugs.launchpad.net/juju-core/+bug/1567676
[11:37] <mup> Bug #1567676: windows: networker tries to update invalid device and blocks machiner from working <juju-core:Triaged> <https://launchpad.net/bugs/1567676>
[11:37] <bogdanteleaga> help? :D
[11:38] <dimitern> bogdanteleaga, hey - the networker is gone for a while now; which version of juju?
[11:39] <bogdanteleaga> dimitern, latest, I wasn't sure what terminology to use
[11:39] <dimitern> bogdanteleaga, I see (might have helped to actually read the bug desc first :)
[11:39] <dimitern> bogdanteleaga, I know the issue
[11:40] <dimitern> bogdanteleaga, but that's already fixed in master IIRC - is that on a feature branch?
[11:41] <bogdanteleaga> dimitern, it's from yesterday's master
[11:41] <bogdanteleaga> dimitern, https://github.com/juju/juju/commit/b631875ff9ed67ec321f1d6fa90bc65d839b9e93
[11:43] <dimitern> bogdanteleaga, so the error in comment #1 is no longer happening
[11:43] <dimitern> bogdanteleaga, but the "<nil>" value of the address is troubling
[11:45] <bogdanteleaga> dimitern, I'll try the latest master
[11:47] <dimitern> bogdanteleaga, it might be better to use --config logging_config='<root>=TRACE' to bootstrap to get more context (and paste in the bug please)
[11:49]  * dimitern steps out for a couple of hours
[11:59]  * perrito666 steps out for an hour too
[12:36] <rogpeppe> an update to juju-core to fix the resources API used by charmrepo: http://reviews.vapour.ws/r/4490/
[12:37] <rogpeppe> should be trivial and I've very much like to get it landed to day if poss...
[12:37] <rogpeppe> so reviews welcome :)
[12:58] <axw> fwereade: 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/
[13:08] <mup> Bug #1567925 opened: help text for juju add-unit needs improving <helpdocs> <juju-core:New> <https://launchpad.net/bugs/1567925>
[13:09] <voidspace> babbageclunk: frobware: dimitern: maas2 has landed on master - all further maas2 work to be against master
[13:17] <mup> Bug #1567934 opened: uploadSuite.TestMockBuildTools relies on exact archive/zip output <juju-core:New> <https://launchpad.net/bugs/1567934>
[13:17] <mup> Bug #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:27] <alexisb> morning all
[13:29] <voidspace> alexisb: o/
[13:30] <babbageclunk> voidspace: nice!
[13:30] <katco> yaaaay life-giving internet
[13:30] <alexisb> katco, hey
[13:30] <voidspace> katco: morning
[13:30] <katco> alexisb: hey
[13:30] <alexisb> heh
[13:30] <katco> voidspace: morning from across the way
[13:30] <alexisb> is what I mean
[13:30] <voidspace> katco: o/
[13:32] <katco> alexisb: 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.0
[13:33] <alexisb> katco, sounds like it is time to move ;)
[13:33]  * katco wry smile
[13:34] <katco> perrito666: i have a patch to propose for the keystone issue... will you have a chance to tal?
[13:36] <babbageclunk> alexisb, katco: morning!
[13:36] <katco> babbageclunk: o/
[13:36] <katco> babbageclunk: good to hear from you again. how are you getting on with things?
[13:38] <babbageclunk> katco: settling in ok, I think. Heaps and heaps to learn, obviously!
[13:38] <katco> babbageclunk: that's what makes it fun :D
[13:39] <babbageclunk> katco: :)
[13:53] <mup> Bug #1567951 opened: help text for juju login needs improving <helpdocs> <juju-core:New> <https://launchpad.net/bugs/1567951>
[13:53] <mup> Bug #1567952 opened: container/lxd: TestDetectSubnetLocal fails with link/none <juju-core:New> <https://launchpad.net/bugs/1567952>
[13:54] <perrito666> katco: sure, shoot
[13:57] <katco> perrito666: http://reviews.vapour.ws/r/4492/
[13:58] <katco> perrito666: ta
[14:03] <natefinch> ericsnow: 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 type
[14:03] <alexisb> cmars, you around yet?
[14:04] <natefinch> ericsnow: I still have to update the tests for that code, but shouldn't be too bad.
[14:06] <perrito666> katco: shit it with a doubt
[14:07] <natefinch> perrito666: lol
[14:07] <perrito666> meh
[14:07] <perrito666> sorry typo
[14:07] <katco> perrito666: i think we only care about the major to resolve the client
[14:07] <cherylj> jam: you still around?
[14:07] <katco> perrito666: i.e. if the major is 2, we'd settle on 2.x whatever that may be
[14:09] <cherylj> frobware: ping?
[14:10] <perrito666> katco: I had miss interpreted your code, sorry
[14:11] <katco> perrito666: ah no worries... so is it still OK?
[14:11] <perrito666> yes
[14:13] <katco> perrito666: awesome... thanks for reviewing
[14:13] <perrito666> tx for that fix
[14:14] <katco> perrito666: weird situation... i don't really understand why an openstack deployment would report a version that isn't working yet
[14:15] <perrito666> cherylj: speaking of reviews, could I get a rev on http://reviews.vapour.ws/r/4481/ ?
[14:15] <cherylj> perrito666: yes, I will take a look this morning  (about to have a call right now :/)
[14:33] <katco> natefinch: ericsnow: https://plus.google.com/hangouts/_/canonical.com/retrospective?authuser=1
[14:39] <katco> ericsnow: ping?
[14:40] <frankban> cherylj: hey morning? any news on merging embedded-gui?
[14:40] <katco> natefinch: ericsnow: deferred half an hour; sorry, thought ericsnow was already on
[14:41] <mup> Bug #1567963 opened: help text for juju remove-unit needs improving <helpdocs> <juju-core:New> <https://launchpad.net/bugs/1567963>
[14:41] <cherylj> frankban: let me look at the CI reports, one sec
[14:42] <frankban> cherylj: sure ty
[14:54] <perrito666> https://www.mongodb.com/blog/post/mongodb-2-6-end-of-life?utm_content=buffere388e&utm_medium=social&utm_source=twitter.com&utm_campaign=buffer
[14:55] <perrito666> meh we could use a bot that says the title: "MongoDB 2.6 End of Life"
[14:55] <natefinch> perrito666: the url was pretty obvious ;)
[14:55] <perrito666> lol, I did not realize, I could have very well have stripped the get params
[14:56]  * dimitern whew... almost 1h stuck in traffic :/
[15:02] <dimitern> voidspace, frobware, babbageclunk, btw the sync call is not happening
[15:02] <dimitern> perhaps next week
[15:16] <alexisb> cherylj, fyi, I was able to deploy a few services yesterday off master
[15:21] <dooferlad> frobware: 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:22] <alexisb> dooferlad, that is good news
[15:22] <dooferlad> alexisb: :-)
[15:59]  * alexisb changes locations
[16:09] <frankban> cherylj: can we stick a little trivial last change to the embedded-gui branch? is it too late?
[16:09] <babbageclunk> dimitern: sync call? I don't think I'm on it - can you please add me?
[16:09] <evilnickveitch> cherylj, we may have a problem with a lot of the updated help text
[16:09] <frankban> hatch: you dropped
[16:09] <cherylj> frankban: dont' put anything else into embedded gui
[16:10] <cherylj> wait for us to merge it and put it directly in master
[16:10] <hatch> :'(
[16:10] <hatch> :D
[16:10] <frankban> cherylj: sounds good
[16:10] <cherylj> evilnickveitch: what's going on with the help text?
[16:10] <frankban> cherylj: when is the deadline for putting changes into master?
[16:10] <evilnickveitch> cherylj, a lot of the user related commands reference 'share-model'
[16:10] <evilnickveitch> 'sahre-model' seems to have disappeared
[16:11] <evilnickveitch> i mean, as a command
[16:11] <cherylj> frankban: needs to be done today
[16:11] <cherylj> evilnickveitch: yeah, it was a late change to be juju grant
[16:12] <evilnickveitch> cherylj, :(
[16:13] <evilnickveitch> cherylj, well, be on the lookout for it in case we miss changing any - it appears in a lot of 'see also' text
[16:13] <cherylj> evilnickveitch: ok, thanks, I'll keep an eye out
[16:13] <evilnickveitch> cherylj, I will just go and rewrite all the user docs...
[16:14] <dimitern> babbageclunk, sure - invite sent
[16:15] <babbageclunk> dimitern: thanks!
[16:18] <frankban> cherylj: actually I am lucky enough that the code I need to change is the GUI server already in master, will be just a trivial improvement
[16:19] <cherylj> frankban: great!
[16:20] <redir> cherylj: evilnickveitch there appears to be one unshare-model too
[16:32] <cmars> i can review some things now. what should i look at first?
[16:33] <cmars> alexisb, cherylj ^^ ?
[16:34] <alexisb> perrito666, are you still waiting for a review?
[16:34] <alexisb> cmars, this one is important: http://reviews.vapour.ws/r/4491/
[16:36]  * cmars is reviewing
[16:37] <alexisb> cmars next inline would be this one: http://reviews.vapour.ws/r/4481/
[16:38] <mup> Bug #1568028 opened: Juju fails to deploy with 2.0-beta3-xenial-arm64 <juju-core:New> <https://launchpad.net/bugs/1568028>
[16:40] <alexisb> frobware, I am back online if you are still looking for me
[16:58] <alexisb> cmars, if you are done with the two reviews above, the next in priority are any from jam dealing with lxd and containers
[16:58] <alexisb> like: http://reviews.vapour.ws/r/4478/
[17:00] <cmars> alexisb, still on the first one
[17:01] <alexisb> cmars, understood, thanks
[17:02] <mup> Bug #1568028 changed: Juju fails to deploy with 2.0-beta3-xenial-arm64 <juju-core:New> <https://launchpad.net/bugs/1568028>
[17:24] <cmars> 4491 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 with
[17:28] <alexisb> cmars, what about 4881>
[17:28] <alexisb> ?
[17:29] <cmars> alexisb, you mean 4481? that looks more manageable. i'll review over lunch
[17:30] <alexisb> yes that is what I meant sorry
[17:30] <alexisb> and cool, thank you
[17:32] <frobware> alexisb, just to say... great few days with christian, have some sort of fix (need testing) for bug #1566801
[17:32] <mup> Bug #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:36]  * frobware EOW 
[17:51] <katco> ericsnow: you have ship its
[17:51] <ericsnow> katco: thanks
[17:52] <natefinch> katco, ericsnow: down to 4 comments btw
[17:52] <ericsnow> natefinch: nice
[17:52] <katco> natefinch: yeah just looked... that is a ton of code lol
[17:52] <natefinch> katco: yeah, there's a reason it took a while
[17:53] <katco> natefinch: ericsnow: thanks for pairing on that
[17:53] <natefinch> katco: still need some new tests for some of the new code eric and I wrote yesterday
[18:00] <mup> Bug #1568069 opened: Container networking cannot ssh <ci> <lxc> <network> <regression> <juju-core:Triaged> <https://launchpad.net/bugs/1568069>
[18:06] <mup> Bug #1568069 changed: Container networking cannot ssh <ci> <lxc> <network> <regression> <juju-core:Triaged> <https://launchpad.net/bugs/1568069>
[18:20] <alexisb> katco, I have a few minutes if you would like ot meet
[18:21] <mup> Bug #1568069 opened: Container networking cannot ssh <ci> <lxc> <network> <regression> <juju-core:Triaged> <https://launchpad.net/bugs/1568069>
[18:21] <mup> Bug #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:24] <katco> alexisb: sure
[18:26] <alexisb> katco, 1x1 HO
[18:33] <perrito666> alexisb: sadly I am
[18:37] <cmars> perrito666, reviewed 4481
[18:37] <perrito666> cmars: tx
[18:38] <perrito666> cmars: tx a lot
[18:45] <mup> Bug #1568090 opened: help text for juju sync-tools needs improving <helpdocs> <juju-core:New> <https://launchpad.net/bugs/1568090>
[18:54] <natefinch> ericsnow, 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] <ericsnow> natefinch: k
[18:54] <ericsnow> natefinch: let me know if I can help; as I mentioned, I did the same thing for channels
[18:55] <natefinch> definitely
[19:06] <mup> Bug #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] <katco> natefinch: sorry was otp. whoa, what does "fixing model migrations" mean? that doesn't sound fun =|
[19:09] <natefinch> katco: 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:10] <katco> natefinch: oh b/c you're storing new info in state you have to support it when migrating models
[19:10] <natefinch> yep
[19:10] <katco> natefinch: gotcha... makes sense. sounded scary o/0
[19:10] <natefinch> still a little scary ;)
[19:10] <katco> ha
[19:14] <natefinch> uh hmm
[19:14] <natefinch> there's nothing in there about charms
[19:14] <natefinch> ericsnow: ^
[19:15] <ericsnow> natefinch: if we're not migrating charms then I'm guessing that Tim & co. haven't gotten to it yet
[19:16] <ericsnow> natefinch: however, I'd be surprised by that
[19:16] <ericsnow> natefinch: perhaps it's folded into the service representation? (and they will re-download the blobs)
[19:17] <natefinch> ericsnow: I thought of that, but then they can't do that for local charms
[19:17] <ericsnow> natefinch: yuck
[19:17] <ericsnow> natefinch: so blobs have to be migrated too I suppose
[19:18] <mup> Bug #1568101 opened: help text for juju deploy needs improving <helpdocs> <juju-core:New> <https://launchpad.net/bugs/1568101>
[19:18] <natefinch> ericsnow: I would assume so....
[19:19] <natefinch> ericsnow: hang on... I see some code related to charms in core/migration/migration.go .... reading
[19:19] <ericsnow> natefinch: ah, no MigrationSuite.TestCharmDocFields()
[19:29] <natefinch> ericsnow: 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 there
[19:29] <ericsnow> natefinch: will do
[19:31] <alexisb> katco, ericsnow: has this documentation made ericsnow's google doc obsolete?: https://github.com/juju/juju/wiki/Implementing-environment-providers
[19:33] <katco> alexisb: 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 did
[19:34] <natefinch> alexisb 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:35] <katco> natefinch: i don
[19:35] <katco> 't
[19:35] <katco> natefinch: menn0 is working on that primarily... thumper should still be able to answer questions as well
[19:35] <natefinch> ..... do our tests create virtual ethernets on the machine they're running on?
[19:36] <katco> natefinch: omg... PLEASE tell me they don't
[19:36] <natefinch> sure looks like it
[19:36] <natefinch> I have like 4 virtual ethernet devices on my machine right now
[19:36] <katco> what. the. hell.
[19:36] <natefinch> I noticed because I saw a popup that say "connected to veth<somguid>
[19:38] <alexisb> natefinch, fwereade sometimes is around late
[19:38] <alexisb> otherwise they are all on their weekend
[19:38] <alexisb> natefinch, the MM work is not complete so it may be that it is not in master yet
[19:39] <alexisb> though with menno's latest branch I was able to successfully migrate a basic mediawiki deploy successfully
[19:40] <natefinch> alexisb: ok
[19:43] <natefinch> katco: http://i.imgur.com/d1ZKuPz.png
[19:44] <ericsnow> katco: gah, I forgot to mention a patch I have up against charmrepo:  https://github.com/juju/charmrepo/pull/84
[19:45] <katco> ericsnow: k tal
[19:45] <ericsnow> katco: ta
[19:46] <natefinch> that feeling when you're pretty sure you're waiting for the 10 minute testing timeout :/
[19:46] <cherylj> I know that feeling
[19:47] <redir> natefinch: everytime I run all the tests.
[19:48] <natefinch> or not: ok  	github.com/juju/juju/state	414.273s
[19:49] <natefinch> almost 7 minutes just for state tests.  Huzzah.
[19:57] <alexisb> o the state tests...
[19:58] <perrito666> natefinch: the state tests are quite shroedinger cat-ish unthil they finish they have and have not failed in your mind
[19:59] <katco> ericsnow: lgtm... fairly straightforward changes :)
[20:00] <ericsnow> katco: yep :)
[20:00] <ericsnow> katco: thanks
[20:00] <katco> ericsnow: ty
[20:09] <mup> Bug #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] <mup> Bug #1568122 opened: help text for juju remove-machine needs improving <helpdocs> <juju-core:New> <https://launchpad.net/bugs/1568122>
[20:13] <cmars> alexisb, should i try to continue reviewing 4491 or look at something else
[20:21] <alexisb> cmars, if they are easier you can do the lxd reviews
[20:21] <alexisb> http://reviews.vapour.ws/r/4478/
[20:22] <cmars> alexisb, looking
[20:22]  * perrito666 sees his refactoring interrupted by eye drops :p
[20:25] <ericsnow> natefinch: LGTM with a few small comments
[20:26] <natefinch> ericsnow: ok
[20:26] <natefinch> ericsnow: thanks
[20:26] <ericsnow> natefinch: np
[20:26] <ericsnow> natefinch: thanks for doing it :)
[20:51] <ericsnow> natefinch: how close are you to pulling the trigger on that patch?
[20:59] <katco> ericsnow: you were waiting on him to rebase weren't you?
[21:00] <ericsnow> katco: was hoping he'd merge his patch before I land my last one
[21:02] <katco> ericsnow: might have to land yours first... your work needs to be in by EOD
[21:03] <katco> ericsnow: and then we can pair to help through the difficult merge
[21:03] <ericsnow> katco: won't be hard, just tedious :)
[21:03] <ericsnow> katco: we're changing a bunch of the same signatures
[21:05] <katco> ericsnow: he's probably gone to dinner, i'd say go ahead and begin merging
[21:05] <perrito666> aghh I broke the reboot test :(
[21:06] <ericsnow> katco: sounds good
[21:19] <natefinch> ericsnow: thanks, fotgot you were waiting on me
[21:19] <ericsnow> natefinch: np
[21:19] <natefinch> bbl
[21:39] <mup> Bug #1568150 opened: xenial lxc containers not starting <juju-core:New> <https://launchpad.net/bugs/1568150>
[22:00] <alexisb> perrito666, I am seeing reboot test failures too
[22:00] <alexisb> from master
[22:00] <perrito666> meh, It seems broken
[22:00] <perrito666> I believe its breaking mongod but it is not related from my changes
[22:00] <perrito666> this might prevent me from merging :(
[22:01] <alexisb> alrighty all I am out for a few hours will be back this evening
[22:01] <alexisb> perrito666, please open a bug with details
[22:01] <perrito666> ok
[22:03] <alexisb> thanks
[22:03] <alexisb> alexisb-afk
[22:09] <perrito666> alexisb: for when you come back, I cannot reproduce now but I feel we are leaking lxcs and mongos
[22:09] <mup> Bug #1568160 opened: destroying a unit with an error gives no user feedback <juju-core:New> <https://launchpad.net/bugs/1568160>
[22:09] <mup> Bug #1568161 opened: Pass whether service is primary or subordinate via relation <juju-core:New> <https://launchpad.net/bugs/1568161>
[22:30] <mup> Bug #1568175 opened: destroy-controller failed: EOF <ci> <destroy-controller> <regression> <juju-core:Triaged> <https://launchpad.net/bugs/1568175>
[22:30] <mup> Bug #1568176 opened: charm deployment requests invalid revision number <juju-core:New> <https://launchpad.net/bugs/1568176>
[22:34] <mup> Bug #1568175 changed: destroy-controller failed: EOF <ci> <destroy-controller> <regression> <juju-core:Triaged> <https://launchpad.net/bugs/1568175>
[22:34] <mup> Bug #1568176 changed: charm deployment requests invalid revision number <juju-core:New> <https://launchpad.net/bugs/1568176>
[22:40] <mup> Bug # opened: 1568175, 1568176, 1568177, 1568179
[23:01] <mup> Bug #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] <mup> Bug #1568183 opened: GCE timeout pending new instance operation failed <ci> <gce-provider> <jujuqa> <reliability> <juju-core:Triaged> <https://launchpad.net/bugs/1568183>
[23:01] <ericsnow> katco: third patch merging now :)
[23:01] <mup> Bug #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] <katco> ericsnow: woot woot!
[23:31] <mup> Bug #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] <mup> Bug #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] <mup> Bug #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>