[00:10] screw git... [00:10] how to I reset hard? [00:10] ... [00:10] git reset --hard [00:10] leaves me with lots of untraced files [00:10] jw4: did that [00:11] hmm; were they added files? [00:11] how do I get rid of the untracked files? [00:11] from a screwed up pull [00:11] well if you don't have any changes you want to keep: [00:12] git rm -r --cached . [00:12] what does that do? [00:12] step 1 [00:12] I want the equivalent of "bzr clean-tree" [00:12] yep [00:12] wait [00:12] even easier [00:13] git ls-files -o --exclude-standard [00:13] | rm [00:13] *should* remove all the untracked files [00:13] the git rm -r --cached . removes all the files from the stage [00:14] but you just need "git ls-files -o --exclude standard | rm" [00:14] if you did do the rm --cached command you follow it with "git reset --hard " again to get your working copy back [00:15] jcw, thumper: does the "git clean" command not help? [00:15] menn0: ;-p [00:15] thumper: just do git clean [00:16] menn0: I've just spent hours figting line ending conflicts so my head was in the "cleanup stage" mode [00:20] thumper: did 'git clean -fdx' work? [00:23] thumper: http://pastebin.ubuntu.com/9108069/ save that as git-clean-tree (no .sh) in your path [00:23] thumper: you can the go `git clean-tree` [00:23] waigani: I'd prefer not to have a bunch of special aliases [00:23] it makes it hard to work with others [00:24] thumper: okay, it's just two commands [00:24] git clean -df worked [00:24] 1. `git clean -df` removes any [d]irectories and [f]iles not added [00:25] 2. git checkout . discards any uncommited changes [00:27] davecheney, menn0: I've hit a bug in manual testing that I can't reproduce (yet) in the unit tests on this openPorts, so I might not get to the other tasks for a bit [00:28] "on this openPorts branch" I mean === liam_ is now known as Guest89217 [00:48] kk [00:48] ah, menn0 it wasn't just the tests. st.Ports is used in the actual (1.21) upgrade step to get all the ports to upgrade. That explains the bug I'm hitting in manual testing. === Spads_ is now known as Spads [01:19] menn0: http://reviews.vapour.ws/r/497/ [01:19] menn0: if you want to look at it since you looked at the 1.20 one [01:19] menn0: actually, I'll propose it for trunk first [01:20] menn0: that is the 1.21 branch [01:24] menn0: this is the trunk comit - http://reviews.vapour.ws/r/498/ [01:25] thumper: looking [01:25] ta [01:38] * thumper is running all the tests... [01:38] again [01:38] had a missing import statement [01:40] thumper: review done [01:41] * thumper is wondering why the tests aren't seeming to progress... [01:42] normally they are done by now... [01:42] thumper: mongo pooped itself, then you have to wait for the timeout ? [01:43] thumper: and other review done [01:44] * thumper grunts [01:44] ffs [01:44] why are we testing this shit in two places [01:45] thumper: my favorite is the way that api/* and apiserver/* have the same set of tests [01:54] this is ugly ugly ugly [01:56] WT actual F [02:03] I have a test failure where it says it writes the file, but then says it doesn't exist... [02:03] I don't see anywhere where it removes it it between [02:05] and this time it panics [02:06] panic: API not available from this context [02:06] how can that work sometimes and not others? [02:12] fucking weird... the real upgrades work, but the tests for jujud fail [02:15] hmmmm [02:15] you can't destroy a 1.22 env with a 1.20 client [02:15] $ /usr/bin/juju destroy-environment --force --yes local [02:15] WARNING unknown config field "block-destroy-environment" [02:15] WARNING unknown config field "agent-metadata-url" [02:15] WARNING unknown config field "uuid" [02:15] WARNING unknown config field "prefer-ipv6" [02:15] WARNING unknown config field "set-numa-control-policy" [02:15] WARNING unknown config field "block-destroy-environment" [02:15] WARNING unknown config field "uuid" [02:15] WARNING unknown config field "agent-metadata-url" [02:16] WARNING unknown config field "set-numa-control-policy" [02:16] WARNING unknown config field "prefer-ipv6" [02:16] WARNING unknown config field "prefer-ipv6" [02:16] WARNING unknown config field "block-destroy-environment" [02:16] WARNING unknown config field "agent-metadata-url" [02:16] WARNING unknown config field "uuid" [02:16] WARNING unknown config field "set-numa-control-policy" [02:16] ERROR cannot find network interface "": net: invalid interface name [02:16] ERROR failure setting config: cannot find address of network-bridge: "": net: invalid interface name [02:16] 1.22 client works [02:17] wallyworld_: that MAAS API meeting is going to be difficult for me to attend, I have to take my daughter to school then [02:17] ah [02:18] ah... [02:18] * thumper pokes the mock [02:18] axw: would 30 minutes later work? [02:18] wallyworld_: worse :) 2 hours earlier would be good... [02:18] 1 hour earlier at a stretch [02:19] that clashes with the sprint schedule i think, i'll check [02:21] yeah, sprint sessions are earlier [02:21] wallyworld_: I could go an hour later too [02:21] maybe an hour later? that would be 7pm for them [02:21] * thumper stabs this tangled mess repeatedly [02:21] they can delay their dinner maybe, or eat earlier [02:21] i'll check with alexis [02:26] menn0: ok, need some help here [02:28] thumper: ok [02:28] thumper: standup channel? [02:28] yep [03:09] thumper, davecheney: here's the env UUID upgrade for the meterStatus collection [03:09] http://reviews.vapour.ws/r/499/diff/ [03:14] waigani: here's the meterstatus env uuid work http://reviews.vapour.ws/r/499/diff/ [03:15] * menn0 is off to run some errands [03:30] menn0: done === noodles785 is now known as noodles775 [03:55] menn0: http://reviews.vapour.ws/r/497/diff/# has been updated to unblock 1.21 release [03:55] menn0: I'll fix for 1.22 with different patch that splits out the work === Spads_ is now known as Spads [04:18] thumper: looking [04:25] thumper: i see you've already submitted so all good. [04:25] thumper: good idea to get this in and then do the split. [04:28] wallyworld_: ping [04:28] hi, otp, one sec [04:29] wallyworld_: np [04:38] menn0: hiya [04:39] wallyworld_: so I discovered earlier that a 1.20 client can't destroy a 1.22 env [04:39] wallyworld_: would you call that a bug? [04:40] i think so [04:40] yes [04:40] sigh [04:40] raise a bug and we'll fix it for 1.22 [04:40] there emitted error is: [04:40] ERROR cannot find network interface "": net: invalid interface name [04:40] ERROR failure setting config: cannot find address of network-bridge: "": net: invalid interface name [04:40] writing up a bug now [04:40] ok, so some new config was introduced in 1.22 [04:40] ta [04:41] 1.22 needs to be more robust to such situation [04:48] wallyworld_: here tis: bug 1394450 [04:48] Bug #1394450: 1.20 client can't destroy a 1.22 environment [04:48] ta [04:53] menn0:curiosity kills me - would this happen often that a client is of older version than environment? [04:54] menn0: i would have assumed that more often u'd encounter 1.20 env and 1.22 client, for eg... [04:54] menn0: apart from devs?.. [04:55] this also affect 1.21 [04:57] anastasiamac: good question. I guess it's not that common. I just noticed it by accident after I made a mistake while testing upgrades. [04:57] anastasiamac: still worth fixing if we can I think. [04:57] wallyworld_: i haven't tested with 1.21 but you're probably correct. [04:57] wallyworld_: let me check [04:57] menn0: of course ;-) whether fix is needed or not is not even a question :-) [04:58] this fix would be to make the network-bridge attribute have a default of lxcbr0 instead of "" i think [05:00] although it appears the code does use lxcbr0 if network bridge is "" [05:00] so i'm not sure off hand how this occurs === kadams54 is now known as kadams54-away [05:02] wallyworld_, anastasiamac : i've tried destroying with a 1.21 client and that works fine [05:02] so it looks like it's just the 1.20 client that has a problem [05:02] menn0: ah, i meant destroying a 1.21 env ith a 1.20 client [05:03] ok right [05:03] wallyworld_: i'll try that then :) [05:03] wallyworld_: it's coming from utils.network.go ;-( [05:03] not sure if the issue needs fixing in both 1.21 and 1.22 [05:04] network.go won't work though if we pass in "" for networkName [05:05] wallyworld_: we find network interface fine (i dont think we r passing "") [05:05] wallyworld_: we cannot find addrs for the network interface... [05:05] the bug seems to indicate otherwise [05:05] but i could be wrong [05:06] ERROR failure setting config: cannot find address of network-bridge: "" [05:06] seems to indicate that networkBridge := config.networkBridge() returned "" [05:07] wallyworld_, anastasiamac: as suspected, a 1.20 client can't delete a 1.21 env either [05:07] wallyworld_, anastasiamac: ticket updated [05:07] thanks menn0 [05:07] we'll need to fix this for the 1.21 release [05:07] wallyworld_: yep ;( [05:07] wallyworld_: and yes ;( [05:10] wallyworld_: as u suggested, config var "network-bridge" should probably b defaulted to something other than "" [05:10] yeah, do it at the schema level instead of in the getter [05:10] but, i'm not 100% sure why the current code doesn't work [05:12] wallyworld_: well, wher r we assuming that network bridge is lxcbr0 if we get ""? [05:13] wallyworld_: from what m seeing we r relying that network bridge is provided [05:14] before calling getAddressForInterface(), the network bridge value is gotten like so: networkBridge := config.networkBridge() [05:14] that should have resulted in a non "" being returned [05:15] so more digging is needed to find out why it is failing [05:15] wallyworld_: that's easy. network-bridge in config defaults to "" [05:16] yes, but the getter turns it into a value [05:16] the getter has code: [05:16] if name == "" { [05:16] return lxc.DefaultLxcBridge [05:17] wallyworld_: assuming instance.LXC [05:17] wallyworld_: can c.container b something other than LXC or KVM [05:17] no [06:03] Actions refactor PR - large, but much of it is mechanical. PTAL http://reviews.vapour.ws/r/502/ [06:04] fwereade_, TheMue ptal too ^^^ [06:05] but I welcome anyone's feedback :) [08:08] morning all [09:02] morning [09:03] jw4: will take a look (in case you're still awake *g*) [09:44] morning TheMue , mattyw [09:45] dimitern: o/ [09:45] dimitern, TheMue good morning! [09:46] does anyone know off hand how to file bugs for subrepos? I found an issue with juju/utils.. maybe I'll just file it against juju-core [09:49] dimitern: depends [09:49] i've logged them against the repo on GH [09:49] ... when it's thumpers stuff :) [09:50] davecheney, right ;) so I'll file an issue in GH and a link to a bug in LP then [09:50] dimitern: sgtm [09:57] davecheney: thanks for editing right for the diary doc [09:58] TheMue: no worries [09:58] i was surprised that @canonical didn't ahve edit already [09:58] ¯\_(ツ)_/¯ [09:59] urgh [09:59] davecheney: yes, I wondered too [09:59] y are none of you in the hangout ? [10:00] Y U NO JOIN HANGOUT [10:00] oh, now? sh** [10:00] joining [10:00] omw [10:02] fwereade_: team call ? [10:02] dimitern: when you get a chance [10:02] dimitern: http://reviews.vapour.ws/r/504/ [10:03] voidspace, will look in a bit [10:03] dimitern: and then I need to talk to you about "what next" [10:03] dimitern: there are several tasks that look achievable for me, but all with some "hand holding" :-) [10:04] axw: ping for team call ? [10:05] doh [10:05] jam: thanks [10:06] wwitzel3: perrito666: are you coming to the team meeting ? [10:06] mattyw, team call? [10:07] cold in my room - time to run the whole juju test suite and warm up the computer a bit [10:08] dimitern, mattyw has taken an early lunch [10:30] davecheney: thanks for the review [10:34] jam: sorry had to solve a couple of last minute issues did not make it [10:35] the public transport went on strike the same day the city is flooding with rain.. have to love these people [10:38] voidspace: no wuks [11:04] ahh crap, I guess the core team meeting moved an 3 [11:04] hour earlier? [11:05] stupid daylight savings [11:21] dammit, utopic broke fullscreen.... any app that goes fullscreen now lags like hell [11:22] natefinch: from my viewpoint it has not moved [11:22] perrito666: likely because your country does do stupid daylight savings crap [11:23] but 3 hs is a bit much [11:24] has anyone tried ubuntu.next ? [11:26] davecheney: like post utopic? [11:28] dammit now my mouse is invisible [11:28] yup [11:28] the one that jane talked about in the canonical newsletter [11:29] tried switching video drivers to see if that would help the fullscreen problem [11:29] unity 8 is the default [11:29] maybe you should ask first if anyone has read the newsletter ;) [11:30] I haven't heard of anyone using ijt. [11:30] natefinch: just pat down your desk, must be around [11:30] lol [11:30] :-) [11:30] brb gonna reboot [11:31] davecheney: I am curious about unity yet I do need some stability on the rest of my system [11:31] I'm willing, I'm a glutton for punishment ;) [11:40] perrito666: natefinch don't be a stooge, install it in a vm [11:40] you'll know what I mean === natefinch is now known as nate-tablet [11:41] well this is going well [11:42] can't log in to Ubuntu because it errors when trying to switch monitor config and dumpsme back at login [11:42] nice error handling there [11:45] is there a trick to login without the log in screen? [11:45] gotta enable auto login ? [11:45] that is part of gdm [11:47] or is there a way to with from the guest session? that log in works fine [11:47] s/with/with/ [11:47] auth [11:48] stupid tablet [11:51] * nate-tablet goes to google "switch display driver from terminal" [11:52] nate-tablet: so, the utopic upgrade went well then... [11:53] it might be mostly a driver issue and one egregious [11:53] ly [11:53] bad error handling [11:53] right [11:53] I hope you get it sorted [11:53] sounds like the opposite of fun [11:53] i don't mean to swear in church [11:54] but ubuntu only really works on thinkpads [11:54] it works great on my desktop [11:54] intel graphics ? [11:54] but then I picked all the components for compatibility and built myself [11:54] nope, amd I think [11:54] maybe I've just been lucky [11:54] driving four monitors from one card though [11:57] nate-tablet: enter a session in your shell [11:57] then DISPLAY=:1 startx& [11:58] then DISPLAY=:1 whateverthewmiscalledthesedays [11:59] davecheney: to be honest it sometimes fails on mine, but yes, I have a preference for thinkpads when using linux, just in case [12:01] davecheney - this is the first major problem I've had [12:02] (that wasn't caused by juju) [12:04] besides, I figured it's pretty safe using the same model laptop as Mark S ;) [12:05] nate-tablet: well, you seem to have a bit different one [12:06] nate-tablet: the right trick, is to have the same laptop linus torvalds has, so when everything else fails, at least the kernel still works [12:07] maybe he's not running two external monitors [12:08] nate-tablet: maybe, I can only run one and it works well enough [12:08] to solve that I am tempted to get a desktop machine, but I am to picky about hardware [12:13] nate-tablet: i like that qualification [12:13] ther is a reason i have a 8gb /tmp partition [12:13] in ram [12:13] (stupud test runner bug) [12:14] my two biggest problems with Ubuntu have been back error handling [12:14] bad [12:16] no etc/networks? no network! ....what ever happened to sane defaults? [12:16] nate-tablet: yeah, no desktop [12:16] no network manager [12:16] no network [12:16] catch 22 [12:46] any reviewers willing to take a look at a trivial fix for bug 1394524? https://github.com/juju/utils/pull/91 [12:46] Bug #1394524: juju/utils/apt retrying logic is poorly tested and can fail the second time [12:47] dimitern: will do [12:48] TheMue, cheers [12:49] jw4, bodie_, hey guys, if you can have a look perhaps ^^ as the fix might have some implications for windows? [12:51] dimitern: done [12:52] TheMue, thanks [12:52] rogpeppe, ping [12:52] yw [12:52] dimitern: pong [12:52] rogpeppe, hey, do you know if juju/utils is running under the landing bot? How to land a patch on that subrepo? [12:53] dimitern: i don't think so [12:53] dimitern: i can land patches there [12:54] dimitern: isn't the landing bot intended for repos that actually build binaries? [12:54] dimitern: (rather than just packages) [12:54] dimitern: i think the landing bot for juju-core should actually run go test github.com/juju/... [12:55] dimitern: or at any rate for the packages juju-core depends on [12:55] rogpeppe, yeah it does [12:55] dimitern: ah, cool [12:55] rogpeppe, ok, so how will you land it? [12:55] dimitern: i push the big green button :) [12:56] rogpeppe, ha :) ok, I'll try, thanks! [12:56] but first let's try a $$merge$$ comment to see if the bot will pick it up [13:21] folks - sorry I missed the core hangout - I have my reasons - but there is no excuse [13:22] mattyw: there is *no* excuse, you let us all down. In fact, we could barely even have the meeting at all without you. [13:22] :) [13:22] mattyw: happens, I did too, anyway I think there is no room for all of us [13:22] TheMue, another related trivial - http://reviews.vapour.ws/r/506/ ? [13:23] jam, most of all I let myself down [13:23] dimitern: ok, if it's trivial I can jump in. ;) [13:23] TheMue, :) it's just updating dependencies.tsv [13:24] dimitern: ic, and where is the unit test for it? *lol* [13:24] TheMue, :D I wish we had some [13:24] dimitern: also all those leading spaces/tabs [13:25] dimitern: somehow 6bfed5692f0d524bab3be5976f26d24bd4d3f677 is very self-documenting [13:25] TheMue, yeah, indeed [13:25] dimitern: so, done [13:25] TheMue, thanks! [13:25] dimitern: always a pleasure [13:26] * TheMue currently does another, larger review, so has been in the right mode [13:35] TheMue: I trust your other larger review is mine :) [13:36] jw4: eh, ehm, why do you think so? *grin* [13:36] TheMue: :) [13:40] jw4: you already work on daves feedback? [13:40] TheMue: I'm starting to go through it... [13:41] jw4: fine, it's catching most of the points [13:41] TheMue: a lot of his feedback is about things that are bigger than my change [13:41] TheMue: but I'll respond to each one in the PR [13:42] jw4: yep, so for some of the we may need to create a follow-up [13:42] kk [13:53] hmm, so i've just discovered that juju deploy local:trusty/foo can totally ignore the directory $JUJU_REPOSITORY/trusty/foo if there's another charm in there with Meta().Name == "foo" [13:54] fwereade_: does that seem right to you? it certainly surprised me [13:54] rogpeppe, it's insane but As Designed :/ [13:54] fwereade_: can we just change it please? :) [13:55] fwereade_: can you think of something it might break? [13:56] rogpeppe, well, it opens the issue of meta name not matching the url we use [13:56] fwereade_: we could make it complain if the meta name didn't match the dir name [13:57] rogpeppe, it's never been *quite* annoying enough to reach the top of the list; in principle I'd be glad to see it improved; in practice I think we'd need to talk to the people who actually use local repos [13:57] fwereade_: i think that having meta name in there is silly anyway, but at least then we wouldn't be in such a ridiculous situation (i created the directory, deployed the charm, then wondered wtf was going on) [13:58] rogpeppe, the other reason not to do it is the hope that we'll be able to transition to just publishing directories directly, and forget the whole local repo thing entirely [13:59] fwereade_: i'm guessing we'll still want a local repo [13:59] fwereade_: i guess it depends how important the naming-after-series thing is [14:00] rogpeppe, don't we end up needing series in the metadata in the general case though? [14:00] fwereade_: that is, as we know, a controversial topic :) [14:00] fwereade_: FWIW I'm +1 [14:00] rogpeppe, well, it's only lp-based charms where we get the magic series-from-name thing -- where else is it going to come from? [14:01] rogpeppe, I'm probably just blanking, but I don't recall the controversy [14:01] rogpeppe, multiple-series charms are controversial for usre [14:01] rogpeppe, (lp-based and local-based, I suppose, but local is just aping lp, I think) [14:02] fwereade_: except... all the official charms are multiple series AFAIK. [14:02] rogpeppe, IMO the only ones where that's really defensible are the subordinates [14:03] fwereade_: i'd like to robustly defend the ability to create portable software :) [14:04] fwereade_: and we're not even talking portability across different unix flavours here [14:04] rogpeppe, I'd like to dismissively pooh-pooh the notion that anyone cares what OS they use, so long as their workloads function properly [14:04] ;) [14:04] rogpeppe, subordinates come crashing hard against that statement though [14:05] fwereade_: i care that i understand something about the OS so i can debug it. i.e. not windows.. [14:05] fwereade_: yeah, subordinates imply we want a single service that can be deployed to multiple series [14:05] rogpeppe, and if we can fix it for subordinates, we can fix it for everything [14:05] rogpeppe, it's just not looming particularly large in my mind atm [14:06] rogpeppe, because, well, subordinates potentially need to be *really* cross-platform [14:06] fwereade_: it's something that's looming large in my mind, with some possibly significant changes in the charm store needed [14:06] fwereade_: well, yes [14:07] fwereade_: i'd like to take the series out of the charm name entirely. that would help quite a lot of things. [14:09] rogpeppe, series feels like a pretty useful part of a charm url tbh [14:09] rogpeppe, but I'm looking at it from a different perspective, I suspect, keen to hear more [14:09] [14:04:13] rogpeppe, I'd like to dismissively pooh-pooh the notion that anyone cares what OS they use, so long as their workloads function properly [14:09] :) [14:09] fwereade_: it assumes that the OS is just as important as the workload [14:10] rogpeppe, "juju deploy whatever" [14:10] fwereade_: yup [14:10] rogpeppe, *we* have to care about the charm url, so that our users don't ;) [14:10] rogpeppe, the series, I mean [14:10] rogpeppe, and having the series part of the url is convenient for us, at least [14:11] fwereade_: yup. that, for me, means that the URL (*the* user-visible part of a charm) shouldn't contain the series [14:11] fwereade_: it's *kinda* convenient for us, but i don't think we gain that much from it [14:12] rogpeppe, well, we want some way to distinguish between charms for the same workload with different targets, don't we? [14:12] rogpeppe, series does that [14:12] fwereade_: isn't that what constraints do? [14:12] rogpeppe, and it's easier to read than a hash or something [14:13] fwereade_: i see series as just one aspect of the "aspects of the target that i care about" continuum [14:14] rogpeppe, the centos, windows, and ubuntu apache charms will actively differ in a number of respects, though, right? [14:14] fwereade_: oh yes. but that's our problem - the user should be oblivious to their differences, i think. [14:14] rogpeppe, distinguishing by version number is terrifyingly opaque [14:15] rogpeppe, and jamming the target into the name feels strictly inferior to making it a separate component [14:15] rogpeppe, I can't really see other options [14:16] fwereade_: i think i'd provide a way to publish several charms (with different series) under the same name, and let juju select the appropriate one. [14:16] fwereade_: that's *kinda* what happens already [14:16] rogpeppe, right, but if you happen to deploy the same workload on different OSs, you want to be aware that they're running different charms [14:17] fwereade_: i'm not sure [14:17] fwereade_: only when things go wrong [14:17] fwereade_: but there are all sorts of things you want to know in that case [14:19] rogpeppe, well, they *are* different charms... if every charm "url" really points to an opaquely context-sensitive redirect (to something without a url?), I think it will be... confusing [14:20] fwereade_: i guess it depends how much you care about the workload vs how it's implemented [14:21] rogpeppe, for me it's more about a charm url being unambiguous [14:22] rogpeppe, I *don't* actually care about the series, any more than I care about the revision [14:22] rogpeppe, but both those components disambiguate usefully === katco` is now known as katco [14:24] fwereade_: in the end, what the charm *does* can be arbitrarily different depending on platform anyway, regardless of what the contents of the charm are [14:24] dimitern, does bug 1394524 need to merged into the 1.21 branch? [14:24] Bug #1394524: juju/utils/apt retrying logic is poorly tested and can fail the second time [14:24] rogpeppe, as it can be depending on revision too, right? [14:25] fwereade_: yeah. [14:29] sinzui: ping === enterprisedc_ is now known as enterprisedc [14:31] hi anastasiamac [14:32] sinzui: about bug 1394450 [14:32] Bug #1394450: 1.20 client can't destroy a 1.21 or 1.22 environment [14:32] sinzui: m not convinced it needs backporting [14:32] :) [14:32] sinzui: different version had different defaults for network bridge names [14:33] sinzui: this ione is specifically about default in 1.22 changing to "" [14:34] sinzui:1.22 works fine because we specify container specific defaults in the getter [14:34] anastasiamac, so 1.22 needs to change to allow any older juju to destroy it? [14:34] sinzui: however, older clients don't use getter but rather schema default and hence get "" [14:34] TheMue: responded to davecheney 's feedback - any further feedback on this PR? http://reviews.vapour.ws/r/502/ [14:35] sinzui: kind of - 1.22 needs to set container specific defaults when it sets up env config [14:35] fwereade_: I'd appreciate your review of ^^ too [14:35] sinzui: so I am hoping that it's just the change on 1.22 ;-) [14:35] me too. thank you anastasiamac [14:36] sinzui: gr8 :) thnx! [14:36] jw4: so far mostly more simple comments, dave already found most [14:36] tx TheMue [14:37] jw4: will submit in a few minutes [14:37] kk [14:39] sometimes diffs are really strange, when new functions are shown as placed inside an existing function and only two lines of the old code are shown afterwards [14:39] TheMue: I know, I've noticed that too :( === kadams54 is now known as kadams54-away [14:54] jw4: so, published my comments and now go through your answers to daves comments [14:54] TheMue: perfect [14:57] jw4, sent a few thoughts/questions [14:57] fwereade_: excellent, thanks [14:59] fwereade_: if i do a juju upgrade-charm --force when a unit is an error state, should i expect to get an upgrade-charm hook called at some point in the future? [15:02] rogpeppe, no [15:03] rogpeppe, if upgrade-charm is important you basically have to prefix all your hooks with an effective maybe-upgrade-hook :/ [15:03] fwereade_: how do you know if the charm has been upgraded? [15:04] rogpeppe, heh. [15:04] rogpeppe, copy the charm url file somewhere in the upgrade-charm hook? [15:05] rogpeppe, not that, as a charm author, you're really meant to know about or depend upon that file [15:05] fwereade_: yeah, i didn't know that existed [15:05] fwereade_: i guess i could hash the contents of the charm directory [15:05] fwereade_: or look at the mtime of the metadata.yaml file [15:06] rogpeppe, or, hmm, copy the revision file somewhere? [15:06] rogpeppe, although that *could* fail if you --switch [15:06] fwereade_: is there guaranteed to be a revision file? maybe, i guess [15:06] rogpeppe, in a deployed charm, yes, I think so [15:07] fwereade_: yeah, doesn't seem too reliable given switch === liam_ is now known as Guest73106 [15:08] fwereade_: perhaps this is worth fixing. i think it may be woth guaranteeing that upgrade-charm will be called when the charm is upgraded [15:09] fwereade_: the problem is there's no decent way around it. if a charm dies in a relation hook and the only way to fix it is to upgrade the charm, there's no way of triggering the upgrade-charm hook *and* retrying the relation hook AFAICS [15:12] nate-tablet: did you fix your computer? [15:12] I'm still on my ta let [15:12] tablet [15:13] support is trying to help [15:15] rogpeppe, just a mo, sorry, laura just got home [15:15] fwereade_: np [15:16] fwereade_: at your convenience, after reading my responses to your review, we can chat about your issues raised? [15:18] rogpeppe, so, yes, that was essentially the issue [15:18] rogpeppe, we don't really want to run upgrade-charm in the middle of an incomplete, errored hook [15:19] fwereade_: my expectation was that you'd upgrade the charm, retry the hook, and then upgrade-charm would be run [15:19] rogpeppe, really, honestly, there are only two answers -- either don't depend on upgrade-charm, or make your charm bulletproof enough that no user will have to force-upgrade it :/ [15:20] rogpeppe, yay preserving compatibility [15:20] rogpeppe, fwiw that *is* documented, I'm pretty sure [15:20] rogpeppe, not saying it's *good* though [15:20] rogpeppe, ok, that is not an unreasonable expectation, but let's dig in [15:20] rogpeppe, if we do that [15:21] rogpeppe, we're implicitly denying the need for an upgrade-charm to ever happen, aren't we? [15:21] fwereade_: i wouldn't mind your "treat every hook as if it might be run after an upgrade" solution *if* there was a way of finding out whether a charm's been upgraded [15:21] rogpeppe, because we're assuming that any hook can be rerun safely after an upgrade *without* an upgrade-charm [15:21] fwereade_: yeah [15:21] fwereade_: i'd realised that [15:23] rogpeppe, it would really feel *safer*, I think, to require that upgrade-charm be able to handle running in any state... [15:23] fwereade_: yes, and then just run it and leave the current error state unaffected [15:24] rogpeppe, but then we guarantee that config-changed will run after upgrade-charm, so we can either relax *that* restriction or raise the spectre of a double-nested error state with a queued config-changed lurkingin there somewhere too [15:24] fwereade_: ha [15:24] rogpeppe, happy fun times, eh? [15:24] fwereade_: happy fun times indeed [15:25] fwereade_: when we have charm status, i'm going to write charms such that their hooks never fail [15:25] rogpeppe, +100 to that [15:25] nate-tablet: FYI, I'll be here sporadically again today (feeling a bit better) [15:29] ericsnow: ok [15:30] now I have managed to install lubuntu (which didn't help) and can't get it to go away [15:31] * nate-tablet is a special flower [15:32] my upgrade went about the same as nate's [15:32] looked ok, then just progressively degraded everytime i used my laptop, until I had to reinstall [15:34] luckily I just backed everything up to a USB drive [15:34] so if worse comes to worse..... [15:45] nate-tablet: borked upgrade from 10.04 .10? [15:45] jw4, yeah, I think all those things are fine for followups [15:46] fwereade_: okay; I'll add issues to our leankit board and drop the issues here [15:47] jw4, a thought about the notifications, though -- would it be sane to encode the statuses in the notification _ids, and to add unconditional remove-other-possible-notification-kind ops to the transactions that notify? [15:47] fwereade_: interesting [15:47] jw4, that might be insane or it might be brilliant or it might just be meh, I just want us to think about the possibility space a bit here [15:47] rogpeppe: ping [15:48] fwereade_: yeah I like it. Requires a little more bookkeeping but seems worth it [15:48] jw4, and saves us the expensive-style watcher [15:49] fwereade_: and the beauty is there is a logic hole in the expensive watcher that would be addressed by this approach [15:49] rogpeppe: could you tell me a bit more about the yaml change so that I change the test? [15:49] TheMue: pong [15:49] * fwereade_ cheers [15:49] TheMue: you need to change the tests so that they don't depend on the specific marshaling details of YAML [15:49] TheMue: have you run the tests yet, with the latest version of yaml.v1 ? [15:50] rogpeppe: no, want to start now [15:51] rogpeppe: afaik today the tests are based on yaml strings. so it would be better to let them base on types that are marshalled and unmarshalled? [15:51] TheMue: no, you don't need to do that in general [15:51] TheMue: i pointed you to a possible solution yesterday [15:52] rogpeppe: have to see if I have a log [15:54] jw4, actually, though [15:54] jw4, I am wondering about the nolongeravailable thing [15:55] fwereade_: as in what kind of error it should be? [15:55] jw4, it feels kinda overspecific -- should we maybe be using NotFound directly there? [15:55] rogpeppe: ah, the YAMLEquals checker, found the log ;) [15:55] TheMue: yeah [15:55] fwereade_: that sounds plausible [15:56] jw4, it will be all the clearer, I think, when we call it PendingActions [15:56] jw4, agree? [15:56] fwereade_: +1 [15:59] sinzui, hey [15:59] sinzui, I'll backport the fix for bug 1394524 into 1.21 as well [15:59] Bug #1394524: juju/utils/apt retrying logic is poorly tested and can fail the second time [16:00] Thank you dimitern [16:04] sinzui, hmm looking at how much 1.21 has changed from 1.22 wrt juju/utils/ it might be safer not to backport perhaps? [16:05] dimitern, I respect your decision, and I am happy to remove the issue from the milestone if you think it is best [16:05] sinzui, juju/utils for 1.21 is more than 10 commits behind master and some interface changes were introduced during some of these [16:06] sinzui, yeah, thanks - I think it's safer [16:07] okay [16:15] dimitern: ping [16:15] dimitern: you still around? [16:16] voidspace, yep [16:18] huzzah [16:18] and the answer is... fucked up .profile [16:19] no idea why that caused the whole "could not switch monitor configuration" error... but there was a little .xsession-errors file in my home directory that said "your .profile is fubared" [16:20] /usr/sbin/lightdm-session: 30: /home/nate/.profile: source: not found [16:20] /usr/sbin/lightdm-session: 32: /home/nate/.profile: function: not found [16:20] /usr/sbin/lightdm-session: 35: /home/nate/.profile: __git_ps1: not found [16:20] /usr/sbin/lightdm-session: 36: /home/nate/.profile: Syntax error: "}" unexpected [16:20] this is what I get for copying a shell script into my .profile [16:21] natefinch: wb :) [16:22] ironcically, while this all happened, my mother called me to ask for help getting malware off her laptop [16:22] one of those "malware masquerading as anti-malware" things [16:31] fwereade_, TheMue LGTM on review 502? [16:37] fwereade_: just seen your all-watchers-in-one-file-statement in jw4's proposal, laughed loud [16:40] jw4: from my side it's ok now as the cards for the missing parts are created. but would like to wait for fwereade_ to check your answers on his review [16:42] jw4, if you've done what I suggested and it LGT TheMue I'm happy [16:43] fwereade_ TheMue cool tx [16:44] (yes, I've addressed all comments and issues) === kadams54_ is now known as kadams54-away === kadams54-away is now known as kadams54_ === kadams54_ is now known as kadams54-away === kadams54-away is now known as kadams54_ [17:54] night all [18:11] cmars: http://juju-ci.vapour.ws:8080/job/github-merge-juju/1368/ plus 1369 plus 1370 [18:33] * perrito666 is back to the land of the bad but bearable internet [18:54] g'night all [19:06] does anyone have a minute to help debug proxy issues with a users juju env? [19:15] lazyPower: you're on the wrong time of day to find people that know about proxies and juju, unfortunately [19:15] i figured as much - but i think its a config issue fortunately [19:15] not something juju isn't respecting [19:16] thanks for the reply natefinch [19:16] lazyPower: wish I had a better reply. [19:21] wiii, I have an upgrade resistent restore, I am a happy man [19:21] :D [19:22] yay [19:26] fwereade_: don't know if the new name I picked is the best, but it's clearer : http://reviews.vapour.ws/r/508/ [19:30] go fmt not taking a list of paths as parameter makes me sad [19:31] you can do go fmt ./... [19:32] (or other package wildcards) [19:32] natefinch: that I did not know [19:32] note, that's go fmt not gofmt [19:33] natefinch: http://reviews.vapour.ws/r/298/ [19:35] perrito666: also, you can give multiple paths to gofmt gofmt *.go foo/*.go seems to work, for example [19:35] (note that one is gofmt, not go fmt [19:35] natefinch: go fmt, which I prefer does not like multiple paths === kadams54 is now known as kadams54-away [19:56] well, being blessed as I am with having no power and using internet from a cellphone tethering I think I might call a stop here and return later [19:56] wow, reviewboard sucks badly in bad internet situations === kadams54-away is now known as kadams54 === kadams54 is now known as kadams54-away [20:29] not sure if waigani is trolling or actually means it [20:31] natefinch: cmon it wont affect you unless you decide to code in an old english typeset [20:31] :p [20:33] natefinch: I put it to the vote, and this was the result. It's a stupid minor point and the best way to avoid ongoing time loss on the issue is to reach a decision and get on with coding. [20:35] waigani: isn't the best way to avoid wasting time on it.... to not waste time on it? [20:36] natefinch: isn't that what we are doing now? [20:36] natefinch: or to make a decision and not revisit it? [20:36] +1 [20:36] waigani: my point is, we're adding something else we have to check for in a code review that offers ZERO benefit to the quality of the code. [20:36] or to the quality of the comments [20:37] natefinch: except that new folks not party to this debate may raise it again unless it's spelled out [20:37] is this about the one-space after periods? [20:37] katco: yep [20:37] why is this something we're even discussing? [20:38] I have never (previously) in my 15 years of development experience had anyone ask about number of spaces after a period. [20:38] natefinch: sigh. The reason I put it to the vote is because there were debates about it, that were distracting. The solution to such debates is to make a decision and not revisit it (without good reason) [20:38] that is the role of the style guide I think [20:39] i'm with nate on this... my solution would be to tell the person bringing this up that there are more important things to discuss [20:39] why are we reviewing the # spaces in a comment? [20:39] katco: because it's fun! [20:39] katco: except you'd have to say it again everytime someone who didn't know asked [20:39] OMG [20:40] yay bikeshedding [20:40] jw4: i don't know... i've never seen this ever come up either [20:40] if we want to end the discussion, state in the style guide that we don't care [20:40] the intention was to put the issue to bed and move on - that is what I'm doing now. If it does not get a ship it does not land. I'm not fussed. [20:40] katco: I know, but Go is the first language I've used that has one-true-way of formatting code too [20:41] katco: for this exact reason I believe [20:41] jw4: yeah, but that's to handle major stylistic differences in actual code [20:41] jw4: this is... a space... b/t sentences [20:41] jw4: all languages have one true way of formating code... mine [20:41] jw4: writers can't even agree on this lol [20:41] katco: :) [20:42] I for one favour one sentence per line [20:42] natefinch: i really like that stance. "we don't care" [20:42] natefinch: removes the additional review point, and puts the issue to bed as well [20:42] we don't even have style guides for things that actually kinda sorta matter, like formatting long lines of function declarations [20:42] natefinch: I missed that - I'm okay with that stance too [20:42] "implementation defined" [20:42] natefinch: +1 +1 +1 [20:42] we just said "as long as it passes gofmt, it's ok" [20:43] waigani: i'm sorry if i'm causing you some pain. this isn't directed at you at all [20:43] waigani: i was just very surprised to see this [20:44] katco, natefinch my main support of this is to address it so that newcomers know what the stance is. "We don't care" is great if it's in the docs. (IMVHO) [20:45] let's make it a tab after a space, that way the spacing is customizable to everyone's own tastes ;) [20:45] sorry, tab after a period [20:46] natefinch: oh no you didn't [20:46] :) [20:46] lol [20:46] I have never seen someone dont care so strongly about something [20:46] perrito666: :) [20:46] tabs are better than spaces, but only if you're using the best editor: emacs [20:46] ^ the most contentious sentence ever written [20:46] katco: oh no you didn't [20:46] even more this seems to be the longest not argument about something I have ever been [20:47] perrito666: i respectfully hear your viewpoint! [20:48] for the record, if we have to pick one, I do prefer 2 spaces after a period. But I'd much rather just let everyone do whatever they want. [20:48] oh, no, I feel offended, I demand that you ignore me in the spirit of this conversation amd I will definitely not say that using vim I dont care because I can easily turn one into the other [20:48] natefinch: I think the ship sailed [20:48] natefinch: i believe in the annals of typography, one space is actually correct [20:48] natefinch: 2 spaces? who are you, guttemberg= [20:48] ? [20:49] but i guarantee you i'm not going to check for this lol [20:50] katco: lets ignore typography in favor of proper language and just read thoroughly the comment as we should [20:50] said by the guy that sucks at english <-- [20:50] haha [20:50] i've found your english to be quite good perrito666 [20:50] better than some americans [20:51] katco: heh, most of the reviews I get are people correcting my grammar [20:51] also my english is like a health bar in videogames, certain things can make me loose it [20:51] well, you know... go is a unicode language. write in Portuguese :) [20:51] perrito666: "most of the reviews I get are *from* people correcting my grammar" [20:52] katco: my kb lacks a few chars for it I believe [20:52] perrito666: i'm actually afraid of that happening. right now code is wonderfully unified with latin characters [20:53] i read code this japanese guy wrote all the time. in the future, maybe it's written in hirigana [20:53] and that avenue will be closed to me [20:53] katco: here, code in spanish is is usually considered bad [20:53] i mean i know it's a bit ethnocentric, but it is nice that anyone anywhere can read any code [20:54] katco: no offense intended, we consider English to be a simple, undecorated and sufficiently small language to be practical in technical things [20:54] I think it's pretty great that people in other languages can write code that uses their own alphabet. [20:54] I do find it quite amusing that the standard time format in Go is in US month/day/year order, though [20:54] perrito666: none taken... wow is that really true? [20:55] perrito666: funny, some people say that same thing about Go :) [20:55] katco: well spanish has accents and this -> ñ and º for degrees and many things like the fact that words have gender [20:56] which makes a pain to code and comment in it [20:56] i see [20:56] and also, you most likely never thought of this because you speak english but programming languages are also in english :p [20:56] natefinch: that is odd [20:56] which makes constructions in prog lang+my lang very ugly [20:57] it will be interesting to see what happens as time goes on [20:57] katco: rob pike apologized for it, he just wasn't thinking. The canonical date is 01/02 3:45pm 2006 tz +7 [20:57] I spent some time reading code in spanish, and the SQL sentences where hilarious [20:58] (that's jan 2) [20:58] natefinch: i kind of like how they implemented the parsing stuff, but i wish they would have picked a date that is easier to remember [20:58] natefinch: oh god [20:58] Is there any rationale why you ppl use mm/dd/yy? [20:58] natefinch: and as i say that i just now realize it's 1-8 [20:58] ^7 [20:58] yep [20:59] yuk yuk [20:59] good job katherine [20:59] perrito666: i don't know that there is... it's just what i grew up with [20:59] I like the idea, but the actual date is hard to remember unless you know the order in which the timestamp parts are put in the canonical string [20:59] i prefer coarse to fine, e.g.: yyyy-mm-dd [20:59] katco: absolutely [20:59] katco: +1 [21:00] it's just like numbers... i don't know why we don't do that [21:00] i think that is the ISO standard though isn't it? [21:00] there's a lot of ISO standards for time :) [21:01] katco: I belive so. I try to always use that format : writing cheques, notes, etc. [21:01] a better time format might be 9999 08 07 06:54pm tz +3 [21:01] jw4: i admit i use what i grew up with, unless i'm producing documents for the interwebs [21:02] for august 7th, 9999 [21:02] katco: I grew up with dd/mm/yyyy and I still have to make a concious choice every time I write a date [21:03] I just noticed I can (possibly?) delete waigani's review request on reviewboard. I guess there's no permissions for that kind of thing? [21:03] natefinch: what about 9988 07 06 etc. [21:03] jw4: don't need it, you only use the amount of resolution in the year as you get, so 99 would be a 2 year date [21:03] natefinch: review by fiat? [21:04] ericsnow: ping? alive? [21:04] natefinch: "i DIS-AGREE!" (delete) [21:04] katco: I wouldn't, I just thought it was amusing that it was possible :) [21:04] natefinch: interesting [21:05] haha, I AM willing to change the title of the review, though [21:05] thumper, ping [21:05] alexisb: hey [21:05] hey there thumper I have a couple of things I need to discuss with you [21:06] can you spare me a few minutes [21:06] ? [21:06] yep [21:06] ok, let me find a corner [21:07] ok thumper our 1x1 hangout [21:07] ok [21:10] sinzui: slight typo in that email subject ;) [21:11] yeah, we well need a few years to get to that number :) [21:30] davecheney: do you have any more details on how you're reproducing bug 1394066 ? [21:30] Bug #1394066: FAIL: action_test.go:254: ActionSuite.TestActionsWatcherEmitsInitialChanges [21:57] if api/* is finding my facade, but not my method in apiserver/*, what have i most likely overlooked? [22:02] thumper: can we delay by 1 hr today? I have a meeting clash [22:02] wallyworld: sure [22:02] \o/ === arosales__ is now known as arosales [22:22] oh fuck... [22:22] fuckity fuck fuck [22:22] * thumper head desks [22:22] mooommm; thumper is doing it again [22:24] ah, no, panic over [22:24] haha [22:24] i think we're still good [22:24] * thumper goes back to the 1.22 upgrade malarky [22:25] * fwereade_ driveby: any less than 7 spaces after a period is unacceptable [22:25] fwereade_: smart alec [22:28] fwereade_: but surely it's 8.5 if you're using a comic sans font? ;-p [22:29] fwereade_: you mean "fewer"; spaces are countable [22:29] menn0, comic sans users have to use at least 37 as a penance [22:29] thumper: are all the docstrings in factory.go wrong? they all say, "If more than one params struct is passed to the function, it panics." [22:29] * fwereade_ fails at pedantry forever, tips hat to benji [22:30] :) [22:30] menn0: yeah [22:30] menn0: obviously the person that changed the factory failed to update the doc strings [22:30] thumper: but there's no way to pass in more param structs [22:30] thumper: is that left over from a previous implementation? [22:30] yep [22:32] thumper: k, i'll fix. i'm in there anyway [22:34] oh god... [22:35] * thumper just had a horrible thought [22:35] * thumper tries to work out if he cares [22:36] shit shit shit [22:36] bugger [22:36] balls [22:36] * thumper pretends the problem doesn't exist [22:37] * jw4 can't help but be intrigued when thumper goes off [22:37] thumper: that works... sometimes [22:37] it's like a shiny object or a squirrel to me [22:48] asking again in the hopes someone can help me out: if api/* is finding my facade, but not my method in apiserver/*, what have i most likely overlooked? [22:48] fwereade_: ^ ? [22:48] katco: is it a new end point? [22:48] thumper: it is [22:48] then you have probably forgotten to hook up the end point with the root [22:49] thumper: i've been using the usermanager as a guide... can you point me to a line of code maybe? [22:49] * thumper goes to look [22:50] thumper: ty sir. hopefully i am providing a distraction from what sounds like fun problems :) [22:50] thumper, waigani: here's Factory.MakeEnvironment() https://github.com/juju/juju/pull/1201 [22:53] katco, thumper: if you're seeing the facade but not the method, failing to register seems unlikely, and I don't immediately have good ideas... mm, versioning maybe? [22:54] fwereade_: i registered as 0 [22:54] * thumper goes back to his horrible problem [22:54] thumper: haha i'm sorry. ty so much for responding [22:54] fwereade_: common.RegisterStandardFacade( [22:54] FacadeName, [22:54] 0, [22:54] NewLeadershipServiceFn(leaderMgr), [22:54] ) [22:54] where FacadeName is "LeadershipService" [22:55] perrito666: you still looking for me? [22:55] fwereade_: the only thing i could think of is i am returning an interface, so maybe the reflection is failing [22:55] fwereade_: so i returned a pointer to the actual type, and that didn't work [22:56] fwereade_: but the type is not exposed [22:56] fwereade_: not sure if that matters [22:56] katco, I don't *think* the type matters so long as the methods are exposed [22:57] fwereade_: they're there on the interface alright [22:57] ericsnow: nope, solved it [22:58] :D [22:58] fwereade_: i should have mentioned this; i tracked the failure to rpc/rpcreflect/type.go Method(...) [22:58] \o/ [22:58] fwereade_: line 180 [22:59] menn0: done [23:00] * fwereade_ scratches head expressively at katco [23:00] katco, hold on a moment [23:00] waigani: thanks [23:01] katco, I'm unconvinced that we need a new facade for what you're doing -- but then I don't *actually* know *exactly* what you're doing [23:01] katco, oh wait scratch that [23:01] katco, we agreed leadership could & shoudl be handled by a worker that's not the uniter, hence a separate facade [23:01] katco, ignore me [23:01] * katco always prefers to have hard conversations when it's late where fwereade_ is [23:01] ;) [23:03] in vino veritas, or sometimes just nonsense [23:03] ah ffs [23:03] haha [23:05] fwereade_: is the translation of that "in wine we find the truth?" [23:06] thumper, yeah :) [23:06] I agree [23:06] bit early for wine here [23:06] being just after noon [23:06] but I'm tempted [23:09] thumper: if you get a moment, you you please look at https://code.launchpad.net/~wallyworld/golxc/support-cmd-env/+merge/242438 [23:09] wallyworld: kinda deep in it right now [23:10] np [23:10] fwereade_: ahoy... all the functions live in the "discarded" array [23:10] katco, ahhh [23:10] fwereade_: they feel so abandoned :( [23:10] katco, you will have failed to have "appropriate" signatures [23:10] fwereade_: yeah well the troublemakers are always the interesting ones [23:10] fwereade_: how do i know what's appropriate? [23:11] katco, I feel somewhat justified in my ages-ago belief that this would cause someone trouble sometime [23:11] katco, looking, oping I can remember [23:12] fwereade_: i think in rpc/rpcreflect/type.go newMethod(...) there lies a truth [23:12] katco, look at newMethod in rpcreflect/type.go I thinl? [23:12] first! [23:12] katco, jinx [23:12] first again! [23:12] * fwereade_ is evidently outclassed [23:12] ahaha [23:13] fwereade_: i will pursue this. thank you, as always, for your support :) [23:14] katco, cheers :) [23:14] fwereade_: do you mind if when i find out what is wrong i add some helpful error message? [23:14] katco, go for it [23:15] fwereade_: it would appear i do not fit the template for # of return values or some such thing [23:15] * fwereade_ looks at the code he was writing before he started the evening, remembers he decided to change the signature of an abundantly-used method, grumbles quietly to himself [23:16] katco, it's either (result, err) or (err) IIRC [23:16] katco, I think it would have been nicer if it'd been (err) only tbh, at least that would have matched the golang rpc package [23:16] * fwereade_ shrugs, resolves not to whine further [23:17] menn0: ship it [23:18] fwereade_: gosh... is it because i'm using *params.Error, and not *error? [23:18] katco, ha [23:18] katco, most likley [23:18] fwereade_: so now i am thoroughly confused. i thought i was supposed to use params.Error b/c that's our over-the-wire protocol? [23:18] katco, the params.Errors should probably be in a slice in the result somewhere [23:19] fwereade_: aha [23:19] katco, errors out of the actual methods are OMG-EVERYTHING-IS-BROKEN errors [23:19] fwereade_: right [23:19] fwereade_: thought we were still supposed to use params.Error for some reason [23:21] katco, I *think* that's handled elsewhere -- it's the distinction between a this-element-is-problematic (where we expect you to have created a sane result) and this-whole-call-is-broken where the transport does something about it [23:21] * fwereade_ waves hands vaguely, tries to look wise [23:21] fwereade_: mmm yes, i see, i see. i do not comprehend your brilliance of course, but i think i can fix this. [23:23] menn0: FWIW the jujud upgrade test starts with a version of 1.16.something and upgrades to current [23:23] menn0: and makes sure everything passes [23:24] menn0: so it is testing full upgrades [23:24] and jumping versions [23:25] thumper: that sounds good [23:25] thumper: given that Factory is your baby, can you PTAL at http://reviews.vapour.ws/r/511/? waigani has already given it the ok. [23:25] ericsnow: if you have some time, many of your changes just pushed to http://reviews.vapour.ws/r/298 [23:25] menn0: ok [23:26] ericsnow: make notes for all you feel needs moving so we do it after this lands [23:30] * thumper bangs and the desk and leaves the office for a bit [23:31] thumper: before you go [23:32] bangs hard enough that the desk leaves the office? [23:32] that's some serious table flip [23:33] thumper: never mind