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