/srv/irclogs.ubuntu.com/2014/01/30/#juju-dev.txt

davecheneyhttp://paste.ubuntu.com/6840930/00:00
davecheneygreat, how am I supposed to shut down this environment ?00:00
wallyworld_i would kill the machines by hand and delete the control container, then delete the jenv00:18
=== thumper-afk is now known as thumper
thumperdavecheney: really, WTF?00:21
thumperhow do I remove the govet check?00:22
thumperwallyworld_: I upgraded to trusty last night00:22
thumpertrunk starts the local provider00:22
wallyworld_hmmm00:22
thumperalthough my http-proxy wasn't on the right port00:22
thumpertrunk mind, not release00:23
thumperrelease is all fubared00:23
wallyworld_when i tried on friday, the agent ran, but no lxc container would start00:23
thumperfor some reason I've lost the all-machines.log00:23
wallyworld_yeah, me too00:23
thumperneed to figure that out00:23
thumperI have a branch to submit for destroy00:23
wallyworld_\o/00:23
thumperif I can get lbox shite working00:24
wallyworld_thumper: with go vet, i installed from source and got it to work that way00:24
thumperwallyworld_: you installed go from source?00:24
* wallyworld_ tries to remember00:24
wallyworld_go get -u ...something00:24
wallyworld_let me see if my history shows it00:24
wallyworld_i think the error message gave a hint00:25
thumperit tries to open the /usr/lib/go pkg dir00:25
thumperfor some reason it isn't honouring the GOPATH00:25
thumperdavecheney: can I just change an environment variable?00:26
wallyworld_not sure if this was the attempt that worked00:26
wallyworld_sudo bash -c "export GOPATH=/usr/lib/go; go get code.google.com/p/go.tools/cmd/vet"00:26
wallyworld_something like that was necessary00:26
wallyworld_i just assumed it was my fucked up set up00:26
wallyworld_didn't realise it was a general issue for trusty00:26
thumperhmm...00:27
thumperyeah, that looks weird to the extent I don't want to do that00:28
wallyworld_i may have had to do it with a GOPATH pointing to a dir off my home, not usr/lib00:28
wallyworld_but i recall sudo and messing with GOPATH was necessary00:28
thumpera fuck it...00:28
thumpernah, that doesn't work00:30
wallyworld_hmmm sorry. i can't recall exactly what worked. i *think* i had to point GOPATH to a local src dir, but use sudo. not sure thpugh00:31
* thumper stabs .lbox.check00:33
thumperFARK00:34
thumpercan't propose with a dirty branch00:34
thumperso either I commit the lack of vet00:35
thumperor don't use lbox00:35
* thumper decides to skip lbox00:35
wallyworld_\o/00:36
thumperwallyworld_: https://code.launchpad.net/~thumper/juju-core/fix-lxc-destroy/+merge/20386100:39
wallyworld_looking00:39
wallyworld_thumper: done. btw did you see google offloaded motorola to lenovo. those lenovo guys are really going on a shopping spree00:42
thumperno I didn't know that00:42
thumperwallyworld_: did google keep all the patents?00:43
wallyworld_yep :-)00:43
wallyworld_of course00:43
thumperheh00:43
wallyworld_they licenced them to lenovo00:43
thumperwallyworld_: I can't land it yet, it didn't fix the problem00:43
wallyworld_so they paied $12bil, sold for $3bil00:43
thumperah...00:46
thumperhang on00:46
thumperI see what's happened now00:46
thumperI have a running juju machine agent for zero00:46
thumperrebuilding a local juju doesn't help destroy environment00:46
thumperbecause the environment is already running00:47
thumperand using an old jujud00:47
thumperwhich doesn't have my fix00:47
thumperFFS00:47
thumpermanual hackery FTW00:52
wallyworld_i've done the same thing before too00:53
thumperok, at least that is the local provider starting and stopping in trusty00:57
thumpernow to poke and see why we have no all-machines.log00:57
thumperhmm... I think I may see00:58
axwthumper: I looked into that the other day, it was permissions01:00
thumperyeah, I saw that too01:00
thumperaxw: have you a fix ready?01:01
thumperif not, I'll do it01:01
axwthumper: I think we may have to do something yucky, like symlinking to /var/log/juju01:01
axwno01:01
axwit passed out of my memory on the flight back01:01
thumperwallyworld_: I'm witting in our weekly hangout...01:07
wallyworld_oh yeah01:07
hazmatthumper, how's proxy support coming?01:12
thumperhazmat: trunk is almost feature complete01:12
thumperthere are a few things outstanding01:13
thumper- remove old apt proxy cruft in container setup01:13
thumper- make sure containers get the proxy info for the container providers01:13
thumper- some upgrade code01:13
hazmathmm.. that sounds pretty good... does that include things like the cli respecting std http_proxy env vars for tool lookup?01:16
thumperhazmat: in a hangout just now01:16
thumperhazmat: how long have you got?01:16
hazmatthumper, i'll make some time.. i'm pushing through on a late night01:17
thumperwaigani: hangout? standup01:32
waiganiyep01:32
davecheneycap01:33
davecheneycrap01:33
davecheneynow destroy-environment cannot save me01:33
davecheneyhttps://bugs.launchpad.net/juju-core/+bug/127435501:33
thumperschool run02:01
thumperbbs02:01
* axw rejoices and promptly stops thinking about windows cli02:04
axwdavecheney: try destroy-environment --force02:05
axwdavecheney: actually... what version are you on?02:05
davecheneytrunk02:05
davecheneyaxw: yup02:06
davecheneythat worked02:06
axwdavecheney: destroy-environment now goes and talks to the API server to destroy the environment "cleanly"02:06
axwif it's borked, --force is required02:06
axwwe just had a chat on our scrum, and figured we should probably tell that to the user if it fails and --force wasn't specified02:07
davecheneycouldn't hurt02:13
* thumper sighs02:51
* thumper created a broken local provider02:52
* thumper deletes things02:52
thumperfark!!!03:31
thumperI think it is apparmour03:31
axwthumper: doing what?03:32
thumperstopping writing to ~/.juju/log/all-machines.log03:32
axwhm ok03:32
* thumper pokes and tweaks03:33
* thumper does something icky03:36
=== wedgwood is now known as Guest44693
* axw goes out for lunch03:46
axwbbl03:46
axwif I don't return, the heat has claimed me03:47
thumperwallyworld_: ping03:47
thumperwallyworld_: who owns your /var/log/syslog file?03:47
axwthumper: mine is syslog:adm03:48
thumperhmm...03:48
thumpermine is messagebus:adm03:48
thumperno idea where that came from03:48
wallyworld_thumper: um, let me check04:14
thumperwallyworld_: that's ok04:14
wallyworld_i don't have one right now04:15
thumperall these permission problems are due to rsyslog apparmor profiles04:15
wallyworld_ah04:15
thumperI've got a work-around04:15
thumperbut I need to go make dinner04:15
thumperso it'll have to wait04:15
wallyworld_f*cking apparmor04:15
wallyworld_:-)04:15
thumperwaigani: is there a bug for the "no all-machines.log for the local provider"?04:15
waiganithumper: I have not created on04:16
waigani*one04:16
=== thumper is now known as thumper-afk
axwwallyworld_: when you're not busy, can you please poke the bot?06:16
wallyworld_sure06:16
wallyworld_will do it now06:16
wallyworld_axw: poked06:17
axwthanks06:17
axwwallyworld_: I thought there was a reason we needed FullPath for the manual bootstrap - the reason eludes me though06:38
wallyworld_axw: FullPath is filled in when the particular tools are read from simplestreams. but is was unnecessary to fill in that value when writing metadata files caused it is ignored then06:42
wallyworld_we just store ralative path in the json06:43
wallyworld_and construct fullpath when needed as the metadata is loaded06:43
axwyeah, I thought it was used in-memory... my memory may just be faulty though06:43
axwwelp, seems to work anyway06:44
wallyworld_yay06:44
axwwallyworld_: in provider/maas/storage.go, there's talk about 404 and IsNotFoundError06:53
axwdoes this not work?06:53
wallyworld_um06:53
wallyworld_that works06:54
wallyworld_but the issues is URL()06:54
axwah sorry06:54
axwURL, not Get06:54
wallyworld_yeah, maas is the only one that can't give a url for a non existent fle06:54
wallyworld_so i changed dummy storagr to match that06:54
axwwallyworld_: ahh, so MAAS 404s even for URL06:54
axwgot it06:55
wallyworld_yeah :-(06:55
wallyworld_sadly yes06:55
axwwallyworld_: why do we no longer set imagemetadata.DefaultBaseURL in bootstrap? and does tools.DefaultBaseURL continue to be updated?07:00
axw(sorry for dumb questions, just trying to be thorough...)07:00
wallyworld_axw: np. the image metadata is uploaded from a local dir to cloud storage07:00
wallyworld_it was a mistake to set th default url07:00
wallyworld_in the cloud storage, the uploaded metadata is in the search path07:00
wallyworld_and keeping default url unchanged means we fall back07:01
wallyworld_if no images found in user metadata07:01
axwok07:01
axwwhereas if we change it, it'll try to look in cloud storage and in the local dir07:01
axwright?07:02
wallyworld_yeah, but it will error out later in the boot process07:02
wallyworld_i didn't look into why07:02
axwmk07:02
wallyworld_i decided it was wrong to change the default url07:02
wallyworld_tools url is changed cause that's what sync tools needs07:03
wallyworld_end result - tools and image metadata is uploaded to cloud storage if local dir is used07:03
axwthanks07:04
wallyworld_and cloud storage is in simplestreams search path, behind config urs and ahead of default locations07:04
axwgot it07:04
axwI remember why FullPath/URL was required before: we used to do a URL fetch, rather than go through storage07:04
axwso we previously had to keep the URL in memory to pass along to the merging bit07:05
wallyworld_axw: just to check - i'm pretty sure that we don't need full url now right07:32
axwwallyworld_: right07:32
wallyworld_cool, that matches my understanding too07:33
axwsince we use the path & storage now07:33
wallyworld_yep07:33
wallyworld_just wanted to be 1000% sure :-)07:33
axw:)07:33
wallyworld_axw: with the abs(), i thought it better to report the whole dir to the user to avoid confusion07:34
axwokey dokey07:35
jamaxw: do we actually have a solution for people trying to use juju-core with Canonistack and setting up 'sshuttle' while they are trying to bootstrap?08:41
jamor does bootstrap itself complete because it uses the SSH redirection, (I thought one of the steps was to directly connect to the API to know that it is up, so now that step will fail)08:42
axwjam: bootstrap works because it goes through SSH, sshuttle is still required as it was before post-bootstrap08:42
axwthere's no API interaction during bootstrap08:43
jamaxw: so "before" you could use sshuttle to the bootstrap node once it came up, and then everything else would go direct to the API, that stil works?08:43
axwwe did talk about that during SFO - juju status at the end - but it hasn't been implemented08:43
jamah, k08:43
axwjam: yep08:43
axwsshuttle -r ubuntu@bootstrap-node subnet/cidr08:44
axwI would kinda of like it if API connections did some magical tunnelling as a fallback, but it's probably too messy08:45
jamaxw: It would be helpful for people at Canonical, I'm not sure that it is a general solution.08:46
jamSaying "you need public access to the cloud machines you are using" doesn't seem like a particularly bad thing to me.08:47
axwyeah I guess so. I don't really know how many of our private cloud users will have the same issues08:47
axwprobably not that many08:48
mgzmorning08:49
axwmorning mgz08:49
jamaxw: I think you'll generally need some-sort-of-VPN but we actually have 2 different ones inside Canonical. (the Server Stack needs OpenVPN to access, and doesn't work via SSH at all)09:01
axwtrue, though our bootstrap won't even work without ssh access09:02
axwI was just thinking it'd be nice to have guaranteed API access if bootstrap succeeds09:03
axwit's not a big deal to have to sshuttle though, really09:03
mgzI may be a min or two late for meeting later09:06
rogpeppewallyworld_: ping09:33
rogpeppeaxw: you reviewed this branch, i think: https://codereview.appspot.com/5851004409:36
rogpeppeaxw: i'm having difficulty understanding how it works - did you work it out?09:37
=== thumper-afk is now known as thumper
axwrogpeppe: sorry, which part?09:52
axwdoh, good catch on the dummy storage boolean...09:53
axwrogpeppe: re "How does the URL get into the metadata now?" -- simplestreams metadata doesn't encode the URL09:54
axwwe used to store it in the in-memory object when tools metadat was fetched by URL09:55
axwbut now we use the path, and go through storage09:55
axwrogpeppe: and the fact taht we were calling the environment storage'URL09:56
axwoops09:56
axwstorage's URL method, caused an error wtih MAAS09:56
thumperwhere's wallyworld_09:59
wallyworld_here09:59
wallyworld_on the juju call. yay. will look in a bit. i'm also 3/4 drunk :-D10:04
bigjoolswallyworld_: ROFL10:05
thumperwallyworld_: 3/4 drunk and working on maas fixes \o/10:41
wallyworld_no, i'm ok now :-)10:41
bigjoolshaha10:41
wallyworld_it's been an hour or more since my last wine, i was exagerating :-)10:41
wallyworld_a man can drink a glass of wine with dinner :-)10:42
bigjoolsdigging more like10:42
bigjoolsa single glass has been known to get you blotto10:42
fwereaderight, I'm not interviewing horacio right now because we need to turn the power off here to investigate scary burning smells11:03
fwereadehopefully I will be back shortly11:03
natefinchfwereade: good luck11:04
fwereadecheers11:04
jammgz: are you available to just hang out?11:08
rogpeppeaxw, wallyworld_: what's the purpose of ToolsMetadata.FullPath? are we actually setting it now?11:12
rogpeppeah! it's ignored11:12
wallyworld_rogpeppe: it's used when wget fetches the tools11:12
wallyworld_it's ignore writing out11:13
rogpeppewallyworld_: ok11:13
wallyworld_it is composed from the relative path stored in the json11:13
wallyworld_composed when needed11:13
wallyworld_rogpeppe: is trunk working?11:15
rogpeppewallyworld_: i'm going to try it in a mo11:15
* wallyworld_ crosses his fingers11:15
mgzjam: sure11:16
jammgz: I'm just going to grab a snack, mumble or g+?11:16
mgzlets try mumble first, I'll be on there11:17
jamyeah, I can't understand you at all, which means it is messed up, I'll reconnect 1 more time11:26
mgzjam: we'll have to just fall back to hangout11:27
mgzuse the daily standup  one?11:27
rogpeppewallyworld_: sorry, it appears that there are no spare boxes currently, so testing it will have to wait for a little bit11:29
wallyworld_ok, np11:29
jammgz: now ff is freaking out again... just a sec11:29
mgz:)11:29
wallyworld_rogpeppe: i'm quietly confident it will work :-)11:30
jammgz: I'm in the "juju core team" one from earlier today11:30
rogpeppewallyworld_: did you see my remarks on your CL?11:30
wallyworld_no, i'll look11:31
wallyworld_rogpeppe: tomorrow i'll fix the remaining issues, thanks11:35
rogpeppewallyworld_: thanks11:35
wallyworld_rogpeppe: it seems that we're still not quite universally consistent with converting relative paths to absolute when running commands11:36
rogpeppewallyworld_: yeah, we should probably use Context.AbsPath throughout11:36
wallyworld_yeah, i didn't realise we had that method11:37
dimiternrogpeppe, i'm trying to debug this missing GOPATH issue that cause uniter tests to fail in an install hook12:07
dimiternrogpeppe, so i'm using your debug.Callers() thing to dump the stack when I detect GOPATH is empty in testing/charms.go:init()12:07
rogpeppedimitern: ok12:08
dimiternrogpeppe, but what I'm getting is a bunch of files, and the line number for each one is always the last line in the file12:08
dimiternrogpeppe, what does that mean?12:08
rogpeppedimitern: paste?12:08
jamrogpeppe: so if you *don't* supply -test.timeout, then when it times out it just says "test took to long" if you *do* supply a value (like 10s in my testing) then you get a panic12:08
dimiternrogpeppe, http://paste.ubuntu.com/6843438/12:09
dimiternrogpeppe, and my changes to init(): http://paste.ubuntu.com/6843442/12:10
dimiternrogpeppe, for some reason during the install hook the environment gets reset or something and GOPATH is lost12:11
jamrogpeppe: it looks like '-test.timeout' is for each individual test triggering a panic, vs the global timeout being run by the meta-process when running multiple packages12:15
rogpeppejam: interesting12:19
rogpeppedimitern: sorry, i got waylaid by insanity. i have to go to lunch now, sorry, otherwise i won't make it to the discussion in 40 mins. will look when i can.12:20
dimiternrogpeppe, ok12:21
=== BradCrittenden is now known as bac
=== gary_poster|away is now known as gary_poster
dimiternrogpeppe, I found a solution: http://paste.ubuntu.com/6843674/ adding GOPATH to the list of env vars a hook gets13:06
rogpeppedimitern: yeah, i thought that might be the problem13:06
dimiternrogpeppe, what i don't get is why it wasn't a problem before?13:07
rogpeppeaxw: are you around?13:07
rogpeppedimitern: agreed - i'm not sure either13:12
rogpeppedimitern: there's a perhaps-better solution13:12
dimiternrogpeppe, what's that?13:12
rogpeppedimitern: just make it so init doesn't panic13:12
dimiternrogpeppe, but it won't solve the issue - it won't find the repo path still13:13
rogpeppedimitern: it doesn't need the repo path13:13
rogpeppedimitern: it's just re-running the test binary as a hook, right?13:13
dimiternrogpeppe, why don't remove init() altogether then?13:13
dimiternrogpeppe, it seems better to get the absolute path of the charms.go in testing and infer the path from there13:15
dimiternrogpeppe, is that possible?13:15
rogpeppedimitern: how would you do that?13:16
rogpeppedimitern: i think that removing the init altogether is reasonable though13:16
dimiternrogpeppe, well, in python i'll do it with os.path.realpath(".") or something13:16
rogpeppedimitern: the current directory isn't related to where charms.go is13:16
dimiternrogpeppe, no, sorry - i'll use the __file__ magic const that contains the full path to the current file13:17
dimiternrogpeppe, kind of like $0 in bash13:17
rogpeppedimitern: define testing.Charms as a function, say func() *charms.Repo13:17
rogpeppedimitern: no such thing in Go13:17
jamthere is some 'build' runtime magic stuff13:17
rogpeppedimitern: (and probably a good thing too, as we'd be tempted to do this, and it would be a bad idea :-])13:18
rogpeppejam: we're already doing that13:18
jamwe use it in testing/charms.go ? something ilke that13:18
rogpeppejam: but GOPATH isn't set if we re-exec the binary13:18
rogpeppedimitern: something like this perhaps? http://paste.ubuntu.com/6843741/13:20
rogpeppedimitern: that actually makes tests that don't use testing charms slightly faster13:21
rogpeppes/tests/suites/13:21
axwrogpeppe: sort of around, what's up?13:21
rogpeppeaxw: just wondering whether it's currently possible to manually bootstrap on a node, but not use the manual/null provider13:22
axwrogpeppe: bootstrap or "provision"?13:23
axwadd-machine?13:23
rogpeppeaxw: bootstrap13:23
axwno13:23
TheMuerogpeppe: interested in reviewing https://codereview.appspot.com/58510045/? it still misses tests (have to think about it and how to do best), but it works live.13:23
rogpeppeaxw: right, i thought so13:23
rogpeppeaxw: but needed to check13:23
axwrogpeppe: nps. who wants that and why?13:23
rogpeppeaxw: there are currently some people who have duplicated the entire bootstrap logic in shell (and it works... kinda)13:24
rogpeppeaxw: just to get this behaviour13:24
rogpeppeaxw: the specific reason is that they want to bootstrap juju onto a maas controller node13:24
dimiternrogpeppe, it doesn't work13:24
axwoh, the installer thing13:24
dimiternrogpeppe, it still fails if GOPATH is not set13:24
rogpeppedimitern: doesn't matter13:25
rogpeppedimitern: why would that hook code be calling testing.Repo ?13:25
axwrogpeppe: why do they not just use the manual provider...?13:25
axwno ssh?13:25
rogpeppeaxw: because they want the environment to be using the maas provider13:25
axwoh yeah, right13:25
dimiternrogpeppe, it doesn't, it just imports stuff from testing13:26
rogpeppedimitern: right, so it's ok then13:26
dimiternrogpeppe, adding GOPATH to that list of vars fixes the issue, i don't want to make it any more intrusive, and i want to land this already13:27
rogpeppedimitern: that actually makes it *more* intrusive, no?13:27
rogpeppedimitern: because that list of vars is in production code, no?13:27
dimiternrogpeppe, so what - in production it will be empty if you don't have go installed, and even if you do, it won't affect anything13:28
rogpeppedimitern: we should not be changing production code for this kind of reason - it's a testing bug and it should be fixed in the testing code13:29
rogpeppedimitern: there might even be a less intrusive way, one mo13:29
dimiternrogpeppe, what you proposed with making Repo a func doesn't work, because it's a type and it's exported13:31
dimiternrogpeppe, I have to refactor the whole thing13:31
rogpeppedimitern: sorry, i meant Charms13:32
rogpeppedimitern: but, i'm pretty sure there's a much less intrusive way13:32
rogpeppedimitern: just checking13:32
dimiternrogpeppe, in every test that uses testing.Charms.xxx I have to go and change it so it reads testing.Charms().xxx13:33
rogpeppedimitern: sure, but that's one trivial global change: gofmt -r -w 'testing.Charms -> testing.Charms()'13:34
rogpeppedimitern: but, again, i think there's a better way13:34
dimiternrogpeppe, I'm listening13:34
rogpeppedimitern: i'm still testing whether it's viable13:34
rogpeppedimitern: something like this: http://paste.ubuntu.com/6843805/13:40
rogpeppedimitern: those are the only changes necessary, i think13:40
dimiternrogpeppe, I'll try it13:41
TheMuerogpeppe: your last hint to me works fine and can be reviewed ;)13:41
rogpeppeTheMue: i'll have a look13:41
TheMuerogpeppe: thx. unit tests are still missing, have to think how to do it best13:42
=== natefinch is now known as natefinch-afk
rogpeppeTheMue: what's the backward compatibility issue you mention w.r.t. environtag/13:43
rogpeppe?13:43
rogpeppeTheMue: oh, i see13:44
rogpeppeTheMue: why not just do that behaviour server side, i.e. no tags means everything?13:44
TheMuerogpeppe: ah, fine. yes, today the command doesn't need an argument. william wanted, that always a tag has to be passed13:44
rogpeppeTheMue: then you won't need any special casing for the environment tag at all13:44
rogpeppeTheMue: really?13:45
TheMuerogpeppe: but i have to add an error replay from server-side w/o a tag13:45
rogpeppeTheMue: seems odd to me13:45
TheMuerogpeppe: yes, really ;)13:45
rogpeppeTheMue: i don't understand the "error replay" thing13:45
TheMuerogpeppe: I could live with a default choosing the environment too, would make it simpler13:45
TheMuerogpeppe: eh, reply13:45
rogpeppeTheMue: yeah13:46
TheMuerogpeppe: if you initially call w/o entities or later set empty entities13:46
rogpeppeTheMue: it simplifies things13:46
dimiternrogpeppe, this works yes, tyvm13:46
rogpeppedimitern: np13:46
TheMuerogpeppe: definitely, please add a comment regarding this point13:46
rogpeppedimitern: sorry for the pushback, but the GOPATH thing sounded like creeping dependencies, fixing things far away from where the actual problem was13:47
rogpeppeTheMue: will do13:47
dimiternrogpeppe, yeah, I agree - sorry, but I was trying to land this for a day now and getting really frustrated13:47
TheMuerogpeppe: but filtering is really nice ;) tested it with one and multiple tags13:47
rogpeppeTheMue: cool13:47
rogpeppedimitern: i know the feeling13:48
jamrogpeppe: we had a gecko show up inside, which my wife made quite clear that it had to go :)13:53
rogpeppejam: lol13:53
rogpeppejam: from a position firmly on top of a table?13:54
jamthey're a bit harder to deal with than mice, as mice don't climb 2m up the wall13:54
jamrogpeppe: not quite so bad, but certainly back off a ways and quite vocal13:54
mgzgeckos sound fun13:54
rogpeppemgz: +113:54
jammgz: they're actually really quite cute (IMO) but I can understand not wanting to catch unexpected motion out of the corner of your eye. It can be a bit disturbing.13:56
mgzokay, lunch for me, hae proposed a much simplified https://codereview.appspot.com/56560043 along the lines you suggested jam (previous rev has the real cert stuff in)13:57
mrammso anybody got an update for me on the 1.17.2 world14:18
mrammas in, are we able to do a 1.17.2 release anytime soon (as measured in hours)14:19
mrammsinzui: ^^^?14:19
marcoceppimgz: are you still going to be able to make it to cfgmgmt camp?14:21
mgzmarcoceppi: yeah, I'll give you and rbasak my travel details14:22
marcoceppimgz: cool, robbie will be at fossdem but not cfgmgmt camp, that'll just be us14:22
mrammarosales: you around?14:26
sinzuimramm, local provider is very broken in trunk. It is not releasable.14:26
sinzuimramm, but when we have a blessed revision, a release takes 4 hours14:26
mrammjuju dev team, is there a way to get just 1.17.1 plus the MAAS bootstrap fix as a branch that we could release as 1.17.2?14:27
mgzsinzui: so, in the meeting this morning our antipodes were under the impression the only remaining broken things were misunderstandings/script things to fix14:27
mgzsinzui: from the mailing list thread, I'm not clear what's still borked14:27
sinzuimgz, maybe one thing needs to be fixed. local deploys of 1.17.1/2 have not worked since last Friday, so I have nothing to give me confidence that there is only one thing blocking the fix14:29
sinzuimgz, though We can see that changes to trunk are fixing things. Since yesterday, local deploy fails because wordpress fails to start: http://162.213.35.54:8080/job/local-deploy/14:31
sinzuiWe also see this in some of the failing azure tests14:32
rogpeppeTheMue: you have a review14:34
mrammmgz: sinzui: who do we need to get into a hangout to sort out these issues?14:36
mgzmramm: I think ideally the US/AUS overlap time14:36
mrammwell, that is quite a bit from now...14:37
mrammbut if that's where we need a meeting, we can do it14:37
arosalesmramm, hello14:37
mrammbut I'd like to have everything we can sorted out by then14:37
mrammarosales: saw your message regarding MAAS for the charmers -- I have one priority over that which is MAAS for CI14:38
mrammI think we can use the MAAS testing lab for that14:38
mrammjames page and I were just talking about it14:38
arosalessinzui, are you available for the juju cross team?14:38
mrammis that now?14:38
mrammjoining14:38
mrammsorry lost track of time here14:38
sinzuiarosales, now? me in standup14:38
arosalessinzui, ah ok. I think the cross team meeting is an hour early than it was in 201314:39
arosalesmramm, ^14:39
TheMuerogpeppe: thank you14:39
mrammarosales: we can move it later14:40
arosalessinzui, and simple stream workflow is still in progress or waiting on utlemming14:40
TheMuerogpeppe: I like the idea with the regex, it is more powerful. will change it.14:50
TheMuerogpeppe: and also to no-filter-means-show-everything14:50
rogpeppeTheMue: thanks14:51
sinzuimgz, I consistently see a mysql error in local deploy http://162.213.35.54:8080/job/local-deploy/767/console14:56
sinzuimgz, We are not getting logs though...14:57
sinzuimgz, do I dare increase the memory constraints as we did for HP14:57
* sinzui accidentally deployed juju on 512m this week and was surprised to see the tests pass...juju's memory requirements probably shrank14:58
mgzyeah, juju itself is generally good14:59
hatchI am trying to determine which version of juju-core actually removes the requirement for sudo on local deployments? 1.17.1 seems to require it but I'm hearing reports that 1.17.2 does not14:59
mgzmysql still does that "allocate 80% of machine's memory" thing though14:59
mgzwhich is probably not good if there's anything else on the box at all14:59
sinzuihatch, trusty + 1.17.2 (which I have not released14:59
hatchsinzui what about precise?15:00
hatchdo you guys consider the 1.17.x release to be a non-stable release? Should quickstart use sudo up to 1.18 ?15:00
sinzuihatch, precise version will still ask for sudo on demand15:00
hatcheven in 1.18?15:01
sinzui1.18 will request sudo passwords on demand15:01
hatchok so trusty with 1.17.2 or greater will not require sudo, everything else does15:02
hatchsinzui: sorry that was a question :)15:02
sinzuihatch, yes. In general, juju will will ask for input as needed. That means your scripts need to provide sudo passwords after they call bootstrap, or do a trivial sudo op before bootstrapping so that the op is not paused for a password15:04
hatchsinzui right, but we also don't want to run it as sudo when it's not required to so I was hoping for something we could programatically check for to see if sudo will be required15:06
mgzhatch: basically it depends on juju version and lxc version15:07
sinzuiokay, hatch, I understand. "juju version" will return 1.XX.X. if the dotted numbers are greater than 1.17.1, don't use sudo15:07
hatchmgz, well, apparently also the ubuntu series15:07
hatchsinzui even on precise?15:08
mgzthat's only an indication oflxc version15:08
sinzuihatch lxc version is implicit with series15:08
sinzuihatch, yes, even precise that I test15:08
sinzuiThis is was ran 10 minutes ago15:08
sinzuijuju --show-log bootstrap -e local --constraints mem=2G15:08
sinzuion precise local15:09
hatchok perfect, haha wow that was confusing for a bit :)15:09
hatchversion > 1.17.1 ! require sudo :D15:09
sinzuibingo15:10
hatchok on it!15:10
hatchsinzui will 1.17.x be released to the stable ppa or is it just a development version?15:11
sinzuihatch, odd numbers are always unstable and not subtle for production15:12
sinzuihatch, when we think the 17 changes are complete, we will call them 1.18.015:13
hatchahh ok gotcha15:13
hatchdoing the node.js like version releases :)15:13
frankbansinzui: can we assume odd numbers are never published on the stable ppa or on main/universe?15:22
sinzuifrankban, The answer should be yes and yes...15:22
frankbansinzui: cool thanks15:22
sinzuifrankban, BUT15:23
frankbansinzui: I feel like there is a BUT15:25
sinzuifrankban, I had no intention of letting odd numbers into the ubuntu devel (trusty), but jamespage wanted it in trusty to test gccgo15:27
bacrogpeppe: i'm getting a mongo error from juju 1.17.1 when trying to deploy a charm.  was wondering if you've seen it before.  https://pastebin.canonical.com/103823/plain/15:28
bacrogpeppe: on trusty with local provider15:28
frankbansinzui: so 1.17.1 is in trusty?15:28
rogpeppebac: 1.17.1 is broken15:28
rogpeppebac: i can't see your paste, but i know what it's gonna look like15:29
bacrogpeppe: good parts: E11000 duplicate key error index: juju.charms.$_id_ dup key:15:29
frankbansinzui: yes it is, thank you15:31
mrammfrankban: yes it is in trusty15:34
mramm1.17 is the last time we will push a dev release into trusty though15:35
mrammwe need to get better at releasing stable versions on a regular schedule15:36
mrammthat will eliminate the felt need to push devel releases into the release15:36
rogpeppebac: ah, that's a different issue15:36
mrammbut we needed to get that process running, due to the main inclusion process that we need to go through this cycle15:36
rogpeppebac: dimitern knows about that, i believe15:36
rogpeppebac: it's something to do with charm uploads - were you running deploy twice, or something?15:37
rogpeppebac: (twice concurrently, i mean)15:37
bacrogpeppe: no.  tried one deploy from juju-gui and got the error15:38
rogpeppebac: hmm.15:38
dimiternbac, I'm trying to land a fix for bug 1067979 since yesterday - hopefully not long now15:38
_mup_Bug #1067979: race: concurrent charm deployments corrupts deployments <deploy> <race-condition> <test-needed> <juju-core:In Progress by dimitern> <https://launchpad.net/bugs/1067979>15:38
rogpeppebac: can you reproduce?15:38
bacrogpeppe: yes, could yesterday via the gui.  was going to spin up an env by hand and see if i can reproduce15:38
bacrogpeppe: was just checking first to see if it was a known issue that could be waved off.  glad to help with reproducing if needed15:39
rogpeppebac: once dimitern has landed his fix, perhaps you could try to see if you could reproduce the problem from trunk tip15:41
bacwill do, rogpeppe15:41
TheMuerogpeppe1: take a look: http://paste.ubuntu.com/6844672/16:39
TheMuerogpeppe1: ;)16:39
dimiternbac, the fix for bug 1067979 landed, btw16:53
_mup_Bug #1067979: race: concurrent charm deployments corrupts deployments <deploy> <race-condition> <test-needed> <juju-core:Fix Committed by dimitern> <https://launchpad.net/bugs/1067979>16:53
natefinch-afkrogpeppe1: When you get a chance, I'd like to talk about the machine agent stuff again.  The API code has me mystified as to where the info is that I'm supposed to be watching16:57
rogpeppe1natefinch-afk: ok, perhaps soon?16:58
=== natefinch-afk is now known as natefinch
natefinchrogpeppe1: sure thing16:59
natefinchrogpeppe1: just wasn't sure how late you'd be there today16:59
dimiternnatefinch, a quick review? https://codereview.appspot.com/5468004517:02
dimiternrogpeppe1, fwereade, https://codereview.appspot.com/54680045 ?17:04
* dimitern bbiab17:05
rogpeppe1natefinch: i'm here for another hour, but currently in a discussion17:07
arosalesmgz, fwereade using the conf call today for the joyent provider sync17:07
natefinchrogpeppe1: ok, I know that takes precedence17:10
natefinchdimitern: I can review17:10
mgzI'll need to read the wiki about conf call things17:13
mgzarosales: I actually don't have a pin, if I need one, and getting one seems to involve an rt17:15
dimiternnatefinch, thanks17:15
arosalesmgz sent you the callin details and updated the invite17:15
mgzdstroppa: so, first thing to try with the amulet tests is adding sentries=False when constructing amulet.Deployment()17:40
mgzthen just remove the d.sentry.wait() call below, it's not strictly needed17:41
mgzit gives you fewer hooks to actually inspect your charms and hooks, but can be changed back later when you need them17:42
dstroppamgz: tried that, this is what I'm getting now http://paste.ubuntu.com/6844982/17:43
mgzdstroppa: much better17:52
mgzdstroppa: get lp:juju-deployer and put the script on your path17:54
dimiternnatefinch, review poke?17:55
natefinchdimitern: yeah, sorry, had to do one thing in the meantime, but working on it now17:56
dimiternnatefinch, cheers17:56
dstroppamgz: I'm getting syntax errors when installing juju-deployer18:11
rogpeppe1natefinch: hangout?18:12
natefinchrogpeppe1: yeah, standup one?18:12
rogpeppe1natefinch: ok18:13
rogpeppe1natefinch: https://plus.google.com/hangouts/_/calendar/am9obi5tZWluZWxAY2Fub25pY2FsLmNvbQ.mf0d8r5pfb44m16v9b2n5i29ig?authuser=118:13
rogpeppe1TheMue: cool18:14
natefinchrogpeppe1: you're frozen in the hangout?18:16
dpb1Hi, is this a bug?  I'm trying to do juju terminate-machine, and I'm in this weird state.  I think that this instance launched in an 'ERROR' state, fwiw:  http://paste.ubuntu.com/6845201/18:29
fwereadedpb1, you should be able to --force it18:37
dpb1fwereade: nice.  I suppose it will not clean up the instance in the "ERROR" state, though?18:39
dpb1--force appears to at least removed it from juju's brain, which is what I needed.18:40
fwereadedpb1, afraid not, it's just a way to get it out of juju's hair18:43
dpb1fwereade: k, thx18:43
rogpeppe1g'night all18:44
natefinchdimitern: reviewed, sorry for the delay18:56
marcoceppigot a question about api stuff?19:08
marcoceppithat was a astatement19:09
natefinchmarcoceppi: I'll answer if I can19:09
marcoceppinatefinch: nvm, sorted as maas sadness19:11
natefinchmarcoceppi: heh ok19:12
* thumper has a few errands in town to prep for the trip20:07
thumperwill be back soonish20:07
=== thumper is now known as thumper-afk
bacdimitern: you still around?20:28
=== thumper-afk is now known as thumper
* thumper wants to stab something21:21
thumperffs21:21
thumpernatefinch: you around?21:22
natefinchthumper: yep21:22
natefinchthumper: please don't stab me21:22
thumpernatefinch: have you tried the ec2 provider recently21:22
thumpermine won't start21:22
natefinchthumper: nope, I can try it21:22
thumperERROR cannot make S3 control bucket: A conflicting conditional operation is currently in progress against this resource. Please try again.21:22
thumperI'm trying to try out my all-machines fix for the local provider21:22
thumperand want to confirm it doesn't break others21:23
thumperhmm... seems to be working now21:23
thumperafter failing twice21:23
thumperwonder if it was an amazon glitch21:24
natefinchthumper: yeah weird21:24
thumpernatefinch: lbox still hates me (no go vet), can you review on LP? https://code.launchpad.net/~thumper/juju-core/all-machines-trusty/+merge/20410621:31
* thumper also wants to get waigani to test it21:31
thumperas I know he is on saucy still and had a similar problem with no all-machines.log21:31
thumperI'm pretty sure this will fix it, but always good to get confirmation.21:31
natefinchthumper: at least it's a small change, so not so bad w/o side by side diffs21:33
thumperyeah21:33
thumperuh oh21:34
* thumper has a bad feeling21:34
thumperpoo21:35
thumperdoesn't work on precise21:35
* thumper hangs head21:36
thumperFARRRRKKK!!!!!21:37
natefinchthumper: my wife knows the guy that owns fark, actually.21:38
* thumper resists the urge to have series specific rsyslog rendering21:39
natefinchthumper: still want a review on that code, or...21:44
thumpernot just yet21:44
thumperstabby stabby stabby21:47
thumpernatefinch: care to take a look now?21:57
thumperjust confirming on ec221:57
thumperworks for local21:57
* thumper looks for someone not on trusty21:57
natefinchthumper: looking21:59
thumperhoo fucking ray22:03
thumperseems to work on precise22:03
natefinchthumper: maybe other people more familiar with rsyslog and bash would understand it, but personally, I'd like a comment about why you have to set and reset FileCreateMode like you do.22:03
thumpernatefinch: where do you want the comment?22:03
thumperI thought I had one?22:04
thumperah, it is in the description of the merge22:04
thumpercan put it in the code somewhere if you like22:04
natefinchYeah, just the first time you do it.22:05
thumperI'll have it in the block comment for the template itself22:06
thumperI don't want it rendered in the actual config file22:06
natefinchright22:08
natefinchok, time for me to go.  Have fun in cape town22:09
hazmatthumper, so i'm using latest trunk, and juju get-env doesn't show proxy config as an option.. how does one go about setting it?22:18
thumperprobably because the default is to Omit it if not set22:19
thumperjuju set-env http-proxy=http://foo22:19
thumperya know22:19
thumperI'm tempted to change it from Omit to default to ""22:20
hazmatgot it.. revno 222422:20
thumperwhat do you think?22:20
thumperbecause if you have one set, you can't unset it22:20
thumperas it won't allow ""22:20
thumperstupid juju22:20
hazmathmm.22:20
hazmatthumper, a doc merge proposal would do :-)22:20
hazmatits pretty hard to discover otherwise22:20
thumperoh for sure22:20
thumperI want it feature complete before we bang on about it though :)22:21
hazmatthe diff on revno 2224 shows all the options. i needed to find.22:21
hazmatthumper, i'll have users testing it tonight :-)22:21
thumpercool22:21
thumperlet me know if there are problems or surprises22:21
thumperI probably won't see the email until I'm in cape town22:21
thumperbut I will see it22:21
hazmatthumper, will do.. careful what you wish for :-)22:21
thumperyou going to be there?22:22
hazmatthumper, yup.. i should be in saturday late22:22
* thumper nods22:22
thumperI get in 9am ish sunday morning22:22
* thumper has been futzing around with rsyslog configs22:22
hazmatthere's some other fire i'm parachuting in to work on.. which will take up most of the freetime during the sprint week, but this project with proxies will probably still keep kicking and screaming.22:23
sinzuithumper, lxc is looking better today, upgrade tests work. deploys always fail because of a wordpress agent error. DO you have any ideas about r2282 that would break the agent22:28
* thumper looks at that revno22:29
sinzuitrusting on aws is broken too, but I think that is aws since canonistack's trusty passed22:29
thumpersinzui: I just used aws with trusty and trunk ok22:30
thumperI hit a glitch with s3 when I first tried22:30
thumperbut that resolved itself22:30
sinzuithumper, lucky you. I have many tests and 4 personal runs that cannot bootstrap22:30
thumpersinzui: no idea22:30
thumpersinzui: try now22:30
thumperit was working for me22:30
thumpersinzui: we should get a real test that looks for the all-machines.log for the local provider22:31
thumperI have just fixed the bug and merge should be working through tarmc22:31
sinzuithumper, I add recovery of that log today for lxc22:31
thumperlxc or local?22:31
thumperwe have containers in other providers remember :)22:32
sinzuithumper, lxc. since we had no logs and lots of failures, I wrote a hack to find the log in local/log before the machine is torn down.22:32
thumperI have a horrible feeling that I am missing something important that I should be doing today to prepare for the trip tomorrow22:32
sinzuihttp://162.213.35.54:8080/job/local-deploy/780/ is the last failure22:32
thumpersinzui: oh, you'll love my last change then22:33
sinzuithumper, CI scp's the log from the bootstrapped machine, but it has never been able to do that with lxc22:33
thumperI keep the log file around until you re-bootstrap22:33
thumperwill be in /var/log/juju-<user>-<envname>/all-machines.log22:33
sinzuithumper, thank you22:34
sinzuithumper, aws trusty still dies on apt-get upgrade22:35
thumperhmm22:35
thumperare you deploying to trusty?22:35
thumperI'm not doing that22:35
sinzuithumper, I am just boostrapping on trusty and failing.22:35
sinzuiit worked for CI yesterday22:36
thumperhmm...22:36
thumpersorry, but worked for me just an hour ago22:36
sinzuithumper, I think it is aws.22:36
thumperwaigani_: can I get you to pull trunk, and try a local provider?22:37
thumperwaigani_: see if you get all-machines.log now22:37
thumperI expect that you should22:37
=== thumper is now known as thumper-afk
waigani_okay22:37
sinzuithe unittests failed when they removed the ami we were using, so I expect to tweak the tests to keep them current with trusty in aws22:37
bigjoolsmorning wallyworld_ how's the head? :)22:56
wallyworld_fine22:56
wallyworld_jeez22:56
sinzuiwallyworld_, I am seeing azure header issues.23:12
wallyworld_oh23:12
sinzuiwallyworld_, I just started a manual test of stable to reassure myself23:12
wallyworld_you are seeing issues running trunk?23:13
sinzuiwallyworld_, we are see this with recent revs: http://162.213.35.54:8080/job/azure-deploy/758/console23:13
sinzuilooking good for the manual bootstrap23:17
wallyworld_sinzui: i can't think of anything that has changed in juju wrt that error23:17
sinzuiI fear that azure has changed.23:17
wallyworld_oh joy23:18
sinzuiWe find the machines are often left running or stopped in azure after a test.23:18
wallyworld_in azure but not ec2 or other clouds?23:18
sinzuiwallyworld_, azure and canonistack are leaving machines behind. but in the case of azure, I am seeing the header errors and a lot of 307 temporary redirect errors that state the delete failed23:20
wallyworld_hmmmm. off hand, i can't offer any useful insight23:22
sinzuiwallyworld_, good news. 1.16.5 on azure is good. I will try to collate the trunk errors into something meaningful23:22
wallyworld_good or worked when run but might be flakey next time?23:23
wallyworld_trunk could well be to blame. there have been no azure specific changes to my knowledge but destroy for example has changed and maybe that is indirectly causing issues on azure23:25

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