[00:42] <davecheney> here is a small review for someone, http://reviews.vapour.ws/r/4883/
[00:52] <thumper> damn it, no closer to starting this email
[00:52]  * thumper out to walk the dog while the sun is shining
[01:41]  * redir eod
[01:42] <mup> Bug #1585005 opened: list-* commands should be aliases for what they're listing <usability> <juju-core:Triaged by macgreagoir> <https://launchpad.net/bugs/1585005>
[01:51] <mup> Bug #1585005 changed: list-* commands should be aliases for what they're listing <usability> <juju-core:Triaged by macgreagoir> <https://launchpad.net/bugs/1585005>
[02:00] <mup> Bug #1585005 opened: list-* commands should be aliases for what they're listing <usability> <juju-core:Triaged by macgreagoir> <https://launchpad.net/bugs/1585005>
[02:42] <mup> Bug #1585015 opened: worker/terminationworker: test timeout during CI <juju-core:New> <https://launchpad.net/bugs/1585015>
[02:50] <cherylj> thumper, wallyworld - can one of you comment on the proposal in bug 1568944?  (last comment)
[02:50] <mup> Bug #1568944: Failure when deploying on lxd models and model name contains space characters <juju-core:Triaged by reedobrien> <https://launchpad.net/bugs/1568944>
[02:50] <wallyworld> ok
[02:51] <wallyworld> cherylj: want me to create that 2.0 kanban?
[02:51] <cherylj> gracias
[02:51] <cherylj> wallyworld: anastasiamac was kind enough to create one for me
[02:51] <cherylj> I'm working on populating it now
[02:51] <wallyworld> cherylj: ok, great, i have stuff also
[02:52] <cherylj> wallyworld: do you know how to add people into the leankit site?  I noticed that christian and mick need accounts
[02:52] <cherylj> wallyworld: https://canonical.leankit.com/boards/view/122133644#workflow-view
[02:52] <wallyworld> cherylj: i think so, i'll see if i have perms
[02:52] <cherylj> thanks!
[02:52] <thumper> if wallyworld doesn't, I do
[02:57] <wallyworld> cherylj: christian is already a user on the site
[02:58] <cherylj> wallyworld: must've missed him :/
[02:58]  * cherylj checks again
[02:58] <wallyworld> adding mick now
[02:59] <wallyworld> cherylj: mick added
[02:59] <cherylj> yarp, there he is
[02:59] <cherylj> thanks, wallyworld!
[03:00] <anastasiamac> cherylj: i have also added christian to our "other" board :-P
[03:00] <cherylj> thanks, anastasiamac  :)
[03:11]  * thumper heads away for a bit to write emails
[04:31] <menn0> windows batch is a nightmare... bad decisions all the way down
[04:34] <natefinch> menn0: why are you messing with batch files?
[04:35] <menn0> natefinch: we have tests which patch out executables with fake shell scripts/batch files (depending on platform). i'm fixing one. just about done but it's been painful.
[04:35] <natefinch> menn0: omg, don't use those
[04:36] <natefinch> menn0: https://godoc.org/github.com/juju/testing#PatchExecHelper
[04:36] <natefinch> menn0: lightyears bettter.
[04:36] <natefinch> menn0: 100% go..
[04:37] <natefinch> menn0: I sent an email about it a while back.  might require a tiny refactor to the production code, but the testing is SO much better
[04:37] <natefinch> menn0: just forwarded the email to you
[04:40] <menn0> natefinch: that looks pretty great but I'm not sure I can use it in this case
[04:40] <menn0> natefinch: I need to know the contents of a temporary file which gets passed to the executable
[04:40] <menn0> and gets deleted
[04:41] <menn0> right now the patched in replacement executable print out the contents of the file they're given
[04:43] <natefinch> menn0: you could do the same thing that helper does and just code it by hand
[04:43] <natefinch> menn0: in other words, write your own special test function that does whatever you're trying to do in batch
[04:43] <menn0> natefinch: hmm ok. i'll take a look
[04:43] <menn0> ironically, I just got the change working
[04:43] <natefinch> heh
[04:45] <natefinch> menn0: I should add another helper that does this rigamarole but takes the name of a custom test function, so you only need to write the test function, and not the exec hacking code.
[05:16] <bradm> I just hit bug 1537585 if anyone knows how to fix it
[05:16] <mup> Bug #1537585: machine agent failed to register IP addresses, borks agent <2.0-count> <blocker> <landscape> <network> <juju-core:Triaged by natefinch> <juju-core 1.25:Triaged by natefinch> <https://launchpad.net/bugs/1537585>
[05:16] <bradm> seems fairly intermittant, I've deployed a number of times without seeing it
[06:13] <mup> Bug #1585059 opened: juju status tabular format doesn't show controller member status <canonical-bootstack> <juju-core:Confirmed> <https://launchpad.net/bugs/1585059>
[06:16] <mup> Bug #1585059 changed: juju status tabular format doesn't show controller member status <canonical-bootstack> <juju-core:Confirmed> <https://launchpad.net/bugs/1585059>
[06:28] <mup> Bug #1585059 opened: juju status tabular format doesn't show controller member status <canonical-bootstack> <juju-core:Confirmed> <https://launchpad.net/bugs/1585059>
[06:52] <mup> Bug #1583934 changed: revision specified in the charm, how? <juju-core:Invalid> <https://launchpad.net/bugs/1583934>
[08:04] <babbageclunk> voidspace: around?
[08:05] <babbageclunk> fwereade: ping?
[08:06] <babbageclunk> menn0: hey
[08:06]  * babbageclunk tumbleweeds.
[08:27] <voidspace> babbageclunk: yep, sorry
[08:41] <thumper> voidspace, babbageclunk: morning
[08:42] <thumper> frobware: alive?
[08:43] <thumper> dooferlad: feeling better?
[08:46] <voidspace> thumper: morning
[08:50] <frobware> thumper: yep. thankfully. :-D
[08:50] <babbageclunk> voidspace: no worries, worked it out.
[08:51] <voidspace> babbageclunk: I figure that left to your own devices you'll usually figure stuff out
[08:51] <babbageclunk> morning thumper
[08:51] <voidspace> babbageclunk: so I never worry about replying to you
[08:51] <voidspace> babbageclunk: you're probably better off if I don't...
[08:51] <voidspace> ;-)
[08:51] <babbageclunk> voidspace: generally with a bit more swearing
[08:51] <voidspace> babbageclunk: :-)
[13:11] <mup> Bug #1568079 changed: github.com/juju/juju/apiserver/client unit tests fail if xenial is the LTS <xenial> <juju-core:Fix Released by reedobrien> <juju-core 1.25:Fix Released by reedobrien> <https://launchpad.net/bugs/1568079>
[13:20] <mup> Bug #1568079 opened: github.com/juju/juju/apiserver/client unit tests fail if xenial is the LTS <xenial> <juju-core:Fix Released by reedobrien> <juju-core 1.25:Fix Released by reedobrien> <https://launchpad.net/bugs/1568079>
[13:23] <mup> Bug #1568079 changed: github.com/juju/juju/apiserver/client unit tests fail if xenial is the LTS <xenial> <juju-core:Fix Released by reedobrien> <juju-core 1.25:Fix Released by reedobrien> <https://launchpad.net/bugs/1568079>
[14:33] <frobware> alexisb: ping - audio/HO trouble... brt
[14:34] <alexisb> frobware, np
[14:35] <alexisb> frobware, I am going to take the opportunity to get coffee, brb
[14:37] <alexisb> frobware, heh
[14:38] <alexisb> frobware, did you want to reschedule?
[14:38] <alexisb> while you work out audio?
[14:38] <frobware> frobware: one min
[14:38] <alexisb> k
[15:12] <natefinch> dooferlad: you around?
[15:19] <katco> fwereade: ping
[15:19] <fwereade> katco, pong
[15:19] <katco> fwereade: hey, re. http://reviews.vapour.ws/r/4882/
[15:20] <katco> fwereade: eric is trying to mechanically move this code over to state to enable cmr for tim
[15:20] <katco> fwereade: can we please land as is and address your comments as bugs. i think eric has opened
[15:21] <fwereade> katco, what are the odds that my concerns will actually be addressed this year?
[15:21] <katco> fwereade: lol, i don't know the answer to that. but the priority comes from the top-down
[15:22] <katco> fwereade: we don't have resources to address these comments right now
[15:22] <katco> fwereade: we're just trying to move the code because tim wants it over in state
[15:25] <fwereade> katco, because it needs to be part of a solid migration process... which, there is every likelihood, will be breakable by components that skip referential integrity, because AIUI the migration actually validates the model it sends over
[15:25] <katco> cherylj: alexisb: i need to pull nate off of bug 1537585 so he can ramp up on what ericsnow is working on b/c ericsnow will be out for pycon
[15:25] <mup> Bug #1537585: machine agent failed to register IP addresses, borks agent <2.0-count> <blocker> <landscape> <network> <juju-core:Triaged by natefinch> <juju-core 1.25:Triaged by natefinch> <https://launchpad.net/bugs/1537585>
[15:25] <alexisb> katco, ok
[15:27] <fwereade> katco, I very much appreciate the structural work ericsnow has done, but it *actually does* need both referential integrity and detailed race testing
[15:27] <mup> Bug #1585059 changed: juju status tabular format doesn't show controller member status <canonical-bootstack> <juju-core:Confirmed> <https://launchpad.net/bugs/1585059>
[15:27] <katco> fwereade: i understand, but we need to address those as bugs, separately. eric needs to land this so he can help hand off to nate so he can go to pycon. this is a PM thing, not a code quality thing
[15:29] <katco> ericsnow: can you demonstrate to fwereade that you have bugs open for the issues he's bringing up?
[15:29] <fwereade> katco, this is not code quality, this is level 0 "does it actually work"
[15:30] <fwereade> katco, I cannot convince myself that it works correctly on the basis of the code, and AFAICS there is no test that actually exercises the functionality I am worried about
[15:31] <katco> fwereade: OK, can we please address those concerns separate from moving the code? it will unblock tim, eric, and nate
[15:34] <fwereade> katco, I have yet to be convinced that they will be addressed
[15:35] <fwereade> katco, IIRC, I expressed these concerns a *very long time ago* when this code was first written, and reminded you on several occasions, and they have remained untouched
[15:37] <fwereade> katco, and if it goes into state like this people will use it as a model and spread the badness
[15:38] <katco> fwereade: ok, that is a valid concern. we'll have to drop this patch for the moment then. ericsnow needs to be working with natefinch before he leaves
[15:38] <fwereade> katco, ack, thank you
[15:38] <katco> fwereade: i'll send an email to tim and cc you explaining the reasoning
[16:54] <mup> Bug #1585289 opened: Juju2: websockets API should alert, error or warn about ignored parameters <landscape> <usability> <juju-core:New> <https://launchpad.net/bugs/1585289>
[17:15] <mup> Bug #1585300 opened: environSuite invalid character \"\\\\\" in host name <ci> <regression> <test-failure> <unit-tests> <windows> <juju-core:Incomplete> <juju-core 1.25:Triaged> <https://launchpad.net/bugs/1585300>
[17:31] <redir> fwereade: yst?
[17:33] <perrito666> hello, sorry all I got a small relapse last night, I am well and alive now, anyone needs a review?
[17:39] <katco> perrito666: welcome back to the land of the living
[17:39] <katco> perrito666: you're working on acl stuff, correct?
[17:40] <perrito666> katco: I am
[17:41] <perrito666> I have been in the world of the living for a few hours but was working without looking at the chat
[17:45] <katco> perrito666: cool.
[17:54] <natefinch> ericsnow: do you want to pair?
[17:54] <ericsnow> natefinch: sure
[17:54] <ericsnow> natefinch: give me a couple minutes
[17:55] <natefinch> ericsnow: np
[18:34] <jcastro> katco: heya
[18:34] <katco> jcastro: o/
[18:34] <jcastro> I noticed the last few releases the controller is now much more robust inbetween reboots, etc.
[18:34] <jcastro> it hasn't failed on me at all
[18:34] <katco> jcastro: i'm sorry, we'll try to introduce more bugs.
[18:34] <jcastro> I just wanted to tempt fate
[18:35] <alexisb> :)
[18:36] <katco> jcastro: seriously, that's good to hear :)
[18:52]  * perrito666 ads a call to ohYouHadToSayIt(jcastro)
[18:52] <natefinch> lol
[19:01] <ericsnow> natefinch: sorry about that
[19:01] <ericsnow> natefinch: ready?
[19:02] <natefinch> ericsnow: ye[
[19:02] <natefinch> yep
[19:10] <perrito666> bbl
[19:28] <redir> is the mergebot alive?
[19:29] <redir> yes it is
[20:51] <thumper> morning
[20:53] <lazyPower> o/ thumper
[20:54] <mup> Bug #1585359 opened: juju debug-log prints some log messages, but quickly stops streaming <juju-core:New> <https://launchpad.net/bugs/1585359>
[20:54] <mup> Bug #1585361 opened: megawatcher delta is missing data (service & relation) <juju-core:New> <https://launchpad.net/bugs/1585361>
[21:02] <thumper> hmm..
[21:03] <alexisb> thumper, hmm... ??
[21:03] <thumper> I was 2 minutes late for the standup and no one was there...
[21:03] <thumper> then michael turned up
[22:01] <wallyworld> alexisb: sts call? you should attend this one
[22:01] <alexisb> wallyworld, be there soon
[22:05] <sinzui> thumper: bug 1585388 is the new form of broken ssh to a container. the sxample shows --proxy was used.
[22:05] <mup> Bug #1585388: Container networking cannot ssh after machine is ready <ci> <lxc> <lxd> <network> <regression> <juju-core:Triaged> <https://launchpad.net/bugs/1585388>
[22:07] <mup> Bug #1585388 opened: Container networking cannot ssh after machine is ready <ci> <lxc> <lxd> <network> <regression> <juju-core:Triaged> <https://launchpad.net/bugs/1585388>
[22:16] <mup> Bug #1585388 changed: Container networking cannot ssh after machine is ready <ci> <lxc> <lxd> <network> <regression> <juju-core:Triaged> <https://launchpad.net/bugs/1585388>
[22:19] <perrito666> mmm, we error out mentioning a model by its uuid, that is a ux faux pas
[22:28] <mup> Bug #1585388 opened: Container networking cannot ssh after machine is ready <ci> <lxc> <lxd> <network> <regression> <juju-core:Triaged> <https://launchpad.net/bugs/1585388>
[22:39] <alexisb> ug sorry wallyworld, cherylj I missed that call
[22:40] <wallyworld> alexisb: am sending an email
[22:40] <alexisb> thanks
[23:16] <wallyworld> anastasiamac: perrito666 axw be right there
[23:56] <natefinch> davecheney: online but encumbered by a sleeping toddler, if you want to talk fslock