[08:36] <tasdomas> marcoceppi, do you think https://github.com/juju/amulet/pull/80 needs any further changes?
[09:52] <tasdomas> hi
[09:52] <tasdomas> does bundletester keep a cache of charms anywhere?
[09:53] <tasdomas> it seems to ignore an updated local charm
[10:37] <Odd_Bloke> stub: Might you be able to have a look at https://code.launchpad.net/~daniel-thewatkins/charm-helpers/unison_patches/+merge/265633?
[10:40] <Odd_Bloke> marcoceppi: If stub can't, might you be able to have a look at ^?
[10:42] <Odd_Bloke> stub: marcoceppi: Oh, hold on, that looks like I've screwed something up pulling the changes across to my branch.
[10:50] <Odd_Bloke> stub: marcoceppi: OK, fixed now.
[11:07] <stub> Odd_Bloke: I've got a MP adding hookenv.peer_relationship_id(), which would allow you to drop the peer relation argument entirely if you want (not sure who is using this or how important backwards compatibility is)
[11:08] <stub> Odd_Bloke: I'm no longer using this, but I'll review since I was probably the last person to touch it.
[11:08] <Odd_Bloke> rcj: You might be interested in ^
[11:08] <Odd_Bloke> stub: Thanks!
[11:09] <stub> Odd_Bloke: Are you just using the ssh bit btw? That is all I was using, and at the time thought it should really be separate from unison.
[11:10] <Odd_Bloke> stub: I can't tell you for sure, I haven't really touched that bit of the charm.
[11:10] <Odd_Bloke> I'm trying to get a completely separate change merged, but need this to land in charm-helpers to remove our local modifications to charm-helpers to get that merged. :p
[11:13] <stub> Odd_Bloke: I haven't seen someone assigning to mock.call_args_list before. I use mock.reset_mock()
[11:13] <stub> I guess this way preserves return_value etc.
[11:23] <stub> niedbalski: You broke trunk. 'make test' fails on core.file with Python3.
[11:28] <stub> Fixed.
[11:34] <stub> And the intermittent openstack failure, cause that's the sort of guy I am.
[11:34] <Odd_Bloke> stub: Masochistic? :p
[11:34] <stub> Odd_Bloke: Avoiding my real TODO list.
[11:34] <Odd_Bloke> ^_^
[11:42] <Odd_Bloke> stub: Thanks for the merge. :)
[11:42] <stub> np
[11:44] <stub> Should I have landed that? I just spotted tarmac_tests.sh
[11:57] <Odd_Bloke> marcoceppi: If you could have a look at https://code.launchpad.net/~daniel-thewatkins/charms/trusty/ubuntu-repository-cache/update_charm-helpers/+merge/265645, it would be much appreciated.
[11:57] <Odd_Bloke> That's just a charm-helpers update (and removal of the patches we were using which have now been merged).
[12:20] <Odd_Bloke> marcoceppi: https://code.launchpad.net/~daniel-thewatkins/charms/trusty/ubuntu-repository-cache/add-rsync-timeout/+merge/261424 is also approved by a couple of people and ready to merge.
[12:31] <tasdomas> does anyone know what could be the cause of bundletester deploying an older revision of a local charm?
[12:33] <lazyPower> tasdomas: git or bzr based charm?
[12:37] <tasdomas> lazyPower, bzr
[12:37] <lazyPower> tasdomas: do you have uncommitted changes in the repository? iirc there's a bug open about it not deploying the uncommitted changes
[12:58] <tasdomas> lazyPower, that was it - thanks!
[12:58] <lazyPower> tasdomas: sorry you ran into that, but glad I could help :)
[12:58] <lazyPower> tasdomas: https://github.com/juju/amulet/issues/55 might want to subscribe to this bug so you know when its resolved.
[13:07] <tvansteenburgh> lazyPower: i just realized why that's happening
[13:07]  * tvansteenburgh goes to fix bug
[14:25] <jrwren> lazyPower: I really like that branch name
[14:25] <lazyPower> @jrwren - btw saw you starred gogs. You should alsos tar gitea
[14:25] <lazyPower> its the fork thats under active dev, gogs maintainer isn't accepting patches.
[14:26] <lazyPower> jrwren: https://github.com/go-gitea/gitea
[14:26] <jrwren> lazyPower: thanks. I will.
[14:27] <lazyPower> just FYI - i have a gogs server deployed that will be replaced by this when they land a stable.
[14:27] <lazyPower> its not ready to replace it, but will be in short order.
[14:27] <lazyPower> and re: branch name - its self explanatory.
[14:34] <lazyPower> and i was having a rough day to boot
[14:42] <tasdomas> jrwren, are you writing a gogs charm?
[14:43] <jrwren> tasdomas: NO!
[14:43] <tasdomas> jrwren, sheesh, no need to shout ;-]
[14:49] <jrwren> tasdomas: I want to be very clear. :)
[14:49] <jrwren> tasdomas: sorry, even the thought of this nearly gave me panic attacks.
[14:50] <tasdomas> jrwren, yeah - I spent half a day trying to charm gogs once, started with the bright idea of "I know, I'll first package it to a ppa"
[14:51] <jrwren> tasdomas: hahaha, and its go code, so it doesn't package to deb without a fight :)
[14:52] <tasdomas> jrwren, not without trying to drive you mad first
[14:52] <jrwren> tasdomas: well, if you need help with that step, let me know. I've gotta good at it.
[14:54] <tasdomas> jrwren, yeah - I decided I don't really need gogs, so there went the motivation to charm it
[14:54] <tasdomas> jrwren, built an action-based git server charm instead
[14:55] <jrwren> tasdomas:that sounds interesting. What is an action-based git server?
[14:55] <tasdomas> jrwren, the charm uses actions to manage a git server
[14:55] <jrwren> tasdomas: what kind of actions? and to what end?
[14:56] <tasdomas> jrwren, juju actions - https://jujucharms.com/u/tasdomas/git/trusty/4
[14:56] <tasdomas> jrwren, still ironing out a few kinks (and writing amulet tests for it)
[14:58] <jrwren> tasdomas: that is really interesting
[14:59] <tasdomas> jrwren, thanks
[15:12] <marcoceppi> tvansteenburgh: any other thoughts on this? https://github.com/juju/amulet/pull/80 lgtm
[15:16] <tvansteenburgh> marcoceppi: weird, never got an email about that, looking...
[15:25] <tvansteenburgh> marcoceppi: lgtm
[15:26] <marcoceppi> tvansteenburgh: cool
[15:26] <marcoceppi> I'm going to merge and do a release
[15:27] <tvansteenburgh> ack, although i have more fixes coming
[15:28] <marcoceppi> tvansteenburgh: oh, I'll hold off then
[17:05] <ddellav> jamespage, whenever you get a chance if you could take a look at my MP: https://code.launchpad.net/~ddellav/charms/trusty/glance/upgrade-action/+merge/265592
[19:57] <niedbalski> stub, could you expand?
[20:01] <niedbalski> stub, this is not my change's fault, is in fact an issue with the latest mock package (please see: https://code.launchpad.net/~freyes/charm-helpers/mock/+merge/264884)
[20:02] <niedbalski> stub, if you are refering to the same issue.