/srv/irclogs.ubuntu.com/2012/11/14/#juju-dev.txt

davecheneylucky(~/src/launchpad.net/juju-core) % time juju status00:02
davecheneymachines:00:02
davecheney  0:00:02
davecheney    agent-version: 1.9.200:02
davecheney    dns-name: ec2-54-252-34-17.ap-southeast-2.compute.amazonaws.com00:02
davecheney    instance-id: i-b9c8ac8300:02
davecheneyservices: {}00:02
davecheneyreal    0m6.679s00:02
davecheneyuser    0m0.148s00:02
davecheneysys     0m0.020s00:02
davecheneymuch better00:02
davecheneywow, m1.smalls are slow, my arm host can compile faster than it00:36
davecheneyevenin'08:05
davecheneywhy is thunderbird so crap ?08:06
davecheneyi could possibly do a better job screaming into an acoustic modem08:06
TheMuemorning08:08
davecheneyhmm, a dopleganger08:09
TheMuedavecheney: hi, original ;)08:10
davecheneyhttps://codereview.appspot.com/6853048/08:38
davecheneyhttps://codereview.appspot.com/6846050/08:38
davecheney2nd is trivial08:38
davecheney1st i feel is straight forward08:38
davecheneybut so few things in this world at these days08:38
rogpeppedavecheney: 1st LGTM08:55
davecheneyrogpeppe: ty08:56
rogpeppedavecheney: it seems a pity we have to duplicate all the charm code for a single word change, but i guess that's where we're at08:56
davecheneydidn't think there would be much debate about that one08:56
davecheneyrogpeppe: i could probably do some symlink shenanigans08:56
davecheneymaybe I don't need to duplicate it08:56
davecheneythe charms don't seem to care about their names08:56
davecheneyi've been hapily putting precise charms on quantal all day08:57
rogpeppedavecheney: i know - it's not surprising, there's actually very little difference between series08:57
davecheneyrogpeppe: having said that, we do need a separate hook/install for each different build of mongo-2.2.0 we do08:57
davecheneysee my email08:57
davecheneyi thought that the logic to choose a vesion of mongo was the same as the toos08:57
davecheneytools08:58
rogpeppedavecheney: 2nd CL LGTM too08:58
rogpeppedavecheney: what's wrong with uploading the version of mongo build for quantal as version 2.2.0 ?09:00
rogpeppedavecheney: isn't it built from the same source code as the precise build?09:00
rogpeppedavecheney: and therefore justifies the same version number?09:01
davecheneyit comes from my years as a build engineer09:01
davecheneyyou can't reuse a release number09:01
davecheneynot having that 4th digit is the same as reusing juju-core-1.9.209:01
davecheneywithout a -$BUILD suffix09:01
davecheneyyou have two things in the world with the same name, that are actually different09:02
davecheneyin this case mongo-2.2.0-quantal-amd64 was not actually that09:02
davecheneyit was renmaed from the precise version09:02
davecheneyso to replace it with the real one compiled on quantal felt wrong09:02
davecheneynot to mention was subject to http caching nonsense09:02
davechen1yrogpeppe: could i trouble you to putyour LGTM for ap-southeast-2 in writing09:17
davechen1yfor some reason it didn't stick09:17
rogpeppenp09:17
davechen1ya09:17
davechen1yta09:17
* davechen1y goes to fix the long broken golang-src PPA's09:18
rogpeppedavechen1y: done09:18
davechen1yta muchly09:18
TheMuedavechen1y: Twice LGTM09:19
davechen1ythat zone is ~30 ms away from me09:19
davechen1ybut the cloud archives are being served from the UK for the moment09:19
davechen1ywhich makes cloudinit sucky09:19
davechen1yrogpeppe: in rc10:28
davechen1yis there an unset ?10:28
davechen1yie, unset GOMAXPROCS ?10:28
rogpeppedavechen1y: x=()10:28
rogpeppedavechen1y: (the empty list is equivalent to unset)10:29
davechen1y$GOROOT/src/run.rc:4610:29
davechen1yso I can do GOMAXPROCS=10:29
rogpeppeGOMAXPROCS=()10:29
rogpeppehold on10:29
rogpeppeyeah, that works10:30
davechen1yinside that subshell type thing ?10:30
rogpeppeor should do10:30
davechen1yrelated: windows batch files are hard10:31
rogpeppedavechen1y: yeah, @{} is equivalent to () in sh10:31
rogpeppedavechen1y: ha10:31
rogpeppedavechen1y: paste me the rc script before you submit it and i'll give it a once-over10:31
davechen1yrogpeppe: https://codereview.appspot.com/684705010:32
davechen1ythis has come up a few times before10:32
davechen1ybut fullung keeps nagging me about it10:33
rogpeppedavechen1y: i guess the rc script should probably call the api checking tool like the others too10:34
rogpeppedavechen1y: BTW GOMAXPROCS=0 is probably equivalent to unset10:35
davechen1yrogpeppe: the sad thing is Uriel made a C version of run.bash10:36
davechen1ybut it got shot down10:36
rogpeppedavechen1y: the rc script should do @{ ... } || exit $status, but that's for another CL10:37
rogpeppedavechen1y: yeah, poor uriel10:38
rogpeppedavechen1y: one more push.10:38
davechen1ytoo painful10:39
davechen1yrogpeppe: if you want to fix the rc script, it wouldn't hurt10:39
davechen1yi don't think anyone but ality is working on it10:39
* rogpeppe should probably fire up plan 9 again once in a while...10:40
rogpeppeactually, i still have a remote login on my old vitanuova account10:40
* rogpeppe avoids getting distracted.10:41
davechen1yrogpeppe: good man10:42
niemeyerMornings!11:26
TheMueniemeyer: Hello.11:30
fssniemeyer: morning :-)11:31
TheMueAram: Moin.12:16
rogpeppeniemeyer: hiya13:32
niemeyerrogpeppe: yo13:33
rogpeppeanyone up for giving a second opinion on this CL? https://codereview.appspot.com/6811095/13:33
TheMuerogpeppe: *click*13:34
niemeyerrogpeppe: Will have a look too13:39
TheMuerogpeppe: You've got a review.13:39
rogpeppeTheMue, niemeyer: ta13:39
niemeyerrogpeppe: CertAndKey?  Is there a PEM format that has both a cert and a private key?13:40
rogpeppeniemeyer: yes13:40
rogpeppeniemeyer: PEM includes an arbitrary number of blocks13:40
rogpeppeniemeyer: we can split 'em up if you'd like13:41
rogpeppeniemeyer: but this is the format that mongo requires13:41
rogpeppeniemeyer: which is why i chose it13:42
niemeyerrogpeppe: If that's the case, sounds good13:42
niemeyerrogpeppe: Done, cheers13:46
rogpeppeniemeyer, TheMue: there's a followup if you're in the mood: https://codereview.appspot.com/6819115/13:48
TheMuerogpeppe: Will have a look.13:49
niemeyerrogpeppe: Looking13:54
rogpeppeniemeyer: i have a slight problem with just renaming things to *PEM - the PEM format doesn't have any implications for what might be in it. So actually StateServerCertAndKeyPEM would be more useful, though long-winded. similarly for pemPath vs serverCertAndKeyPath13:54
rogpeppeniemeyer: not sure though13:54
TheMuerogpeppe: It seems the arguments of startInstance() grow and grow. ;)13:55
niemeyerrogpeppe: I don't mind too much myself.. looks better than StateServerCertAndKe13:55
niemeyerrogpeppe: We know what it is, and it should be documented13:55
rogpeppeTheMue: yeah, next time it happens, i'll make a struct for it.13:55
rogpeppeniemeyer: ok, seems reasonable.13:55
TheMuerogpeppe: You already made.13:56
rogpeppeTheMue: oh sorry, i thought you meant the public api...13:56
TheMuerogpeppe: No, the private one.13:57
rogpeppeniemeyer: the "certificate" thing is awkward, because some places use "certificate" to mean both the private key and the certificate itself (the tls package is a culprit in this respect). i'm happy to be less ambiguous though.13:58
niemeyer_rogpeppe: That's wrong13:59
niemeyer_rogpeppe: A certificate doesn't contain a private key13:59
rogpeppeniemeyer_: yeah, i agree.13:59
rogpeppeniemeyer_: tls.Certificate.PrivateKey is a wrongness, i think13:59
niemeyer_rogpeppe: It contains a public key14:00
rogpeppeniemeyer_: it contains a private key too14:00
rogpeppeniemeyer_: http://golang.org/pkg/crypto/tls/#Certificate14:00
niemeyer_rogpeppe: http://en.wikipedia.org/wiki/X.50914:01
niemeyer_rogpeppe: One can put whatever one pleases in a struct14:01
rogpeppeniemeyer_: i know that x509 certs don't contain a private key.14:01
niemeyer_rogpeppe: My point is that a X509 certificate doesn't contain a private key14:01
rogpeppeniemeyer_: but some places use a "certificate" name to hold both a certificate and a private key. tls being one such place.14:02
niemeyer_rogpeppe: Where is it doing that?14:02
rogpeppeniemeyer_: as i said, i'm happy not to do that.14:02
=== niemeyer_ is now known as niemeyer
rogpeppeniemeyer: http://golang.org/pkg/crypto/tls/#Certificate14:02
niemeyerrogpeppe: This is a struct14:02
niemeyerrogpeppe: A certificate has a respective private key14:02
rogpeppeniemeyer: sure. it's a struct called "Certificate" :-)14:02
niemeyerrogpeppe: It's a fine design to be able to refer to it14:03
niemeyerrogpeppe: Note that, though:14:04
niemeyer    Certificate [][]byte14:04
niemeyer    PrivateKey  crypto.PrivateKey // supported types: *rsa.PrivateKey14:04
rogpeppeniemeyer: yeah. it did confuse me a little to start with though.14:04
=== TheMue_ is now known as TheMue
* niemeyer => lunch14:27
* TheMue => lunch too14:37
TheMueBack again15:20
niemeyerTheMue: Welcome back15:45
TheMueniemeyer: u2 ;)15:47
niemeyerTheMue: Cheers15:47
TheMueniemeyer: Did you had a good lunch? We today simply had french fries, the kids love them.15:47
TheMueniemeyer: And their dad. :D15:48
niemeyerTheMue: LOL15:49
niemeyerTheMue: Top notch :)15:49
niemeyerTheMue: It was great here too.. Ale cooked us some great Brazillian style food15:50
TheMueniemeyer: We definitely have to visit you for a sprint.15:51
niemeyerTheMue: Agreed! :)15:51
* rogpeppe had a sandwich and a banana :-)15:51
niemeyerrogpeppe: Not bad, not bad :)15:52
TheMuerogpeppe: I had a banana and two clementines this morning15:52
rogpeppeit *was* my special chilli-cheese sandwich, which i never seem to grow tired of15:52
rogpeppeBrazillian-style food would have been better though :-)15:53
niemeyerrogpeppe: You guys will have to come over to try that out15:54
rogpeppeniemeyer: i'd love to15:54
niemeyerWe should prepare something for next year15:55
TheMueniemeyer: +115:55
rogpeppeit's a bit of a pity there appears to be no way to marshal a *x509.Certificate16:02
rogpeppeniemeyer: juju.Bootstrap implementation. hopefully not *too* crackful :-] https://codereview.appspot.com/6843059/17:37
niemeyerrogpeppe: Cheers17:37
niemeyerrogpeppe: I'm in another CL now, but hopefully I can get there today17:37
rogpeppeniemeyer: thanks17:37
TheMuerogpeppe: Will also look at it, but will last a bit longer.17:50
rogpeppeTheMue: thanks17:51
rogpeppeoh dear, looks like i broke the bootstrap tests in cmd/juju some time ago...18:21
rogpeppefairly trivial CL to end the day: https://codereview.appspot.com/684805218:30
rogpeppeand with that, i'm off for the evening.18:30
rogpeppenight all.18:30
TheMuerogpeppe: N818:41
niemeyerrogpeppe: nn18:44
niemeyerI'll also step out for some time, actually.. will be back later to continue reviewing18:56
=== Aram2 is now known as Aram

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