[05:46] hey hey [05:46] the good news, juju can now bootstrap from quantal to precise [05:46] the bad news, do-release-upgrade on my precise system totally hosed it :( [07:07] davecheney, heyhey [07:08] fwereade: hey [07:09] davecheney, reviewed one, with unhelpful whining about tests, I'm afraid [07:09] understood [07:09] it is a non trivial amount of work to add 409 response behavior to s3test [07:09] i'll man up and do it [07:10] davecheney, on the other, trying to figure out what the deal is with the arch change in cloudinit_test [07:11] davecheney, thanks very much :) [07:11] davecheney, I expect it makes sense somehow, but I just cannot figure out where that i386 came from [07:14] fwereade: i386 is just to double check that cloudinit isn't amd64 only [07:15] davecheney, I just can't see what change caused it to use i386 where once it used amd64 [07:15] fwereade: i can remove the i386 thing if it is confusing [07:15] it isn't testing anything important [07:16] davecheney, sorry, I have just come to realise I am being stupid [07:17] davecheney, so that one basically LGTM -- but the description should be fixed if it is now actually intended for submission ;) [07:18] fwereade: cool [07:18] that is the one i really care about [07:19] as i upgraded to quantal today (twice) [07:21] davecheney, ah, cool [07:21] davecheney, you have an LGTM with one nitpick [07:23] bbiab [07:47] fwereade: hiya [08:09] rogpeppe, heyhey [08:13] morning [08:39] TheMue: hi [08:40] rogpeppe: hiya [08:40] fwereade: what's our current position on watchers returning updates for any changes, not just lifecycle? (i'm looking at MachinePrincipalUnitsWatcher here) [08:43] fwereade: it seems a bit silly that when we're told about a unit changing, we fetch its unit doc twice. ISTM that hardly any logic in the client (worker/machiner) would have to change, and we could get rid of a big chink of logic in the watcher [08:45] rogpeppe: I like the idea of getting notified on any change, but I then like to get the watched entitiy (not only the id) and a useful information what has changed (e.g. the lifecycle state) [08:46] rogpeppe, I am still undecided tbh [08:46] TheMue: from what i see of the clients, they don't make that much use of that info. [08:47] rogpeppe, the trouble is that different use cases are different and I think that both behaviours are wanted in different places [08:47] rogpeppe: The old added/removed separation has been more helpful than today iterating over ids, retrieve the state and compare it. [08:47] fwereade: if we're just sending ids, then i think it makes sense that the client is responsible for doing the refresh and doing the diffs itself [08:48] TheMue: i agree, but since we've moved away from that, i think it makes sense to take advantage of that [08:48] rogpeppe, this is true, up to a point -- let me read some code a mo [08:48] TheMue: there's no need for both the watcher *and* the client to be independently fetching the doc [08:48] rogpeppe: That's true [08:50] rogpeppe, I consider it to be a good thing, though, that the ServiceRelationsWatcher filters out irrelevant changes and only tells me about life changes [08:50] fwereade: how much would your logic need to change if it told you about other changes too? [08:50] rogpeppe, it could be more efficient -- it only needs to grab 2 fields to do the comparison -- but I think we need to consider the HTTP API here [08:51] fwereade: after all, you need to fetch the doc to find out exactly *what* has changed [08:51] rogpeppe, I think it *is* worth getting the document twice, because once the watcher is implemented behind the API, right next to the state, it is cheap for it to get a document [08:52] rogpeppe, and by doing so it can send fewer updates to remote clients on the other side of the API [08:52] rogpeppe, and those clients, for whom getting a document is expensive, don't have to repeatedly get and inspect the documents to do their own filtering [08:53] rogpeppe, sane? [08:53] fwereade: hmm [08:55] fwereade: that's a reasonable point [08:57] fwereade: o [08:57] fwereade: darn, i thought we could get rid of large chunks of code [09:04] rogpeppe, annoying, innit ;) [09:05] * rogpeppe grumbles [09:05] rogpeppe, TheMue: so, given that, I think the answer really is annoyingly situational [09:05] rogpeppe, TheMue: for the firewaller, it's perfect to get life and ports changes only [09:06] rogpeppe, service relations we want life only [09:06] rogpeppe, TheMue: again, for the (?)deployer and/or machiner, life changes only [09:07] fwereade: i'm happy with that now. [09:07] rogpeppe, cool, sorry :) [09:07] :D === mcclurmc_away is now known as mcclurmc [10:29] morning. [10:32] Aram: hi [10:41] Aram: you've got a couple of reviews [10:41] Aram, fwereade: what's the story regarding https://codereview.appspot.com/6811091 vs https://codereview.appspot.com/6820112 ? [10:42] looks like there are quite a few conflicts there [10:42] rogpeppe: unfortunate duplication of effort [10:44] Aram: well, yours does more, as it uses WatchUnits rather than WatchPrincipalUnits. maybe fwereade's should go in first. [11:06] hmm, looks like my gmail account has been hacked [11:07] rogpeppe: Yesterday there also has been a strange twitter message by you. [11:07] TheMue: yeah, that was hacked too. seems a bit strange - they both had different passwords [11:07] TheMue: although, to be fair, both passwords were terribly weak [11:08] rogpeppe: OK, so maybe just brute force. [11:08] TheMue: i think so [11:09] rogpeppe: I like the xkcd, where long curious sentences are better than "P4ssw0rd". ;) [11:09] TheMue: yeah, that's the approach i'm using [11:10] TheMue: with a couple of weird chars thrown in for good measure [11:10] rogpeppe: I've changed it and now have stupid sentences. [11:12] rogpeppe: But today I had to set one with 8 to 15 chars and no specia characters. Why those constraints? *sigh* [11:12] TheMue: yeah, that's outrageous. don't they know that you can run sha1 in javascript? :-) [11:13] rogpeppe: Yep ;) [11:24] interesting to see the spam machine in action [11:24] none of the sent messages addressed more than 5 recipients [11:25] and they all sent a different link [11:27] rogpeppe: Yeah, they do their best to not being catched by spamassassin or postgrey. [11:29] they got as far as "j" in my address book [11:30] 65 spam messages sent in total === dimitern is now known as dimitern_lunch [12:38] fwereade: ping [12:44] rogpeppe: I want acme integration with codereview. [12:45] Aram: how would you want it to work? [12:46] rogpeppe: somewhat like mail. [12:46] in fact even using that with the email you get now from codereview is almost usable, aparte from the fact that you can't reply. [12:47] Aram: do you still use acme mail? [12:47] no. [12:47] Aram: i used it for years actually, but the lack of threading finally got to me [12:48] Aram: and the benefits aren't so great if you're not running under plan 9 [12:48] yeah... [12:48] rogpeppe, deferred pong [12:48] rogpeppe, I have to pop out for a while and will be somewhat spottily around this afternoon and evening [12:48] acme mail is not so useful at this point anymore, but I think for specialized things, an acme mail thing would be useful. [12:48] rogpeppe, I will grab you this pm though [12:48] fwereade: ok. sometime i'd like a chat about some new attributes in config [12:49] fwereade: but i think i've decided on a reasonable approach actually [12:49] rogpeppe, cool -- I look forward to seeing it :) [12:49] tytyl [12:49] fwereade: have fun [12:50] Aram: go for it; i'd like to see what you come up with. === dimitern_lunch is now known as dimitern === flaviamissi is now known as flaviamissi_ === TheMue_ is now known as TheMue === flaviamissi_ is now known as flaviamissi [15:13] fwereade: i'll just run the config stuff by you to check for crackfullness, if that's ok [15:13] rogpeppe, sgtm [15:14] fwereade: so, we want to be able to specify a root CA when bootstrapping and when connecting to an environment [15:14] rogpeppe, ok [15:15] fwereade: when bootstrapping we'll need the private key too, so we can sign a new key/cert pair and hand it off to the first instance [15:15] fwereade: so my thought is to add three new attributes to config [15:15] fwereade: root-cert-path, root-cert and root-private-key [15:15] fwereade: root-cert-path acts a little like authorized-keys-path [15:16] fwereade: that is, if it's set, the cert/key pair is read from that file; otherwise if root-cert isn't set in config, we'll try to read from ~/.juju/rootcert.pem [15:17] rogpeppe, yep, that makes sense [15:17] fwereade: we verify that the cert parses and that, if given, the key matches the cert. [15:18] fwereade: which prevents going a long way down the road with a bogus cert/key pair [15:18] rogpeppe, ok, yep [15:18] fwereade: i think that's it, for the time being [15:19] fwereade: i'm just doing the tests [15:19] fwereade: it's a pity that it's yet one more step to getting a juju up and running [15:20] fwereade: i wonder if we might have a juju helper that prompts for the necessary information and does everything else necessary (e.g. generating the key, generating an admin secret, etc) and writes an environments.yaml file. [15:21] rogpeppe, just one of many, really, we'll get there :) [15:21] rogpeppe, yeah, new-user experience would be very much helped by that [15:22] fwereade: juju new-environment ? [15:22] fwereade: i'm not sure that we can rewrite environments.yaml losslessly though, sadly [15:22] rogpeppe, I'm pretty sure we can't [15:23] fwereade: me too. +1 to json :-) [15:23] fwereade: actually, maybe it could just spit out an environments.yaml to stdout [15:23] fwereade: anyway, that's way down the line. [15:23] fwereade: if all that SGTY, i'll carry on [15:24] rogpeppe, I *think* it does [15:24] rogpeppe, I am not sure how to handle changing the root cert [15:24] rogpeppe, or in general how to (or whether we should) change any of those new fields [15:25] rogpeppe, (I guess root-cert-path is not really a setting, at least, just a sort of input filter) [15:25] fwereade: yeah, changing the root cert is hard [15:25] fwereade: and what about certificate revocation [15:25] ? [15:26] rogpeppe, quite so [15:27] rogpeppe, I guess it would be worth trying to think a little way through those questions to ensure we're not closing any future doors, even if we don't support anything like that at this stage [15:30] fwereade: tbh, i think that both with changing the root cert and revocation, what we're building now is a necessary prerequisite. we are always going to need to pass at least one certificate into the cloud. [15:31] rogpeppe, that does sound solid to me [15:31] fwereade: revocation, i think happens automatically, as long as clients are checking revocation lists appropriately. [15:33] fwereade: changing the root cert we'd probably do by leveraging the existing root cert to produce another cert. you'd probably do it by using a level of indirection and revoking the original certificate but not the original *root* certificate (he says, hand-waving wildly) [15:35] rogpeppe, I think that's outside the realm of things I can claim to have a reasoned opinion on ;) [15:52] rogpeppe, incidentally, I was wondering something related... ISTM that the user is already adequately identified with the authorized key, and I'm suddenly unsure what we get out of admin-secret... what am I missing? [15:53] rogpeppe: significantly refined version https://codereview.appspot.com/6820112/ , when you have time [15:54] fwereade: that may well be true in the future. for the time being, i don't know of a way to get mongodb do client authentication [15:55] rogpeppe, ok, cool, so it's purely for admin auth with mongo? [15:55] fwereade: yes, i *think* so [15:56] Aram: looking [16:50] rogpeppe: also https://codereview.appspot.com/6814108/ [16:51] Aram: sorry, i got some way through, then realised i really wanted to propose my CL by the end of the day... [16:51] rogpeppe: don't worry, I'm only pinging you, I don't expect reviews right away. [16:52] Aram: that's find. just letting you know in case you expected one after my "looking" [16:52] s/find/fine/ [18:03] fwereade: ping [18:03] rogpeppe, pong [18:03] fwereade: i've just realised, i think, that i'm slightly askew in my assumptions [18:03] rogpeppe, oh yes? === mcclurmc is now known as mcclurmc_away [18:04] fwereade: i've been assuming that it's mandatory to have a root certificate [18:04] fwereade: but actually, i think we perhaps want the capability to connect to a server without checking [18:04] rogpeppe, expand please [18:04] rogpeppe, naively it seems like a good idea [18:05] fwereade: well, say we have a juju environment but we've lost the root cert key [18:05] fwereade: the checking that juju does is, i think, something that should be a choice (the default choice) [18:05] fwereade: 'cos otherwise we can't connect [18:06] fwereade: a bit like the way ssh automatically adds a key if you ask for it [18:06] rogpeppe, I kinda favour just advising people that "keys are important; don't lose them" [18:07] fwereade: i was just wondering if i really want the root cert to be mandatory in config [18:07] fwereade: (i've just spent the last while putting it everywhere...) [18:08] rogpeppe, ha, yeah, it is usually at such junctures that I start to question things [18:08] rogpeppe, hmm [18:08] rogpeppe, honestly I don;t know what is really meant by "everywhere" so it is hard for me to follow intuitively [18:09] fwereade: i got to environs/ec2, and realised that it uses your actual authorized keys, so i'm wondering whether we want to the same approach for root cert, or if we should generate one on the fly [18:09] fwereade: everywhere we've got a set of attributes for a config [18:09] fwereade: pretty much everywhere we've got an authorized-keys attribute now [18:10] fwereade: i'm thinking that it might be fine just to hard-code a root cert for testing. [18:11] fwereade: there's another thing which is a source of discomfort currently [18:12] fwereade: which is that the cert and key are specified in a single file, but they get two attributes in the config. [18:12] rogpeppe, yeah, that definitely seems wrong [18:12] rogpeppe, except hmm [18:13] rogpeppe, maybe not [18:13] fwereade: there are +s and -s [18:13] rogpeppe, but, yeah, it doesn't sit quite right [18:13] fwereade: i was trying to avoid having *two* more path attributes in the config. [18:13] rogpeppe, hmm, just a mo [18:14] fwereade: but perhaps that makes more sense in fact; keep them separate at all times. [18:14] rogpeppe, if that is as convenient that sgtm [18:15] fwereade: it's not convenient at the moment - i'd have to rewrite some code and lots of tests :-) [18:15] rogpeppe, ha, indeed [18:16] fwereade: but on balance, i think it's probably a better approach [18:16] fwereade: bugger [18:16] * fwereade sympathises [18:20] rogpeppe, a strawmanny thought [18:21] * rogpeppe gets out the wooden axe [18:22] rogpeppe, how would it be if the administrator who created the environment were to have keys and certificates automatically generated and stored in ~/.juju/environments/? [18:22] rogpeppe, they could of course be overridden by config settings [18:23] fwereade: i think that's a reasonable thought, and i've thought about something similar myself [18:23] fwereade: but i think it's orthogonal to what i'm doing currently [18:24] fwereade: which enables that, trivially [18:24] rogpeppe, agreed... maybe a suggestion for the lists, see if it draws popular support, if it does it should indeed be easy to build on top [18:25] fwereade: yeah. actually, i don't think there needs to be a key per environment [18:25] fwereade: i think one root key for all environments is just fine [18:26] fwereade: so the command line tool could generate ~/.juju/rootcert.pem if it doesn't already exist [18:27] rogpeppe, I dunno, I think one per environment is saner in a multi-admin situation [18:27] rogpeppe, but I don't feel very strongly about it, haven't really thought it through [18:27] fwereade: if you want one per environment, that's easy to do. but a single name space also makes sense, i think. [18:28] fwereade: it means there's only one public key to distribute, for one [18:28] rogpeppe, yeah, also sounds reasonable :) [18:30] right, i can't launch into changing all this now; time to stop for the night, i think. [18:30] a pint beckons [18:31] fwereade: have a great weekend, see ya monday [18:31] night all [18:31] rogpeppe, have fun :) === mcclurmc_away is now known as mcclurmc