smoserblackboxsw: 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
smoserblackboxsw: https://bugs.launchpad.net/cloud-init/+bug/174249417:22
ubot5Launchpad bug 1742494 in cloud-init "network reporters generate WARNING in local stage" [Medium,Confirmed]17:22
blackboxswahh thx smoser17:22
blackboxswsmoser: maas code claims it's adding apt configuration (even though they set preserve_sources=False) for earlier cloud-init versions...hmm17: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
smoserthe nwere apt format is more expressive17:56
smoserand allows them to say some thigns that they could not say previously17:56
smoserbut curtin udnersands the new format17:57
smoserand writes it into the target for everything (i'm pretty sure)17:57
blackboxswlanded https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/33510820:56
powersjblackboxsw: if you are looking at merges, want to respond to smoser on https://code.launchpad.net/~powersj/cloud-init/+git/cloud-init/+merge/33577421:01
smoserpowersj: yeah, that was on his list.21:02
powersjok :)21:02
blackboxswumm, 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
blackboxswbut looks like maybe we rebased the cii-requirements branch or something21:08
powersjread my latest comment21:08
blackboxswpowersj: I've conveniently forgotten our conversation. and will read the latest comment21:09
blackboxswhrm.. 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/misremembered21:12
blackboxswpowersj: you talked at standup about maybe pulling a commit rev or git hash on certain dependencies.21:13
powersjyeah I should update pylxd==2.2.4 to be something like git://github.com/lxc/pylxd.git@0722955260a6557e6d2ffde1896bfe0707bbca2721:15
blackboxswwere 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 o21:15
blackboxswbecause I'm all about the short-sighted solutions21:15
powersjand 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 packages21:16
powersjand I don't think I want to go installing pypi packages on anyone's system21:16
blackboxswpowersj: 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
blackboxswor you feel that's muddying the waters too much in read-dependencies script?21:20
powersjyeah trying to remember why we have all these exceptions seems like asking for a nightmare21:21
powersjdon'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 everything21:21
smoseri was just fine to push 'install c-i requiremenst' to a later date.21:25
* smoser has to run21:25
blackboxswyeah 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 branch21:27
blackboxswpowersj: 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
powersjblackboxsw: thanks :) and yes we will21:46
blackboxswpowersj: did you say pylxd was snapped/21:47
powersjblackboxsw: no, lxd is21:47
blackboxswahh ok. gotcha21:47
powersjand to use the lxd snap we needed newer version of pylxd21:48
blackboxswwhich we can only easily get from here right? https://github.com/lxc/pylxd21:49
powersjblackboxsw: correct22:05
blackboxswpowersj: tox env change for pylxd? http://paste.ubuntu.com/26363025/23:19
blackboxswI can put up a branch unless you already have that23:19
blackboxswjust tested with: tox -r -e citest -- run --os-name=artful --platform=ec2 --preserve-data --data-dir=../results --verbose -t modules/runcmd23:20
blackboxswsorry post that irccloud got in the way23:20
blackboxsw 44.9 78.1  47:21.26 chrome23:21
blackboxswgeez man.23:21
blackboxsw78 % MEM for chrome23:21
blackboxswmy machine is almost locking up23:21
powersjI found chrome recently was taking all my swap too23:22
powersjI was out of memory, locking up23:22
powersjanyway... trying your patch, but I think that works23:22
powersjyeah that looks good, can you change the comment to https://paste.ubuntu.com/26363038/23:24
powersjblackboxsw: that way I know why and can update as necessary23:24
blackboxswfunny I had realized that after I posted the pastebin.... I had # lxd backend master tp from 01/10/2018  ..... I like yours better23:36
blackboxswpushing a silly branch for this, just so we all get karma (I mean so it can be tracked)23:36
blackboxswhttps://code.launchpad.net/~chad.smith/cloud-init/+git/cloud-init/+merge/335963 powersj23:44
powersjblackboxsw: thx!23:45
blackboxswwill land it23:46
blackboxswfastest branch ever23:46
blackboxswif only I'd review your branches that quickly23:46
* blackboxsw waits for ci23:47
blackboxswjust because23:47

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