/srv/irclogs.ubuntu.com/2015/06/02/#juju-dev.txt

waigani_wallyworld: lp:gomaasapi/enum.go only has 7 node status consts, where lp:maas/maasserver/enum.py has 15. Do you know if there is a reason for that, or is gomaasapi out of date? If so, I'll add the missing 8 statuses, along with the deployed failed status needed to fix this bug.01:23
axwwallyworld anastasiamac: 1.24 release notes are here: https://docs.google.com/document/d/1qKWvSZ06Vx3ZI2RxYg6P7sWdIvOD5QXPfvpUNEtImMA/edit01:24
cheryljthumper: ping?01:36
thumperhey01:37
cheryljshould DestroyEnvironment be moved from apiserver/client to apiserver/environmentmanager so it could be called when you're just logged into a system?01:37
thumperah...01:38
thumperno01:38
thumperdon't think so because it actually operates on an environment...01:38
thumperwhich makes me rethink slightly01:38
thumperbugger01:38
cheryljheh01:38
thumperah FFS01:39
* thumper thinks01:40
thumpershit shit shit shit shit shit shit shit shit shit shit shit shit shit shit shit shit shit shit shit shit shit shit shit shit shit shit vv01:40
cherylj:(01:40
* thumper rethinks the option of splitting the command up01:41
thumperdamn it01:42
cheryljthumper: need to chat about it?01:44
thumperyeah01:44
cheryljokay, one sec.01:44
thumpergimmie a few minutes to finish munching01:44
cheryljsure01:45
thumpercherylj: https://plus.google.com/hangouts/_/canonical.com/destroy-all-the-environments01:50
wallyworldwaigani_: yes, gomaasapi out of date, which is part of the reason for the bug01:54
waigani_wallyworld: I've fixed the bug, now I just have to refresh my memory on bzr to propose to gomaasapi...01:54
wallyworldwaigani_: does the fix include updating pending machine status to error as per the bug report?01:55
wallyworldbzr push lp:~waigani/gomaasapi/your-branch01:56
waigani_wallyworld: it adds a TestBootstrapNodeFailedDeploy to the maas provider - that is bootstrap returns an error01:56
wallyworldwaigani_: i see, and so that propagates through to make start instance error01:57
wallyworldit works for any instance starting up, not just bootstrap?01:57
waigani_wallyworld: yes, the fix is in waitForNodeDeployment - but l'll follow up with more testing to make sure01:59
wallyworldwaigani_: thanks, i'm just being cautious01:59
waigani_wallyworld: yep, of course02:00
axwanastasiamac: storage-add is all done on 1.24 right?02:07
axwanastasiamac: just updating the docs02:07
wallyworldaxw: anastasiamac had to go out to the school for a bit; add is done except for the 2 remaining PR's for the hool tool fixes02:11
axwwallyworld: thanks02:11
axwwallyworld: is it MAAS 1.8 that's required for disk constraints?02:26
wallyworldaxw: yep02:26
axwwallyworld anastasiamac: updated docs, would appreciate your eyes over it when you have time: https://github.com/juju/docs/pull/44302:31
wallyworldaxw: ty, am finishing a branch will look soon02:32
anastasiamacaxw: tyvm!02:32
anastasiamacaxw: will look soon :D02:32
axwthanks02:32
anastasiamacaxw: tyvm for comments on PR as well :D02:33
anastasiamacaxw: if u have time to cast ur eyes over this one would gr8!!02:33
anastasiamacaxw: http://reviews.vapour.ws/r/1828/02:33
davechen1ythumper: http://paste.ubuntu.com/11510221/02:45
davechen1ylatest state of play02:45
davechen1ystill very messy02:45
mupBug #1460882 was opened: provider/joyent: multiple data races <juju-core:New for dave-cheney> <https://launchpad.net/bugs/1460882>02:46
thumperdavechen1y: keep at it, you're doing a great job!02:46
waigani_wallyworld: https://code.launchpad.net/~waigani/gomaasapi/faildeploy/+merge/26078602:46
wallyworldwaigani_: thanks will look soon02:46
waigani_wallyworld: once that lands I'll update dependencies.tsv and propose the juju-core pr02:47
davechen1ythumper: /me salutes02:47
axwanastasiamac: :(  it occurred to me that storage-add is a bit too permissive with the constraints. unit should not be able to specify which pool to add storage from02:47
axwanastasiamac: that's an operator concern. units should only be able to request more of storage that has already been assigned02:48
axwanastasiamac: i.e. they should be able to specify count -- not even sure about size02:48
axwI think disallowing pool would be sufficient for now02:48
axwanastasiamac: added a comment to review - when doing dependent branches, please use rbt to set the parent branch02:52
menn0thumper: here's the PR that adds the logsink log file03:14
menn0thumper: http://reviews.vapour.ws/r/1835/03:15
thumperk03:15
menn0thumper: my only concern with it is that in a large env there's potentially 1000's of request handlers using the same lumberjack.Logger03:16
menn0thumper: it's goroutine safe (using a mutex internally) so that's not a problem03:17
thumpermenn0: are we writing one file per stateserver or one file per environment?03:17
menn0thumper: but I do wonder about the perform impact for the logsink API03:17
menn0thumper: one per stateserver03:17
thumperhmm.....03:17
menn0thumper: the env UUID is included in each log line03:18
thumperI suppose it is only for post-mortems03:20
menn0thumper: separate log files would be tricker to manage when envs are destroyed03:21
* thumper nods03:21
menn0thumper: re the performance aspect I was thinking there could be a single goroutine which writes to the file, using a buffered channel to help cope with bursts03:21
menn0thumper: not sure if it's worth the complexity (and it might not really help that much)03:22
thumperhmm...03:22
thumperyeah, probably not worth it just yet03:22
menn0thumper: the way things are now if the server end starts slowly down it should be fine because there's tons of buffering on the client side03:22
thumperI guess we'll see how it performs :)03:24
menn0thumper: i could run the performance test again03:25
thumperup to you...03:25
thumperI'm not too worried at this stage03:25
menn0ok03:25
menn0i'll make a note to run the test again - maybe when i'm writing specs later this week03:26
natefinchmenn0, thumper: you can easily toss a bufio.Writer around lumberjack and get buffered writing03:27
natefinchbasically zero complexity.  I know another application using lumberjack did the same thing, because they were doing a ton of small writes.03:27
wallyworldaxw: could you look at this for me? discussing with curtis we have been wanting this for a little while - bootstrap command waits till api server is ready before exiting http://reviews.vapour.ws/r/1836/03:28
menn0natefinch: not a bad idea ...03:29
wallyworldwaigani_: gomaasapi change looks good03:30
axwwallyworld: looking03:32
waigani_waigani_: cool - that will allow the maas provider to report the right error03:32
menn0natefinch: the problem is making sure things make it to disk when the agent dies03:32
natefinchmenn0: yep, that's a problem03:33
menn0natefinch: looking at the Writer code it looks like unless you call Flush or Close before things finish up the contents of the buffer at the end won't make it to disk03:33
menn0natefinch: so i'd still need something watching the api server's tomb which calls flush as it's dying03:33
menn0natefinch: anyway i'll leave things as they are for now. the OS's own caching may be just fine.03:34
menn0thumper: do you have time to look at that PR?03:35
thumpermenn0: yeah03:35
thumperin a few minutes03:35
menn0thumper: thamks03:37
axwwallyworld: reviewed, probably good to ship but I have a few questions first03:39
wallyworldsure, ty03:39
wallyworldaxw: answers published03:43
davechen1ythumper: here is a nasty one https://bugs.launchpad.net/juju-core/+bug/146089303:45
mupBug #1460893: many unhandled assigned values <juju-core:New> <https://launchpad.net/bugs/1460893>03:45
axwwallyworld: I'll shipit, but what I had expected was that when you bootstrap, Juju wouldn't even export a limited API until bootstrap+possible upgrade had finished03:46
wallyworldaxw: but then that would not allow status to run03:47
axwwallyworld: why does that matter? if we're blocking until the API is available?03:47
wallyworldaxw: and it's even harder because the upgrade check needs to api03:47
wallyworldso we need to start the api for the upgrade workr03:48
wallyworldhence everything can see it also03:48
wallyworldand so log in03:48
wallyworldis that making sense03:49
axwwallyworld: it's not unsolvable, but I understand that that makes it more difficult03:49
wallyworldit would need changes down at the rpc leve;03:50
axwyep03:50
wallyworldso messy this late in 1.2403:50
wallyworldand we've wanted bootstrap to behave like this anyway03:50
axwit could use loopback-only during bootstrap-upgrade, but that probably wouldn't help for local provider03:50
wallyworldlikely, yes03:51
wallyworldwe can take another stab in 1.25 or something03:51
axwwallyworld: it's got a rubber stamp now03:51
wallyworldty03:52
wallyworldtested on amazon, just testing on local to be sure03:52
menn0natefinch: how do the log files managed via lumberjack end up with the "correct" ownership and perms?03:55
menn0natefinch: i'm not seeing where that happens03:55
natefinchmenn0: https://github.com/natefinch/lumberjack/blob/v2.0/lumberjack.go#L20003:57
menn0natefinch: ok right. so b/c the shell script used with upstart/systemd sets the initial mode and owner lumberjack perpetuates it04:00
natefinchmenn0: yep04:00
* menn0 needs to do something similar for this other log file04:00
natefinchmenn0: that's like 2/3rds of the reason I left the upstart script alone, was because I didn't want to have to recreate that logic somewhere else.04:01
mupBug #1460893 was opened: many unhandled assigned values <juju-core:New> <https://launchpad.net/bugs/1460893>04:04
thumpertime to head off and give a juju talk04:58
wallyworldaxw: if you have time, i'd like another set of eyes on a problem05:10
axwwallyworld: yup05:11
wallyworldso in that recent PR with bootstrap - it fails on local provider because the isAgentUpgrading() never returns false05:11
wallyworldso bootstrap polls and gives up05:12
wallyworldand yet the bootstrap is ok because deploy works05:12
wallyworldso i can't quite see right now why the channel to signal agent upgrade checks are finished isn't closed05:13
axwwallyworld: I don't see any isAgentUpgrading(), did you change something and not upload?05:13
wallyworldno, don't think so. that's in jujud/agent/machine.go05:13
axwisAgentUpgradePending?05:14
axwok05:14
wallyworldseems to all work fine on aws05:14
wallyworldif i hard code isAgentUpgradePending() to return false it works05:14
wallyworldso for some reason the agent upgrade worker is not closing the channel on local provider05:15
wallyworldthat's where i've got to so far05:15
axwwallyworld: I'm guessing it's something to do with local provider auto-bumping the version05:16
wallyworldah, yeah maybe, that version is stored as agent-version in state i guess05:16
axwyes05:16
wallyworldi'm sure i tried this, let me try closing that channel at the start of the upgrade worker05:17
menn0wallyworld: could you have a quick look at this? http://reviews.vapour.ws/r/1838/05:18
wallyworldok05:18
menn0wallyworld: thanks. it's an easy one.05:18
davechen1ymenn0: sorry, -1 on that PR05:19
davechen1ymenn0: delete the tests and i'll LGTM it05:20
menn0davechen1y: fine by me I guess05:21
menn0davechen1y: the tests are kind pointless but i was expecting a reviewer to require them05:21
menn0:)05:21
menn0kinda05:21
menn0davechen1y: the original func with the juju codebase wasn't tested either05:22
wallyworldaxw: if i hard code the closing of the agentUpgradeComplete channel at the very start of the agent upgrade worker loop, it still is unhappy, so the version bump may be a red herring05:23
axwwallyworld: I'll pull your branch and see if I can spot anything05:24
wallyworldaxw: thanks, i'm sure it's something ovious05:24
menn0wallyworld: you ok with me deleting the test as per davechen1y (above)05:24
menn0wallyworld: ?05:25
wallyworldmenn0: was looking at test05:25
wallyworldi guess so05:25
davechen1ymenn0: os.Chown has tests05:25
davechen1ythe gynmastics to mock out the call just to prove that we can call that call05:25
davechen1yseem unnecessary05:25
menn0davechen1y: I agree it's a little contrived05:25
menn0davechen1y: i could go either way on this one05:25
menn0davechen1y: the test at least shows that the right things are called05:26
davechen1yif it didn't work, there are other tests that would break in other packages05:26
davechen1yif you want to ahve tests05:26
davechen1ytest that calling the os.Chown wrapper changes permissions on disk05:26
davechen1yotherwise all you're testing is function dispatch works05:27
menn0davechen1y: yeah but you can't do that meaningfully if the tests aren't running as root05:27
davechen1yhave two tests05:27
davechen1yif os.GetUid() != = {05:27
davechen1y  t.Skipf("test skipped, run as roo")05:27
davechen1y  t.Skipf("test skipped, run as root")05:27
davechen1yor seomthing05:27
menn0davechen1y: yeah but that will never happen05:27
menn0and you shouldn't have to run tests as root05:28
davechen1yagain, it comes down to what is the test testing ?05:28
davechen1yif the test _requires_ that root can os.Chown, then we have to run as root05:28
davechen1yotherwise, mocking the funciotn doesn't test anyhting05:28
davechen1y(apart from function dispatch)05:28
menn0davechen1y: sure but it's checking that the correct funcs are called with the correct args which is something05:29
menn0davechen1y: anyway, i don't actually mind much05:29
menn0davechen1y: i'll delete the test05:29
axwwallyworld: you have to get a whole new client05:31
axwwallyworld: also, you should close the client in that method05:31
davechen1ymenn0: sgtm05:31
davechen1ynot worth the argument05:31
wallyworldaxw: let me check what i did05:31
axwwallyworld: would be nice if the "Waiting for API..." message came before "Bootstrap complete" too05:31
wallyworldaxw: yeah, i thought of that, the bootstrap complete is down in environs - i could move it from there05:33
wallyworldso what do you mean by a new client? is NewAPIRoot(0 not enough?05:33
axwwallyworld: or change environs/bootstrap.Bootstrap to do the waiting - I think that would be preferable, rather than adding more logic to the command code05:34
axwwallyworld: I mean, when you retry, you need to get a new client05:34
axwwallyworld: otherwise it uses the same apiserver root05:34
wallyworldwhy a new client when retrying?05:34
wallyworldoh right, yes05:34
axwwallyworld: ^^05:34
wallyworldaxw: i thought about putting the check in environs, but that would have meant putting client code down there05:35
wallyworldand i didn't think that to be appropriate05:35
axwwallyworld: well to be truthful the bootstrap *is* complete, so I guess it can stay as it is05:35
wallyworldyeah, i sorta came to the same conclusion05:36
wallyworldso let me retry getting another client each retry05:36
wallyworldaxw: yeah, that was it, thank you. i think i was hoping / expecting that a server side change would be noticed by existing api server roots05:38
axwwallyworld: the current impl doesn't strike me as ideal05:39
wallyworldme either05:39
wallyworldi keep thinking it works differently05:39
wallyworldaxw: just read the release notes - the link in the note to the wip doc - the doc is out of date eg says placement not supported06:41
wallyworldnot sure if we should just cut and paste the wip doc into the release notes as doc06:41
axwwallyworld: where? I updated that06:42
axwwallyworld: sorry, I updated the doc: https://github.com/juju/docs/pull/44306:42
wallyworldin Unimplemented Cavearts06:42
wallyworldoh, ok06:42
wallyworldso just needs to be acted on06:42
wallyworldaxw: makes more sense now. i think maybe would also be worthwhile to mentin the storage hook tools in the google doc so that at least the doc has mentions of the key features?06:44
axwwallyworld: seeing as it's the first release, I was trying to avoid having all the same information in two places. I can if you really want.06:46
wallyworldaxw: the issue is that the release notes are really quite comprehensive, so give the impression they cover "all the things"and so a reader might see them and not realise hook tools are there. i agree agbout 2 places. so perhaps the release notes need to be wound back?06:47
axwwallyworld: originally it was just "we've done storage, here's a link to the docs" - yes, I think we should go back to that06:48
wallyworldok, sounds good. i have to head to soccer, will be back later. can you liaise with anastasia if necessary to get her stuff landed?06:48
wallyworldand get her to re-read the release notes and wip doc?06:48
axwwallyworld: no worries06:49
wallyworldtyvm06:49
* anastasiamac cleans storage-add stuff06:52
anastasiamacericsnow: ping :D07:19
fwereadewell, that was interesting08:59
* fwereade just found a small lizard hiding in his dressing gown09:00
voidspacefwereade: I'm sure it's not that small09:00
voidspacedon't be so hard on yourself09:00
fwereadehaha09:01
=== mgz is now known as mgz_
rogpeppefwereade, dimitern, voidspace: fancy a little review? (a couple of bug fixes to the juju/schema package) https://github.com/juju/schema/pull/609:39
dimiternrogpeppe, LGTM09:46
rogpeppedimitern: thanks!09:46
anastasiamacaxw: in case u miss my pmsg - doc is LGTM :D loving your writting style!!10:30
perrito666morning11:51
rogpeppeanyone know if there's any documentation for the possible environment config attributes anywhere?12:03
anastasiamacperrito666: morning!12:04
rogpeppedimitern, jam, fwereade, perrito666, mgz_, evilnickveitch: ^12:04
rogpeppeanastasiamac, perrito666: hiya12:05
anastasiamacrogpeppe: o/12:05
perrito666rogpeppe: sorry dont know12:05
evilnickveitchrogpeppe, maybe...12:05
evilnickveitchdo you mean this:12:06
evilnickveitchhttps://jujucharms.com/docs/stable/config-general12:06
rogpeppeevilnickveitch: that looks good, thanks. except... a some of those values don't seem to have any descriptions.12:08
rogpeppeevilnickveitch: it's a good starting point though, thanks12:09
=== liam_ is now known as Guest92132
evilnickveitchrogpeppe, yes indeed, if you could fill them in as you go along, that would be a great help :)12:09
rogpeppeevilnickveitch: i'm just about to make a big table inside environs/config that describes them all :)12:09
evilnickveitchhurrah!12:10
Syed_AHello Folks, How can i force remove a service stuck with a hook failed.12:27
Syed_Anevermind12:51
dooferladvoidspace, dimitern: https://plus.google.com/hangouts/_/canonical.com/maas-juju-net13:06
dimiterndooferlad, voidspace, it's over isn't it?13:10
dooferladdimitern: yea13:10
dooferladdimitern: I was the only Juju guy who turned up and didn't have much to add.13:11
dimiterndooferlad, too bad :)13:11
dimiterndooferlad, yeah, np13:11
natefinchperrito666: it ended up being faster to just write a dumb program than try to figure out how to install that bzr plugin: https://github.com/natefinch/bsmrt13:40
perrito666lol13:40
katconatefinch: as ocr, can you give the reviews for this bug precedence? https://bugs.launchpad.net/juju-core/+bug/145162613:41
mupBug #1451626: Erroneous Juju user data on Windows for Juju version 1.23 <1.23> <blocker> <juju> <oil> <regression> <windows> <juju-core:Triaged> <juju-core 1.24:Triaged> <https://launchpad.net/bugs/1451626>13:41
katconatefinch: it's been sitting for a bit, and we would like to land it ASAP for 1.2413:42
natefinchkatco: yep.13:42
katconatefinch: ty sir13:42
perrito666man being able to hangout and build juju is so cool13:42
katco:p13:42
natefinchperrito666: what, did you upgrade to a supercomputer?13:43
wwitzel3lol13:43
perrito666I have a backpack cluster to run hangout13:44
perrito666the level of annoyance I have to go to get local provider working despite not having juju local installed is.. well annoying :p13:50
natefinchkatco: lol, Gabriel has gsamfira has been trying to get me to review that stuff for ages... he'll be glad it's officially my duty to do so now.13:50
katconatefinch: haha13:51
katconatefinch: sorry... you just happened to be ocr ;p13:51
natefinchkatco: I've wanted to review it, but we've just been so tight on deadlines, I haven't felt like I could spare the time, so I'm glad to be able to now.13:51
rogpeppeevilnickveitch: FWIW, all these config attrs are available but don't seem to be documented: block-remove-object provisioner-safe-mode rsyslog-ca-key block-destroy-environment tools-metadata-url storage-default-block-source block-all-changes lxc-use-clone tools-stream allow-lxc-loop-mounts lxc-default-mtu14:04
rogpeppeev: ah, tools-metadata-url is actually documented as deprecated14:05
evilnickveitchrogpeppe, yup. thanks for the others though.14:06
evilnickveitchrogpeppe, what is the difference between lxc-clone and lxc-use-clone?14:07
evilnickveitchor are they the same?14:07
rogpeppeevilnickveitch: i've no idea :)14:07
evilnickveitchhehehe14:08
rogpeppeevilnickveitch: i need to find out though14:08
mupBug #1461111 was opened: Allow status-set/get to a service by its leader unit <juju-core:Triaged by hduran-8> <juju-core 1.24:In Progress by hduran-8> <https://launchpad.net/bugs/1461111>14:11
wallyworldevilnickveitch: the same, lxc-use-clone is deprecated14:14
wallyworldrogpeppe: ^^^14:14
rogpeppewallyworld: is that documented anywhere?14:14
evilnickveitchwallyworld, thanks!14:14
wallyworldrogpeppe: not sure tbh14:14
rogpeppewallyworld: is lxc-clone itself documented, in fact?14:15
wallyworldrogpeppe: evilnickveitch: also, anastasiamac sent out notes on the block config14:15
rogpeppewallyworld: i'm trying to gather info on all the config attributes to put into a table inside environs/config14:15
wallyworldrogpeppe: it would have been in release notes but i sadly suspect we didn;t do more than that14:15
evilnickveitchwallyworld, cool, I will sync with her on updating the table14:16
wallyworldty14:16
evilnickveitchrogpeppe, lxc-clone is documented, but as it is provider specific, it is on the lxc page14:18
evilnickveitchhttps://jujucharms.com/docs/stable/config-LXC14:18
rogpeppeevilnickveitch: it's not really provider-specific, is it? all environments can have lxc containers14:19
wallyworldevilnickveitch: rogpeppe: it used to be but not anymore14:19
wallyworldit was changed in 1.2014:19
rogpeppewallyworld: ok14:19
wallyworldi think that's when use-clone was deprecated also14:19
* wallyworld is really going away now to sleep14:20
evilnickveitchheheheh14:20
evilnickveitchnight night wallyworld14:20
wallyworldttyl14:20
rogpeppewallyworld: thanks for the pointer14:21
evilnickveitchrogpeppe, provisioner-safe-mode is deprecated too, as of 1.21.114:25
rogpeppeevilnickveitch: ok, thanks14:25
rogpeppeevilnickveitch: presumably superceded by harvest-mode14:26
evilnickveitchyes14:26
evilnickveitchrogpeppe, tools-stream was replaced with agent-stream14:27
=== brandon is now known as web
=== kadams54 is now known as kadams54-away
=== kadams54-away is now known as kadams54
perrito666is anyone willing to run a test in 1.24 to confirm something?16:15
mupBug #1461150 was opened: lxc provisioning fails on joyent <blocker> <ci> <joyent-provider> <lxc> <regression> <juju-core:Triaged> <juju-core 1.24:Triaged> <https://launchpad.net/bugs/1461150>16:24
perrito666I need to leave for a moment can anyone check if, while in debug hooks,  you can call any helper at all?16:28
perrito666for 1.24 that is16:28
natefinchI'm leaving for a while too, or I would16:29
perrito666I think something is seriously broken there16:29
rogpeppemgz_: ha, looks like 445a79b25d7d7a95127ec36a1f4c41674718a98f changed a little more than you hoped it would16:32
rogpeppemgz_: for instance, i just found a bug link to https://bugs.github.com/juju/juju/+bug/122449216:32
mupBug #1224492: environs/config: zero-valued port settings are allowed but ignored <tech-debt> <juju-core:Triaged> <https://launchpad.net/bugs/1224492>16:32
rogpeppemgz_: which should be the link that mup pointed to...16:33
fwereadeDAMMIT: ... Error: Account should have balance of 25000, got 2499916:44
fwereadeI really thought I might have had it that time16:44
wwitzel3fwereade: trying to pull a Superman III?16:57
Syed_AI am trying to reduce the memory of containers by setting up lxc.cgroup.memory.limit_in_bytes in /var/lib/lxc/conatiner/config, but it is throwing errors.17:15
Syed_AHow can i change the default memory for containers in juju ?17:16
Syed_Alxc_cgmanager - cgmanager.c:cgm_setup_limits:1250 - Error setting cgroup memory:lxc/juju-machine-6-lxc-3 limit type memory.memsw.limit_in_bytes17:16
lazyPowerquestion: I'm running 1.24-beta5.1 and i've found some weird behavior today. I don't have unit output as i tore the env down and stood it back up wiping the logs in teh process - but i've run into edge cases where it appears that juju did not upload is-leader as part of the toolchain17:22
lazyPoweris this known behavior that i've missed a bug on? I dont want to file a bug without additional info to support the claim other than lazy's gone skitzo17:23
lazyPowerkatco: btw - ive put aside some time to review the status doc you sent over, will get you feedback before i EOD, ta for sending that over17:23
fwereadelazyPower, I'd still go ahead and report it17:24
lazyPowerfwereade: ack, will do. Sorry about the zero info bug in advance :|17:25
fwereadelazyPower, no worries, that points to a pretty specific area of the code, the inputs to which have been changing a bit lately17:26
=== kadams54 is now known as kadams54-away
=== kadams54-away is now known as kadams54
=== kadams54 is now known as kadams54-away
=== kadams54-away is now known as kadams54
Syed_AIs it possible to reduce the memory of a lxc container started by juju ?18:49
=== liam_ is now known as Guest52471
Syed_ACan i increase the memory of a lxc container started by juju ?19:09
=== kadams54 is now known as kadams54-away
natefinchabentley: btw, I updated the merge proposal, and I believe I addressed all your concerns (for real this time).19:54
abentleynatefinch: Thanks.  I'll have a look soon.19:54
natefinchabentley: thanks19:54
marcoceppikatco: can we chat about min version?19:56
katcomarcoceppi: sure: https://plus.google.com/hangouts/_/canonical.com/moonstone?authuser=120:00
abentleysinzui: Could you please review https://code.launchpad.net/~abentley/juju-reports/web-enqueue/+merge/260879 ?20:01
natefinchkatco, marcoceppi: we  should rename it to "Capability Flags" since that better reflects what we're really implementing (until someone decides we need a different color shed).20:02
sinzuiyes abentley20:06
abentleynatefinch: You still have this wait_for_started that you haven't explained the need for or removed.20:07
natefinchabentley: oh, I thought I deleted that.. honestly, I was just copying what deploy_stack did when it was deploying charms.20:07
natefinchabentley: I can delete it, that's fine.  Running a test with it gone right now, just for a sanity check.20:10
abentleynatefinch: I wish I had a perfect script to point you to.  The problem with deploy_stack is that, as the oldest script, it is out-of-date in places.20:10
natefinchabentley: yeah, I didn't realize that when I was copying everything it did :/20:11
natefinchabentley: that's part of why I tell people to follow what assess_log_rotation.py does for new CI tests in the wiki page.... because at least that'll be less wrong.20:12
mupBug #1461246 was opened: UpgradeSuite.TestUpgradeStepsHostMachine consistently fails on utopic <blocker> <ci> <regression> <unit-tests> <utopic> <juju-core:Triaged> <juju-core 1.24:Triaged> <https://launchpad.net/bugs/1461246>20:12
natefinchabentley: test passes, removed and pushed20:13
abentleynatefinch: That seemed like a premature choice given that assess_log_rotation hadn't yet passed code review.20:13
natefinchabentley:  but I knew it was being reviewed carefully, so once it was in, it should be a good example.20:15
natefinchabentley: and if not, then I knew someone could just go change the wiki page anyway ;)20:15
abentleysinzui: Could you tell me what you think of the indentation changes in https://code.launchpad.net/~natefinch/juju-ci-tools/logrot/+merge/259750 ?20:37
=== kadams54-away is now known as kadams54
natefinchabentley, sinzui: for the record, everything was run through autopep8... apologies for the apparent spurious indentation changes.20:39
abentleynatefinch: The updated style is more my preference, but I believe the previous style is sinzui's preference and I've been writing to that.20:43
=== natefinch is now known as natefinch-afk
sinzuinatefinch-afk: abentley I have pondered switching to autopep8. I dissagree with the unpythonic closing brace that is NOT specified in PEP8.21:04
sinzuinatefinch-afk: abentley The formatting is fine. I accept the change.21:04
sinzuiabentley: natefinch-afk setting "ignore": "E123, E133", for pep8 and autopep8 removed trailing brace from discussion21:06
* sinzui uses both to not take sides when he reviews/updates other people’s code21:07
sinzuir=me abentley. I missed the submit button 30 minutes ago21:08
wallyworldwaigani: quick status update on the maas fix? should be done today?21:19
wallyworldor this morning even :-) ?21:20
mupBug #1460171 changed: Deployer fails because juju thinks it is upgrading <blocker> <ci> <deployer> <regression> <tech-debt> <upgrade-juju> <juju-ci-tools:Fix21:27
mupReleased by sinzui> <juju-core:Fix Released by wallyworld> <python-jujuclient:Fix Committed by wallyworld> <https://launchpad.net/bugs/1460171>21:27
waiganiwallyworld: yes, in stand up - first thing after21:29
wallyworldty :-)21:29
=== kadams54 is now known as kadams54-away
=== kadams54-away is now known as kadams54
=== kadams54 is now known as kadams54-away
waiganiwallyworld: what's the best way to merge the branch into gomaasapi?21:46
wallyworldwaigani: sorry, in meeting, will need to pull trunk and merge and repush, can you ask thumper21:47
waiganiwallyworld: right, thanks21:47
wallyworldsinzui: i don't think this one will be fixed for 1.24 either https://bugs.launchpad.net/bugs/145722521:52
mupBug #1457225: Upgrading from 1.20.9 to 1.23.3 works, but error: runner.go:219 exited "machiner": machine-0 failed to set status started: cannot set status21:52
mupof machine "0": not found or not alive <cts> <sts-stack> <upgrade-juju> <juju-core:Triaged> <juju-core 1.24:Triaged> <https://launchpad.net/bugs/1457225>21:52
menn0natefinch-afk: did you see this: https://bugs.launchpad.net/juju-core/+bug/146124621:53
mupBug #1461246: UpgradeSuite.TestUpgradeStepsHostMachine consistently fails on utopic <blocker> <ci> <regression> <unit-tests> <utopic> <juju-core:Triaged> <juju-core 1.24:Triaged> <https://launchpad.net/bugs/1461246>21:53
sinzuioh goody. wallyworld I don’t see an action we can take on it. Agreed.21:53
waiganiwallyworld: gomaasapi updated, dependencies.tsv updated, Tried to land on 1.24, but it's blocked. Shall I JFDI?22:10
wallyworldwaigani: yeah, i think so22:10
waiganiwallyworld: done22:11
wallyworldtyvm22:11
whitif I'm running off 1.24-beta5-trusty-amd64 locally and juju bootstrap --upload-tools, the tools on my deployed machines should match right?22:30
* whit is seeing tools == 1.24-alpha1.1-trusty-amd6422:31
* whit is wondering because is-leader is fairly consistently not present22:31
wallyworldwaigani: did you see the gomaasapi dep looks wrong because build failed?22:34
waiganiwallyworld: on it22:35
waiganiwallyworld: I don't get it. TestDependenciesTsvFormat passes and the revision id is from here: http://bazaar.launchpad.net/~juju/gomaasapi/trunk/revision/6222:40
wallyworldwaigani: sorry, was in another meeting and about to start another but gotta do something first, can you ask thumper (it's not like he doesn't know bzr :-)22:55
waiganiwallyworld: found it. there was a space before the revis id but after the tab - the format test didn't pick that up.23:00
wallyworldwaigani: you should always run godeps locally before pushing23:01
waiganiwallyworld: lesson learnt. I'll update the test also.23:02
wallyworldty :-23:02
wallyworld)23:02
mwhudsondavechen1y: fwiw, go 1.4 is in wily now23:02
perrito666ericsnow: ping23:09
ericsnowperrito666: hi23:09
alexisband sinzui and team is working on 1.4.2 verification with juju23:09
perrito666ericsnow: heyhey23:09
perrito666ericsnow: do you have any extra info on https://bugs.launchpad.net/juju-core/+bug/1434437 besides whats on the bug?23:10
mupBug #1434437: juju restore failed with "error: cannot update machines: machine update failed: ssh command failed: " <backup-restore> <maas-provider> <juju-core:Invalid> <juju-core 1.22:Fix Released by ericsnowcurrently> <https://launchpad.net/bugs/1434437>23:10
ericsnowperrito666: nope23:10
perrito666ericsnow: I am quite curious abut the original bug, as that would indicate that at least one of the machines was not provisioned?23:10
perrito666I am not entirely sure a machine of juju can be missing a /var/lib/juju23:11
ericsnowperrito666: yeah, not sure23:15
ericsnowperrito666: could be that thing where juju uninstalls itself23:16
stokachusinzui: any issue with juju failing to create lxc containers? http://paste.ubuntu.com/11530516/23:16
stokachuive hit this in 2 separate labs, one with proxy and one without23:16
stokachuon both trusty and precise23:16
perrito666ericsnow: not sure, although the issue seems to have been resolved since then and replaced by other23:17
stokachuboth using maas23:17
wallyworldperrito666: standup?23:18
perrito666wallyworld: I am there23:18
davechen1ymwhudson: right23:30
davechen1yso what was all the hubbub about23:30
mwhudsonnot sure23:30
* mwhudson is fighting monitors23:30
alexisbdavechen1y, mostly my lack of understanding in the process, I apologize for the unneeded alarm23:31
davechen1yalexisb: all good23:32
davechen1yi don't care how it happens23:33
davechen1yonly that it happens23:33
davechen1ysinzui: so the next step is to get the W 1.4.4 package backported into the juju ppa23:33
davechen1yhow would you like to track that work ?23:33

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