| thumper | I added a constraint that a new environment should have the same provider type as the state server environment | 00:00 |
|---|---|---|
| thumper | well, default | 00:00 |
| thumper | not constraint | 00:00 |
| jw4 | thumper: I've not been able to reproduce the bug 1412292 - are those errors happening consistently for you? | 01:37 |
| mup | Bug #1412292: Intermittent test failure in ActionSuite.TestActionsWatcherEmitsInitialChanges <action> <intermittent-failure> <test-failure> <juju-core:Triaged> <https://launchpad.net/bugs/1412292> | 01:37 |
| thumper | jw4: about one time in three | 01:37 |
| jw4 | thumper: hmm; do you have a fast machine, or a slow one - or is it PPC - or anything like that? | 01:38 |
| thumper | two year old fast i7, lots of ram and fast ssd | 01:38 |
| jw4 | thumper: kk - I'll try picking apart the logic to see what I can find | 01:38 |
| thumper | ok | 01:38 |
| menn0 | thumper: Fix it, then Ship it | 01:59 |
| jw4 | thumper: two quick questions - 1) I assume the defect report has the full output of the test run. Is that true? 2) what version of go are you using? | 01:59 |
| jw4 | thumper: fwiw I suspect the problem is the watcher isn't getting updated in time that the two actions were removed | 02:00 |
| thumper | menn0: ta | 02:42 |
| thumper | jw4: using what ever version is in trusty | 02:42 |
| thumper | jw4: I'm guessing it is a race condition | 02:42 |
| thumper | jw4: but I've not looked at the code | 02:43 |
| thumper | man... | 03:20 |
| thumper | getting endorsements on linked in by folk I don't remember linking to | 03:20 |
| thumper | oh... he is a juju gui peep | 03:21 |
| axw | wallyworld_: about to send an email to fwereade for some advice on the state model, I've CC'd you in case you have ideas on it | 03:25 |
| wallyworld_ | ok | 03:25 |
| axw | the relations to units and services are feeling a bit messy | 03:26 |
| * thumper finishes the apiserver httphandler before enfixorating the branch menno reviewed | 03:26 | |
| thumper | menn0: I have another branch that fixes the rest of the apiserver | 03:50 |
| thumper | menn0: I just need to write tests for each of the api end points to exercise the additional environment | 03:50 |
| menn0 | thumper: sweet | 03:52 |
| menn0 | thumper: i'm deep into integrating envWorkerManager into the machine agent | 03:52 |
| thumper | awesome | 03:52 |
| thumper | I feel the pieces are slowly coming together | 03:52 |
| menn0 | thumper: lots of awkward details to figure out | 03:52 |
| thumper | menn0: yep, I know | 03:53 |
| thumper | menn0: so many fiddly bits | 03:53 |
| menn0 | thumper: e.g the singular runner | 03:53 |
| thumper | boo... hiss... | 04:15 |
| thumper | damn | 05:08 |
| thumper | I bet that two line tweak might break something | 05:08 |
| thumper | I'm done | 05:14 |
| thumper | these tests are taking ages to write | 05:14 |
| thumper | laters | 05:14 |
| dimitern | morning all | 07:05 |
| anastasiamac | dimitern: morning :) 5pm here tho | 07:13 |
| dimitern | anastasiamac, afternoon then :) | 07:13 |
| anastasiamac | dimitern: thnx for consideration :) | 07:14 |
| anastasiamac | dimitern: one thing i miss about Nothern hemisphere - r long dusks in summer | 07:14 |
| dimitern | anastasiamac, oh, they're not so long there? | 07:14 |
| dimitern | anastasiamac, I'd have a hard time adjusting to summer in january :) | 07:15 |
| anastasiamac | dimitern: with +40 and an awful humidity, u will not mistake this for winter! | 07:16 |
| anastasiamac | dimitern: there is no dusk :( | 07:16 |
| dimitern | anastasiamac, *lol* | 07:16 |
| anastasiamac | dimitern: one minute it's day, the next night | 07:16 |
| anastasiamac | dimitern: there r millions of other advantages tho :) | 07:16 |
| dimitern | anastasiamac, that is a bit weird | 07:17 |
| anastasiamac | dimitern: k... to be fair it takes about 20 minutes for day to turn to night :D | 07:17 |
| dimitern | anastasiamac, I bet there are, although with all the creepy crawlers in AU.. | 07:17 |
| anastasiamac | dimitern: and don't forget flying cockroaches :P | 07:17 |
| dimitern | anastasiamac, oh we have those as well - nasty creatures | 07:18 |
| anastasiamac | dimitern: certainly very exotic | 07:19 |
| wallyworld_ | axw: i've made some changes; i was taking some shortcuts previously to try and get it done more quickly; tests should have better coverage now | 07:23 |
| dimitern | wallyworld_, axw, hey guys - can one of you review this goamz PR please? https://github.com/go-amz/amz/pull/20 | 07:24 |
| axw | wallyworld_: ok cool | 07:24 |
| axw | wallyworld_: looking | 07:24 |
| axw | dimitern: will take a look in a bit | 07:25 |
| wallyworld_ | dimitern: i'm about to get ready for soccer, but can look when i get back if not done | 07:25 |
| dimitern | axw, thank you | 07:25 |
| dimitern | wallyworld_, cheers | 07:25 |
| dimitern | wallyworld_, isn't it too hot for soccer? :) | 07:25 |
| wallyworld_ | dimitern: inly if you're soft :-) | 07:26 |
| dimitern | wallyworld_, :D | 07:26 |
| wallyworld_ | axw: i gotta dash, but i might next add some basic framework for registering storage provider types eg loop with a skeleton implementation. then the blanks can be filled in while work is done on making the loop storage available to env providers, and fallback logic implemented etc | 07:33 |
| wallyworld_ | axw: tnks for review, i just used state in tests to get end-end tested for now, can mock out later | 07:33 |
| axw | wallyworld_: that SGTM | 07:33 |
| axw | thanks | 07:33 |
| axw | dimitern: reviewed | 07:57 |
| dimitern | axw, ta! | 07:57 |
| dimitern | voidspace, standup? | 10:03 |
| voidspace | dimitern: omw | 10:03 |
| axw | dimitern: when you're free, can you please review https://github.com/juju/names/pull/37 ? | 11:13 |
| dimitern | axw, sure thing | 11:14 |
| dimitern | axw, lgtm | 11:20 |
| axw | dimitern: do you mean vary the storage name? i.e. something other than "store"? | 11:20 |
| axw | thanks | 11:21 |
| dimitern | axw, yeah | 11:21 |
| axw | dimitern: okey dokey, will do | 11:21 |
| dimitern | axw, I only later read the description of what it is :) | 11:21 |
| perrito666 | morning | 11:50 |
| anastasiamac | perrito666: morning... | 11:51 |
| axw | dimitern: another small one http://reviews.vapour.ws/r/766/ | 12:00 |
| dimitern | axw, ship it! :) | 12:02 |
| axw | dimitern: thanks | 12:02 |
| perrito666 | bogdanteleaga: there is something wrong with http://reviews.vapour.ws/r/632/ | 12:21 |
| perrito666 | it lacks code | 12:21 |
| perrito666 | jam2: inet problems? | 12:21 |
| jam2 | perrito666: its been happening for a couple of days now. They "upgraded" my local router in preparation for changing something about the networking to the building, and now every 15-30min I get a SSL session negotiation failure. | 12:22 |
| bogdanteleaga | perrito666: hmm, it hasn't been created automatically so I tried to do it manually, apparently it didn't work | 12:23 |
| bogdanteleaga | perrito666: the code is on github however can I update it from the web interface? | 12:23 |
| perrito666 | bogdanteleaga: if you did the post manually probably it worked but you need to go to reviewboard and publish it | 12:25 |
| dimitern | voidspace, maas meeting? | 14:04 |
| * perrito666 actually had to prepare a brew to review fwereade's branches | 14:07 | |
| fwereade | perrito666, oh dear, is it that bad? | 14:07 |
| perrito666 | nope, I just checked the lenght and decided that i better prepare my brew before and not after | 14:08 |
| fwereade | perrito666, good thinking :) | 14:08 |
| fwereade | perrito666, if it helps, I think the test changes outweigh the code in both of them | 14:08 |
| perrito666 | abentley: this was the failing test of the last cursed email I got http://juju-ci.vapour.ws:8080/job/win-client-deploy/1319/console that is the test right? | 14:12 |
| abentley | perrito666: That looks like a problem with the machine running the test. | 14:13 |
| * perrito666 stops peeking at the local newspaper before getting an ulcera | 14:14 | |
| natefinch | Morning everybody | 14:26 |
| dimitern | hey natefinch! | 14:26 |
| natefinch | dimitern: howdy :) | 14:27 |
| perrito666 | hey natefinch back? | 14:27 |
| natefinch | perrito666: back, yes | 14:27 |
| natefinch | 058524 | 14:27 |
| perrito666 | natefinch: we wherent expecting you for a couple of weeks | 14:27 |
| perrito666 | natefinch: tx nice pass | 14:28 |
| natefinch | lol | 14:28 |
| natefinch | just my google authenticator number | 14:28 |
| natefinch | yeah, this is about when I expected to be back.... 2 weeks after the company shutdown | 14:28 |
| voidspace | dimitern: damn! | 14:39 |
| voidspace | dimitern: sorry | 14:39 |
| dimitern | voidspace, :) np | 14:40 |
| voidspace | dimitern: just forgot :-( | 14:40 |
| voidspace | dimitern: I need to tattoo it to my hand or something | 14:40 |
| dimitern | voidspace, *lol* | 14:40 |
| dimitern | voidspace, nothing major - non-installable nodes can be tested very soon - check the minutes doc if you like | 14:41 |
| sinzui | natefinch, perrito666 dimitern : I don't see any 1.21-beta5 specific issues in play in the this bug. I don't think it should block the release of 1.21-rc1 https://bugs.launchpad.net/juju-core/+bug/1410876 | 14:41 |
| mup | Bug #1410876: Error executing lxc-clone: lxc_container: utils.c: mkdir_p 220 Not a directory - Could not destroy snapshot %s - failed to allocate a pty; Insufficent privileges to control juju-trusty-lxc-template <lxc> <oil> <trusty> <juju-core:Incomplete> <juju-core 1.21:Incomplete> <juju-core | 14:41 |
| mup | 1.22:Incomplete> <https://launchpad.net/bugs/1410876> | 14:41 |
| voidspace | dimitern: I will do, cool | 14:41 |
| voidspace | dimitern: sounds good | 14:41 |
| dimitern | sinzui, looking | 14:42 |
| voidspace | dimitern: we need an api to assign resources (like ip addresses) to them for non-installable nodes to be directly useful to us | 14:42 |
| perrito666 | sinzui: I agree thumper and Í documented our findings there and it seems it is outside juju | 14:42 |
| sinzui | thank you perrito666 . I will try to remove the bug from rc1 | 14:43 |
| dimitern | sinzui, yep, I was just checking the code - it seems 1.21-rc1 shouldn't be affected | 14:52 |
| sinzui | fab | 14:53 |
| wwitzel3 | perrito666: ping | 15:06 |
| sinzui | natefinch, dimitern:do you have a moment to review this branch http://reviews.vapour.ws/r/768/ | 15:22 |
| natefinch | sinzui: ship it! | 15:26 |
| sinzui | thank you natefinch | 15:26 |
| jw4 | OCR PTAL http://reviews.vapour.ws/r/769/ - fix for bug 1412292 | 15:44 |
| mup | Bug #1412292: Intermittent test failure in ActionSuite.TestActionsWatcherEmitsInitialChanges <action> <intermittent-failure> <test-failure> <juju-core:Triaged by johnweldon4> <https://launchpad.net/bugs/1412292> | 15:44 |
| dimitern | jw4, looking | 15:44 |
| jw4 | dimitern: thanks | 15:45 |
| dimitern | jw4, if the issue is with the watcher, why not fix it there? | 15:46 |
| jw4 | dimitern: 1) I'm not sure it's a problem with the watcher 2) making a change there would have widespread effect | 15:50 |
| jw4 | dimitern: the issue in the watcher is that it waits up to 10 ms every time after a change to see if there are more changes before returning | 15:50 |
| jw4 | dimitern: that's probably not bad in real life, but in the tests we were expecting it to be a bit more deterministic | 15:50 |
| dimitern | jw4, right, there might be an issue lurking in there, but the fix seems fine - almost ready with the review | 15:51 |
| jw4 | dimitern: tx | 15:51 |
| dimitern | jw4, done | 15:55 |
| jw4 | dimitern: ta | 15:55 |
| === kadams54 is now known as kadams54-away | ||
| === kadams54-away is now known as kadams54 | ||
| voidspace | just spent about an hour trying to work out if I can get the MACAddress out of a network definition in lshw | 18:29 |
| voidspace | only to realise a few minutes ago that it's there, it's just called "serial" instead of mac address | 18:29 |
| voidspace | d'oh... | 18:29 |
| voidspace | at least I have it | 18:30 |
| hazmat | is usermanagement so to be functional | 18:36 |
| hazmat | nevermind | 18:38 |
| === kadams54 is now known as kadams54-away | ||
| === kadams54-away is now known as kadams54 | ||
| voidspace | right | 19:10 |
| voidspace | g'night all | 19:10 |
| voidspace | EOD | 19:10 |
| === kadams54 is now known as kadams54-away | ||
| katco | when we create our lxc templates for cloning, we start them and then wait for them to stop. what causes them to eventually stop so cloning can continue? | 19:44 |
| === kadams54-away is now known as kadams54 | ||
| natefinch | katco: not sure | 19:51 |
| katco | for reference: https://github.com/juju/juju/blob/1.20/container/lxc/clonetemplate.go#L162 | 19:52 |
| katco | trying to dx https://bugs.launchpad.net/juju-core/+bug/1348386 | 19:52 |
| mup | Bug #1348386: lxc template fails to stop <clone> <lxc> <oil> <juju-core:Fix Released by cox-katherine-e> <juju-core 1.20:Fix Released by cox-katherine-e> <https://launchpad.net/bugs/1348386> | 19:52 |
| hazmat | are tags supposed to leak through the api? | 19:56 |
| katco | hazmat: they are not | 19:57 |
| hazmat | cause it seems like we started off without doing that, but now their leaking all over the place | 19:57 |
| hazmat | actions, usermanager, etc | 19:57 |
| hazmat | ssh key manager.. basically anything using Entities is expecting tags. | 19:58 |
| hazmat | param.Entities that is | 19:58 |
| katco | hazmat: in the api/* packages? or apiserver/*? | 19:58 |
| hazmat | katco, in apiserver | 19:59 |
| katco | hazmat: ah. that is designed to take tags | 19:59 |
| katco | hazmat: tags are OtW format, api/* is what is client-facing | 19:59 |
| hazmat | well that is the exposed api | 19:59 |
| katco | hazmat: as i understand it anyway | 19:59 |
| hazmat | api/* is an implementation of a client.. server is the api so we have a rather unfortunate situation on server that things are a mix of tags and not tags. | 20:00 |
| jw4 | hazmat, katco as I recall from discussion w/fwereade the apiserver takes string representations of tags | 20:00 |
| katco | hazmat: this could be wrong, but how it was explained to me is that tags are OtW format and so as jw4 said we pass the string-representation of tags into apiserver/* | 20:01 |
| jw4 | hazmat: there was some confusion about tags vs. string representation of tags - are you referring to the string representation or the type names.Tag ? | 20:01 |
| katco | hazmat: i.e.: tags are how clients communicate with the RPC server | 20:01 |
| hazmat | none of the core api does (deploy/add-machine/remove-machine/add-units/remove-units/setcharm/etc), but all the things that have been added recently are using tags heavily | 20:02 |
| hazmat | so we now have proper mismash of different ids in the api | 20:02 |
| jw4 | hazmat: as I recall it that is because of a shift in focus too - the intent being that all ids are internal only and string representation of tags used ubiquitously | 20:03 |
| jw4 | hazmat: we just haven't migrated everything yet | 20:03 |
| hazmat | or maybe its just client facade shouldn't use tags | 20:03 |
| hazmat | and other facades are all tag centric | 20:03 |
| jw4 | hazmat: but I could be wrong - hoping fwereade notices and weighs in | 20:04 |
| fwereade | hazmat, all facades should be tag centric -- but the client ones were mostly written before tags were a thing, and many have not been (needed to be) updated yet | 20:04 |
| fwereade | hazmat, tags were introduced as a way to encode id and type for things like the annotations api | 20:04 |
| hazmat | tags have been around a while, we just never exposed them through to client facade consumers | 20:04 |
| hazmat | fwereade, which means they were in the first release of the api really.. ie ever since gui supported juju-core | 20:05 |
| fwereade | hazmat, so we didn't need to ensure that every entity namespace was disjoint from every other | 20:05 |
| fwereade | hazmat, near enough, yes, it was specifically for annotations that we added them iirc | 20:05 |
| hazmat | okay.. i guess i'm going to just keep with other client impl not forcing users to care about tags, just automatically append per appropriate call | 20:07 |
| hazmat | the only place there was ever a need for tag disambiguation was the annotations (and perhaps not there), i'd rather not have api consumers actually have to concern themselves with it unless by choice. | 20:08 |
| === kadams54 is now known as kadams54-away | ||
| natefinch | morning thumper | 20:52 |
| natefinch | wwitzel3, ericsnow: can you guys post a review for the GCE stuff so I can get started on it? | 21:01 |
| ericsnow | natefinch: almost there | 21:02 |
| natefinch | ericsnow: cool... feel free to make a PR with a big WIP on the front of it if that's helpful. | 21:02 |
| aznashwan | ericsnow: have a moment? | 21:11 |
| ericsnow | aznashwan: almost :) | 21:11 |
| ericsnow | aznashwan: gimme a minute | 21:11 |
| * thumper headdesks | 21:12 | |
| thumper | damn... damn... damn.... | 21:12 |
| katco | thumper: good morning | 21:13 |
| thumper | I've just realised that although the tests pass, this will fail under real load | 21:13 |
| thumper | hi katco | 21:13 |
| ericsnow | aznashwan: what's up? | 21:17 |
| aznashwan | ericsnow: i was the loonie with the systemd service management PR from Christmas Eve (namely this one: http://reviews.vapour.ws/r/671/) | 21:18 |
| ericsnow | aznashwan: yep :) | 21:19 |
| aznashwan | ericsnow: the thing is it's done exactly as upstart (namely I used exec calls to systemctl for managing the services), and since have been told that a more native approach would be more favorable... | 21:19 |
| aznashwan | I've recently been show this package from the coreOS guys: https://github.com/coreos/go-systemd which might be exactly what we're looking for. | 21:21 |
| ericsnow | aznashwan: yeah, that is what we were planning on using | 21:21 |
| aznashwan | It would not take too long to adapt my PR to use that, however i'm in the middle of exam season right now, and am not able to get to it sooner than this weekend | 21:22 |
| aznashwan | plus however much time responding to review requests and etc... would take | 21:23 |
| aznashwan | is that timeline ok for you guys? | 21:23 |
| === kadams54-away is now known as kadams54 | ||
| ericsnow | aznashwan: it depends on what we end up getting done this week with regards to systemd | 21:24 |
| ericsnow | aznashwan: I expect that we will already have take care of things by this weekend | 21:24 |
| ericsnow | aznashwan: we were planning on incorporating changes from your patch in whatever we did | 21:25 |
| wwitzel3 | natefinch: http://reviews.vapour.ws/r/771/ | 21:26 |
| aznashwan | ericsnow: as you will, apart from changing the exec calls and (sadly) having to redo the tests, my patch is pretty much on par with the upstart one | 21:27 |
| ericsnow | aznashwan: cool | 21:27 |
| aznashwan | ericsnow: are you guys also planning to do anything about how the service management system is selected in the (very) near future? | 21:28 |
| ericsnow | aznashwan: yeah, that will be a part of what we are working on | 21:29 |
| ericsnow | aznashwan: we also have to refactor various bits of upstart-specific code throughout juju core to be init-system agnostic :( | 21:30 |
| aznashwan | ericsnow: yes, the places were services were used seemed very upstart-specific in the first place... | 21:31 |
| ericsnow | \me gears up for paying off some technical debt | 21:31 |
| ericsnow | natefinch: for fun take a look at the list of commits on that PR | 21:32 |
| aznashwan | ericsnow: well, best of luck to you guys. as i've said, I would be more than happy to do it on Saturday of you don't get around to it by then | 21:32 |
| ericsnow | aznashwan: sounds great :) | 21:33 |
| aznashwan | ericsnow: keep in touch. godspeed :D | 21:33 |
| natefinch | ericsnow, wwitzel3: cool, looking | 21:34 |
| katco | / | 21:54 |
| === lazyPower is now known as lp|outy5000 | ||
| menn0 | axw: i'm currently reviewing your charm storage PR | 22:15 |
| thumper | menn0: where is the statesuite that you created? | 22:37 |
| thumper | state/testing | 22:38 |
| thumper | nm | 22:38 |
| menn0 | thumper: :) | 22:39 |
| jw4 | thumper: I'm pretty sure I've fixed the watcher test - fyi :) | 22:40 |
| wwitzel3 | ericsnow: ping | 23:06 |
| ericsnow | wwitzel3: present | 23:07 |
| wwitzel3 | ericsnow: going to jump in to moonstone for a bit | 23:07 |
| ericsnow | k | 23:07 |
| === kadams54 is now known as kadams54-away | ||
| === kadams54-away is now known as kadams54 | ||
| === kadams54 is now known as kadams54-away | ||
| anastasiamac | thumper: super command of super command.. | 23:47 |
| anastasiamac | thumper: for eg. juju storage pool list.... | 23:47 |
| anastasiamac | thumper: is there a precedence?.. | 23:47 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!