/srv/irclogs.ubuntu.com/2015/06/21/#juju-dev.txt

=== StoneTable is now known as aisrael
=== stokachu_ is now known as stokachu
=== \b is now known as benonsoftware
=== wgrant_ is now known as wgrant
=== Ursinha_ is now known as Ursinha
=== kadams54 is now known as kadams54-away
=== mwhudson_ is now known as mwhudson
=== mwhudson is now known as Guest80728
jamfwereade: you would remember that we discussed writing something like that, but I don't think we ever actualyl did08:33
fwereadejam, ah right :)08:33
jamfwereade: what are you doing around now ? :)08:33
fwereadejam, taking the opportunity to code a little while (almost) nobody's around ;p08:34
jamunderstood08:34
fwereadejam, (although if you fancy reviewing either of my branches (or just conversing on the open issues in the first one) I'd be most grateful08:36
=== mwhudson is now known as mwhudson__
=== mwhudson__ is now known as mwhudson
fwereadejam, hey, I thought I needed something like this: http://paste.ubuntu.com/11749789/ ...but it turns out it's not quite as simple as that09:38
fwereadejam, but that general structure looks very familiar09:39
fwereadejam, do you think there's a place for a worker that just serializes ops onto a single goroutine and lets us focus on the op implementation?09:41
fwereadejam, possibly I should be calling them commands not ops?09:41
=== kadams54 is now known as kadams54-away
=== Beret- is now known as Beret
=== Tribaal_ is now known as Tribaal
=== kadams54 is now known as kadams54-away
=== kadams54 is now known as kadams54-away
=== _thumper_ is now known as thumper
mupBug #1467331 opened: configstore lock should use flock where possible <juju-core:Triaged> <https://launchpad.net/bugs/1467331>22:29
mupBug #1467331 changed: configstore lock should use flock where possible <juju-core:Triaged> <https://launchpad.net/bugs/1467331>22:38
mupBug #1467331 opened: configstore lock should use flock where possible <juju-core:Triaged> <https://launchpad.net/bugs/1467331>22:41
thumperwallyworld: https://github.com/juju/juju/pull/261323:20
thumperwallyworld: review board seems to be not picking up new pulls23:20
wallyworldthumper: will look after standup23:21
wallyworldthumper: lgtm23:37
thumperwallyworld: cheers23:37
thumperwallyworld: the other thought I had was the 1 second timeout23:38
thumperwallyworld: do you think they might be hitting that?23:38
wallyworldthumper: which timeout? is that mentioned in the bug report?23:39
thumperno... but it is the timeout that that fslock uses23:40
thumperthe lock is only held during the duration of a read or write of the file23:40
thumperso I originally thought 1s should be more than enough23:40
thumperhowever if there are many ops running concurrently, they are serialized through this file23:40
thumperso perhas 2s?23:41
thumperI'm not sure23:41
thumpercould just leave it as it is for now23:41
thumperI suppose23:41
thumperpretty sure we never hit this before23:41
* thumper asks the bot to merge23:42
thumperwill forward port to master23:42
thumperdamn it is going to be cold at the gym23:42
* thumper psychs himself up23:42
wallyworldthumper: yeah leave it for now if we haven't seen it before. but if the system has lots to do. may be a factor. i had to introduce reties to lxc ops due to i/o ontention recently23:42
thumpermeh, just noticed bug 146435623:50
mupBug #1464356: TestCloudInit fails <blocker> <ci> <gccgo> <regression> <test-failure> <windows> <juju-core:Invalid> <juju-core 1.24:Triaged> <https://launchpad.net/bugs/1464356>23:50
* thumper off to the gym23:50

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