| smoser | blackboxsw: thank you. i'm not sure why that appears sometimes and is not crippling. | 01:43 |
|---|---|---|
| === shardy is now known as shardy_lunch | ||
| === shardy_lunch is now known as shardy | ||
| === shardy is now known as shardy_mtg | ||
| === shardy_mtg is now known as shardy | ||
| smoser | blackboxsw: https://bugs.launchpad.net/cloud-init/+bug/1742494 | 17:22 |
| ubot5 | Launchpad bug 1742494 in cloud-init "network reporters generate WARNING in local stage" [Medium,Confirmed] | 17:22 |
| blackboxsw | ahh thx smoser | 17:22 |
| blackboxsw | smoser: maas code claims it's adding apt configuration (even though they set preserve_sources=False) for earlier cloud-init versions...hmm | 17:41 |
| blackboxsw | # Add APT configuration for new cloud-init (>= 0.7.7-17) | 17:41 |
| * blackboxsw checks how 0.7.7-17 behaved regarding apt.... maybe this is an issue on trusty for some reason? | 17:42 | |
| smoser | the nwere apt format is more expressive | 17:56 |
| smoser | and allows them to say some thigns that they could not say previously | 17:56 |
| smoser | but curtin udnersands the new format | 17:57 |
| smoser | and writes it into the target for everything (i'm pretty sure) | 17:57 |
| blackboxsw | landed https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/335108 | 20:56 |
| powersj | blackboxsw: if you are looking at merges, want to respond to smoser on https://code.launchpad.net/~powersj/cloud-init/+git/cloud-init/+merge/335774 | 21:01 |
| smoser | powersj: yeah, that was on his list. | 21:02 |
| powersj | ok :) | 21:02 |
| blackboxsw | umm, https://code.launchpad.net/~powersj/cloud-init/+git/cloud-init/+merge/335774 so, I'm forgetting something here josh. I thought I had seen a changeset at some point that did still provide a mechanism by which we could install system packages via 'make ci-ubuntu-deps' | 21:08 |
| blackboxsw | but looks like maybe we rebased the cii-requirements branch or something | 21:08 |
| powersj | read my latest comment | 21:08 |
| blackboxsw | powersj: I've conveniently forgotten our conversation. and will read the latest comment | 21:09 |
| blackboxsw | hrm.. yeah I guess I thought at the end of our conversation you were working up a solution to handle the outliers which couldn't be installed bit pypi or system packages. looks like I misunderstood/misremembered | 21:12 |
| blackboxsw | powersj: you talked at standup about maybe pulling a commit rev or git hash on certain dependencies. | 21:13 |
| powersj | yeah I should update pylxd==2.2.4 to be something like git://github.com/lxc/pylxd.git@0722955260a6557e6d2ffde1896bfe0707bbca27 | 21:15 |
| blackboxsw | were you thinking about that as 'futures'? we might be able to extend read-dependencies script to handle some of that lookup for us. /me digs back into context here to look for shortsighted/quick-wins o | 21:15 |
| blackboxsw | because I'm all about the short-sighted solutions | 21:15 |
| powersj | and yes I was thinking about doing something special for integration tests, but it just didn't make sense because 1) it is only ubuntu and 2) I would need some mix of system packages + pypi packages | 21:16 |
| powersj | and I don't think I want to go installing pypi packages on anyone's system | 21:16 |
| blackboxsw | powersj: what if we left that packages/pkg-deps.json rename and the bzr+lp system package translation in tools/read-dependencies... then at least our "make ci-ubunt-deps" could still additionally call "read-dependencies --requirements-file integration-requirements.txt" | 21:19 |
| blackboxsw | ... with the exception of pylxd right? | 21:19 |
| blackboxsw | or you feel that's muddying the waters too much in read-dependencies script? | 21:20 |
| powersj | yeah trying to remember why we have all these exceptions seems like asking for a nightmare | 21:21 |
| powersj | don't get me wrong though, having a install-deps for integration tests would be nice, and I would like it a lot, especially since I want to get two new jenkins systems up, I don't want to have to worry about deps for everything | 21:21 |
| smoser | i was just fine to push 'install c-i requiremenst' to a later date. | 21:25 |
| * smoser has to run | 21:25 | |
| blackboxsw | yeah that's fine there too. will land powersj as is. we can bikeshed on making make ci-deps-ubuntu work for integration tests in a subsequent branch | 21:27 |
| blackboxsw | powersj: landed, https://code.launchpad.net/~powersj/cloud-init/+git/cloud-init/+merge/335774. Thanks for the rehash. We'll go in circles about this again later :) | 21:45 |
| powersj | blackboxsw: thanks :) and yes we will | 21:46 |
| blackboxsw | powersj: did you say pylxd was snapped/ | 21:47 |
| powersj | blackboxsw: no, lxd is | 21:47 |
| blackboxsw | ahh ok. gotcha | 21:47 |
| powersj | and to use the lxd snap we needed newer version of pylxd | 21:48 |
| blackboxsw | which we can only easily get from here right? https://github.com/lxc/pylxd | 21:49 |
| powersj | blackboxsw: correct | 22:05 |
| blackboxsw | powersj: tox env change for pylxd? http://paste.ubuntu.com/26363025/ | 23:19 |
| blackboxsw | I can put up a branch unless you already have that | 23:19 |
| blackboxsw | https://www.irccloud.com/pastebin/uIisHdgJ/ | 23:19 |
| blackboxsw | just tested with: tox -r -e citest -- run --os-name=artful --platform=ec2 --preserve-data --data-dir=../results --verbose -t modules/runcmd | 23:20 |
| blackboxsw | sorry post that irccloud got in the way | 23:20 |
| blackboxsw | 44.9 78.1 47:21.26 chrome | 23:21 |
| blackboxsw | geez man. | 23:21 |
| powersj | lol | 23:21 |
| blackboxsw | 78 % MEM for chrome | 23:21 |
| blackboxsw | my machine is almost locking up | 23:21 |
| blackboxsw | #toomanytabs | 23:22 |
| powersj | I found chrome recently was taking all my swap too | 23:22 |
| powersj | I was out of memory, locking up | 23:22 |
| powersj | anyway... trying your patch, but I think that works | 23:22 |
| powersj | yeah that looks good, can you change the comment to https://paste.ubuntu.com/26363038/ | 23:24 |
| powersj | blackboxsw: that way I know why and can update as necessary | 23:24 |
| blackboxsw | funny I had realized that after I posted the pastebin.... I had # lxd backend master tp from 01/10/2018 ..... I like yours better | 23:36 |
| blackboxsw | pushing a silly branch for this, just so we all get karma (I mean so it can be tracked) | 23:36 |
| blackboxsw | https://code.launchpad.net/~chad.smith/cloud-init/+git/cloud-init/+merge/335963 powersj | 23:44 |
| powersj | blackboxsw: thx! | 23:45 |
| powersj | +1ed | 23:46 |
| blackboxsw | thx | 23:46 |
| blackboxsw | will land it | 23:46 |
| blackboxsw | fastest branch ever | 23:46 |
| blackboxsw | if only I'd review your branches that quickly | 23:46 |
| * blackboxsw waits for ci | 23:47 | |
| blackboxsw | just because | 23:47 |
| powersj | :) | 23:50 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!