* babbageclunk goes for a run, bye!00:12
* babbageclunk comes back, just want to $$merge$$ that PR first00:13
katcomenn0: thanks for the feedback. i responded to a few of your comments and i'm implementing the others00:31
* babbageclunk is actually foing for a run now.00:33
* babbageclunk gah, going00:33
anastasiamacaxw: as an OCR, there is also this PR that's worth looking at :D https://github.com/juju/juju/pull/656300:36
menn0katco: looks good. i've approved the pr.00:40
katcomenn0: ta00:40
mupBug #1468752 opened: "juju ssh" adds an additional strings to all commands when used on Windows, in interactive mode <ssh> <windows> <juju:In Progress by gz> <juju 2.1:In Progress by gz> <juju-core:Triaged> <juju-core 1.25:Triaged> <https://launchpad.net/bugs/1468752>00:41
anastasiamacwallyworld: u r off the hook - all is good in the land :)00:49
anastasiamacmenn0: ping me when u want a break from migrations (not in the next 30mins tho)00:50
menn0anastasiamac: ok00:52
thumpermenn0: you'd think that finding this would be easy...01:37
* thumper grumbles01:37
menn0anastasiamac: now?01:52
menn0babbageclunk: your last merged PR was perfect timing. I needed exactly the changes you made for something I'm doing.01:53
anastasiamacoh menn0'01:56
anastasiamaci mean 'ok menn0'... h is the new k, right?01:56
anastasiamacho or irc?01:56
menn0anastasiamac: HO is probably more efficient01:57
babbageclunkmenn0: :)02:01
menn0anastasiamac: just noticed this: https://github.com/natefinch/lumberjack/pull/1602:32
menn0anastasiamac: someone already has a PR up that does gzip compression on rotation02:33
menn0seems like it's close to ready02:33
anastasiamacmenn0: \o/ this will make asking Nate nicely even easier :D02:33
menn0anastasiamac: and it will give him permission to work on it during work hours02:34
anastasiamacaxw: this one does not seem to be transacation related... https://bugs.launchpad.net/juju/+bug/164439602:38
mupBug #1644396: machine-0 takes up massive memory, slows down to unusable, "upgrade in progress" loops <juju:New> <https://launchpad.net/bugs/1644396>02:38
anastasiamactransaction* (who needs to type today?)02:39
axwanastasiamac: interesting. I don't have time to look into it atm, doing OCR02:41
babbageclunkthumper, menn0: Should I be deleting logs for the migrated model from the source controller after the transfer is complete?03:30
thumperbabbageclunk: I'd say yes03:30
thumpermenn0: ?03:30
babbageclunkmigrating back and forth between two controllers, now I've got quite a lot of logs.03:31
menn0babbageclunk: yes I think so03:33
menn0otherwise they'll never go away03:33
menn0babbageclunk: in fact, that should be part of the REAP phase03:33
wallyworldanastasiamac: i cannot reporduce any "unknown remoteApplications collection" errors. The only bug I can see refers to the cmr feature branch which has been obsolete for some time now. Do you have any recent occurrences?03:40
anastasiamacwallyworld: let me have a look at CI logs.. this were i saw them yesterday :)03:41
anastasiamacwallyworld: it's behind feature flag on both 2.0 and develop, right?03:42
wallyworldalways has been :-)03:43
wallyworldunless i missed a place03:43
wallyworldbut i diffed 2.0 and develop03:43
anastasiamacwallyworld: k.. i'll ping u if i find anything :) pretend everything is awesome for now... \o/03:45
wallyworldi couldn't see anything03:45
anastasiamacwallyworld: so... in the latest develop run, http://reports.vapour.ws/releases/4591, go to first failure - azure - http://reports.vapour.ws/releases/4591/job/azure-arm-native-deploy-bundle/attempt/101703:55
anastasiamacwallyworld: under artifcats, open up controller/machine-0/machine-0.log.gz03:55
anastasiamacwallyworld: lines from 1447 :)03:55
jammenn0: are you still able to meet today?04:03
wallyworldaxw: i've not used the new worker catacomb before - i didn't know about Plan.Init. but that makes it much easier04:23
axwwallyworld: cool04:31
axwwallyworld: catacomb makes this particular worker a whole lot easier to write in general04:32
wallyworldaxw: even though i wasn't using Init, the logic resulted in te same thing but the code was clumsy04:32
natefinchmenn0, anastasiamac: just saw your ping about log compression04:32
wallyworldyeah, the catacomb is nice04:32
axwwallyworld: not quite, because if Invoke failed, you weren't killing the watcher (unless I missed something)04:32
wallyworldaxw: the invoke could only fail if things were not properly set up. and those same failures would not see the init thngs registred either04:33
axwwallyworld: Invoke *always* kills the Init workers, regardless of how it fails. see the defer at the top of Invoke04:33
wallyworldoh, so it does04:34
wallyworldaxw: testing the sub worker wait errors will be a pita since there's no easy way right now to override the Wait() methods05:11
wallyworldi could override the new() methods in export test and use a shim where i control the error05:13
wallyworldor go all the way and pass in the new methods via the worker config05:13
wallyworldor i could ignore for now05:13
axwwallyworld: that doesn't sound quite right. I think you should have ObserveRelationUnitsChange return an error; trigger a RU change; trigger a relation change, and arrange for it to be NotFound05:15
axwwallyworld: that would cause it to try and stop the RUW, at which point it'll get the error that ObserveRelationUnitsChange returned05:15
wallyworldaxw: aren't we talking about the worker.Stop() errors being propagated?05:16
wallyworldthey are the ones i was ignoring05:16
wallyworldi can also add that other scenario05:17
axwwallyworld: if you cause the RUW to fail atm (e.g. by having Observe... return an error), what happens to the remote application worker?05:18
wallyworldit should exist with that error, but there's no test for that, i'll add one05:20
wallyworldaxw: i've pushed some changes to fix some races, should be good now i think06:13
axwwallyworld: ok, looking06:13
wallyworldaxw: thanks for review. so many pieces to this cmr puzzle. will be several more prs yet06:32
wallyworldaxw: i have a one liner as well https://github.com/juju/juju/pull/661006:33
anastasiamacaxw: i have a small but volatile one too :-P ... yes, wallyworld :) https://github.com/juju/juju/pull/642206:49
axwanastasiamac: reviewed07:01
frobwareaxw: thanks for the review comments07:24
anastasiamacaxw: thnx \o/07:47
=== frankban|afk is now known as frankban
SimonKLBwould it be out of the question to allow some kind of generic relation where the only thing youre really interested in is the name of the other charm you add the relation to? a relation that didnt require an interface that is08:08
macgreagoirjam frobware voidspace: Any need for the regular juju-core-team HO?10:01
jammacgreagoir: given Turkey day, I'd guess not many will show, thoughts?10:02
macgreagoirLikely just the same people as the stand-up later. I say can it.10:02
jammacgreagoir: nobody is there right now, at least.10:03
jamjust me10:03
macgreagoirI just left :-)10:03
anastasiamacaxw: add-model help PR updated.. :)10:43
jamfrobware: I need to grab some food, but I'll join the hangout in a moment.11:03
frobwarejam: np11:03
jamI ended up doing back-to-back meetings today and haven't carved out food time yet.11:04
axwanastasiamac: thanks, your wording on name uniqueness is very clear11:18
jamfrobware: I'm joining the hangout now11:26
anastasiamacaxw: \o/ made my day! m going to land it ;)11:31
anastasiamacaxw: (especially since it's ur wording all the way!) :D11:31
* perrito666 just realizes that today is a holiday in the US12:16
mupBug #1644566 opened: juju bootstrap hangs forever at "Attempting to connect to" <amd64> <apport-bug> <zesty> <juju-core:New> <juju-core (Ubuntu):New> <https://launchpad.net/bugs/1644566>14:40
mattywevilnickveitch, it's there, many thanks17:27
=== frankban is now known as frankban|afk
mupBug #1644566 changed: juju bootstrap hangs forever at "Attempting to connect to" <amd64> <apport-bug> <zesty> <juju-core:Invalid> <golang (Ubuntu):New for mwhudson> <juju-core (Ubuntu):Invalid> <lxd (Ubuntu):Invalid> <https://launchpad.net/bugs/1644566>18:37
babbageclunkhey menn0, I proved to myself that mongo's sort isn't stable, quelle surprise. Do  you think it would be feasible to add _id to the model,time index?20:24
menn0babbageclunk: so make the index model,time,_id?20:25
menn0that should be fine20:25
menn0queries that use model,time will still take advantage of the index20:25
babbageclunkmenn0: I'm just making a test to show the problem and then I'll try that.20:26
babbageclunkmenn0: quick review? https://github.com/juju/juju/pull/6612/files21:49
menn0babbageclunk: give me 5 mins21:51
babbageclunkmenn0: no rush21:54
babbageclunkmenn0: I meant quick in the sense that it should be straightforward, not "hey do this quickly"! :)21:55
menn0babbageclunk: got it :)21:59
menn0babbageclunk: your change looks great but something needs to remove the previous index22:41
menn0babbageclunk: the correct way to do that is to add an upgrade step22:41
menn0babbageclunk: have you done one before? I can guide you through it.22:42
babbageclunkmenn0: No, I haven't22:47
menn0babbageclunk: quick HO?22:48
babbageclunkmenn0: Sure22:49
babbageclunkmenn0: https://hangouts.google.com/hangouts/_/canonical.com/menno22:51
blahdeblahAnyone around to give some quick advice/explanation on how relation scoping works in juju 1.x?  Here's my environment: https://pastebin.canonical.com/171809/23:34
blahdeblahThe canonical-livepatch charm has an nrpe-external-master relation which allows it to relate to nrpe.  However, there are two nrpes in the environment with different service names.23:34
blahdeblahI've found relating it to both doesn't work; do I need two separate canonical-livepatch services to make that happen?23:35
menn0blahdeblah: let me dig into that for you23:36
blahdeblahmenn0: thanks; I'm guessing this is something to do with relation scopes, which is an area that I've always found a bit head-scratchy23:37
menn0blahdeblah: you're not the only one :)23:37
blahdeblahI'm glad. :-)23:37
* blahdeblah goes to re-read charm author doc on that stuff23:38
menn0blahdeblah: when you say "relating it to both doesn't work", is Juju giving you an error or the relation doesn't do what you expect?23:38
blahdeblahit just never seems to build the relation23:39
blahdeblahNo errors or anything; it's just that nothing seems to happen in debug-log23:39
menn0blahdeblah: so in your environment what is canonical-livepatch subordinate to?23:43
blahdeblahthe main primary charm23:43
blahdeblahbut we need to add the relation to the nrpe charm so that canonical-livepatch produces a nagios check which we can monitor23:43
blahdeblahI've looked at the source of both charms, and they're both set to use container scope for the nrpe-external-master relation, which is correct from my reading of the documentation.23:44
* menn0 looks at the code23:45
menn0blahdeblah: actually, i've got a standup now... back in a bit23:46
blahdeblahno worries - thanks anyway23:46

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!