/srv/irclogs.ubuntu.com/2014/08/15/#juju-dev.txt

hazmatwallyworld, ping00:31
wallyworldhazmat: otp, one sec00:32
thumpertime to wrap up warm and take the dog for a walk00:41
davecheneythumper: still snowing ?00:43
* hazmat wonders if the dog gets wrapped00:43
davecheneyhttp://www.nancysdogpawsboutique.com/images/products/Italian%20Greyhound%20English%20Style%20Fleece%20Dog%20Coat%20brown%20001.jpg00:45
davecheneysinzui: is wolfe-01 you box ?01:04
thumperhazmat: no, not that cold, dog is fine01:35
thumperdavecheney: not snowing, just cold01:35
davecheneythumper: this one is for you, https://github.com/juju/loggo/pull/301:39
* thumper looks01:39
thumperdavecheney: what is that?01:40
thumpergodoc badge01:40
axwjust a link to the godoc page for loggo01:41
thumperah...01:41
thumperok01:41
davecheneywaigani: are you working on https://bugs.launchpad.net/juju-core/+bug/1356899 ?01:53
mupBug #1356899: manual provider add-machine fails if no ubuntu user on machine <manual-provider> <regression> <juju-core:Triaged> <https://launchpad.net/bugs/1356899>01:53
waiganidavecheney: the revert branch has merged01:54
thumperwaigani: did it? I thought it was rejected01:55
waiganiPull request successfully merged and closed01:56
waiganiYou're all set—the waigani:revert-manual-prov  branch can be safely deleted.01:56
thumperok, good01:56
waiganidavecheney: ^ from github01:56
davecheneywaigani: please assign yourself to the issue and mark it in progress01:56
* thumper scrolls up01:56
thumperoh, there it goes01:56
davecheneyif it's fixed, mark it as fix-comitted to unblock the build01:56
thumperactually, mark it fix committed01:56
waiganidone01:57
thumperwaigani: are you going to re-propose without the deleted line?01:57
waiganithumper: yep01:57
wallyworldaxw: finished02:09
thumpercoffee needed02:18
hazmatcoffee finished02:29
hazmatwaigani, btw.. if you want a functional test case  for that regression try... using juju digitalocean plugin02:31
waiganithumper: axw: https://github.com/juju/juju/pull/52402:40
thumperhazmat: do DO machines not have the ubuntu user?02:41
hazmatthumper, no ubuntu image thats not cpc cloud does02:41
thumperEPARSE02:41
thumperreally?02:42
thumperso, since DO are not CPC, they have no ubuntu user?02:42
thumperwhy is that?02:42
hazmatthumper, you've installed ubuntu desktop02:42
thumperwhat do they have instead?02:42
hazmatthey have a sane version of ubuntu server02:42
thumperthey need some user right?02:42
hazmatthumper, yeah... its called root02:42
axwwaigani: LGTM, but please test it live02:42
thumperreally?02:43
thumperew02:43
waiganiaxw: okay, will do now.02:43
thumper:-)02:43
hazmatthumper, ew.. like a user who can admin stuff without typing sudo..02:43
thumperyeah, yeah02:43
davecheneyscott/tiger02:43
* hazmat bows to the oracle 02:43
* thumper missed that reference02:44
hazmatthumper, default/doc'd/example'd password on oracle02:44
davecheneythumper: http://xforce.iss.net/xforce/xfdb/97002:44
thumper?!02:44
thumperheh02:45
menn0davecheney: do you have time to cast your eyes over this again? https://github.com/juju/juju/pull/50803:06
* davecheney looks03:10
waiganiaxw: $ juju add-machine ssh:scott@10.0.3.24403:21
waiganiscott@10.0.3.244's password:03:21
waigani[sudo] password for scott:03:21
waiganiinstall: cannot create directory /home/ubuntu: Permission denied03:21
waiganiERROR subprocess encountered error code 1 (Connection to 10.0.3.244 closed.)03:21
waiganiaxw: scott is in the sudoers file03:21
waiganiaxw: I just ran the same with trunk, I get same result03:26
axwwaigani: sorry was afk having lunch03:59
waiganiaxw: np, I've commented on PR03:59
axwwaigani: can you confirm you can "sudo mkdir /home/ubuntu" on the machine when logged in as scott?03:59
waiganiaxw: yep I could03:59
waiganiaxw: but it asks for scott's password04:00
axwwaigani: add-machine should prompt you for that, is it not doing so?04:01
waiganiaxw: yep it did04:01
waiganiaxw: same behaviour on trunk btw04:02
axwwaigani: I'll try to repro, sounds odd04:03
waiganiaxw: that would be great, thanks04:04
axwwaigani: I cannot reproduce on master. did you remove /home/ubuntu? userdel does not do that by default04:11
ericsnowHere's a patch that fixes the blocking bug if anyone has a minute: https://github.com/juju/juju/pull/52504:13
ericsnowmgz_: you have a minute?04:15
thumperericsnow: done04:15
ericsnowthumper: thanks04:16
ericsnowwallyworld: ping04:37
wallyworldericsnow: hey04:38
ericsnowyou have a minute to talk about reviewboard04:38
wallyworldericsnow: sure, hangout?04:39
ericsnowsure04:39
wallyworldericsnow: https://plus.google.com/hangouts/_/canonical.com/tanzanite-stand04:40
waiganiaxw: does it work for you?04:50
axwaxw> waigani: I cannot reproduce on master. did you remove /home/ubuntu? userdel does not do that by default04:50
waiganiaxw: yes04:50
waiganistrange04:50
axw*shrug*, just worked for me04:50
axwplease look into what's going on there, because it doesn't sound right04:51
waiganiyep04:51
waiganiaxw: just to confirm, 'scott' just needs to be in the sudoer's file04:52
axwwaigani: well, I just s/ubuntu/scott/ in the juju file in /etc/sudoers.d04:53
waiganiaxw: did you delete or rename home dir?04:54
axwwaigani: I deleted /home/ubuntu04:54
waiganiaxw: I did sudo adduser scott sudo04:54
axwwaigani: I'll try again with that method.04:56
waiganiaxw: and I'll try swith your04:57
waiganiaxw: ah04:58
waiganiaxw: it's still ubuntu in sudoers.d04:59
axwwaigani: ?04:59
axwwaigani: it should be. if you just did "adduser scott sudo" that's just adding it to the group04:59
waiganiaxw: right, I thought that was enough05:00
axwwaigani: what that doesn't do is enable passwordless login05:00
axwbut if you have a password that should be fine05:00
waiganiaxw: scott has a password and I gave it to juju add-machine05:02
axwwaigani: works for me using that approach too.05:04
axwwaigani: my /home is 075505:04
waiganiaxw: and *not* changing /etc/sudoers.d ?05:04
axwwaigani: correct, I just did "sudo adduser scott sudo" and used the password05:05
waiganihmph05:05
waiganiaxw: I can't reproduce it now, everything seems to be working as expected.05:25
axwhrm ok.05:25
axwwell, I say land it. CI and/or hazmat will complain if it's still broken ;)05:26
waiganiokay, I'll jfdi it05:26
voidspacemorning all06:14
dimiternmorning voidspace06:19
voidspaceo/06:19
TheMuemorning07:16
voidspaceTheMue: morning07:18
stubReading the pain points spreadsheet, and I was thinking a manifest file in the charm, listing one or more URIs and the path to install or unpack them, could solve several items. It that the sort of thing already being talked about?07:55
dimiternmorning TheMue08:08
TheMuedimitern: heya, seen my mail?08:09
dimiternTheMue, yep, I haven't had time to reply yet, but I'll get back to you soon (I have a call with Mark S in 15m)08:14
TheMuedimitern: ok, np, and thanks already08:15
TheMueHmmmpf, shitty net, currently it dislikes me.09:26
* fwereade_ is on a public holiday and not working10:03
* fwereade_ would still like to express his frustration at not being able to express a `chan foo` as a `chan interface{}`10:03
dimiternTheMue, I think we need a branch to expose all capabilities on the apiserver/agent facade10:27
TheMuedimitern: I found a way10:27
dimiternTheMue, including the new one; no need to expose the environ config, just the part of the interface defined as state.EnvironCapabilities10:27
TheMuedimitern: I’ll paste the part, one moment10:28
dimiternTheMue, the provisioner and upgrader api facades have access to the environ config, but that's a hacky approach10:28
TheMuedimitern: http://paste.ubuntu.com/8052638/10:29
TheMuedimitern: I’m using the config too10:30
dimiternTheMue, ah, this looks nice10:31
dimiternTheMue, cheers10:31
TheMuedimitern: so right now the providers only return constants, now the second part letting them return real values10:32
dimiternTheMue, what constants?10:33
TheMuedimitern: as I understand the local bootstrap as well as manual and manually provisioned machines on other providers need the safe networker10:33
dimiternTheMue, yes, all the other cases (maas, ec2, etc.) will use the normal networker10:34
TheMuedimitern: true, but nothing I’ll push before the change descibed above. only wanted to see it compiling10:34
TheMuedimitern: do manually provisioned machines exist on maas too? today this isn’t covered10:35
dimiternTheMue, to make it reusable, I'd add a method to the agent that returns state.EnvironCapability, and internally uses the st.Environment().EnvironConfig() + config.New, etc.10:36
dimiternTheMue, manually provisioned machines can exist regardless of the provider the environment uses10:36
TheMuedimitern: ok, sounds good10:36
dimiternTheMue, cheers10:37
dimiternvoidspac_, standup?10:47
voidspac_dimitern: sorry!10:47
voidspac_dimitern: TheMue: omw10:47
dimiternmgz_, hey11:15
mgz_hey11:16
dimiternmgz_, so you're switching to the juju ci team?11:16
mgz_yup11:16
dimiternmgz_, congrats! :)11:16
dimiternmgz_, that was kinda sneaky hehe11:16
katcogood morning, team :)12:21
mgz_hey katco12:23
katcocongratulations on the move, mgz_ :)12:23
wallyworldkatco: hey, i'm back from soccer if you wanted to chat12:44
katcowallyworld: you're pretty much up to speed w/ what i'm doing. but we can definitely chat if you'd like :)12:44
wallyworldkatco: only if you need to, happy to let you keep going... :-)12:45
katcowallyworld: $$greatful it landed$$12:45
katcoerr *grateful12:45
wallyworld:-)12:45
katcoi do have one question: did you win? ;)12:45
wallyworldkatco: nope, our stikers kept missing the net :-(12:49
katcoaw bummer12:49
katcowhat position do you play?12:49
wallyworldleft right out :-)12:49
wallyworldcentre back12:49
katcoah! that was my old position :) sweeper/left fullback12:49
katco<3 defense12:50
wallyworldi like that position, less running12:50
katcohaha12:50
wallyworldplus i read the game well, so it suits me12:50
katcoyeah it always struck me as an analytical position12:50
* wallyworld wonders if fwereade_ is around12:51
TheMuewallyworld: public holiday, but has been around this morning once12:52
wallyworldTheMue: tanks12:53
wallyworldthanks12:53
TheMueyw12:54
=== ChanServ changed the topic of #juju-dev to: https://juju.ubuntu.com | On-call reviewer: see calendar | Open critical bugs: None
mattywfwereade_, I'm working on that api to get a list of metrics if you fancy doing some bikeshedding with me ;)13:14
jrwrenwho are the eco folks?13:19
ericsnowmgz_: ping13:32
mgz_ericsnow: hey13:40
ericsnowmgz_: just following up on my reviewboard email :)13:41
mgz_yeah, 's in my queue of many emails to respond to...13:41
ericsnowmgz_: no worries (wanted to be sure it didn't get lost in the ether)13:42
mgz_ericsnow: short answer is past the charm and some minor changes, most of the work is in just deploying the thing13:46
ericsnowmgz_: awesome!13:46
mgz_so, ther's not *much* to pass over, one thing to sort out is where to host it13:46
ericsnowmgz_: I was considering pinging Curtis about that (after I'd gotten info from you)13:47
=== hatch__ is now known as hatch
alexisbericsnow, on and ready when you are17:01
ericsnowalexisb: brt17:02
katcoalexisb: hey!17:38
sinzuihi katco ericsnow do either you have a minute to review https://github.com/juju/juju/pull/52618:45
* katco looking18:45
ericsnowsinzui: sure18:45
katcosinzui: looks benign to me, although i'm not sure exactly what18:49
katcohangs off of version.Version18:49
ericsnowsinzui, katco: yeah, same here18:51
sinzuiericsnow, katco Both version changes define the internal version juju or the installer reports. It is also the version that packaging will adopt18:52
sinzuiI make this change after ever release, though since I haven't really released, I am a little disgruntled I need to make this change18:52
ericsnowsinzui: why don't we use some tagging mechanism in our versions (e.g. 1.20.0-a1, 1.20.0-rc3)?18:54
katcoericsnow: +118:54
natefinchdon't we do that?  we have like 1.20.0-alpha18:55
katcoericsnow: i mean presumably versions are free, but there's a cognitive overhead people have when you jump from 1.20.2 to 1.20.4 or 1.20.518:55
ericsnownatefinch: not if version/version.go has anything to say about it18:56
sinzuinatefinch, we cannot go back in time to tell 1.18 not to panic when it sees an alpha in the version18:57
ericsnowsinzui: yuck18:57
natefinchsinzui: oh, I see... so for 1.20.x we can't use our fancy new system for alphas, but for the release after, we can, correct?18:58
katcosinzui: so if we wanted to make that change, we need a major point release?18:58
sinzuinatefinch, yep, all this pain of internal testing wouldn't be happening if we were already on 1.2118:59
natefinchsinzui: can we just call the next release 1.21? :)18:59
ericsnowsinzui: ah, it's just a 1.20 issue19:00
sinzuinatefinch, well, we could all agree to abandon 1.20.x dev becomes 1.22-alpha1 and stable becomes 1.21-rc1...because that is how we are actually treating these branches/releases19:00
natefinchsinzui: version numbers aren't important to developers... but I presume there's someone else who would raise a stink if we did that19:01
sinzuinatefinch, they are important to debian/ubuntu. Since 1.20.x is not in either, I don't think anyone would notice19:02
* sinzui is not sure how to announce a major version that doesn't offer new features19:03
natefinchsinzui: "hey guys, this makes my life 100x easier and will mean we get releases out faster"  ... I think everyone will understand.19:04
katcosinzui: it supports v4 AWS signing i.e. China region :)19:04
natefinchthere, see?19:04
katconot that i have enough information to have a horse in this race, but there's that19:04
katco_although_ we did not have a china instance to test against19:05
katcoso i would hate to call that out as a major value-add and then have it not work (even though all signs say that it will)19:05
natefinchsinzui: the new feature is that dev revisions are now -alpha :)19:05
katcohaha19:05
sinzuinatefinch, the dev features really are alpha. Given the problems getting them to pass, I think we can say we are a month away from a  beta19:06
sinzuii even have plans to release an alpha until we have sorted out the utopic stable release19:07
katcosinzui: hey is there a doc that describes our release strategy? versions, process, etc.?19:13
sinzuikatco, not versions since we just switched the process.19:13
katcosinzui: ah ok. i'd love to grok that at some point (like i don't already have enough to learn) :p19:14
sinzuikatco, CI tests the release process so this doc and all the test scripts document what must happen to be a blessed https://docs.google.com/a/canonical.com/document/d/1o0au1ZNPpA8sGBNj2yNgOV3PH7Oxpyqw3hQNuIFaS-w/edit19:15
sinzuikatco, This template illustrates the automated and manual steps of a release https://docs.google.com/a/canonical.com/document/d/1aPOHa_JNKjKd5lAcOlFahjbNl8V_KgtMx6rgYdC2f7k/edit19:15
katcosinzui: cool, thank you!19:16
sinzuikatco, this is the log of the 1.20.1 release which is a little easier to read https://docs.google.com/a/canonical.com/document/d/1BOWKU4Iqy363pp4w8IWBwabDT2TdA9fb4futQVjtuVQ/edit19:16
natefinchkatco: odd minor versions used to be dev releases, even minor versions were stable.  But luckily we realized that was dumb and non-obvious, so we switched to "anything N.N.N is stable, anything N.N.N-<some_words> is a dev release"19:17
katconatefinch: ah ok19:17
natefinchkatco: and stable releases only upgraded to new stable releases, and dev->dev.  so 1.20 is the cross-over, where it knows about the new naming scheme, but 1.18. which wants to upgrade to 1.20, doesn't.19:18
katconatefinch: gotcha. thus we can't fiddle with versions in 1.20 b/c 1.18 wouldn't know what to do?19:19
natefinchkatco: correct19:20
katcomakes sense19:20
jcw4TheMue, axw https://github.com/juju/juju/pull/527 PTAL :)19:25
katcofun friday fact: i just realized i have 3 8 core machines at my disposal here at home. that's quite nice :)20:36
katcoi wonder if i could set up my own juju ci to continually run tests as i commit so it's not a big bang at the end :p20:37

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