/srv/irclogs.ubuntu.com/2013/08/21/#juju-dev.txt

davecheneybigjools: sup00:52
davecheneythumper: do you have any contacts on u1 ?00:52
davecheneysidnei: ping00:53
sidneidavecheney: pong01:10
davecheneysidnei: nate is looking for someone that can help him with windows installer tools01:11
wallyworld_thumper: did you write the DiskManager stuff for tools?01:11
davecheneydoes u1 use anything for producing it's windows installer that would be of use ?01:11
sidneidavecheney: uhm, brian curtin was our windows guy, i think mike mccraken is in charge now. let me see if i can figure out what they are using.01:12
davecheneysidnei: what timezone are they in01:12
davecheneyi wanna try to cross connect nate and those two guys01:13
sidneimike is in cali01:13
davecheneykk01:13
davecheneythat'll work01:13
sidneii think brian left recently01:13
davecheneyoh snap01:13
davecheneyok, i'll write to mike01:13
sidneidavecheney: seems to be bitrock: http://bitrock.com/01:17
sidneihttp://bazaar.launchpad.net/~ubuntuone-control-tower/ubuntuone-windows-installer/trunk/view/head:/scripts/build_installer.py fwiw01:17
davecheneysidnei: thanks01:17
* sidnei disappears again01:18
thumperwallyworld_: no01:34
wallyworld_yeah, i saw john did it01:34
thumperdavecheney: yes01:34
wallyworld_but nothing seems to use it apart from tests01:34
davecheneythumper: yes what01:34
* thumper shrugs01:34
davecheneyi think I forgot the question01:35
thumperdavecheney: yes I have contacts on u101:35
thumperdavecheney: beuno01:35
davecheneythumper: ahh, this is a question from Nate01:35
davecheneywho wants to know what we use for win32 installers01:35
thumperah01:35
davecheneysidnei: said talk to Miek McCracken01:35
thumperthey are in very similar time zones01:35
davecheneythumper: anyone else who delivers one win32 in the company I can point him too ?01:36
thumpersorry, don't really know01:36
davecheneythumper: s'okj01:36
davecheneyit's an unusual request01:36
thumperalthough jamesh used to work on u101:36
thumperhe may know01:36
* thumper pings him01:37
davecheneyaxw: do you get much chance to talk with Nate ?01:37
davecheneyhe's in a pretty shitty timezone for you01:37
davecheneyWA <> Central time is not awesome01:37
axwdavecheney: haven't spoken to him since I've started01:37
axwyeah it's like, the opposite01:37
axw:)01:37
thumperthey must be ~12 hours apart01:38
axwyep01:38
davecheneyaxw: it is the fall that enslaves us all01:38
axw"most isolated city in the world", even when you're working remotely01:39
davecheneyaxw: you have your own currency over there, right ?01:40
axwyeah, the WA bux01:41
axwactually we just trade in iron ore01:41
davecheneywith a picture of the little creatures brewery on the $20 bill01:41
thumperdavecheney: <jamesh> The win32 code is at https://launchpad.net/ubuntuone-windows-installer01:41
davecheneyha! raw coke iron01:41
davecheneythumper: ta01:41
thumperheh, might be using py2exe01:42
davecheneyaxw: do the ATMs dispence iron ore and uranium ?01:42
thumperbut should still use a .msi somehwere01:42
axwlooks the same as bzr's01:42
axwdavecheney: what is an ATM? I keep my money under my bed01:43
axwanyway... ;)01:43
thumpermoney... or opals and ore?01:43
axwyep, not a very comfortable bed01:44
axwI am shattered01:53
axwmy daughter woke up twice last night, my son three times01:53
axw:(01:53
thumper:(01:53
thumperI remember those days01:53
thumperaxw: don't stress, and sleep if you need to01:54
thumperthe world won't fall apart if you miss a day :)01:54
axwthumper: thanks, I'm kinda used to it, just a bit worse than usual01:54
* thumper nods01:55
axwthumper: have you ever tried upgrading a local provider? :)01:56
axwthumper: the bootstrap machine agent restarts repeatedly01:56
axwI'll make sure I can reproduce it again, and log a bug..01:59
thumperaxw: we decided that we didn't support upgrading with the local provider02:06
thumperaxw: however, if it is easy to fix02:06
thumperthen we could do it I suppose02:06
thumperI just wasn't going to waste cycles on it02:06
axwthumper: ah ok02:06
thumperwe should either document the limitation or fix it I guess02:07
axwdo we want a bug in LP anyway? at least then we can decide to tell "upgrade-juju" to not attempt02:07
axwBug #1214676  -- logged anyway, at least then there's a record for the next unsuspecting user02:10
_mup_Bug #1214676: upgrade-juju in local environment causes bootstrap machine agent to restart continuously <juju-core:New> <https://launchpad.net/bugs/1214676>02:10
davecheneyjamespage: thanks for keeping https://blueprints.launchpad.net/ubuntu/+spec/servercloud-s-juju-2-delivery up to date02:39
davecheneyi'll try to find out when we're going to deliver 1.14 stable as soon as I can02:39
bigjoolso/03:07
davecheneyaxw: thanks for 12742045/03:56
davecheneyhow come none of the tests have to change ?03:56
davecheneydoes that mean we don't have a test that checks that the listener file exists on disk ?03:57
axwdavecheney: because you don't need to manually delete unix sockets in Go03:57
axwClose() unlinks them03:57
davecheneyaxw: you can still leave socket turds around03:58
axwif the process terminates, yes03:58
davecheneyservice jujud-unit-agent stop03:58
davecheneywill leave a turd around03:58
davecheneythat was how I ran across this03:58
axwthe existing tests all specify the path as non-abstract04:00
axwi.e. they don't go through that codepath that specifies them as abstract04:00
davecheneyhmm, ok04:00
davecheneyi'm not going to complain04:00
davecheneybut maybe someone will04:00
axwideally we'd have a test for it, but it seemed like an awful lot of work for very little gain04:00
davecheneyaxw: yup04:01
bigjoolsdavecheney: so, you are prepared for an Azure onslaught?04:02
bigjoolsand axw :)04:02
davecheneybigjools: yes, bring it on04:02
bigjoolsbecause bugs are getting filed already (not sure if they are real bugs yet...)04:02
davecheneybigjools: yeah, we'll manage04:03
bigjoolsI'll help as much as I can of course04:03
bigjoolswill take you through it next week04:03
davecheneyok04:03
bigjoolswill prob make Monday after all04:03
davecheney\o/04:03
davecheneyyour weed delivery came through early ?04:04
bigjoolsnah sending the wife to get it04:04
axwbigjools: I know nothing about Azure, so ... probably not prepared. But I'll look forward to learning04:04
bigjoolsaxw: there is indeed much to learn04:04
bigjoolson the bright side, the weather here is marvellous04:05
axwhere too - spring is slightly early04:05
jamhas anyone been trying canonistack with 'use-floating-ip: true' ? I saw the machine come up, and I can ssh to the private address, but the public address tells me invalid public key06:03
axwI haven't, but I can give it a try06:05
jamaxw: if you would, to give me a point of comparison06:05
axwjam: it works for me if I specify the identity file (~/.canonistack/axw_lcy02.key) and user as ubuntu06:15
axwi.e. what's in my .ssh/config for canonistack, for the private IP range06:15
jamaxw: ah, user ubuntu, probably my fault06:16
jamdifferent machine, I don't have my normal setup06:16
jamaxw: thanks for the reminder06:17
axwjam: no worries06:17
jamdefault key was fine, didn't have the config for ubuntu@ for 172.* addresses.06:17
jamaxw: interestingly, username isn't part of the 'ssh -v' output.06:18
axwso it would seem - helpful!06:18
bigjoolsarosales: hopeful ping06:25
arosalesbigjools, hello06:26
bigjoolsarosales: wow you're up!06:26
bigjoolsarosales: I updated your config doc with my version of things, let me know how it looks06:26
arosalesfor a little bit longer06:26
arosalesbigjools, thanks06:27
* arosales is waiting for agent-state to go to started in Azure06:27
arosalesusing --upload-tols06:27
arosalesbigjools, I saw your and gavin's reply so I was mistaken there. I was unaware the --upload-tools built juju-tools locally to match the version being used06:28
bigjoolsarosales: no worries06:28
bigjoolsI'll see if I can re-create your deployment bug06:28
arosaleshttps://bugs.launchpad.net/juju-core/+bug/1214636 may be due to mismatch juju-tools06:30
bigjoolsI highly suspect so yes06:30
arosalesI am trying with --uploads-tools now06:30
arosaleshttps://bugs.launchpad.net/juju-core/+bug/1214178 I think we can make better by having the user specify the settings file06:30
bigjoolsI am just bootstrapping then I'll try to deploy06:30
bigjoolsarosales: this is the first time I ever saw a settings file :)06:31
arosaleshttps://bugs.launchpad.net/juju-core/+bug/1214181 ~should~ be resolved once we get the correct tools in Azure06:31
_mup_Bug #1214181: Azure Provider always uploading 1.12 tools <juju-core:Incomplete> <https://launchpad.net/bugs/1214181>06:31
arosalesbigjools, I take it you also saw Ben's tool in the google set up doc06:31
bigjoolsI did06:31
bigjoolsnot looked at the code06:31
arosalesok06:31
bigjoolswe just saw the stuff on the management UI about creating and uploading a certificate06:32
arosalesAzure complained my pem didn't have the right structure when I openssl command06:32
arosalesI too went that route initially06:32
bigjoolsI love the way their ui insists your file must be a .cer, like that means anything06:33
arosalesbut  I think the settings file makes for a better user experience.06:33
arosalesbigjools, I feel your guys' pain06:33
bigjoolsarosales: all of Red used the openssl tool to generate a certificate fine06:33
arosalesto a _very_ small degree06:33
bigjoolsha :)06:33
bigjoolsarosales: the settings file can only be downloaded once you upload a certificate, right?06:33
davecheneyarosales:   openssl req -config /usr/share/ssl-cert/ssleay.cnf -x509 -nodes \06:33
davecheney    -days 3650 -newkey rsa:2048 -keyout azure.pem -out azure.pem06:33
davecheney   openssl x509 -inform pem -in azure.pem -outform der -out azure.cer06:34
arosalesbigjools, that may be my pilot error on the pem gen .  .06:34
bigjoolsdavecheney: exactly06:34
arosalesI thought I was following Azure instructions, but the settings file helps simplify that.06:34
bigjoolsis the settings file documented anywhere?06:34
arosaleshm . . . I'll have to check with utlemming on that06:35
bigjoolsI can't see anything on manage.windowsazure.com about it06:35
davecheneywhat is the settnigs file ?06:35
arosalesAzure hands down the pems for a subscription in a xml based file06:35
bigjoolsAFAICT it's some xml that contains a subscription ID and the certificate06:36
bigjoolsbut given that you have to generate and upload a certificate I question its usefulness for juju06:36
arosalesbigjools, aiui this settings file generates the need certificate.06:37
arosalesso I didn't have to upload a cert to have juju work.06:37
arosalesjust needed to parse the settings file and put the pem in the correct path06:38
bigjoolsarosales: ok, would love to see any docs on that06:38
bigjoolshow did you find out about it?06:38
arosalesbigjools, ok I'll follow up with utlemming on it.06:38
bigjoolsthanks06:38
arosalesutlemming also says this helps solve the China Azure endpoint problem too06:38
bigjoolsarosales: what problem?06:39
arosalesutlemming told me about it when I was having some initial bootstrap issues06:39
arosalesmanaging multiple certificates for different Azure end points06:39
arosalesbigjools, I'll start a thread with utlemming so when he gets up the morning he can shed some more light on it.06:40
bigjoolsarosales: I honestly can't work out what the problem is with that, if Azure needs a separate cert then we just config it in a separate juju env06:40
bigjoolsok cool06:40
arosalesubg state still in pending with --upload-tools06:41
arosalesbigjools, http://pastebin.ubuntu.com/6009219/06:42
arosalesI seem to not be able to get out of pending06:42
arosaleswhat version of juju should I be using?06:42
arosalesand is there any other special juju set up I need?06:43
* arosales using 1.13.2 (compiled yesterday)06:44
bigjoolsarosales: I am using tip of trunk06:44
bigjoolsI just did a deployment, so let's see how it goes06:45
arosalesbigjools, ok thanks06:45
bigjoolsarosales: you can try ssh-ing into machine 106:45
bigjoolscheck the agent log06:45
* arosales just destroyed06:46
arosaleswill redeploy again06:46
bigjoolsI just deployed deploy cs:precise/juju-gui on saucy06:47
arosalesbigjools, juju didn't complain about the series mismatch?06:47
bigjoolsit does unless you force the series06:48
arosalesI was just deploying wordpress out of a local saucy repo06:48
bigjoolsok06:49
arosalesbigjools, I guess you just set you "default-series" to "precise" in your env.yaml, correct?06:49
bigjoolsarosales: no, that's saucy still.  I literally just did "juju deploy cs:precise/juju-gui"06:49
bigjoolsand off it goes06:50
arosaleshuh ok06:50
arosalesbigjools, have you had success with a local deploy?06:51
bigjoolsI haven't tried06:51
arosalesI just downloaded the latest wordpress charm and did06:51
arosales--repository=/home/arosales/devel/local-charms/   local:saucy/wordpress06:52
* arosales stating the obvious06:52
bigjoolsI'm getting public key error trying to ssh into machine 106:53
bigjoolsthis means cloud-init is probably hosed06:53
bigjoolseither that or juju's user data went wrong - and given the bootstrap worked I suspect the latter06:54
arosalesugh06:54
bigjoolsI've definitely deployed before so something has recently broken06:54
arosalesthere was a recent fix for cloud-init ssh access on azure . .  .06:55
bigjoolssadly no way of finding out since I can't ssh in ...06:55
bigjoolsoh do you have a reference?06:55
bigjoolsthe fix might not be in the image06:56
arosaleshttps://bugs.launchpad.net/cloud-init/+bug/121272306:56
_mup_Bug #1212723: cloud-init fails to set user password on Windows Azure <amd64> <apport-bug> <cloud-images> <saucy> <cloud-init:Fix Committed by smoser> <cloud-init (Ubuntu):Fix Released by smoser> <https://launchpad.net/bugs/1212723>06:56
arosalesI had thought that had gone into the Monday's daily . . .06:56
bigjoolsarosales: ah that's not relevant here06:56
bigjoolswe don't use passwords06:56
bigjoolsso I can ssh into machine 0 but not 106:57
arosalesah ok06:57
bigjoolssomething catastrophic has gone wrong in cloud-init if it hasn't picked up the ssh key06:57
bigjoolsI'll write this up on the bug arosales06:57
arosalesya 0 has been able to go to a started state for me, its just subsequent services that get stuck in pending06:57
arosalesbigjools, if you can point to something in cloud-init I can pick this back up with utlemming and smoser in the morning (us time)06:58
bigjoolsit could be cloud-init or juju's fault06:58
bigjoolsso I'll write up as much as I can and then you can get smoser to take a look I guess06:59
bigjoolswe could do with a debug setting to put a password on the account06:59
arosalesbigjools, which bug are you documenting in?07:00
bigjoolsarosales: https://bugs.launchpad.net/juju-core/+bug/121463607:00
_mup_Bug #1214636: Azure Provider: Deployed service never goes to started <juju-core:New> <https://launchpad.net/bugs/1214636>07:00
arosalesok07:00
arosalesbigjools, did that fail with the charm store gui deploy and the local charm or just the local?07:01
bigjoolsarosales: with the cs one07:02
arosalesok07:02
bigjoolsnot sure it matters07:02
bigjoolsit's not getting that far07:03
bigjoolsarosales: I'm going to try with a different (older) image that I used successfully before and see if that helps07:05
arosalesbigjools, ack, thanks for looking into and the help07:05
bigjoolsarosales: not a problem07:07
arosalesgeesh deploys are taking about 9 minutes07:08
arosalesand that just waiting for the service to connect to the state server07:08
bigjoolsarosales: azure is slow :/07:09
arosalesbigjools, is there a particular API call that takes long07:10
arosalesI would like to bring this up with msft on why deploys take so long07:10
bigjoolsarosales: most of them return quickly, it's just the provisioning process07:10
bigjoolswaiting for a machine to come up and then boot and then provision.07:11
bigjoolsa certain amount of time is wasted if the image is not new enough as the apt-get update/upgrade takes a while07:11
arosaleswe should be hitting a local mirror for apt-get updates on the order of less than a minute, especially for a daily image.07:12
arosalesIt seems the console provisions vms faster than 5 minutes, but I haven't done one recently07:13
arosaleswould be good comparison07:13
bigjoolsarosales: the api calls that are slow are mostly deletion of stuff07:14
bigjoolsbut there's plenty of bugs filed about that already07:14
arosalesok, but that shouldn't affect the deploy times07:14
arosalesand there are fixing delete07:14
arosalesI think you saw the latest API for that07:15
bigjoolsyep07:15
axwjam: I can't make the standup. should I just send comments to the list?07:23
axw(for shared review)07:23
noodles775allenap: Hi! I saw your branch with improvements to the Makefile... Another thought I had the other day is adding -y to the apt-add-repository and apt-get installs (or at least allowing -y via env or similar?). What do you think?07:26
arosalesbigjools, on a quick test from the console gallery it takes about 4 minutes to create a virtual machine07:28
bigjoolsarosales: sounds about right07:28
bigjoolsarosales: huh, my known-good image no longer works07:32
bigjoolsdavecheney: why would mongo be coming up on port 27017 and juju wanting to connect on 37017?07:33
arosales:-(07:33
=== tasdomas_afk is now known as tasdomas
bigjoolsarosales: you should probably call it a night! If you can grab Scott tomorrow maybe he can help debug based on my bug comments07:38
arosalesbigjools, will do07:41
arosalesok good night fellas07:42
bigjoolsnn arosales07:42
arosalesbigjools, I'll touch base with smoser tomorrow morning07:42
bigjoolsok07:42
bigjoolsthanks07:43
* bigjools eats07:43
noodles775507:54
jamaxw: sounds good. I think we're going to try and have Tim run a shared review conversation sometime in the AU-friendly timezones. But I haven't seen him to coordinate it.07:55
axwjam: okey dokey, thanks07:57
rogpeppemornin' all08:03
fwereaderogpeppe, heyhey08:04
rogpeppefwereade: yo!08:04
TheMuerogpeppe: heya08:06
rogpeppeTheMue: hiya08:06
fwereadeTheMue, ping08:13
allenapnoodles775: I think that's a fair point. I'll add the -ys.08:13
fwereadeTheMue, actually I'll just mark https://codereview.appspot.com/12347043/ WIP, please reject it yourself if you're doing a fresh branch for unset08:14
noodles775allenap: thanks (it wasn't really a point about your branch, just something I needed/wanted yesterday, and since you were there... :) ).08:15
TheMueGna, update just made me reboot.08:27
TheMuefwereade: Seen that you pinged me?08:27
bigjoolshey, anyone know why would mongo be coming up on port 27017 and juju wanting to connect on 37017?08:59
jambigjools: "apt-get install mongodb" brings it up on 2701709:01
jambigjools: but jujud coming up sets an upstart config that puts a different one on 3701709:01
jamwith TLS enabled, etc.09:01
bigjoolsjam: this was after I did a bootstrap09:01
jamb09:01
bigjoolsbootstrap node didn't finish coming up09:01
jambigjools: that sounds like cloud-init successfully installed mongodb-server, but jujud did not successfully finish bringing itself and mongodb up09:01
bigjoolsand ssh-ing in showed this09:01
bigjoolsyeah09:01
bigjoolsin cloud-init-output.log it just shows lots of failed connections09:02
bigjoolsno other errors obvious09:02
jamcan you tar up and post the cloud-init-output ?09:02
jamI can take a look at it09:02
bigjoolsI an do better and put your public key on the machine if you want?09:02
jambigjools: ssh-import-id jameinel should wokr09:02
jamwork09:02
bigjoolsone sec09:03
bigjoolsssh ubuntu@juju-azure-9nz3pmvw7e.cloudapp.net09:03
jamsuccess09:03
bigjoolsjam: I'll come clean on something - this is using an old-ish saucy image but I needed to work out why I can't deploy on the latest daily.  This old image used to work.09:04
bigjoolsthe latest daily shows a publickey error, so can't ssh in and see09:04
jambigjools: root@default:~# mongod --version09:06
jamdb version v2.0.4, pdfile version 4.509:06
jamWed Aug 21 09:05:32 git version: nogitversion09:06
jamthat looks like a really old db version09:06
jam2.0.409:06
jamvs 2.4+ should be in new saucy09:06
bigjoolsweird - I have deployed with this image before09:06
bigjoolswhy would this stop working?09:06
jam||/ Name                     Version                  Description09:06
jam+++-========================-========================-===================================================09:06
jamii  mongodb-server           1:2.0.4-1ubuntu2.1       object/document-oriented database (server package)09:06
jambigjools: /etc/lsb-release says this is a Precise image, not saucy09:07
bigjoolsyeah was about to sya09:07
bigjoolsbut should still work nonetheless09:07
bigjoolsoh wait ... haha09:07
jambigjools: if we know it is precise, we add-apt-repository ppa:juju/stable  to get newer mongodbx09:07
bigjoolsI know what;s up, I think my default series is wrong :)09:08
bigjoolsright :)09:08
bigjoolsthanks for helping me see the end of my nose!09:08
jambigjools: so your machine thinks it is deploying saucy, but it is actually precise09:08
davechen1yjam: bigjools you need to install juju:ppa/stable09:26
davechen1yto pick up the mongodb dep09:27
bigjoolsdavechen1y: jam figured it out, I had a dodgy config09:27
davechen1ybigjools: also09:29
davechen1yi wasn't thinking09:29
davechen1ywe always insert that ppa via cloud-innit on the remote bootstrap machine09:29
davechen1yif required09:29
bigjoolsunderstandable09:29
* bigjools heads towards TV to watch Aussies lose at cricket again09:30
davechen1ybigjools: we're going to get on like a house on fire next week09:30
davechen1yi'm super confident of that fact09:30
davechen1yjam: yolanda has a questoin about bootstrapping on canonistack09:30
bigjoolsdavechen1y: :D09:30
yolandahi, i just upgraded to 1.12 version of juju, trying to deploy on canonistack, but deployment is stuck on INFO juju open.go:69 state: opening state; mongo addresses: ["10.55.60.49:37017"]; entity ""09:31
davechen1yshe's seeing it trying to contact the bootstrap node on the private 1009:31
davechen1yip09:31
yolandagives timeout09:31
davechen1yjam: mgz any ideas ?09:31
davechen1yhow do you setup the port forwarding for canonistack ?09:31
rogpeppe1fwereade, jam, mgz, TheMue: https://codereview.appspot.com/1308904509:34
rogpeppe1fwereade: i hope this addresses your concerns about the lax stuff.09:34
jamdavechen1y: you can either use 'sshuttle' or you can just set "use-floating-ips: true'09:35
jamdavechen1y: canonistack got 2 new /24 for public IPs, so we should have enough now.09:35
TheMuerogpeppe1: *click*09:36
jamdavechen1y: but the port forwarding I use is: apt-get install sshuttle; sshuttle -r ubuntu@$BOOTSTRAP 10.55.0.0/1609:36
yolandajam, so i'll try it09:37
jamyolanda: k, if you have any more questions, feel free to ask09:38
yolandajam, that worked!09:40
davechen1ywoot09:40
davechen1yyolanda: i the default image type on openstack machines is 1core 1gb ram09:43
davechen1yjam: and mgz wrote this09:44
davechen1ythey would be able to tell you09:44
rogpeppe1i just realised that i reviewed (comprehensively) entirely the wrong file09:48
jamrogpeppe1: we'll still appreciate that review :)09:48
yolandatried a juju debug-log, having that error: http://paste.ubuntu.com/6009664/09:48
rogpeppe1jam: i reviewed constraints.go09:48
rogpeppe1jam: (a small file, but with primitives used in quite a few places)09:49
jamrogpeppe1: I can make that the assignment for next week :) I was going to do relations.go but constraints is worthwhile for the group, too.09:49
* davechen1y looks09:49
yolandathat url that it complains about, works nice in a browser09:49
davechen1yyolanda: could be sporadic09:49
davechen1ycan you try again09:49
jamyolanda: you can try a) swift list and b) just try juju debug-log again09:49
yolandajam, davechen1y, different answer now09:54
davechen1yjujy deploy --constraints="cores=4 mem=4G" $SERVICE09:54
davechen1ysomething like this should ask for a different instance09:54
davechen1yi forget if constraints are best fit or absolute match09:55
TheMuerogpeppe1: you've got a review09:55
rogpeppe1TheMue: thanks!09:55
davechen1yyolanda: if you need more disk i *beleve* you get it by matching an instance type that has more cores and more ram09:55
jamdavechen1y: they should be a "minimum" so if there is only a 4-core 8G service we'll give it to you09:56
davechen1yjam: /me can't remember the instance types available on canonistack09:56
rogpeppe1TheMue: as I said in the description, the plan is to rename Sync to StartSync - i didn't want to confuse matters by doing it in this CL09:56
jamyolanda, davechen1y: sidnei was putting together a patch to add a "root-disk" constraint to specify how much disk space the OS gets. But that isn't in a released version. (It might have landed and will be in 1.13.2, though)09:56
rogpeppe1jam, fwereade: i would very much appreciate your feedback on this CL https://codereview.appspot.com/1308904509:57
davechen1yjam: yolanda do all canonistack instances get the same root disk partitoin ?09:58
davechen1yi thought it was larger if you asked for more cores/ram09:58
yolandadavechen1y, i normally got more disk by setting a larger instance-type, for example m1.small instead of m1.tiny09:59
jamdavechen1y: can you try running your mongodb with --no-unix-socket? We have a really hard time in the test suite if we try to start mongodb with a flag it doesn't recognize. So if it is ~roughly sane, we can go with it, but if there is a chance it will be hard to debug, I don't think it is worth it.09:59
jamaxw: ^^09:59
jamdavechen1y, yolanda: 'nova flavor-list' shows about 1300 options, so it isn't like you can remember them all :)10:00
jamsorry 30010:00
jamthey all start with a 110:00
axwjam: --nounixsocket works (I tested it before across all the code), --no-unix-socket isn't a thing10:00
jamaxw: I realize it works on your machine, I'm concerned about it running on all developers and platforms we want to run the test suite on.10:01
axwright sorry, misunderstood10:01
jamWe ran into trouble in the --no-ssl switch (where having an old mongo just hangs the test suite for 600s before the test times out, with poor information about why it is failing)10:01
axwfwiw I verified it's on the PPA version10:01
axwso it'll be no worse than lacking SSL10:01
TheMuerogpeppe1: rename Sync to StartSync? that's what IMHO would be wrong.10:01
rogpeppe1TheMue: no, rename StartSync to Sync.10:02
yolandajam, i normally use a instance-type=m1.small and that's all, or flavor 2, that works for canonistack10:02
rogpeppe1TheMue: there should be no occurrences of Sync left (other than calls to presence Sync)10:02
jamaxw: I can confirm that with the 2.2.4 that we produced in the tarball on S3, --nounixsocket works10:02
axwjam: thanks10:02
axwgtg to dinner, adios10:03
TheMuerogpeppe1: oh, you written the other direction above. so the first step is the new StartSync() and rename all Sync()s to StartSync() and then later to Sync()?10:03
jamaxw: have a good evening10:03
axwcheers, you too jam10:03
rogpeppe1TheMue: yes10:03
rogpeppe1axw: g'night10:03
TheMuerogpeppe1: ah, then absolute +1, my fault10:03
rogpeppe1TheMue: np10:04
jamrogpeppe1: "the watcher loop will not do anything else until it finishes sync" is that appropriate? (It sounds like we are blocking when we should be doing something in a background thread)10:04
davechen1yaxw: ship it10:04
rogpeppe1jam: the watcher would never do anything else while syncing10:05
davechen1ylucky(~/src/launchpad.net/juju-core/provider) % mongod --whogivesafuck10:06
davechen1yerror command line: unknown option whogivesafuck10:06
davechen1yuse --help for help10:06
davechen1ylucky(~/src/launchpad.net/juju-core/provider) % mongod --nounixsocket10:06
davechen1yWed Aug 21 20:05:53 [initandlisten] MongoDB starting : pid=23438 port=27017 dbpath=/data/db/ 64-bit host=lucky10:06
davechen1yWed Aug 21 20:05:53 [initandlisten] db version v2.2.4, pdfile version 4.510:06
rogpeppe1jam: the only change now is that there's no time interval between asking for a sync and it actually starting one10:06
davechen1yworks for me10:06
jamdavechen1y: thanks for confirming, thats what I see here as well.10:06
davechen1yjam: jolly good10:06
davechen1ywas just a bit gun shy after last time10:06
jamdavechen1y: mongodb's start flags are surprisingly hard to expose and respond to.10:07
* davechen1y sobs10:08
yolandadavechen1y, i also tried a juju destroy, debug showed unit is dying, but service doesn't die, and it has been a long time, there should be some problem?10:08
davechen1yfffuuuu web 2.010:08
davechen1yyolanda: weird, pastebinit ?10:09
yolandadavechen1y http://paste.ubuntu.com/6009736/10:11
yolandajust got that "unit is dying" and no more notice10:11
yolandabut service is still there10:11
davechen1yyolanda: did you do remove-unit or destroy-service ?10:13
yolandadestroy-service10:13
yolandain status it just shows "dying"10:14
davechen1yyolanda: juju status $YOURSERVICE should show10:15
davechen1yone service10:15
davechen1ywith no units10:15
davechen1yrogpeppe1: +1 for nuking Sync10:16
rogpeppe1davechen1y: cool10:16
davechen1yfor no other reason than we always use StartSync10:16
davechen1yso it's unneeded10:16
rogpeppe1davechen1y: well, we *did* use Sync in quite a few places10:16
rogpeppe1davechen1y: but it's always unnecessary10:17
davechen1yexactly10:17
davechen1yand one way to do things is better than two10:17
rogpeppe1davechen1y: and i think my changes give StartSync the same amount of useful guarantee that we had from Sync previously10:17
rogpeppe1davechen1y: (also, i plan to rename StartSync to Sync once the dust has settled)10:18
yolandadavechen1y, i destroyed environment and redeployed again, as charm was in a failed state, but no error in the debug-log10:18
davechen1yyolanda: im sorry it's not working on canonistack10:26
davechen1yi do not test on canonistack10:26
davechen1ythis is my faliing10:26
davechen1yfailing10:26
yolandadavechen1y, you mean the debug-log? or which issue?10:28
jammgz: natefinch: given you are adding Addresses to lots of providers, are we missing a LiveTest across all implementations that it is available and gives a "sane" result?10:29
jamor is the idea that you have to build it up first and then implement the conformance test?10:29
jam(I'd rather see a patch that adds a conformance test and stubs out the ones that don't implement it until we fix them, since that clearly records the current state)10:30
natefinchgreetings all10:46
TheMuerogpeppe1, fwereade, natefinch: https://codereview.appspot.com/1275204410:46
TheMuenatefinch: good morning10:46
rogpeppe1natefinch: hiya10:46
natefinchjam: Yes, I believe we're missing a live test to make sure they all work... though I think they all work, since we're explicitly implementing them to get at information we know exists10:47
fwereaderogpeppe1, (btw I cast a quick eye over it, and I think I like, but I've got a lot of reviews to churn through today)10:48
rogpeppe1fwereade: should i hold on for your review?10:48
jamfwereade: fwiw I've done quite a few of them for you already :)10:48
davechen1yyolanda: not working well on canonistack in general10:48
fwereadejam, <310:48
yolandadavechen1y, noticed some problems, yes10:49
* TheMue => lunchtime10:49
jamfwereade: of course, the only one that I didn't dig into is yours :)10:50
fwereaderogpeppe1, go ahead and merge it, I'll keep a tab open and throw a fit after the fact if I spot something awful, but I don;t expect to -- I did sneak a look and it seems solid to me10:50
rogpeppe1fwereade: cool, thanks10:50
fwereadejam, haha10:50
fwereadejam, I'm a little conflicted about it anyway, I don't think it's our top priority10:50
jamfwereade: unfortunately when you comment via Reitveld, LP doesn't move the branch from "Requested reviews" to "Reviews I am doing".10:51
jam:(10:51
rogpeppe1jam: you ok with the StartSync changes going in? (from your question earlier, i presume you've at least had a glance)10:51
jamfwereade: can you land or reject: https://code.launchpad.net/~fwereade/juju-core/errors-cleanup/+merge/16892810:51
jamrogpeppe1: I haven't looked at the patch yet.10:52
rogpeppe1jam: np10:52
jamI've been reading IRC to follow along with the discussion, though.10:52
fwereadejam, hell, sorry, hat was rotted a month ago :( rejecting10:52
rogpeppe1jam: i can hold on for you if you'd like10:52
jamrogpeppe1: first thing I saw was the "string => interface" change. Is the collect logic actually correct for things that aren't strings?10:54
jam(10:54
rogpeppe1jam: i believe so, as long as they can be used as map keys10:54
fwereadeallenap, ping10:56
jamrogpeppe1: I'm slightly uncomfortable about the change to Lifecycle watcher being bundled with changing the semantics of Sync.10:56
rogpeppe1jam: in the case of Cleanup, the ids are of type ObjectIdHex10:56
allenapfwereade: pong10:57
fwereadeallenap, do you recall, how does maas count cores? does hyperthreading count?10:57
rogpeppe1jam: it seemed like a fairly trivial change, but i could split the collect change out into another CL if you like10:57
rogpeppe1jam: (it's only 4 lines)10:58
allenapfwereade: Let me see....10:58
jamrogpeppe1: just conceptually it is something I need to think about, and seems very orthogonal to what is going on10:58
rogpeppe1jam: i needed to use collect to fix the Cleanup watcher, and it had the wrong type10:59
* fwereade hopes it's counting logical cores, not physical, because that seems to fit best with the ec2/openstack situation10:59
rogpeppe1jam: so the lifecycleWatcher needed to change for that, so it's not entirely orthogonal, but i can propose that change independently as a prereq if that's your preference11:00
fwereadeand actually... jam, mgz: do you know offhand how openstack counts a flavor's cores?11:01
allenapfwereade: I think it counts cores. It evaluates the xpath count(//node[@id='core']/node[@class='processor'][not(@disabled)]) against an lshw XML dump.11:01
jamfwereade: well they are all virtual there, right?11:01
jamI don't know how it maps hyperthreading into available cores.11:01
fwereadejam, indeed, and overcommit is a whole new can of worms11:01
bigjoolswhatever the kernel reports11:01
rogpeppe1jam: essentially it's just moving a dynamic type cast out of collect and into the watcher-specific logic11:02
jamrogpeppe1: so given that "StartSync" actually just puts the "reqSync" into the queue, and when req := <-w.request sees it, it calls handle() and *then* flush() before we loop around to 'if w.needSync', doesn't that mean your statement about "it syncs and waits for it to complete" isn't actually true?11:03
natefinchfwereade: nproc on my hyperthreaded 4 core processor returns 8, btw.11:05
rogpeppe1jam: i don't *think* so11:05
rogpeppe1jam: because handle of a reqSync will not actually add anything to be flushed11:06
natefinchfwereade: I would hope we somehow discount hyperthreaded cores, since they don't even come close to actually doubling processing power11:06
fwereadenatefinch, indeed, I'm really just fretting about whether consistency is even possible given some of our providers11:06
jamrogpeppe1: in the time since we start handling the reqSync, more might come into the channel, which will be flushed before we finish handling the reqSync.11:07
jamthat time window is very small11:07
jamsince we only really need to set the bool11:07
jambut it does exist, doesn't it?11:07
rogpeppe1jam: how could that happen? if there are no events to be flushed, then flush will never read on the channel11:08
jamnatefinch: when I was testing it 7 years ago, 2+2 hyperthreaded cores were easily equivalent to 3 cores (as in, enabling hyperthreading allowed my threaded code to run 50% faster on 2 physical cored11:08
jamcores)11:08
rogpeppe1jam: and so i'm fairly sure there are no places where new events can arrive between handling a request and calling sync()11:09
natefinchjam: last I remember it was like a +25% and then only for jobs that have a lot of thread switching.... but regardless, we shouldn't treat them as fully powered cores11:09
jamnatefinch: I think the precision of this stuff is such that people basically need to test it and figure out what works for them. :) I don't think MaaS or Juju need to grow all that aware of what the actual benchmark results for a given workload map into "cores=X".11:11
jamWe just want to provide a way for people to give their input.11:11
jamIgnoring hyperthreading completely doesn't seem quite correct, though neither is considering them 100%. But I think doing "something" is reasonable, as long as it is consistent.11:12
jamnatefinch: in the case of MaaS most likely people who *really* cared would use hardware-based tags to essentially define their own flavors, and deploy based on that.11:12
natefinchjam: yep, totally makes sense.11:14
TheMuestandup11:31
jamrogpeppe1: fwereade: https://plus.google.com/hangouts/_/f497381ca4d154890227b3b35a85a985b894b47111:32
jammgz: ^^11:33
=== ChanServ changed the topic of #juju-dev to: https://juju.ubuntu.com | On-call reviewer: - | Bugs: 7 Critical, 92 High - https://bugs.launchpad.net/juju-core/
smoserbigjools, or arosales i just tested a saucy daily from today and it seems functional. you could have been bit yesterday by bug 121454112:42
_mup_Bug #1214541: hostname setting is erroring out <amd64> <apport-bug> <cloud-images> <saucy> <cloud-init (Ubuntu):Fix Released> <https://launchpad.net/bugs/1214541>12:42
smosertested == tested from cloud-init's perspective (it ran user-data, provisioned user ... )12:42
rogpeppe1jam: "13:04
rogpeppe1LifecycleWatcher coalescing its changes is a good thing, and a pretty notable13:04
rogpeppe1change to have it silently added.13:04
rogpeppe1"13:04
rogpeppe1jam: Lifecycle watcher *was* previously coalescing its changes13:05
rogpeppe1jam: my CL doesn't change that13:05
rogpeppe1jam: it's quite possible the testing for that wasn't great though13:07
jamrogpeppe1: so is your change just making it use a common helper, or ? Either way it is a modest change that should probably make it into the "and I changed LifecycleWatcher to XXXX". It helps frame an understanding of what is actually changing. Given that I clearly got it wrong 2 times now :)13:07
jam"make it into the *summary*"13:07
rogpeppe1jam: lifecycleWatcher is still calling the same helper it always called13:07
rogpeppe1jam: it's just that i needed to change the signature of that helper so i could use it with the CleanupWatcher13:08
rogpeppe1jam: i could put "I changed the collect helper function to use interface{} keys" into the summary if you think that's worth it13:09
jamrogpeppe1: so I guess some of it is diff context: looking here: https://codereview.appspot.com/13089045/patch/5001/600913:09
jamit certainly looks like lifecycle watcher is changing13:09
jambut I realize that last diff block could be CleanupWatcher13:09
jamwhich I'm *pretty* sure means CleanupWatcher is now collecting when it didn't before13:10
rogpeppe1jam: hmm, i don't see that in my diff13:10
jamrogpeppe1: https://codereview.appspot.com/13089045/diff/5001/state/watcher.go is a different way to look at it.13:10
jambut collect() is now getting called where it wasn't before (unless I'm *completely* misreading this)13:11
rogpeppe1jam: i think you're misreading the unified diff13:11
rogpeppe1jam: the collect code is being added to cleanupWatcher13:11
rogpeppe1jam: occupational hazard with unified diffs, i fear13:11
jamrogpeppe1: so I misread that it was lifecycle, but the code is added to cleanupWatcther which is the same thing I'm mentioning (adding collection to something that wasn't collecting before)13:11
jamrogpeppe1: so s/Lifecycle/Cleanup/ and my comment still applies, I think.13:12
rogpeppe1jam: you're right. sorry, i was thrown off by the name13:12
jamrogpeppe1: we seem to be missing a test for CleanupWatcher now, given that if you revert that change no tests will break, right?13:12
jamso we don't *know* that cleanup is collecting13:12
rogpeppe1jam: no, i made the change because tests broke13:13
rogpeppe1jam: in particular, TestWatchCleanup failed13:13
rogpeppe1jam: i'll retry to make sure of that13:14
rogpeppe1jam: hmm, it doesn't seem to fail any more!13:18
jamfwereade: just a gentle nudge to remind you to submit a couple bugs about container.go13:18
* jam is off for the evening, though I'll probably respond at some point later13:19
fwereadejam, still talking to ian, have a doc open ready to convert :)13:19
rogpeppe1jam: i'll add a specific test for cleanup event coalescence13:21
smoserrvba, are you able to reproduce https://bugs.launchpad.net/juju-core/+bug/1214636 ?13:51
_mup_Bug #1214636: Azure Provider: Deployed service never goes to started <cloud-init:New> <juju-core:Confirmed> <https://launchpad.net/bugs/1214636>13:52
=== tasdomas is now known as tasdomas_afk
rogpeppe1i'm going to be offline for an hour or so, then i should have sporadic network access for 5 hours after that14:00
=== rogpeppe1 is now known as rogpeppe
rogpeppe1fwereade: ping15:08
natefinchanyone here familiar with lxc?  I used juju to deploy locally, and I'd like to open up the service to computers on my network, but I don't really know how to do it. Right now the services just have local IP addresses like 10.0.3.187... .how do I expose that to my local network?15:46
mgzthere's not a trivial way of doing that15:51
natefinchhuh ok15:52
mgzyou can use iptables or similar to manually route traffic on a port in, for instance15:53
mgzand we've briefly discussed making juju expose do something like that15:53
natefinchmgz: yeah, manually routing the traffic is pretty much what I was thinking15:55
mgzI'd give you the iptables command you need, but you can probably google it as easily as me :015:55
natefinchmgz:  haha yeah, that's what I was just doing, no worries15:56
arosalesfwereade, or rogpeppe1 any core folks interested/have time to join our weekly charm sync.16:33
arosalesfwereade, rogpeppe1 wed at 16:0016:33
arosalesutc, that is.16:33
fwereadearosales, yes please, sign me up16:34
arosalesfwereade, thanks I'll add you to the invite. Please feel free to delegate and/or let me know if I should any other folks.16:34
fwereadearosales, I might end up doing so occasionally, but I'm very interested personally16:35
arosalesfwereade, be great to have some core folks there, thanks! :-)16:36
arosalesfwereade, invite sent.16:38
=== BradCrittenden is now known as bac
fwereadehey, it's after 6; I'm tired, see you all later :)17:07
arosalesgot a juju ssh question if any folks are around17:44
natefincharosales: I'm around, but you probably know more than I do.  However, might as well ask :)17:47
arosaleson juju core I am sure you have me beat :-)17:47
natefincharosales: We'll see :)  I can at least search the codebase pretty easily for answers that can be answered that way17:48
* arosales getting17:49
arosales2013-08-21 17:43:25 ERROR juju supercommand.go:282 command failed: required environment variable not set for credentials attribute: User17:49
arosaleserror: required environment variable not set for credentials attribute: User17:49
arosalesbut I am using keys values not user17:49
arosalesalso odd it is opening up my hp environment when I state an aws one . . .17:50
arosalesjuju --debug ssh 1 -e aws-go17:50
arosales2013-08-21 17:43:25 INFO juju provider.go:121 environs/openstack: opening environment "hp-go"17:50
arosaleshmm, that may be the problem . . .17:50
natefinchheh17:50
arosalesthats odd17:50
natefinchthe code looks like it'll say "user" even with keys17:50
natefinchwhich looks like a copy and paste error17:51
natefinch(or at the very least, the error message should be made clearer)17:51
natefinchnah, itlooks like copy and paste.... looks like the section on username/password got copied for keys and then didn't change the error message17:52
natefinchBut, it sounds like you may have figured it out anyway?17:52
arosalesjuju seems to be picking up my bash env setting when trying to ssh instead of my command line option, and just for ssh17:52
arosalesjuju --debug stat -e aws-go17:53
arosales2013-08-21 17:52:08 INFO juju ec2.go:137 environs/ec2: opening environment "aws-go"17:53
arosalesbut ssh tries my "hp-go" environment even though I state "-e aws-go"17:53
natefincharosales: sounds like a bug, though it's weird that it would happen only in one subcommand, I'd expect that to be shared logic... but I can see if I can find where that's set17:54
arosalesi'll open a bug and see if I can work around this by taking out my env  setting in my bashrc file17:55
natefinchcertainly if you say -e and it doesn't use that environment, that's a bug :)17:56
arosaleshttps://bugs.launchpad.net/juju-core/+bug/121505218:02
_mup_Bug #1215052: juju ssh ignores the command line "-e"  and instead uses JUJU_ENV in my .bashrc <juju-core:New> <https://launchpad.net/bugs/1215052>18:02
arosalesbug filed18:02
arosalesodd issue18:02
natefinchyeah, pretty weird18:04
smoserarosales, are you sure it wouldn't work with18:07
smoserjuju -e aws-go --debug ssh 118:07
arosalessmoser, yup18:07
arosalesthat the exact command I ran18:07
smoserwell, its not what you typed above18:08
smoserits not completely unreasonable if juju stopped looking for flags to it after it saw 'ssh', and instead passed those forward to something else.18:08
arosales<arosales> juju --debug ssh 1 -e aws-go18:08
smoserie18:08
smoser<smoser> juju -e aws-go --debug ssh 118:08
arosalessmoser, so you are saying order of arguments matter18:09
smoserorder maybe important18:09
smoseryes. its not terribly uncommon.18:09
arosalesgood point18:09
smoserespecially if juju was going to pass other options on to ssh18:09
smoserie:18:09
smoserjuju --debug ssh 1 run this command18:09
arosalessmoser, I'll try your order here18:10
smoserarosales, also, icame here wondering if you tested aws and saucy18:10
smoser(which i suspect you were trying :)18:10
arosalessmoser, btw I was working on trying to confirm precise ssh is still working18:10
arosalesper the Azure bug <arosales> juju --debug ssh 1 -e aws-go18:11
natefinchsmoser: good point, ssh may be different because it expects to be passed arbitrary arguments to be run18:11
arosaleshttps://bugs.launchpad.net/juju-core/+bug/121463618:11
_mup_Bug #1214636: Azure Provider: Deployed service never goes to started <cloud-init:New> <juju-core:Confirmed> <https://launchpad.net/bugs/1214636>18:11
arosaleslol18:13
arosalesjuju -e aws-go --debug ssh 118:13
arosaleserror: flag provided but not defined: -e18:13
smoser?18:13
arosalesbut, the following works18:13
smoserodd.18:13
arosalesjuju --debug stat -e aws-go18:13
arosales2013-08-21 18:13:05 INFO juju ec2.go:137 environs/ec2: opening environment "aws-go"18:13
arosalessmoser, if I put -e towards the front of the command, juju doesn't recognize it18:13
smoserput it after ssh and before '1'18:14
smosermaybe. just try that.18:14
arosalessmoser, so I can't run your command example, 'juju -e aws-go --debug ssh 1"18:14
smoserarosales, try18:14
smoserjuju --debug ssh -e aws-go 118:14
smoserthat would not be terribly unreasonable if the '-e' was not a juju global flag, but was a flag to the 'ssh' sub command.18:15
smoser(and just happened to be a flag to many subcommands)18:15
arosalessmoser, that does work18:15
smoseralright.18:15
smoserwell thats at least moderately sane18:16
smoser(i'd even argue "perfectly fine")18:16
arosalesbut from a mere mortal like me totally unreasonable18:16
smoserso did you verify that juju works with saucy ?18:16
arosalessmoser, so precise works on aws18:16
arosalesI can ssh18:16
natefinchyeah... seems like juju -e should work, even if the command you're running doesn't care about the environment18:16
smoserarosales, bigjools reported precise (with his custom image) worked.18:16
arosalessmoser, I went back to precse on aws as I was getting the above error18:16
smoseri wanted to see if saucy worked on aws. to rule out general saucy error.18:17
smoseras it really does seem to me that cloud-init is functioning correctly.18:17
arosalessmoser, trying that now18:17
smoserarosales, can you ssh to your precise aws ?18:18
arosalessmoser, yes18:18
smoseri was going to ask you to give me output of 'ec2metadata --user-data'18:18
arosalesbootstrapping with saucy now on aws18:18
smoserin a secure chanel18:18
arosalesah, just destroyed18:19
arosaleslet me see if saucy works and I can get you user data off precise18:19
arosalessmoser, saucy ssh with juju works on aws18:41
arosalessmoser, so sounds like the issue is Azure specific . .  . ?18:41
smoserarosales, can you point me at the doc you have so far ?18:43
smoserand i'll try to reproduce it?18:43
smosermarcoceppi, http://marcoceppi.com/2013/07/compiling-juju-and-the-local-provider/19:33
smoserexport GOPATH="~/.juju/"19:33
smoseris wrong19:33
smoserquoting the '~' explicitly creates a directory called '~'19:33
marcoceppismoser: that's...odd.19:34
smoserits expected :)19:34
marcoceppiIt's odd that I put that quoted in the post19:34
marcoceppiI'll update it!19:34
smoser2 other things19:35
smoserhm..19:35
smoseryou need bzr for 'go get'19:35
marcoceppismoser: ack, that was already installed on my system19:36
marcoceppiI didn't run these against a "clean" machine19:36
marcoceppiWill update the post19:36
smosermarcoceppi, becaus eyour'e the expert (i'm follooing your blog)19:39
smoserdo you know19:39
smoserhttp://paste.ubuntu.com/6011504/19:39
smoseranyone else maybe ?19:39
marcoceppiI've not recieved that error before. What version of ubuntu? I can try to replicate19:39
natefinchsmoser: almost certainly because you're running go 1.02 and juju uses 1.1 now19:40
natefinchunfortunately apt-get still installs 1.0.219:40
marcoceppinatefinch: ah, was that a recent switch? I've not tried to compile since 1.11.419:40
natefinchmarcoceppi: yeah in the last month or so, I think19:41
marcoceppinatefinch: ah, sorry smoser, I'll update my blog to reflect the golang ppa too19:41
marcoceppinatefinch: actually, I thought there was a ~gophers ppa, but I don't see 1.1 in there19:43
natefinchmarcoceppi: https://groups.google.com/forum/#!topic/golang-nuts/iJFhI8K5a2Y19:44
marcoceppinatefinch: bummer :\19:45
smoseryeah, there is no raring either in golang ppa19:45
smosersaucy!19:46
natefinchmarcoceppi: when I started I think I had to use the tarball from golang.org19:51
=== BradCrittenden is now known as bac
sidneismoser, marcoceppi: https://launchpad.net/~james-page/+archive/golang-backports20:03
marcoceppisidnei: awesome, thanks@20:03
marcoceppisidnei smoser updated the blog post20:04
smoserhow do you normally tell juju about ssh public keys?20:06
smoserfor maas it seems to have explicitly (in config)20:06
smoser    authorized-keys-path: ~/.ssh/authorized_keys # or any file you want.20:06
smoser    # Or:20:06
smoser    # authorized-keys: ssh-rsa keymaterialhere20:06
smoserbut that is provider specific ?20:06
marcoceppismoser: it normally just uses whatever is in ~/.ssh/id_rsa.pub20:07
marcoceppiYou can add additional ones using either the two keys above20:07
natefinchmarcoceppi: I deployed discourse using juju and it looks great except that in the registration confirmation emails, it's using the internal hostname of the EC2 instance, instead of the external hostname.... any thoughts on how to fix that?20:22
marcoceppinatefinch: Yeah, you'll need to edit the config/database.yml file20:35
marcoceppiI need to add "external hostname" configuration option to the charm20:35
marcoceppito automatically add that information in20:35
natefinchmarcoceppi: I'd be happy to contribute to the charm :)20:36
marcoceppinatefinch: please do!20:36
* marcoceppi syncs his local version to charm branch20:36
natefinchmarcoceppi: I didn't think you'd say no ;)20:36
natefinchmarcoceppi: I am completely new to the charms, so... it'll take me some ramp up time. But especially for this particular charm, I want to use it for personal reasons. So I have skin in the game that it works well :)\20:37
marcoceppinatefinch: I've got several fixes to the auto-thin configuration that I need to land, so I'm not sure how far behind my cs:~marcoceppi/discourse version actually is20:37
marcoceppibut the github branch is always a little ahead20:37
marcoceppiand then I sync when the charm is "stable" again to the charmstore branch20:37
* marcoceppi enjoys convoluted processes20:38
natefinchmarcoceppi: lol fair enough20:38
marcoceppibut yeah, that part could use a bit of work (ie, update database.yml file and the nginx config with the hostname)20:39
marcoceppiI tried to keep the charm pretty simple, as I wanted to use it as an example charm, but it's kind of grown a bit beyond that, so if you have any questions as to why I've backed bits of crack in to the charm, don't hesitate to ask20:40
marcoceppibaked*20:40
natefinchhaha20:40
natefinchok20:40
smoserany ideas?20:56
smoserhttp://paste.ubuntu.com/6011776/20:56
sidneiwhoa20:58
sidneilooks like it choked on the cert?20:58
smoserhttp://paste.ubuntu.com/6011792/21:00
smoserit sure does look like that. i agree21:01
smoserthat above is what i've done so far to get here.21:01
bigjoolsmorning21:51
bigjoolshi arosales21:53
arosalesbigjools, morning22:03
arosalesbigjools, smoser was unable to reproduce what you and I found on saucy22:03
bigjoolsarosales: because he wasn't using juju22:04
arosalesbigjools, actually he was22:04
bigjoolsI only saw the azure command line tool being used22:04
arosalescompiled the latest even22:04
* arosales not sure if he updated with his latest22:04
bigjoolsarosales: the last comment on https://bugs.launchpad.net/bugs/1214636 doesn't show juju22:05
_mup_Bug #1214636: Azure Provider: Deployed service never goes to started <cloud-init:New> <juju-core:Confirmed> <https://launchpad.net/bugs/1214636>22:05
arosalesbigjools, smoser had to run so he may not have updated his bug with the latest22:05
bigjoolsrighto22:08
thumperhi folks22:44
thumperthis branch is giving me the shits22:44
thumperI thought it would be a small, simple branch22:44
thumperbut OH, NO, no it isn't at all22:44
thumperjust replacing agent.Conf with an interface, and unexporting the structure22:46
thumperFFS it is tedious22:46
thumperI seem to have all tests passing except for agent/agent_test22:46
thumperwhich is because there are shed loads of explicit struct tests22:46
thumperso I left it for last22:46
thumpertrying to do the bare minimum to get this landed22:47
thumperaargghh!22:47
thumperthat and no ubuntu edge to make me feel better22:47
fwereadethumper, ha, I only just got round to thinking "bah, I'd better get one in case a mystery benefactor kicks in 20M at the last moment", but paypal can go fuck itself, so meh22:57
thumper:)22:57
thumperso no comment on the rest of the rant then :)22:57
thumperfwereade: I may throw it at you for review22:58
fwereadethumper, just reading backwards22:58
* thumper looks at the size22:58
fwereadethumper, I won't say it'd be a *pleasure*, but I won't complain too much ;p22:58
* thumper sucks wind22:59
thumper1600 lines ATM22:59
fwereadeouch22:59
thumper 17 files changed, 441 insertions(+), 446 deletions(-)22:59
thumperand this is the simplest thing22:59
thumperthat made sense22:59
fwereadefucking structs22:59
thumperlong live interfaces22:59
* fwereade girds his loins in preparation for the morrow then23:01
sidneithumper: im trying to figure out why local provider is not setting the hostname of the containers it creates, and if this is an lxc bug or not23:25
thumpersidnei: it normally does23:25
thumpersidnei: using clone?23:26
sidneithumper: nope, trunk without my changes23:26
sidneithumper: but im using the daily lxc ppa23:26
sidneiso might be a change there23:26
thumpersidnei: it should, and is a template param23:26
thumper--hostid23:27
thumperline 154 of container/lxc/lxc.go23:27
wallyworldthumper: i'm going to move the Broker interface from worker/provisioner to (somewhere, probs instance), and make Environ be composed from Broker and remove the duplicated Environ methods23:32
wallyworldas part of some refactoring23:32
thumperok23:32
wallyworldsince part of it is extracting common start nstance code23:32
wallyworldi am sharing some of your refactoring pain :-)23:32
wallyworldi still don't fully understand why the fcuk we used structs and not interfaces23:33
wallyworldmaybe whoever did it did not read software enginerring 10123:33
thumperI believe it was because "we don't need it yet" was the rationale23:37
thumperhowever, the retrofitting of interfaces is a royal PITA23:37
wallyworldthere is no such thing as "we don't need it yet" with interfaces23:39
wallyworldthey are needed for all sorts of things from day one, not the least of which is for tests23:40
wallyworldand extensible, refactorable code23:40
wallyworldand Go's design almost mandates their use unless you want gobs of cut and paste boilerplate everywhere23:41
thumperwallyworld: I agree23:43
* wallyworld sighs23:43
thumperwallyworld: but then again, you and I often do23:43
wallyworldoften but not always :-)23:43
wallyworldlike with rugby23:43
thumperno, that would be boring23:44
thumperso, think you have a shot this weekend?23:44
wallyworldnope :-(23:44
thumperheh23:44
* thumper heads to the gym23:44
wallyworldwe're fooked23:44

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