/srv/irclogs.ubuntu.com/2013/06/12/#juju-dev.txt

thumpercontainer "machine-1-lxc-0" created00:00
thumperwell that bit worked...00:00
wallyworldthumper: ParentId() committed, looking into bootstrap machine container issue00:12
thumperack00:16
wallyworldthumper: i just bootstrapped on canonistack and everything works as expected00:48
thumperwallyworld: try ec200:48
wallyworldi tried on ec2 and got an error running status- error: cannot log in to admin database: auth fails00:48
wallyworldbootstrap seemed to work00:48
thumperwhich zone?00:48
wallyworldus-east-100:48
thumperhmm...00:49
thumpermy status works00:49
wallyworldstill, there should be no difference between add-machine on canonistack vs ec200:49
wallyworldi wrote a test for inject machine and containers and will add that test as a driveby00:50
wallyworldthumper: did you remember to "go install launchpad.net/juju-core/..."00:50
thumperaye00:51
wallyworldif not, that would explain your issues perhaps00:51
thumperhmm...00:51
thumperv.weird00:51
wallyworldwhy does my brightness keep maxing out. frustrating when trying to conserve power00:52
* thumper grabs the parent bits as needed now00:53
wallyworldthumper: status just started working on ec2. go figure00:53
fwereade__wallyworld, ec2 sometimes just pretends you have no instances00:54
wallyworld:-(00:54
fwereade__wallyworld, it's just one of those fun things than brighten your day every now and then00:54
thumperec2 sucks00:54
wallyworldwould that explain the auth failure on loggin to admin db?00:54
wallyworldthumper: add-machine works for me on ec200:54
thumperwallyworld: how about 'juju add-machine 0/lxc'00:55
thumperbecause that is what failed for me00:55
wallyworldthumper: yes, that works00:55
wallyworldthat's what i tried00:55
* thumper pulls a face00:55
* thumper tries00:55
thumperjust merged latest, will rebuild and try00:55
wallyworldian@wallyworld:~$ juju add-machine 0/lxc  -e amazon00:55
wallyworldian@wallyworld:~$ juju status -e amazon00:55
wallyworldmachines:00:55
wallyworld  "0":00:55
wallyworld    agent-state: started00:55
wallyworld    agent-version: 1.11.1.100:55
wallyworld    dns-name: ec2-107-22-111-226.compute-1.amazonaws.com00:55
wallyworld    instance-id: i-e77d5b8500:55
wallyworld    series: precise00:55
wallyworld  0/lxc/0:00:55
wallyworld    instance-id: pending00:56
wallyworld    series: precise00:56
wallyworldservices: {}00:56
=== wedgwood is now known as wedgwood_away
thumperwallyworld: hmm...00:58
thumperwallyworld: you should have quotes around 0/lxc/000:58
wallyworldah yes. true00:59
wallyworldi didn't change status or anything. i wonder why they are issing. will need to look00:59
wallyworldi mean, a container is just a machine00:59
wallyworldi'll fix it when i update status to indent containers etc01:00
fwereade__wallyworld, thumper: no you shouldn't. yay yaml01:01
thumperfwereade__: eh?01:01
thumperfwereade__: so "0" shouldn't have quotes?01:01
fwereade__thumper, "0" needs them lest it be interpreted as an int01:01
thumperew01:02
thumperthat is orrible01:02
wallyworldfwereade__: sure, but when printing to console, shouldn;t the output be "nice"?01:03
wallyworldiw leave off the "01:03
fwereade__wallyworld, then it wouldn't be yaml01:03
wallyworldsure, but it's just for the user to look at? or are we expecting to pipe it into a file? or?01:04
fwereade__wallyworld, you can almost certainly customise the yaml output but I don't consider the benefit to be worth the hassle of finding out how to do so01:04
wallyworldany anyway, what are you still doing awake?01:04
fwereade__er, no good answer there01:04
fwereade__wallyworld, it's yaml because that's meant to be nice to look at01:04
fwereade__wallyworld, it's less nice than might be hoped in this case01:05
wallyworldyeah01:05
fwereade__wallyworld, we're not making --format yaml format output other than as yaml, and I question the cost/benefit of figuring out how to goose yaml into doing a slightly nicer thing ;)01:06
wallyworldfwereade__: sure. wasn't seriously suggesting it was The Most Important Thing right now :-)01:06
wallyworldbut if people see it, they do wonder if something is wrong01:07
wallyworldsince it's inconsistent01:07
wallyworldtherefore bad01:07
fwereade__wallyworld, I would certainly not complain about a fix for that01:07
wallyworldi'll add it to my paper cuts list :-)01:08
thumperok, bootstrapping ec2 again01:16
wallyworldgood luck :-)01:16
wallyworldlet me know how you get on01:16
wallyworldi only have 30 mins of battery left :-(01:16
wallyworldstill waiting for tradesman to arrive before i can relocate01:16
bigjoolsI wondered whether you were coming or not wallyworld01:21
wallyworldbigjools: yeah. long story. i'm really pissed off right now01:22
wallyworldi'm never hiring this tiler ver again01:22
bigjoolswhat did I say about builders/tradies? :)01:22
wallyworldwell, i can't repeat it since there may be kids watching :-)01:23
bigjoolsI would leave a note on your gate telling him to come back another day since he was too late01:23
bigjoolsbut I guess you're desperate for a bathroom :)01:23
thumperhi bigjools01:23
thumperwallyworld: it seemed to work this time01:24
* thumper hits it again01:24
bigjoolso/ thumper01:24
wallyworlduser error!!!01:24
thumperwallyworld: bullshit01:24
wallyworldpbkac :-P01:24
thumperalthough no idea why last time01:24
* thumper watches pepper chew on maia's shoe01:25
bigjoolspush it01:25
thumperbigjools: push what?01:27
bigjoolsit was an attempt at a salt and pepper joke01:27
bigjoolspush it real good01:27
thumperhaha01:28
thumperI've nicknamed her pepper pots01:28
thumperwallyworld: ah ffs01:28
wallyworldyeeeees?01:28
thumperwallyworld: cloud-init checks for a valid machine id :)01:28
bigjoolsyou need an iron man now01:28
thumperwallyworld: which 0/lxc/0 fails01:28
wallyworldah balls01:28
wallyworldthumper: you want to comment out that check locally and i'll work up a fix?01:29
* thumper looks for where it fails01:29
wallyworldthumper: i can include in my next branch a s i don't think you need it urgently (if you do a local hack)01:30
thumperstate.IsMachineId01:30
* wallyworld nods01:30
thumperwallyworld: is a regex01:31
wallyworldyes it is01:31
thumperwallyworld: how is your regex?01:31
wallyworldok01:31
wallyworldtrivial fix01:31
wallyworldyou want to fix locally and i'll land a fix later? i don't have much power left01:32
thumperwallyworld: I'll just comment out locally01:32
wallyworldsounds good01:32
thumperfor now01:32
wallyworldyep01:32
wallyworldthumper: bigjools: power almost gone. good bye cruel world01:34
thumperwallyworld: use the power adapter01:34
thumperduh!01:34
bigjoolswallyworld: adios01:34
wallyworldthumper: i said before - we have no electricty01:34
wallyworldtill later today :-(01:34
thumperoh... missed that01:34
thumperusing the phone internet?01:35
bigjoolssmack your tiler in the chops01:35
wallyworldso i'm off to bigjools' house after trademan gets here01:35
thumperack01:35
wallyworldthumper: yes, using phone. big data bill :-(01:35
wallyworldtrademan just arrived \o/01:35
bigjoolsyou get 1.5Gb free!01:35
wallyworldyes, but will weekly video calls at soccer training on thursday......01:35
wallyworldanyways, back online a bit later01:36
* bigjools considers an undercut charge for wallyworld01:36
thumperah ffs02:00
thumperhmm...02:26
=== negronjl` is now known as negronjl
* thumper sighs02:36
thumperbigjools: when wallyworld arrives, tell him that I'm fixing the IsValidMachine as I have to ... for other bits02:37
thumperhi wallyworld02:46
thumperwallyworld: cloudinit bits need to know if machineId is a container or not02:47
thumperwallyworld: as I can't install lxc inside lxc on precise02:47
thumperso the cloud-init bits need to be more careful02:47
wallyworldthumper: you can look at the container type02:48
thumperwallyworld: I don't have a machine, just an id02:48
thumperwallyworld: https://codereview.appspot.com/1021504302:48
thumperwallyworld: I might expose the ParentId function02:48
wallyworldyeah02:49
wallyworldor maybe implement an IsContainer?02:49
wallyworldi think that would be better02:49
thumperwallyworld: yeah...02:50
wallyworldthumper: or pass both id and container type?02:51
bigjoolsthumper: I missed your message because he had just arrived and I had to make the whinging Aussie a coffee02:51
wallyworldi like that even more02:51
thumperbigjools: haha02:51
* wallyworld is waiting for his backrub from bigjools02:51
thumperwallyworld: what do you mean?02:51
wallyworldthumper: just a sec, bigjools is snorting coffee02:52
* wallyworld slaps bigjools02:52
wallyworldSTFU02:52
thumperwallyworld: you don't have to type it to him, he is next to you right?02:52
wallyworldthumper: yes, but i wanted to share the hilarity02:52
bigjoolswithin spitting distance02:53
wallyworldthumper: i mean instead of introspecting an id to see if the machine is a container, pass both the id and container type02:53
wallyworldthat sounds cleaner to me02:53
thumperwallyworld: give me a func prototype02:53
thumperI don't get what you are saying02:54
wallyworldthumper: you say you only have an id in the place where you need to know if a machine is a container, right?02:54
thumperright02:54
wallyworldso, why can't we give that place access to the machine's container type as well as its id?02:55
wallyworldie pass the container type to that logic02:55
wallyworldas well as the id02:55
thumperwallyworld: because...02:56
* thumper thinks02:56
* wallyworld is planning for the case where we only have surrogate ids02:57
thumperwallyworld: getting close to having a machine agent running inside the container03:08
wallyworld\o/03:08
thumperthat one failed because of the valid machine check03:09
thumperand I was using a previously bootstrapped instance03:09
* thumper destroyed, rebuilt, bootstraps03:09
wallyworldthumper: you haven't seen the txn aborted issue again?03:11
thumperwallyworld: I've not tried to create many containers03:11
thumperjust waiting for my new bootstrap node to come up03:11
thumperthen I'll add a couple03:11
wallyworldkk03:11
* thumper looks at how to add a mount point to lxc config03:11
thumperhi mramm03:20
thumpermramm: really you, or just a machine?03:20
mrammreally me03:20
thumpermramm: hey hey03:20
mrammhow goes?03:20
thumperoh ffs03:36
thumperwallyworld: hey04:17
wallyworldhi04:17
thumperwallyworld: the reason the machine agent didn't start properly on the bootstrap node04:17
thumperwallyworld: is that the state server address it had was localhost :)04:17
* thumper kicks a new machine up04:17
wallyworldthumper: so not my fault?04:17
thumperwallyworld: no04:18
wallyworld\o/04:18
* thumper has started machine 104:18
thumperhmm...04:29
* thumper makes notes of extra bits needed04:29
thumperlike, specific root file system location04:30
thumpermounting /var/log/juju04:30
thumperfor some reason, when I run our cloud-init bits, I can't log in... damn password04:30
thumperwallyworld: what I do need is a machine watcher that watches for changes in containers on the machine04:38
thumperwallyworld: parameterised by container type04:38
thumperwallyworld: so "get me a watcher for lxc containers on machine x"04:38
wallyworldsounds reasonable04:39
thumperwallyworld: ok, machine 1 lxc machine agent fails with auth issues04:41
thumperso I'm probably doing something wrong there.04:41
wallyworldthumper: i had auth issues also just bootstrapping04:42
wallyworldand all of a sudden it came good04:42
wallyworldec2 is stupid :-(04:42
wallyworldthumper: i have to run away and do the school pickup04:42
thumperack04:42
=== tasdomas_afk is now known as tasdomas
TheMuemorning06:44
=== TheRealMue is now known as TheMue
* fwereade__ seeks a further review on https://codereview.appspot.com/10166044/08:05
TheMuefwereade__: will do (and good morning btw)08:08
fwereade__TheMue, good morning, and cheers :)08:10
TheMuefwereade__: while I'm looking into your source, could you please add a hint to your review in https://codereview.appspot.com/10148045/diff/10001/state/state_test.go line 1272?08:13
fwereade__TheMue, just add two services which each have a peer relation, remove them both, Sync, and check you only get one event.08:22
fwereade__TheMue, might be nice as a separate test08:22
TheMuefwereade__: ok, good hint, thanks08:31
=== tasdomas is now known as tasdomas_afk
=== tasdomas_afk is now known as tasdomas
* danilos_ steps out, back in ~3009:15
fwereade__rogpeppe1, https://codereview.appspot.com/10166044/patch/1/910:04
rogpeppe1fwereade__: looking10:04
fwereade__rogpeppe1, you mean to just test that it's hooked up by doing something 100% broken and hoping for a characteristic error?10:04
rogpeppe1fwereade__: yes10:04
fwereade__rogpeppe1, I'm not sure that's a very solid technique compared to running the operation and verifying success10:05
fwereade__rogpeppe1, ah10:05
fwereade__rogpeppe1, but it's just the perm test10:05
rogpeppe1fwereade__: because we must get the permission-denied error (which is all this test is checking for) before it even tries to attempt the operation10:05
fwereade__rogpeppe1, gotcha10:05
fwereade__rogpeppe1, thanks10:05
rogpeppe1fwereade__: np10:05
jammgz_, wallyworld: https://plus.google.com/hangouts/_/8868e66b07fa02bdc903be4601200d470dae9ee3  ?11:30
mgz_ta11:32
=== danilos_ is now known as danilos
=== tasdomas is now known as tasdomas_afk
TheMuefwereade__: ping13:11
fwereade__TheMue, pong13:11
TheMuefwereade__: the resumer shall process the ResumeAll() on State.runner?13:12
fwereade__TheMue, yeah13:13
fwereade__TheMue, you could even just have a state.Resumer type that just happened to implement Worker13:13
TheMuefwereade__: ok, and here periodically?13:14
fwereade__TheMue, yes please13:14
fwereade__TheMue, I think I need to go eat some lunch though, didn't quite get round to it yet13:14
fwereade__bbiab13:15
TheMuefwereade__: Resumer in state? OK, so I don't need a function state.ResumeAll()13:15
TheMuefwereade__: enjoy your meal13:15
fwereade__TheMue, I'm not sure13:16
fwereade__TheMue, think through the tradeoffs and just precis why what you pick is more sensible ;p13:16
fwereade__TheMue, we can chat after kanban if you like13:17
TheMuefwereade__: sure, we can do13:23
* TheMue just fetched an espresso13:23
rogpeppe1fwereade__: hasn't we decided to lose the stuttering?13:33
=== wedgwood_away is now known as wedgwood
fwereade__TheMue, https://codereview.appspot.com/10148045/ reviewed, nearly there14:53
TheMuefwereade__: thanks14:53
TheMuefwereade__: I'm a bit confused now. while the tests with the two Destroy() before the last propose only raised one event I now have two :(15:02
TheMuefwereade__: so the coalescing as designed in the watcher doesn't work15:02
TheMuefwereade__: ah, it has to do with the change of Sync() to StartSync() in the asserts15:16
TheMuehave to leave for today. will comeback to you tomorrow, fwereade__15:55
ehwdoes go-juju upload the tools tarball to MAAS automatically, or does I need to upload it manually?17:00
ehwI guess I should phrase that differently: is go-juju actually intended to work with MAAS?17:27
fwereade__ehw, --upload-tools will work if you're building from source; the alternative is `juju sync-tools` which is somewhat inelegant -- and currently requires valid ec2 credentials in env vars -- but that'll copy from the same place other environs get theirs17:51
ehwfwereade__, building from source, as in it works in unreleased versions?17:51
ehwfwereade__ i.e. trunk?17:51
fwereade__ehw, with trunk, you need --upload-tools17:52
fwereade__ehw, we made a mistake a few revs ago and trunk doesn't currently bootstrap released 1.10 tools, so it's a necessity today17:52
fwereade__ehw, in general it should not be17:52
ehwfwereade__, I'm running from ppa:juju/devel, so I'm assuming it's broke?17:53
fwereade__ehw, I'm not certain what version that implies -- 1.11?17:54
ehw1.11.0-1~1240~precise117:54
ehwagainst maas 1.2, from maas-maintainers/stable17:55
fwereade__ehw, if you have ec2 credentials, you can run `juju sync-tools` which should get you matching versions copied into environment storage17:55
fwereade__ehw, this is a sucky story17:55
fwereade__ehw, and it is in hand17:55
fwereade__ehw, but it's the story we have today17:56
ehwfwereade__, will probably need to do some django hacking then; we need to go on the bug warpath over the next couple of weeks17:56
fwereade__ehw, sorry, I 'm missing context -- what is it you're trying to do exactly?17:57
ehwfwereade__, in the larger context, document and debug MAAS, Juju, Openstack, and Landscape in preparation for the 13.09 LDS release17:58
ehwfwereade__, right now, I'm just trying to get juju-core to talk to maas ;)17:58
fwereade__ehw, if it's tools that are the problem you can do it even more low-tech17:59
fwereade__ehw, grabbing this and putting it in your environ's storage under the same path should get you a step further https://juju-dist.s3.amazonaws.com/tools/juju-1.11.0-precise-amd64.tgz18:00
ehwfwereade__, I've got the tarball, but I need to put it where juju is expecting it18:00
ehwfwereade__, which appears to be under MAAS/api/1.0/files/18:01
fwereade__ehw, in that case you *should* just be able to put it in tools/ therein18:01
fwereade__ehw, but: what actual error are you seeing?18:01
ehwfwereade__, error: no tools available18:02
ehwfwereade__, it hits the MAAS api looking for "GET /MAAS/api/1.0/files/?prefix=tools%2Fjuju-&op=list HTTP/1.1"18:02
fwereade__ehw, ok, yeah, putting something under tools/ in there should help18:03
fwereade__brb, let me know if it helps18:03
ehwfwereade__, yeah, but it's not a filesystem, it's binary blogs in the db18:03
ehwprobably planning on using object storage for that at some point18:03
=== niemeyer_ is now known as niemeyer
fwereade__ehw, you would possibly want to talk to bigjools about that, then, because I have not actually deployed on maas myself18:17
fwereade__ehw, but s3 is not, either, it just kinda looks like one if you squint a bit -- if the blob names can have /s you shoudl be fine18:18
fwereade__ehw, sorry, I need to be gone for a while now :(18:18
ehwfwereade__, no worries; I think I might be able to feel my way through this18:19
ehwok, had to pop into maas shell and put the file there manually, but it bootstraps now18:27
=== tasdomas_afk is now known as tasdomas
=== tasdomas is now known as tasdomas_afk
thumperoops22:05
thumperforgot to tear down my ec2 instances yesterday22:05
jcastrothumper: I suppose that's better than forgetting them from 4 months ago22:10
thumperjcastro: haha22:10
thumperfor sure22:10
=== wedgwood is now known as wedgwood_away
* thumper out for lunch23:51

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