anastasiamaccherylj: it looks like gopkg.in/yaml.v1 is missing from the dependencies.tsv00:11
kiltywith juju init deprecated in 2.0, how would one go about setting params for a private openstack cloud?00:24
kiltythe command-changess page links to https://jujucharms.com/docs/devel/juju-config.md as a referenc for the new method, which is a broken link00:26
mupBug #1548564 opened: restart failed upgrade <juju-core:In Progress by anastasia-macmood> <juju-core 2.0:In Progress by anastasia-macmood> <https://launchpad.net/bugs/1548564>00:28
kiltyalternatively, is there an another way to fix the "ERROR juju.cmd supercommand.go:429 cannot set initial environ constraints: index file has no data for cloud {RegionOne} not found" error? (http://pastebin.com/RXKmg2gC)00:29
mupBug #1548564 changed: restart failed upgrade <juju-core:In Progress by anastasia-macmood> <juju-core 2.0:In Progress by anastasia-macmood> <https://launchpad.net/bugs/1548564>00:31
bogdanteleagakilty, by params you mean params that were previously set in environments.yaml?00:41
mupBug #1548564 opened: restart failed upgrade <juju-core:In Progress by anastasia-macmood> <juju-core 2.0:In Progress by anastasia-macmood> <https://launchpad.net/bugs/1548564>00:43
kiltybogdanteleaga: trying to deploy juju on a private openstack cloud - when I install juju-core2, how do I set the auth url, user/pass, tenant name, etc?00:46
bogdanteleagakilty, https://paste.ubuntu.com/15127859/, just at the bottom00:55
bogdanteleagabasically the whole configuration got split up amongst more files, or it can be specified from the cli00:56
kiltythank you - I'll give that a shot tomorrow morning01:01
cheryljanastasiamac: yeah, I saw. Thanks!02:03
* thumper deals with icky merge from trunk02:43
thumperchanges to code I have moderately attacked with a machette during refactoring02:43
* thumper makes a big sad face02:48
wallyworldaxw: whenever you have a chance https://github.com/juju/romulus/pull/1103:08
axwwallyworld: sure, looking03:09
axwwallyworld: hrm, can you hold off a bit? I'm updating the interface to take an account name as well as controller name03:10
axwfor ModelGetter and friends03:10
axwwallyworld: won't be much more of a change03:10
wallyworldnp, i'll wait03:10
wallyworldcherylj: i'd hardly class that "lxd" not being in list-clouds output as critical03:11
wallyworldit's more a kin to a doc issue03:11
wallyworldalong the same lines as the bootstrap help bug03:12
thumperis master currently blocked?03:13
* thumper wants to merge some useful changes03:13
thumperparticularly refactored code I'm using in migrations03:14
thumperto avoid extra conflicts03:14
thumpercan people please stop changing code?03:19
thumperthat'd be swell03:20
natefinchI think my manager would get mad ;)03:20
natefinchmaster is not blocked btw03:20
thumperwallyworld: StopMongoUntilVersion03:21
thumperwallyworld: what can you tell me about it?03:21
wallyworldit's part of the mongo 2-> 3 migration implementation03:22
thumperwhich will have happened by 2.0?03:22
thumperor not?03:22
thumperwhy do we need a worker?03:22
wallyworldyes, we are waiting on the deb packaging03:22
thumpercan't we just say "2.0 needs mongo 3.0"03:23
wallyworldthat's what we are doing03:23
wallyworldbut mongo is not packaged and in the archives03:23
thumperthen why encode into state?03:23
thumperI'm just unclear as to the process03:23
thumperwallyworld: 1:103:23
=== ashipika1 is now known as ashipika
* thumper waits patiently for the local test run to complete before submitting for real muerge08:24
frobwaredimitern, dooferlad: http://reviews.vapour.ws/r/3942/ - choosing to merge blessed builds atm.08:36
dimiternfrobware, LGTM, thanks08:39
jamfwereade: ping about a config location09:34
jamspecifically, the LXD provider has always used $HOME/.config/lxc/$NAMESPACE as the directory that holds the information we need for talking to the LXD daemon09:36
jamhowever, our test suite seems to now be appropriately sanitizing $HOME09:36
jamwhich means the lxdclient code is trying to create /.config/lxc/blah09:36
jamand that is (correctly) failing09:37
jam(note that in the actual providers once you bootstrap, the Juju machine agent was also always using /.config/... because $HOME wasn't set for the Juju agent)09:37
jamI'd like to move those files to somewhere more appropriate, and just wanted to run it past you09:38
fwereadejam, I agree they should not be there :)09:49
jamfwereade: so far as I can tell none of our other Providers/Brokers need to write something to disk09:50
jamso I don't have a lot to reference for where to get the information from and where to save it.09:50
fwereadejam, nor do I really... an agent's datadir feels like the closest equivalent of $HOME?09:51
jamfwereade: so there are 2 aspects. 1 is for the "local" LXD provider which is a provider all by itself09:54
jamand the other is for the agents that then create LXD containers09:54
frobwaredimitern: standup10:03
mupBug #1548799 opened: bootstrap from windows fails with no such file <ci> <regression> <windows> <juju-core:Triaged> <https://launchpad.net/bugs/1548799>12:54
mupBug #1548809 opened: cmd/juju/action: FetchSuite.TestRun takes 18s to run <tech-debt> <juju-core:Triaged> <https://launchpad.net/bugs/1548809>13:30
evilnickveitchso, juju2.0 doesn't set $JUJU_HOME ?13:58
frobwarevoidspace: MAAS ho?14:02
natefinchmorning all14:03
natefinchevilnickveitch: I believe JUJU_HOME is called something different in 2.014:03
natefinchevilnickveitch: to enable using it side by side w/ 1.2514:04
evilnickveitchnatefinch makes sense, but I can't find what it is :)14:04
evilnickveitchI can't find any JUJU*14:05
natefinchevilnickveitch: JUJU_DATA14:05
voidspacefrobware: oops, omw14:06
natefinchevilnickveitch: https://github.com/juju/juju/blob/master/juju/osenv/vars.go#L2114:06
evilnickveitchnatefinch thanks! I will take your word for it. Currently running beta1 and it hasn't set any of those14:07
perrito666evilnickveitch: it is called JUJU_DATA and it is by default $XDG_DATA_HOME/juju or ~/.local/share/juju if XDG_DATA_HOME is not defined14:08
natefinchevilnickveitch: what do you mean by it hasn't set them?  These are what a user would set to control where juju stores its data14:08
perrito666evilnickveitch: juju does not Set those14:08
perrito666evilnickveitch: juju uses them if set14:08
perrito666anyhow if neither of thise is set the path is ~/.local/share/juju14:09
evilnickveitchperrito666, natefinch thanks, I will update the reference pages. I didn't think Juju had picked up my setting but i think it was a typo on my part14:12
natefinchanyone know if there's a plan to remove the "unstable" from the charm.v6 repo?14:46
rick_h_natefinch: only once the new publish process is signed off and as part of 2.0 changes that require updates like resources work14:46
rick_h_natefinch: so "yes, there's a plan" but it's "not yet"14:47
natefinchrick_h_: kk14:47
cheryljfrobware: ping?15:09
cheryljnatefinch: can you do a quick review for https://github.com/juju/juju/pull/4508 ?  It's backporting your proxy fix15:11
natefinchcherylj: no problem15:12
frobwarecherylj: pong15:12
cheryljfrobware: hey, just wanted to check on what you guys are working on in the maas-spaces2 branch, and what milestone / release you were targeting15:13
natefinchcherylj: have you run the tests locally?  I'm worried bumping the utls revision might have unintended side effects.  I mean, certainly we can let CI do the lifting, just curious15:14
cheryljnatefinch: yeah, I forked utils off for 1.25 so it didn't pull in the massive amount of changes that have gone in there since 1.2515:14
cheryljnatefinch: and just applied your proxy change to it15:15
frobwarecherylj: want to sync quickly? I have 15 mins15:15
natefinchcherylj: ahh, ok.15:15
cheryljfrobware: yes - https://plus.google.com/hangouts/_/canonical.com/cheryl-andy?authuser=015:16
natefinchcherylj: in an ideal world we should really make that a separate import path, so there's no confusion, but I understand that's significantly more work than just letting godeps cheat for us.15:17
natefinchcherylj: anyway, LGTM15:18
cheryljthanks, natefinch!15:19
alexisbfrobware, ping15:34
cheryljsorry alexisb, I was keeping him15:34
alexisbah nws, no rush15:34
* natefinch imagines cherylj with frobware in a curio cabinet.15:35
cheryljnatefinch: MRW I had to let him go:  http://goo.gl/blhJiO15:37
natefinchcherylj: lol15:37
cheryljI had to set it to wumbo15:37
natefinchgood lord someone really needs to write a stringdiff checker for gocheck16:05
natefinchlike, don't just tell me these two huge strings are different, show me where16:05
perrito666natefinch: that and pprint for things16:06
natefinchperrito666: yeah, I'd kill for a nicely indented %#v16:07
natefinchperrito666: I think I remember there's a third party package out there that does it16:07
natefinchperrito666: https://godoc.org/github.com/kr/pretty16:09
natefinchthere's a few others, but that one seems most like what I would want as output16:11
natefinchthere's go-spew as well, but it has ugly output IMO16:11
katco`natefinch: ericsnow: could use a +1 on http://reviews.vapour.ws/r/3916/16:27
natefinchkatco`: I'll take a look16:28
katco`natefinch: ty... very short review16:28
natefinchkatco`: ship it16:30
katco`natefinch: ty16:31
katco`ericsnow: are you with us today? :)16:31
ericsnowkatco`: yep, sorry, missed your message16:31
katco`ericsnow: no worries, just hadn't seen you yet16:31
katco`ericsnow: i said good morning, but then my bouncer booted me right away, so don't know if anyone responded16:32
natefinchthe perils of no morning standup16:32
natefinchkatco`: I think your good morning got swallowed by the internet16:33
katco`booo internet16:33
katco`what has it ever done for us16:33
natefinchsmoke signals should be good enough for anyone16:33
ericsnowinternet: she didn't mean it16:33
* ericsnow hopes we haven't angered it16:34
* katco` has quit (Connection timeout and also she smells, ~<3 Internet)16:35
natefinchkatco`, ericsnow: I may have overestimated the time to complete this card: http://reviews.vapour.ws/r/3945/16:35
ericsnowkatco`: I could use a quick re-review on both my patches16:35
katco`natefinch: rock!16:36
ericsnownatefinch: could you look over my responses to your comments on http://reviews.vapour.ws/r/3891/?16:36
ericsnownatefinch: nice16:36
natefinchericsnow: sure thing.16:36
katco`natefinch: ericsnow: i'll be getting to reviews here in a sec. wrapping up my PR for my card16:36
ericsnownatefinch: ta16:36
ericsnowkatco`: np16:36
natefinchactually, I'm on-call reviewer today, so I should spend some time on that anyway16:37
katco`ericsnow: natefinch: furnace guy is here... will be in and out16:49
ericsnowkatco`: k16:50
ericsnowkatco`: good luck16:50
perrito666everything suddenly went almost night dark, so if you dont see me respond is because power went away17:07
perrito666https://pbs.twimg.com/media/Cb6jfLLWEAA_t7O.jpg:large https://pbs.twimg.com/media/Cb6f_tnW4AQySJd.jpg:large  <-- 2 PM on summer17:14
natefinchkatco`, ericsnow: I'll need another card after I do some reviews.  What about the extension validating one?17:15
ericsnownatefinch: what about the overhead card in the iteration backlog?17:16
katco`natefinch: i'd rather you pick up the other critical user story17:16
natefinchericsnow: good point, I was glazing over anything that wasn't a user story.  katco`, what do you think?  That one will fix multiple fairly ugly bugs.17:18
katco`natefinch: that's fine too... need to know how long it's going to take17:19
ericsnownatefinch: FYI, I've reviewed your patch17:20
natefinchkatco`: lemme go investigate for a few, IIRC it was a little annoying to get the left hand to talk to the right hand17:20
natefinchericsnow: thanks17:20
natefinchericsnow: good point about using FormattedCharmResource!17:22
natefinchreusing... feg17:23
katco`natefinch: np17:23
natefinchstupid fingers.17:23
natefinchkatco`: should be doable by tomorrow afternoon17:28
natefinchkatco`: hopefully earlier, but I'm inclined to be conservative at this point.17:28
katco`natefinch: good deal... flag the card if you don't mind17:30
katco`they probably have to replace my entire furnace T.T17:32
katco`right before we move17:32
katco`not cool, internet. not cool.17:33
perrito666katco`: renting house?17:35
katco`perrito666: no, we're relocating17:35
perrito666so you have to fix the furnance for the new home?17:36
katco`no the home we're going to be selling very soon17:36
perrito666ah, it seems its easier to sell it like it is and just make a discount :p17:37
perrito666although I am not entirely sure to what does furnance translate to17:37
katco`perrito666: that's not really an option since it's not working at all17:37
natefinchperrito666: furnace == heat17:39
katco`ah yes, sorry17:39
katco`it's what heats the house17:39
perrito666just googled and I got an idea17:39
natefinchperrito666: no one's going to buy a house without any heat :)17:39
perrito666Ill assume it is not a 300 dollars to replace it as it is here17:39
katco`perrito666: they just quoted me $2k17:40
natefinchperrito666: depends on what you get, but often like $2500-500017:40
perrito666ouch :p17:40
natefinchkatco`: that blows, sorry.17:40
perrito666mine just heats wather for the floor heating system, but it was cheap :)17:40
perrito666this is my street after a few mins of rain http://i.imgur.com/CgSlkF3.jpg17:43
mupBug #1548921 opened: credentials.yaml file should have a stub <docteam> <juju-core:New> <https://launchpad.net/bugs/1548921>17:52
lazyPowernatefinch - i think if oudn the culprit18:29
lazyPowernatefinch - do you recall yesterday how i was having hashsum mismatches delivering the charm  from mc => unit?18:29
lazyPowerthis appears linked to when i use juju upgrade-charm --force-units, with a couple of them deployed... i dont know if the hashes are getting out of sync between what the state server thinks should be there or what, but it doesn't barf if i dont upgrade-charm --force-units in the model.18:30
natefinchlazyPower: interesting18:35
lazyPowernatefinch aside from anecdotal evidence, what could i gather to reproduce and file a bug?18:35
natefinchlazyPower: can you repro with a simple case?18:36
lazyPowerWill work on it and get that + bug# at you then.18:37
lazyPowercheers :)18:37
natefinchlazyPower: cool, thanks18:37
natefinchrick_h_: the spec for showing updates available on resources when running juju resources... should those be shown for all modes of juju resources, or just the generic juju resources servicename?  i.e. should it be shown for juju resources unitName or juju resources servicename --details?18:39
rick_h_natefinch: i think like juju status for all commands. i don't want havevto write a bash for loop to check for updates available across my model.18:47
rick_h_natefinch: thoughts?18:47
natefinchrick_h_: what I mean is,  I already have it coded so that juju resources wordpress will show you warnings for any out-of-date resources for the service.  but my question would be, should we show the same warning if you do juju resources wordpress/0  or juju resources wordpress --details?18:48
natefinchrick_h_: I don't think it's appropriate when you're just viewing a single unit... but not sure about when you're viewing the detailed output for a service.... probably18:49
mupBug #1548938 opened: list-credentials doesn't include 'discovered' creds <docteam> <juju-core:New> <https://launchpad.net/bugs/1548938>18:49
natefinchrick_h_: we don't currently support just running "juju resources" without the service name18:50
natefinchrick_h_: (because it's not in the spec ;)18:50
rick_h_natefinch: ok, taking that one at a time. I definitely think that --details should show since it's about getting into the thorough details18:51
rick_h_natefinch: agree that on a unit, only showing it's current revivsion vs expected revision makes sense18:51
rick_h_natefinch: since you're not scoped to the "service as a whole" which an upgraded resource represents18:51
rick_h_natefinch: and /me loads spec to see what we have/don't for "juju resources" as a lone command18:52
natefinchI wouldn't be totally against it, though honestly, most commands run over the entire model lose usefulness once you have a realistically sized model18:53
natefinchthough if you only showed out of date resources for the entire model, that could be quite usefukl18:53
rick_h_natefinch: well,it seems you'd be able to show your network map, storage pools created, etc.18:54
rick_h_natefinch: but you're right, that never came up in the spec...18:54
natefinchrick_h_: sure, sure18:54
rick_h_natefinch: well any kind of audit function/debugging details would want a 'wtf is running'18:54
rick_h_natefinch: so current resources that have been 'resource-get' and their revisions, along with any upgrades available seems useful18:54
rick_h_natefinch: because if you go to file a bug, knowing which ones are there are important to duplicating. And without having to script a for loop for each service18:55
* rick_h_ hangs head in shame that he missed this in the spec so far18:55
natefinchrick_h_: we missed the existence of juju deploy --resource... I think you're in good company ;)18:56
natefinchthe only perfect spec is the code itself18:56
rick_h_natefinch: ok, so you have what you need for the moment?18:57
natefinchrick_h_: yes... we'll just do it for the service and service with --details18:57
rick_h_natefinch: k, ty for the ping18:57
natefinchrick_h_: and I leave it with you to arm wrestle katco` about listing resources across the model18:57
rick_h_natefinch: heh, /me runs to protective tank18:57
rick_h_natefinch: the good news is that it's not bad since Juju is just looping on myhalf? /me hopes :)18:58
natefinchrick_h_: not the end of the world, but a few days' work, certainly18:59
natefinchrick_h_: one more thing I thought of.... once you override a store resource by uploading a local resource with push-resource, I presume we won't upgrade that resource when you do a charm-upgrade, correct?19:14
rick_h_natefinch: I think that's true. The key there will be indicating that to the user somehow.19:15
rick_h_natefinch: though...if you upgrade-charm...19:15
rick_h_natefinch: thinking19:15
natefinchrick_h_: it seems logical to me to pin it... but then we have no way to un-oin it currently19:15
natefinchun-pin that is19:15
rick_h_natefinch: so since the only way to upgrade that is with upgrade-charm, and that is an explicit call to "get me the latest from the charmstore"19:15
rick_h_(assuming it's not upgrade-charm ./localpath)19:16
rick_h_natefinch: I don't think we pin, I think it's just doing what the user asks19:16
natefinchrick_h_: so the problem is, I'm using a charm that has stub-resources in the charmstore... the charm code gets updated, and there are some minor tweaks to the stub resources, so they get updated too... charm-upgrade will now throw away my real resources and replace them with the stubs in the store19:17
rick_h_natefinch: so but we don't tell 'stub' from 'non-stub'19:17
rick_h_natefinch: you might need that resource for a bugfix and want to get back on 'stable' footing19:17
rick_h_natefinch: I think the trick there is upgrade-charm supporting something...either manually specified resources or something that says "charm w/o new resources"? but then maybe you want one/not all19:18
rick_h_natefinch: so ugh?19:18
natefinchrick_h_: ug is right19:18
natefinchrick_h_: I definitely think there will be times when users want one or the other behavior19:18
rick_h_natefinch: I'm getting tempted to suggest a --without-new-resources flag to upgrade-charm19:19
natefinchrick_h_: I was thinking it might be useful to have a command to revert an uploaded resource to the store-version19:19
* rick_h_ isn't sure on that one19:19
rick_h_natefinch: hmm, actually I think this fits into the stuff I'm writing for the juju resources command.19:21
natefinchrick_h_: I was just thinking that splitting it into two commands might give the most flexibility to the user.  have upgrade-charm keep uploaded resources as they are, but if someone wants to stop using an uploaded resource, they can juju unpush-resource (hopefully with a better name)19:21
rick_h_natefinch: well, the idea would be you could push a resource with a charmstore url.19:22
rick_h_natefinch: that indicates your interest in following the CS blessed paths perhaps19:22
natefinchrick_h_: ooooh, I like that19:22
rick_h_natefinch: though it might not be techincally blessed based on the revision you have...thinking more19:23
natefinchrick_h_: I like that it's explicit about what exactly you'll get19:23
rick_h_natefinch: right19:23
rick_h_I think that falls under the normal charmstore rules of you can leave off the revision, it grabs the latest blessed revision19:23
rick_h_and you'd track the charmstore from there on19:23
rick_h_and if you provide a local file, you'd ignore the charmstore updates from then on, until you went back to a push-resource cs:xxxxx19:23
natefinchyeah, I think that's what I would expect... if juju threw out the thing I uploaded and explicitly told it to use, I'd get mad19:24
rick_h_hmm, this is working ok in my head at least19:24
rick_h_yea...and if I deploy a service with local files, even if I upgrade the charm I've indicated I'm manually handling resources19:24
rick_h_ok, this falls into the lovely "handling upgrades" section.19:25
natefinchmight be worth having a flag on upgrade-charm that forces the use of the store resources... otherwise it might be impossible to upgrade, if the new resources and old resources are incompatible19:25
rick_h_natefinch: how so?19:26
rick_h_natefinch: if you have a new charm, and it wants different resources, it'll error and then you'll have to provide them, rerun hooks, and get from error to normal?19:26
rick_h_natefinch: and there's an upgrade-charm flag with the --resources option19:27
rick_h_natefinch: so you could do it in one swoop?19:27
natefinchrick_h_: anything involving rerunning the hooks is bad.  But yes, --resources could work if you can specify charmstore urls.19:28
rick_h_natefinch: right19:28
natefinchexcept uh..... I think we missed upgrade-charm --resources.  AFAIK we don't have cards for that... ericsnow, katco` ... I think this is work we're missing19:30
ericsnownatefinch, rick_h: correct me if I'm wrong, but "juju upgrade-charm --resources" is new to the spec19:34
rick_h_ericsnow: well it was a "case" in the upgrade cases that was blank19:34
rick_h_ericsnow: those weren't all filled out and natefinch has been asking about those cases right now19:34
natefinchericsnow: so... sort of :)19:34
ericsnowrick_h_: I was looking for a "upgrade-charm --resources" in the spec yesterday but did not find one19:35
ericsnowrick_h_: ah19:35
rick_h_ericsnow: right, it was "Upgrade charm and use supplied resource at upgrade time" without a command in the following bullet point19:35
natefinchericsnow: the actually CLI command was just added, but there was a bullet point labelled "Upgrade charm and use supplied resource at upgrade time" under upgrades19:35
katco`rick_h_: i apologize, i'm having to buy a new furnace so i'm not able to give this my full attention. can the new bullets under "handling upgrades" be pushed to a v2?19:35
rick_h_ericsnow: and I think that logically fits into place there.19:35
rick_h_katco`: understand, let's sync on it when you're free. I've given natefinch what he needs to unblock him for the moment and we'll go through the updates and see what can be bumped.19:36
rick_h_katco`: take care of getting fire first, handy this time of year19:36
katco`rick_h_: you're not kidding. we are getting pretty creative here with space heaters =|19:38
perrito666I dont understand, its pretty warm this time of the year, after a decent rain temp got down to 25C19:40
natefinchwe don't all live in the jungle, perrito66619:40
katco`perrito666: you shut your souther-hemisphere mouth!19:40
* perrito666 closes his mouth, around a straw, and zips a fruit juice19:40
katco`perrito666: rofl19:41
* katco` enjoys the phone transfer tolken ring she seems to be on19:41
perrito666katco`: you are on the infinite call center loop?19:41
ericsnowkatco`: do you mind if I land my 2 patches; natefinch has given ship-its and I've addressed all the open issues19:46
katco`ericsnow: please go for it.19:46
* katco` is frustrated19:46
natefinchericsnow: I have one concern I just found19:47
katco`i can't think of many worse times for this to happen (grumble)19:47
ericsnowkatco`: I left a few open to point out where we might discuss it a bit more, but I feel comfortable with where things are at19:47
natefinchericsnow: I was looking at the updates function19:47
ericsnownatefinch: uh oh19:47
ericsnownatefinch, katco`: FYI, I have a patch up for moving resources persistence to the state package19:48
natefinchericsnow: I don't liek that it assumes the two lists are the same length and in the same order.  I'd rather explicitly check the resource names19:48
ericsnownatefinch: that's fine; I did make it clear in the doc comment that they are one-to-one19:48
natefinchericsnow: yeah, but that guarantee is really really really far removed from that code19:49
ericsnownatefinch: I'll fix it up19:49
natefinchericsnow: thanks... sorry, the only reason I noticed was because I copied your code for doing my testing, and had weird behavior because my lists were out of order19:50
ericsnownatefinch: sounds like a personal problem <wink>19:50
natefinchericsnow: lol19:50
thumperericsnow: thanks for the move, doing payloads too?19:57
ericsnowthumper: wasn't planning on it19:57
ericsnowthumper: I wouldn't mind but you'll have to win over katco` (and you shouldn't mess with her today!)19:58
* thumper keeps his distance19:58
ericsnowthumper: (we're still scrambling with resources)19:58
katco`thumper: :) we'll do it happily if we can get out ahead on resources19:59
ericsnowthumper: (and katco` is dealing with a bad furnace)19:59
ericsnowthumper: FWIW, moving payloads should look just like the patch for resources20:00
thumperericsnow: um... have you seen the size of that?20:02
thumper  47 changed files  with 2,110 additions and 546 deletions.20:02
thumperericsnow: I'm pretty sure you aren't taking across what you think you are20:02
ericsnowthumper: that's because it's based on other branches; in RB you'll see it is ~ +200, -20020:02
natefinchrick_h_: the spec calls the section for updates "[Updates available]" (note the lowercase a in available).  Should that be a capital a for Available?"20:06
rick_h_natefinch: no, it's only the first word caps. /me tries to remember the email/notes from jam on that20:07
natefinchrick_h_: boo20:08
katco`rick_h_: natefinch: i think we can do second word caps too20:08
katco`rick_h_: natefinch: i think the real contention was b/t all caps and camel-case of sorts20:08
rick_h_katco`: natefinch yea, foudn the email20:08
rick_h_that's the rule20:08
rick_h_title case20:08
katco`rick_h_: title case, there we go20:09
rick_h_natefinch: katco` so title case should be Updates Available20:09
rick_h_my bad20:09
katco`ericsnow: natefinch: being on hold has it's advantages: http://reviews.vapour.ws/r/3948/20:43
ericsnowkatco`: nice20:43
natefinchkatco`: lol20:44
natefinchericsnow, katco`: gah, we need to implement FingerPrint.Equal :/20:50
natefinchericsnow, katco`:  that's the second place we'll be writing a custom comparison func20:51
ericsnownatefinch: IIRC, I did in my original proposed patch and you shot it down :)20:51
natefinchericsnow: who, me?  Saying we shouldn't implement things before we need them?  Doesn't sound like me ;)20:53
natefinchericsnow: for that one though, if I did say that, I was definitely wrong. That was pretty inevitable that we'd need it... and it's outside juju/juju, so it's annoying to add later.20:54
ericsnownatefinch: yep20:54
kiltyI'm able to successfully issue a 'juju bootstrap' on my openstack private cloud, but when the new instance starts up the service it returns: "index file has no data for cloud"  Any ideas?20:58
natefinchdoesn't sound familiar21:00
natefinchthumper: ^21:00
mupBug #1548938 changed: list-credentials doesn't include 'discovered' creds <docteam> <juju-core:Invalid> <https://launchpad.net/bugs/1548938>21:08
natefinchericsnow: fixed up my pr for updates available, adding support for doing it during --details as well.  please re-review.  I wrote my own updates function, but I'll wait until your code is in and overwrite mine with yours.21:08
natefinchericsnow: http://reviews.vapour.ws/r/3945/21:08
ericsnownatefinch: it's merging now, BTW21:08
natefinchericsnow: cool, I'll rebase once it's in.21:08
mupBug #1548938 opened: list-credentials doesn't include 'discovered' creds <docteam> <juju-core:Invalid> <https://launchpad.net/bugs/1548938>21:14
mupBug #1548938 changed: list-credentials doesn't include 'discovered' creds <docteam> <juju-core:Invalid> <https://launchpad.net/bugs/1548938>21:17
mupBug #1548938 opened: list-credentials doesn't include 'discovered' creds <docteam> <juju-core:Invalid> <https://launchpad.net/bugs/1548938>21:20
mupBug #1548938 changed: list-credentials doesn't include 'discovered' creds <docteam> <juju-core:Invalid> <https://launchpad.net/bugs/1548938>21:29
natefinchericsnow: here's my approach for returning unitresources for all units, would like your thoughts before I get too far into it: http://reviews.vapour.ws/r/3949/diff/#21:30
natefinch(though obv standup now)21:30
lazyPowerif anyone has a minute to riff on this - https://github.com/juju/charm-tools/issues/91 - is there any explicit reason underscores are not accepted in action parameters/names?22:55
=== philipballew is now known as Guest44851
jw4lazyPower: IIRC it's simply because that wasn't one of the characters allowed.  The 'allowed' characters were a pretty small subset in bodie's initial implementation.23:13
lazyPowercool, thanks jw423:22
perrito666axw: when you are around, could you re-check https://github.com/go-amz/amz/pull/66 please?23:23
axwperrito666: sorry, should have said LGTM with that in the first place. LGTM23:50
* axw disappears again23:50
axwbefore I do...23:50
axwwallyworld: FYI, still a bunch of tests to update for my branch :/   it's a bit of a monster23:51
perrito666axw: tx23:51
wallyworldaxw: quick chat? in standup channel?23:51
axwwallyworld: we were silently ignoring missing current-controller in controller commands. not ignoring == lots of errors23:51
axwwallyworld: ok23:51
axwperrito666: want me to merge?23:54

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