/srv/irclogs.ubuntu.com/2017/10/24/#juju-dev.txt

=== mup_ is now known as mup
wallyworldaxw: i'm missing somthing for sure - doesn't assigning time.Unix(0,0) to a globalEpoch var mean that the var's value will be stale when it comes to be used?01:20
axwwallyworld: it's meant to be constant. it's a fixed point in time (jan 1, 1970)01:21
wallyworldah, doh, ok01:21
wallyworldaxw: if it is changed to a model where each controller has a clock update worker, is the recovery for the mgo session dying (for whatever reason) just to restart the worker and it picks up a new session?01:34
axwwallyworld: that already happens, it'll continue to do that01:34
wallyworldta, just wanted to be sure01:35
babbageclunkwallyworld: I'm still having trouble making a local datasource directory - it never seems to pick up the tarball I have under devel/01:43
wallyworldi'll have to look at the code to see where it searches for the tarballs01:43
wallyworldbabbageclunk: it looks in the top level metadata dir01:45
wallyworldline 16701:45
babbageclunkfile?01:46
wallyworldtoolsmetadata.go01:48
wallyworldthat's what you are running i assume01:48
wallyworldthat's my guess anyway without debugging01:49
babbageclunkwallyworld: hmm, actually I think they need to be in tools/<stream>/.01:51
babbageclunkThanks!01:51
wallyworldah ok, i'll take your word for it01:51
wallyworldbeen ages since i looked at this stuff01:51
babbageclunkwallyworld: Ok, I think that worked - it says it found the file and I can see a new entry for it in com.ubuntu.juju-devel-tools.json02:01
babbageclunkwallyworld: why does it have all the entries for other tarballs that I don't have, though?02:01
* babbageclunk goes for a run in the beautiful sunshine02:14
thumperwallyworld: the invite is on jam's calendar, so you could tell him I'll be a little late :)02:58
jamwallyworld: how late?03:59
wallyworldaxw: swap you reviews? https://github.com/juju/juju/pull/795505:21
axwwallyworld: yup, in a little while after lunch05:23
wallyworldno worries05:23
=== frankban|afk is now known as frankban
wallyworldaxw: thanks for those pick ups, sad that i missed them first time07:51
axwwallyworld: no worries, looking now07:52
axwjam: if you're still looking at the lease PR, I've finished up the code and it's now at https://github.com/juju/juju/pull/795708:27
axwjam: wallyworld's already +1'd, but I'll wait till tomorrow before landing. need to do some more heavy testing anyway.08:27
jamaxw: k08:28
axwjam: and thanks for your comments so far, I'm happier with how it turned out as a result08:29
mupBug #1680787 opened: openstack: add support for neutron networks where port security is disabled <serverstack> <sts> <OpenStack Charm Test Infra:Fix Released by 1chb1n> <juju:Fix Released by hmlanigan> <juju-core:New> <https://launchpad.net/bugs/1680787>08:56
balloonsjam, you are thinking 2.2.6 with force upgrade?15:11
=== frankban is now known as frankban|afk
mupBug #1726945 opened: [2.3] consume feature hard to manage <docteam> <juju-core:New> <https://launchpad.net/bugs/1726945>17:03
jamballoons: I'm not sure that 2.2.6 would be needed for just this. More that it is trivial to do in a 2.2 so we might as well target it, in case there is something that *does* warrant 2.2.617:05
jamballoons: as for you're version question, i'm not positive, I'll try to give it a look. I thought what I saw was someone trying to go to 2.2.5 using a 2.2~rc2 binary, and there 2.2~rc2 is definitely older than 2.2.517:07
balloonsjam, right. I seem to remember it was saying 2.2~rc2  was greater than 2.2.5.. probably useful just to include a unit test for beta and rc versions so we know it behaves properly17:08
balloonsnot a big deal in the sense this will skip that check, heh17:08
jamballoons: IIRC we had a bug in the past where the "greater than" was given the wrong way around17:09
jamthe check was correct, but the message was backward17:09
balloonsjam, lol, hah. Ok, that may have been all it was then. Either way, that stuff is easy to mix uop17:09
wallyworld_balloons: thumper: release call?21:34
wallyworld_babbageclunk: hey! at some time today, would appreciate a fairly simple review. a bit of boiler plate since a new facade method is added https://github.com/juju/juju/pull/796022:23
babbageclunkwallyworld_: sure!22:23
babbageclunkwallyworld_: looking nwo22:23
babbageclunknow22:23
wallyworld_yay, ty22:23
babbageclunkwallyworld_: looks good - kind of depressing how much boilerplate we need to expose one state method though.22:31
babbageclunk(This is not a new observation, obvs)22:32
wallyworld_babbageclunk: yeah, agreed22:32
babbageclunkBrowser's hung - I can hear you but not see or unmute to talk23:48
babbageclunkwallyworld: anyway, I'll try to grab you in a bit.23:48
wallyworldok23:48

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