| tasdomas | marcoceppi, do you think https://github.com/juju/amulet/pull/80 needs any further changes? | 08:36 |
|---|---|---|
| tasdomas | hi | 09:52 |
| tasdomas | does bundletester keep a cache of charms anywhere? | 09:52 |
| tasdomas | it seems to ignore an updated local charm | 09:53 |
| 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:37 |
| Odd_Bloke | marcoceppi: If stub can't, might you be able to have a look at ^? | 10:40 |
| Odd_Bloke | stub: marcoceppi: Oh, hold on, that looks like I've screwed something up pulling the changes across to my branch. | 10:42 |
| Odd_Bloke | stub: marcoceppi: OK, fixed now. | 10:50 |
| 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:07 |
| 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:08 |
| 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:09 |
| 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:10 |
| 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:13 |
| stub | niedbalski: You broke trunk. 'make test' fails on core.file with Python3. | 11:23 |
| stub | Fixed. | 11:28 |
| 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:34 |
| Odd_Bloke | stub: Thanks for the merge. :) | 11:42 |
| stub | np | 11:42 |
| stub | Should I have landed that? I just spotted tarmac_tests.sh | 11:44 |
| 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). | 11:57 |
| 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:20 |
| tasdomas | does anyone know what could be the cause of bundletester deploying an older revision of a local charm? | 12:31 |
| lazyPower | tasdomas: git or bzr based charm? | 12:33 |
| 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:37 |
| 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. | 12:58 |
| === CyberJacob is now known as zz_CyberJacob | ||
| tvansteenburgh | lazyPower: i just realized why that's happening | 13:07 |
| * tvansteenburgh goes to fix bug | 13:07 | |
| === hloeung_ is now known as hloeung | ||
| 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:25 |
| lazyPower | jrwren: https://github.com/go-gitea/gitea | 14:26 |
| jrwren | lazyPower: thanks. I will. | 14:26 |
| 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:27 |
| lazyPower | and i was having a rough day to boot | 14:34 |
| tasdomas | jrwren, are you writing a gogs charm? | 14:42 |
| jrwren | tasdomas: NO! | 14:43 |
| tasdomas | jrwren, sheesh, no need to shout ;-] | 14:43 |
| jrwren | tasdomas: I want to be very clear. :) | 14:49 |
| jrwren | tasdomas: sorry, even the thought of this nearly gave me panic attacks. | 14:49 |
| 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:50 |
| jrwren | tasdomas: hahaha, and its go code, so it doesn't package to deb without a fight :) | 14:51 |
| 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:52 |
| 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:54 |
| 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:55 |
| 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:56 |
| jrwren | tasdomas: that is really interesting | 14:58 |
| tasdomas | jrwren, thanks | 14:59 |
| marcoceppi | tvansteenburgh: any other thoughts on this? https://github.com/juju/amulet/pull/80 lgtm | 15:12 |
| tvansteenburgh | marcoceppi: weird, never got an email about that, looking... | 15:16 |
| tvansteenburgh | marcoceppi: lgtm | 15:25 |
| marcoceppi | tvansteenburgh: cool | 15:26 |
| marcoceppi | I'm going to merge and do a release | 15:26 |
| tvansteenburgh | ack, although i have more fixes coming | 15:27 |
| === lukasa is now known as lukasa_away | ||
| === lukasa_away is now known as lukasa | ||
| marcoceppi | tvansteenburgh: oh, I'll hold off then | 15:28 |
| 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 | 17:05 |
| === 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 | ||
| niedbalski | stub, could you expand? | 19:57 |
| 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:01 |
| niedbalski | stub, if you are refering to the same issue. | 20:02 |
| === natefinch is now known as natefinch-afk | ||
| === CyberJacob is now known as zz_CyberJacob | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!