| * thumper stretches | 00:50 | |
| thumper | that is some tests passing now... | 00:50 |
|---|---|---|
| thumper | time to take the dog for a walk | 00:50 |
| === thumper is now known as thumper-dogwalk | ||
| === natefinch-afk is now known as natefinch | ||
| mwhudson | anastasiamac: building a new juju-mongodb3.2 now in my ppa \o/ | 01:24 |
| mwhudson | at least half the time on the update was spent getting confused by dos line endings, grr | 01:24 |
| anastasiamac | mwhudson: m sure they'd ab lots ppl very happy :) tyvm! | 01:59 |
| === thumper-dogwalk is now known as thumper | ||
| axw | thumper: apart from the two things I've commented on your PR, it looks fine to me | 04:36 |
| thumper | axw: the pubsub one? | 04:37 |
| axw | thumper: sorry yeah | 04:37 |
| axw | thumper: and they're not showstoppers, just my preference to avoid reflection, and I think the channel stuff could be simplified but otherwise it's ok | 04:37 |
| menn0 | thumper: http://reviews.vapour.ws/r/5303/ pls | 05:32 |
| thumper | menn0: done | 05:50 |
| rogpeppe2 | axw: ping | 06:51 |
| === rogpeppe2 is now known as rogpeppe | ||
| axw | rogpeppe: pong | 06:52 |
| rogpeppe | axw: hiya | 06:52 |
| rogpeppe | axw: i was just wondering about user-namespaced model names | 06:52 |
| rogpeppe | axw: if there are two models with the same name in a controller but under two different user namespaces, how would i switch to each? | 06:53 |
| axw | rogpeppe: juju switch owner/model | 06:53 |
| axw | rogpeppe: it was done and dusted, but got backed out as it broke a CI test. waiting on a change to CI being merged before landing again | 06:53 |
| rogpeppe | axw: ah! that's why i don't see it! | 06:54 |
| rogpeppe | axw: i thought you'd done something but couldn't find it | 06:54 |
| axw | rogpeppe: should be back in tomorrow I think | 06:54 |
| rogpeppe | axw: cool | 06:54 |
| axw | assuming master remains unblocked | 06:55 |
| rogpeppe | axw: did you update the docs for juju switch too? | 06:55 |
| axw | rogpeppe: hmm not sure now, but I don't remember doing it. I'll make sure they're up to date when I re-land | 06:55 |
| rogpeppe | axw: thanks | 06:55 |
| * dimitern is going offline for ~30m (scheduled power cut) | 07:26 | |
| hoenir | so if the fix is commited why does it appear on this ? http://juju.fail/ | 09:15 |
| mup | Bug #1606487 opened: apt-mirror does not override security.ubuntu.com for controller <juju-core:New> <https://launchpad.net/bugs/1606487> | 09:16 |
| dimitern | hoenir: because it's not Fix Release yet | 09:34 |
| dimitern | should be once 1.25.7 is out | 09:34 |
| axw | fwereade_: what's wrong with OpenParams? I'm hesitant to change only because it's consistent with just about every other param struct we use | 09:43 |
| fwereade_ | axw, it's not specific to Open | 09:50 |
| fwereade_ | axw, it's naming the type for one of its uses, instead of for what it really *is* | 09:51 |
| axw | fwereade_: environs.New shouldn't even exist. it's just a short-cut around the global provider registry, and EnvironProvider.Open | 09:52 |
| axw | fwereade_: if it didn't exist, it would be specific to Open :/ | 09:52 |
| axw | fwereade_: trying to think of another name. Definition doesn't really sit right with me, because I don't think credentials define an environ | 09:53 |
| fwereade_ | axw, yeah, nor with me | 09:53 |
| fwereade_ | axw, so if you can't think of anything better, go for it -- but I still really doubt this type will be used *exclusively* when calling Open | 09:54 |
| hoenir | thanks dimitern | 09:55 |
| axw | fwereade_: I'm trying very hard to make Open be the only thing that cares about creds/cloud definition, so I really hope it is. I'll ponder, but otherwise I think I'll defer renaming until we have another user - it would probably be clearer what to call it then anyway. | 09:56 |
| fwereade_ | axw, ack, thanks for pondering :) | 09:57 |
| fwereade_ | axw, tyvm for all the recent refactoring, btw, it is gruelling to review but well worth it | 09:58 |
| axw | fwereade_: cool, and thanks for reviewing. pretty gruelling to change too :p | 09:58 |
| axw | I'm especially not looking forward to adding cloud/creds into OpenParams | 09:58 |
| fwereade_ | axw, ha, yeah | 09:58 |
| axw | only 65 call sites | 09:59 |
| * fwereade_ shudders gently | 09:59 | |
| mup | Bug #1606506 opened: juju should tell maas to use a kernel with all required features <oil> <oil-2.0> <juju-core:New> <https://launchpad.net/bugs/1606506> | 10:01 |
| axw | mgz: any chance of a review on the juju-ci-tools MP? | 10:04 |
| mup | Bug #1606506 changed: juju should tell maas to use a kernel with all required features <oil> <oil-2.0> <juju-core:New> <https://launchpad.net/bugs/1606506> | 10:10 |
| mup | Bug #1606506 opened: juju should tell maas to use a kernel with all required features <oil> <oil-2.0> <juju-core:New> <https://launchpad.net/bugs/1606506> | 10:16 |
| dimitern | dooferlad: cheers for the write-up! | 10:21 |
| dimitern | dooferlad: I got a bit carried away suggesting and commenting :) | 10:21 |
| dooferlad | dimitern: thanks! | 10:21 |
| dooferlad | dimitern: no worries! | 10:21 |
| dimitern | dooferlad: it also helped me get into the right train of thought w.r.t. the sprint prep around IPAM | 10:22 |
| dooferlad | dimitern: would you like to hop into a conversation with macgreagoir and me? | 10:22 |
| dooferlad | dimitern: just in a hangout | 10:22 |
| dimitern | can do in 2m though | 10:23 |
| dooferlad | dimitern: https://hangouts.google.com/hangouts/_/canonical.com/release-team | 10:23 |
| dooferlad | dimitern: see you when you get here | 10:23 |
| babbageclunk | dimitern: ping? | 11:13 |
| babbageclunk | fwereade_: you around? | 11:18 |
| dimitern | babbageclunk: I am, but I'm just going out - will be back ~1h though | 11:21 |
| babbageclunk | dimitern: Ok cool - will go for a run now. Talk to you later! | 11:22 |
| * babbageclunk goes for a run. | 11:22 | |
| frankban|afk | fwereade_: hi, can I ask you a question a bout controller's and model's config? | 11:40 |
| rogpeppe | axw: ping | 12:29 |
| katco | everyone, pencils down. we can stop working on juju. emacs has solved the problem: https://github.com/cjohansson/emacs-ssh-deploy | 13:25 |
| mup | Bug # opened: 1606566, 1606568, 1606569, 1606571 | 13:25 |
| natefinch | dooferlad: looks like we screwed windows when we dropped most of our ciphersuites | 13:43 |
| dimitern | rick_h_: standup ? | 14:02 |
| katco | frobware: standup? | 14:05 |
| jam1 | mgz: I just filed https://github.com/juju/juju/pull/5870 but it doesn't look like reviewboard picked it up | 14:19 |
| dimitern | babbageclunk: wanna chat in the other HO (networking tests)? | 14:19 |
| dimitern | https://plus.google.com/hangouts/_/canonical.com/networking | 14:19 |
| babbageclunk | dimitern: oh, yes please! See you there. | 14:19 |
| frobware | balloons: sorry I missed our meeting; was debugging IPv6 and my network lost it! | 14:25 |
| frobware | katco: ^^ ditto | 14:25 |
| katco | frobware: lol that is a good excuse! :) | 14:25 |
| anastasiamac | jam1: reviewed | 14:55 |
| jam1 | thanks anastasiamac, still hasn't shown up | 14:59 |
| === jam1 is now known as jam | ||
| jam | (on Reviewboard) | 15:00 |
| anastasiamac | jam: i have seen some proposals that behave this way. I've reviewed on github, You should b good to land \o/ Do we want to backport it to 1.25 too? | 15:01 |
| jam | anastasiamac: did we backport the other 'status' changes? Like putting the 'model' 'controller' sections? | 15:02 |
| jam | if so, then yes | 15:02 |
| anastasiamac | jam: m not sure if model/controller exist on 1.25 | 15:03 |
| anastasiamac | :) | 15:03 |
| anastasiamac | jam: if address/port order change needs to be backported, it'll need to handcrafted (!) rather than cherry-picked \o/ | 15:04 |
| babbageclunk | dimitern: ipAddressDoc doesn't have mac address! Should I use linkLayerDeviceDocs to populate the interfaceInfo? | 15:33 |
| babbageclunk | dimitern: duh, reading that it's obvious - of course an ip address doesn't have a mac address. | 15:34 |
| dimitern | babbageclunk: yeah :) | 15:34 |
| mup | Bug #1606611 opened: test log spammed with TLS handshake <logging> <tech-debt> <testing> <juju-core:Triaged> <https://launchpad.net/bugs/1606611> | 15:55 |
| dimitern | any reviews welcome btw: https://github.com/juju/juju/pull/5871 | 16:00 |
| * dimitern is outta here | 16:02 | |
| mup | Bug #1606611 changed: test log spammed with TLS handshake <logging> <tech-debt> <testing> <juju-core:Triaged> <https://launchpad.net/bugs/1606611> | 16:04 |
| mup | Bug #1606611 opened: test log spammed with TLS handshake <logging> <tech-debt> <testing> <juju-core:Triaged> <https://launchpad.net/bugs/1606611> | 16:10 |
| mup | Bug #1606617 opened: juju not using image-stream-url for container images <lxd> <juju-core:Triaged> <https://launchpad.net/bugs/1606617> | 16:11 |
| mup | Bug #1606617 changed: juju not using image-stream-url for container images <lxd> <juju-core:Triaged> <https://launchpad.net/bugs/1606617> | 16:23 |
| mup | Bug #1606617 opened: juju not using image-stream-url for container images <lxd> <juju-core:Triaged> <https://launchpad.net/bugs/1606617> | 16:26 |
| frobware | rick_h_: With my ipv6 only environment now working (huzzah) my first bootstrap using the manual provider failed (boo!). Will start to accumulate my setup notes tomorrow. | 17:02 |
| === benji__ is now known as benji | ||
| mup | Bug # changed: 1440097, 1455629, 1572634, 1572637, 1573286, 1584626, 1592537, 1600539, 1602231, 1603133, 1603585, 1604006 | 17:41 |
| natefinch | dude our tests suck | 17:59 |
| natefinch | gaahhhh | 18:26 |
| mup | Bug #1606658 opened: local charm does not deploy updated version after removing application and deploying application <juju-core:New> <https://launchpad.net/bugs/1606658> | 18:44 |
| katco | looks like github is acting up | 18:50 |
| katco | annnd we're back | 18:51 |
| === natefinch is now known as natefinch-afk | ||
| mup | Bug #1606684 opened: upgrade-charm fails to upgrade local charm <juju-core:New> <https://launchpad.net/bugs/1606684> | 20:23 |
| mup | Bug #1606663 changed: cannot juju ssh to lxd machine in non-default model <juju-core:Invalid> <https://launchpad.net/bugs/1606663> | 20:50 |
| mup | Bug #1606569 changed: transient "agent is lost" <oil> <oil-2.0> <v-pil> <juju-core:Invalid> <https://launchpad.net/bugs/1606569> | 21:20 |
| mup | Bug #1606658 changed: local charm does not deploy updated version after removing application and deploying application <juju-core:New for anastasia-macmood> <https://launchpad.net/bugs/1606658> | 21:20 |
| katco | perrito666: ping | 21:41 |
| perrito666 | katco: pong | 21:41 |
| katco | perrito666: hey is that offer to get connected up to my environment's mongo still good? | 21:42 |
| perrito666 | katco: always </snape> | 21:42 |
| katco | hehe | 21:42 |
| mup | Bug #1606706 opened: MachinerSuite.TestMachinerMachineAssignedUnits fails intermittently <intermittent-failure> <test-failure> <juju-core:Triaged> <https://launchpad.net/bugs/1606706> | 21:50 |
| thumper | oh poop | 21:59 |
| thumper | I had assumed that the factory method for creating machines with volumes actually took all the params into account... but it appears not to | 21:59 |
| mup | Bug #1606709 opened: manual provider shows started machine as "pending" <manual-provider> <status> <juju-core:Triaged> <https://launchpad.net/bugs/1606709> | 21:59 |
| thumper | or at least it removes the pool name | 21:59 |
| thumper | ah... go, sometimes you suck badly | 22:14 |
| thumper | ffs | 22:14 |
| * thumper thinks | 22:14 | |
| thumper | nope, that was me | 22:24 |
| mup | Bug #1606711 opened: When add-machine exits with nonzero status, rerunning add-machine can produce a duplicate <add-machine> <ci> <intermittent-failure> <juju-core:Triaged> <https://launchpad.net/bugs/1606711> | 22:29 |
| perrito666 | bb just in time for standup | 22:34 |
| menn0 | thumper: http://reviews.vapour.ws/r/5308/ pls | 22:40 |
| thumper | menn0: shipit | 22:58 |
| menn0 | thumper: thanks | 22:58 |
| thumper | perrito666: ping | 23:16 |
| thumper | volume export tests passing, now to work in imports after lunch | 23:41 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!