| davecheney | here is a small review for someone, http://reviews.vapour.ws/r/4883/ | 00:42 |
|---|---|---|
| thumper | damn it, no closer to starting this email | 00:52 |
| * thumper out to walk the dog while the sun is shining | 00:52 | |
| === thumper is now known as thumper-dogwalk | ||
| * redir eod | 01:41 | |
| === redir is now known as redir_afk | ||
| 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:42 |
| === thumper-dogwalk is now known as thumper | ||
| 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> | 01:51 |
| 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:00 |
| mup | Bug #1585015 opened: worker/terminationworker: test timeout during CI <juju-core:New> <https://launchpad.net/bugs/1585015> | 02:42 |
| 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:50 |
| 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:51 |
| 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:52 |
| wallyworld | cherylj: christian is already a user on the site | 02:57 |
| cherylj | wallyworld: must've missed him :/ | 02:58 |
| * cherylj checks again | 02:58 | |
| wallyworld | adding mick now | 02:58 |
| wallyworld | cherylj: mick added | 02:59 |
| cherylj | yarp, there he is | 02:59 |
| cherylj | thanks, wallyworld! | 02:59 |
| anastasiamac | cherylj: i have also added christian to our "other" board :-P | 03:00 |
| cherylj | thanks, anastasiamac :) | 03:00 |
| * thumper heads away for a bit to write emails | 03:11 | |
| menn0 | windows batch is a nightmare... bad decisions all the way down | 04:31 |
| natefinch | menn0: why are you messing with batch files? | 04:34 |
| 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:35 |
| natefinch | menn0: https://godoc.org/github.com/juju/testing#PatchExecHelper | 04:36 |
| natefinch | menn0: lightyears bettter. | 04:36 |
| natefinch | menn0: 100% go.. | 04:36 |
| 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:37 |
| 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:40 |
| menn0 | right now the patched in replacement executable print out the contents of the file they're given | 04:41 |
| 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:43 |
| 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. | 04:45 |
| 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 | 05:16 |
| 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:13 |
| 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:16 |
| 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:28 |
| === benonsoftware is now known as Guest30663 | ||
| === \b is now known as benonsoftware | ||
| mup | Bug #1583934 changed: revision specified in the charm, how? <juju-core:Invalid> <https://launchpad.net/bugs/1583934> | 06:52 |
| babbageclunk | voidspace: around? | 08:04 |
| babbageclunk | fwereade: ping? | 08:05 |
| babbageclunk | menn0: hey | 08:06 |
| * babbageclunk tumbleweeds. | 08:06 | |
| voidspace | babbageclunk: yep, sorry | 08:27 |
| thumper | voidspace, babbageclunk: morning | 08:41 |
| thumper | frobware: alive? | 08:42 |
| thumper | dooferlad: feeling better? | 08:43 |
| voidspace | thumper: morning | 08:46 |
| frobware | thumper: yep. thankfully. :-D | 08:50 |
| babbageclunk | voidspace: no worries, worked it out. | 08:50 |
| 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: :-) | 08:51 |
| 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:11 |
| 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:20 |
| 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:23 |
| frobware | alexisb: ping - audio/HO trouble... brt | 14:33 |
| alexisb | frobware, np | 14:34 |
| alexisb | frobware, I am going to take the opportunity to get coffee, brb | 14:35 |
| alexisb | frobware, heh | 14:37 |
| 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 | 14:38 |
| natefinch | dooferlad: you around? | 15:12 |
| === Oer is now known as OerHeks | ||
| katco | fwereade: ping | 15:19 |
| fwereade | katco, pong | 15:19 |
| katco | fwereade: hey, re. http://reviews.vapour.ws/r/4882/ | 15:19 |
| 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:20 |
| 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:21 |
| 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:22 |
| 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:25 |
| 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:27 |
| 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:29 |
| 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:30 |
| katco | fwereade: OK, can we please address those concerns separate from moving the code? it will unblock tim, eric, and nate | 15:31 |
| fwereade | katco, I have yet to be convinced that they will be addressed | 15:34 |
| 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:35 |
| fwereade | katco, and if it goes into state like this people will use it as a model and spread the badness | 15:37 |
| 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 | 15:38 |
| === redir_afk is now known as redir | ||
| 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> | 16:54 |
| === bruno is now known as Guest6887 | ||
| 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:15 |
| redir | fwereade: yst? | 17:31 |
| perrito666 | hello, sorry all I got a small relapse last night, I am well and alive now, anyone needs a review? | 17:33 |
| katco | perrito666: welcome back to the land of the living | 17:39 |
| katco | perrito666: you're working on acl stuff, correct? | 17:39 |
| perrito666 | katco: I am | 17:40 |
| perrito666 | I have been in the world of the living for a few hours but was working without looking at the chat | 17:41 |
| katco | perrito666: cool. | 17:45 |
| natefinch | ericsnow: do you want to pair? | 17:54 |
| ericsnow | natefinch: sure | 17:54 |
| ericsnow | natefinch: give me a couple minutes | 17:54 |
| natefinch | ericsnow: np | 17:55 |
| 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:34 |
| alexisb | :) | 18:35 |
| katco | jcastro: seriously, that's good to hear :) | 18:36 |
| * perrito666 ads a call to ohYouHadToSayIt(jcastro) | 18:52 | |
| natefinch | lol | 18:52 |
| ericsnow | natefinch: sorry about that | 19:01 |
| ericsnow | natefinch: ready? | 19:01 |
| natefinch | ericsnow: ye[ | 19:02 |
| natefinch | yep | 19:02 |
| perrito666 | bbl | 19:10 |
| redir | is the mergebot alive? | 19:28 |
| redir | yes it is | 19:29 |
| thumper | morning | 20:51 |
| lazyPower | o/ thumper | 20:53 |
| 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> | 20:54 |
| thumper | hmm.. | 21:02 |
| 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 | 21:03 |
| wallyworld | alexisb: sts call? you should attend this one | 22:01 |
| alexisb | wallyworld, be there soon | 22:01 |
| 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:05 |
| 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:07 |
| 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:16 |
| perrito666 | mmm, we error out mentioning a model by its uuid, that is a ux faux pas | 22:19 |
| 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:28 |
| alexisb | ug sorry wallyworld, cherylj I missed that call | 22:39 |
| wallyworld | alexisb: am sending an email | 22:40 |
| alexisb | thanks | 22:40 |
| wallyworld | anastasiamac: perrito666 axw be right there | 23:16 |
| natefinch | davecheney: online but encumbered by a sleeping toddler, if you want to talk fslock | 23:56 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!