| mup | Bug #1567598 changed: storage-get causes state server to 'error: bad request' <storage> <juju-core:Invalid> <https://launchpad.net/bugs/1567598> | 00:06 |
|---|---|---|
| mwhudson | perrito666: ping? | 00:08 |
| 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:09 |
| 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:10 |
| mwhudson | alexisb: ok | 00:11 |
| mwhudson | alexisb: was actually typing up a bug report now, as it happens :-) | 00:11 |
| alexisb | perfect :) | 00:11 |
| mwhudson | alexisb: https://bugs.launchpad.net/juju-core/+bug/1567708 | 00:14 |
| 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:15 |
| 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:16 |
| mup | Bug #1567708 opened: unit tests fail with mongodb 3.2 <juju-core:New> <https://launchpad.net/bugs/1567708> | 00:21 |
| natefinch | evening all | 00:31 |
| mup | Bug #1567708 changed: unit tests fail with mongodb 3.2 <juju-core:New> <https://launchpad.net/bugs/1567708> | 00:33 |
| 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:36 |
| 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:51 |
| davecheney | cherylj: http://pad.lv/1566303 | 00:53 |
| davecheney | did this pass CI testing ? | 00:53 |
| davecheney | i have access to a windows machine now and can investigate if it's still failing | 00:54 |
| 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:55 |
| 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> | 00:57 |
| * thumper heads out to walk the dog | 01:03 | |
| 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:06 |
| cherylj | davecheney: no, the windows test failed again: http://reports.vapour.ws/releases/3863 | 01:15 |
| davecheney | cherylj: thanks, i'll try to get on the windows test machine now | 01:20 |
| 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:21 |
| mup | Bug #1567458 changed: destroy-controller message and failure is not user friendly <juju-core:New> <https://launchpad.net/bugs/1567458> | 01:27 |
| mup | Bug #1567458 opened: destroy-controller message and failure is not user friendly <juju-core:New> <https://launchpad.net/bugs/1567458> | 01:36 |
| 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:37 |
| 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:38 |
| redir | cherylj: OK. And have you settled on some formatting? indents all with line breaks etc? | 01:39 |
| 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:40 |
| redir | cherylj: ^ | 01:42 |
| cherylj | thanks, redir :) | 01:42 |
| davecheney | menn0: i think this was fixed by william's manifold thinggy | 01:44 |
| 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:45 |
| davecheney | menn0: that's what i'm worried about | 01:46 |
| davecheney | we cannot backport them all | 01:46 |
| 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:48 |
| 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> | 01:54 |
| 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:07 |
| 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:10 |
| 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:12 |
| 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:13 |
| cherylj | menn0: I also still have that mgopurge problem from mario that I haven't done and could use help with :) | 02:14 |
| menn0 | cherylj: ok, I can pick that up next | 02:15 |
| mup | Bug # opened: 1567722, 1567724, 1567726, 1567728, 1567730, 1567732, 1567734 | 02:16 |
| 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:25 |
| 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:26 |
| 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:27 | |
| 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:28 |
| 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:29 |
| 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:30 |
| axw | because the machines and services don't refcount or anything on the model | 02:31 |
| thumper | ok | 02:31 |
| 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:32 |
| * axw nods | 02:33 | |
| 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:34 |
| 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:35 |
| mup | Bug # opened: 1567728, 1567730, 1567732, 1567734 | 02:37 |
| davecheney | base_windows.go makes me sad | 02:48 |
| davecheney | 1. because the registiry api is nuts | 02:48 |
| davecheney | 2. because the way that code enables the api to continue to be nuts, is weird, codependant and unhealtht | 02:49 |
| 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:07 |
| davecheney | menn0: cherylj | 03:12 |
| davecheney | http://reviews.vapour.ws/r/4486/ | 03:12 |
| mwhudson | uh | 03:17 |
| mwhudson | oh | 03:17 |
| mwhudson | nm | 03:17 |
| mwhudson | i should just upgrade to xenial | 03:17 |
| 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:23 |
| 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:24 |
| mwhudson | oh yeah, lxd keeps moving things around | 03:25 |
| 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:26 |
| mwhudson | oh maybe only with the tip go tool | 03:27 |
| mup | Bug #1567747 opened: "juju metadata generate-image" does not validate architectures <juju-core:Triaged> <https://launchpad.net/bugs/1567747> | 03:37 |
| menn0 | davecheney: looking | 03:38 |
| menn0 | davecheney: looks good | 03:39 |
| 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:42 |
| mup | Bug #1567747 changed: "juju metadata generate-image" does not validate architectures <juju-core:Triaged> <https://launchpad.net/bugs/1567747> | 03:49 |
| mup | Bug #1567747 opened: "juju metadata generate-image" does not validate architectures <juju-core:Triaged> <https://launchpad.net/bugs/1567747> | 03:55 |
| redir | night | 04:00 |
| 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:28 |
| davecheney | natefinch ping ? | 05:35 |
| davecheney | hmm, too late | 05:35 |
| frobware | dimitern, voidspace, dooferlad: going to skip standup today. | 08:42 |
| dooferlad | frobware: ok | 08:42 |
| 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:49 |
| 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:50 |
| 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:51 |
| frobware | babbageclunk, ack | 08:52 |
| babbageclunk | voidspace: what I did is in the document from yesterday, though. | 08:52 |
| * thumper is in the hangout | 08:58 | |
| thumper | babbageclunk: good luck | 08:58 |
| * thumper wonders if he will be sitting alone.... | 09:01 | |
| voidspace | babbageclunk: ok | 09:03 |
| voidspace | thumper: omw | 09:03 |
| 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:07 |
| frobware | dimitern, did your change work with my ... | 09:08 |
| dimitern | frobware, testing always appreciated :) yes - it uses the /e/n/i hack | 09:13 |
| frobware | dimitern, not sure my hack is working today; newer lxd since weds... | 09:14 |
| dimitern | frobware, I did a dist-upgrade this morning, so I have rc9 I think | 09:15 |
| dimitern | yep | 09:15 |
| frobware | dimitern, I'm trying to find the right place and time to delete the lxd supplied (and mandatory) eth0.cfg | 09:16 |
| 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:17 |
| 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. | 09:27 |
| voidspace | dimitern: babbageclunk: PR to merge maas2 onto master http://reviews.vapour.ws/r/4487/ | 10:03 |
| dimitern | voidspace, I'll swap it for this - http://reviews.vapour.ws/r/4488/ :) | 10:11 |
| dimitern | voidspace, LGTM | 10:14 |
| voidspace | dimitern: thanks - looking | 10:14 |
| dimitern | voidspace, ta! | 10:15 |
| dimitern | fwereade, hey, are you about? | 10:15 |
| dimitern | frobware, ^^ pushed the fix | 10:16 |
| 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:17 |
| 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:21 |
| dimitern | frobware, well, that's even better - a proper fix, but just saying what i've seen so far | 10:23 |
| 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:24 |
| dimitern | frobware, that upgrade-juju issue - that's on trusty or xenial? | 10:26 |
| 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:30 |
| frobware | dimitern, xenial | 10:31 |
| dimitern | voidspace, well, it's used by discoverspaces, but not yet by others | 10:33 |
| 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:34 |
| 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:36 |
| dimitern | frobware, it's pretty easy with gh web ui | 10:37 |
| 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:38 |
| dimitern | frobware, with my fix ? | 10:40 |
| frobware | dimitern, nope, mine | 10:40 |
| frobware | dimitern, the times I've bootstrapped and added a new machine without reboot seems to work fine with your fix \o/ | 10:41 |
| 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:42 |
| 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:43 |
| 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:44 |
| 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:46 |
| 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:47 |
| 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 | 10:48 |
| 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:17 |
| dimitern | voidspace, \\o// great! | 11:18 |
| * 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:19 |
| frobware | voidspace, very NICE! | 11:20 |
| 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:37 |
| dimitern | bogdanteleaga, hey - the networker is gone for a while now; which version of juju? | 11:38 |
| 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:39 |
| dimitern | bogdanteleaga, but that's already fixed in master IIRC - is that on a feature branch? | 11:40 |
| bogdanteleaga | dimitern, it's from yesterday's master | 11:41 |
| bogdanteleaga | dimitern, https://github.com/juju/juju/commit/b631875ff9ed67ec321f1d6fa90bc65d839b9e93 | 11:41 |
| 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:43 |
| bogdanteleaga | dimitern, I'll try the latest master | 11:45 |
| 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:47 |
| * dimitern steps out for a couple of hours | 11:49 | |
| * perrito666 steps out for an hour too | 11:59 | |
| rogpeppe | an update to juju-core to fix the resources API used by charmrepo: http://reviews.vapour.ws/r/4490/ | 12:36 |
| 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:37 |
| 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/ | 12:58 |
| mup | Bug #1567925 opened: help text for juju add-unit needs improving <helpdocs> <juju-core:New> <https://launchpad.net/bugs/1567925> | 13:08 |
| voidspace | babbageclunk: frobware: dimitern: maas2 has landed on master - all further maas2 work to be against master | 13:09 |
| 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:17 |
| alexisb | morning all | 13:27 |
| voidspace | alexisb: o/ | 13:29 |
| 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:30 |
| 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:32 |
| alexisb | katco, sounds like it is time to move ;) | 13:33 |
| * katco wry smile | 13:33 | |
| katco | perrito666: i have a patch to propose for the keystone issue... will you have a chance to tal? | 13:34 |
| 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:36 |
| 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:38 |
| babbageclunk | katco: :) | 13:39 |
| 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:53 |
| perrito666 | katco: sure, shoot | 13:54 |
| katco | perrito666: http://reviews.vapour.ws/r/4492/ | 13:57 |
| katco | perrito666: ta | 13:58 |
| 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:03 |
| natefinch | ericsnow: I still have to update the tests for that code, but shouldn't be too bad. | 14:04 |
| perrito666 | katco: shit it with a doubt | 14:06 |
| 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:07 |
| cherylj | frobware: ping? | 14:09 |
| perrito666 | katco: I had miss interpreted your code, sorry | 14:10 |
| katco | perrito666: ah no worries... so is it still OK? | 14:11 |
| perrito666 | yes | 14:11 |
| katco | perrito666: awesome... thanks for reviewing | 14:13 |
| perrito666 | tx for that fix | 14:13 |
| katco | perrito666: weird situation... i don't really understand why an openstack deployment would report a version that isn't working yet | 14:14 |
| 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:15 |
| katco | natefinch: ericsnow: https://plus.google.com/hangouts/_/canonical.com/retrospective?authuser=1 | 14:33 |
| katco | ericsnow: ping? | 14:39 |
| 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:40 |
| 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:41 |
| frankban | cherylj: sure ty | 14:42 |
| 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:54 |
| 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:55 |
| * dimitern whew... almost 1h stuck in traffic :/ | 14:56 | |
| dimitern | voidspace, frobware, babbageclunk, btw the sync call is not happening | 15:02 |
| dimitern | perhaps next week | 15:02 |
| alexisb | cherylj, fyi, I was able to deploy a few services yesterday off master | 15:16 |
| 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:21 |
| alexisb | dooferlad, that is good news | 15:22 |
| dooferlad | alexisb: :-) | 15:22 |
| * alexisb changes locations | 15:59 | |
| 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:09 |
| 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:10 |
| 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:11 |
| evilnickveitch | cherylj, :( | 16:12 |
| === redir_afk is now known as redir | ||
| 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:13 |
| dimitern | babbageclunk, sure - invite sent | 16:14 |
| babbageclunk | dimitern: thanks! | 16:15 |
| 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:18 |
| cherylj | frankban: great! | 16:19 |
| redir | cherylj: evilnickveitch there appears to be one unshare-model too | 16:20 |
| cmars | i can review some things now. what should i look at first? | 16:32 |
| cmars | alexisb, cherylj ^^ ? | 16:33 |
| alexisb | perrito666, are you still waiting for a review? | 16:34 |
| alexisb | cmars, this one is important: http://reviews.vapour.ws/r/4491/ | 16:34 |
| * cmars is reviewing | 16:36 | |
| alexisb | cmars next inline would be this one: http://reviews.vapour.ws/r/4481/ | 16:37 |
| mup | Bug #1568028 opened: Juju fails to deploy with 2.0-beta3-xenial-arm64 <juju-core:New> <https://launchpad.net/bugs/1568028> | 16:38 |
| alexisb | frobware, I am back online if you are still looking for me | 16:40 |
| 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/ | 16:58 |
| cmars | alexisb, still on the first one | 17:00 |
| alexisb | cmars, understood, thanks | 17:01 |
| mup | Bug #1568028 changed: Juju fails to deploy with 2.0-beta3-xenial-arm64 <juju-core:New> <https://launchpad.net/bugs/1568028> | 17:02 |
| 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:24 |
| alexisb | cmars, what about 4881> | 17:28 |
| alexisb | ? | 17:28 |
| cmars | alexisb, you mean 4481? that looks more manageable. i'll review over lunch | 17:29 |
| alexisb | yes that is what I meant sorry | 17:30 |
| alexisb | and cool, thank you | 17:30 |
| 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:32 |
| * frobware EOW | 17:36 | |
| katco | ericsnow: you have ship its | 17:51 |
| ericsnow | katco: thanks | 17:51 |
| 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:52 |
| 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 | 17:53 |
| mup | Bug #1568069 opened: Container networking cannot ssh <ci> <lxc> <network> <regression> <juju-core:Triaged> <https://launchpad.net/bugs/1568069> | 18:00 |
| mup | Bug #1568069 changed: Container networking cannot ssh <ci> <lxc> <network> <regression> <juju-core:Triaged> <https://launchpad.net/bugs/1568069> | 18:06 |
| alexisb | katco, I have a few minutes if you would like ot meet | 18:20 |
| 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:21 |
| katco | alexisb: sure | 18:24 |
| === jog_ is now known as jog | ||
| alexisb | katco, 1x1 HO | 18:26 |
| perrito666 | alexisb: sadly I am | 18:33 |
| cmars | perrito666, reviewed 4481 | 18:37 |
| perrito666 | cmars: tx | 18:37 |
| perrito666 | cmars: tx a lot | 18:38 |
| mup | Bug #1568090 opened: help text for juju sync-tools needs improving <helpdocs> <juju-core:New> <https://launchpad.net/bugs/1568090> | 18:45 |
| 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:54 |
| natefinch | definitely | 18:55 |
| 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:06 |
| 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:09 |
| 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:10 |
| natefinch | uh hmm | 19:14 |
| natefinch | there's nothing in there about charms | 19:14 |
| natefinch | ericsnow: ^ | 19:14 |
| ericsnow | natefinch: if we're not migrating charms then I'm guessing that Tim & co. haven't gotten to it yet | 19:15 |
| 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:16 |
| 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:17 |
| 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:18 |
| 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:19 |
| 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:29 |
| alexisb | katco, ericsnow: has this documentation made ericsnow's google doc obsolete?: https://github.com/juju/juju/wiki/Implementing-environment-providers | 19:31 |
| 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:33 |
| 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:34 |
| 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:35 |
| 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:36 |
| 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:38 |
| alexisb | though with menno's latest branch I was able to successfully migrate a basic mediawiki deploy successfully | 19:39 |
| natefinch | alexisb: ok | 19:40 |
| natefinch | katco: http://i.imgur.com/d1ZKuPz.png | 19:43 |
| ericsnow | katco: gah, I forgot to mention a patch I have up against charmrepo: https://github.com/juju/charmrepo/pull/84 | 19:44 |
| katco | ericsnow: k tal | 19:45 |
| ericsnow | katco: ta | 19:45 |
| 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:46 |
| redir | natefinch: everytime I run all the tests. | 19:47 |
| natefinch | or not: ok github.com/juju/juju/state414.273s | 19:48 |
| natefinch | almost 7 minutes just for state tests. Huzzah. | 19:49 |
| alexisb | o the state tests... | 19:57 |
| perrito666 | natefinch: the state tests are quite shroedinger cat-ish unthil they finish they have and have not failed in your mind | 19:58 |
| katco | ericsnow: lgtm... fairly straightforward changes :) | 19:59 |
| ericsnow | katco: yep :) | 20:00 |
| ericsnow | katco: thanks | 20:00 |
| katco | ericsnow: ty | 20:00 |
| 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:09 |
| cmars | alexisb, should i try to continue reviewing 4491 or look at something else | 20:13 |
| alexisb | cmars, if they are easier you can do the lxd reviews | 20:21 |
| alexisb | http://reviews.vapour.ws/r/4478/ | 20:21 |
| cmars | alexisb, looking | 20:22 |
| * perrito666 sees his refactoring interrupted by eye drops :p | 20:22 | |
| ericsnow | natefinch: LGTM with a few small comments | 20:25 |
| natefinch | ericsnow: ok | 20:26 |
| natefinch | ericsnow: thanks | 20:26 |
| ericsnow | natefinch: np | 20:26 |
| ericsnow | natefinch: thanks for doing it :) | 20:26 |
| ericsnow | natefinch: how close are you to pulling the trigger on that patch? | 20:51 |
| katco | ericsnow: you were waiting on him to rebase weren't you? | 20:59 |
| ericsnow | katco: was hoping he'd merge his patch before I land my last one | 21:00 |
| katco | ericsnow: might have to land yours first... your work needs to be in by EOD | 21:02 |
| 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:03 |
| 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:05 |
| ericsnow | katco: sounds good | 21:06 |
| natefinch | ericsnow: thanks, fotgot you were waiting on me | 21:19 |
| ericsnow | natefinch: np | 21:19 |
| natefinch | bbl | 21:19 |
| mup | Bug #1568150 opened: xenial lxc containers not starting <juju-core:New> <https://launchpad.net/bugs/1568150> | 21:39 |
| === urulama is now known as urulama|___ | ||
| 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:00 |
| 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:01 |
| alexisb | thanks | 22:03 |
| alexisb | alexisb-afk | 22:03 |
| 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:09 |
| 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:30 |
| 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:34 |
| mup | Bug # opened: 1568175, 1568176, 1568177, 1568179 | 22:40 |
| 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:01 |
| 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> | 23:31 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!
