[01:49] <blackboxsw> thx smoser, review-mps commit message linting. bah
[02:16] <smoser> blackboxsw: good now. i fixed.
[02:17] <blackboxsw> yeah, I think we'll have to make a cut of cc_ntp tomorrow for bionic. some things need wrap up there and I want a couple of live tests centos/sles which I'm going through now
[02:17] <blackboxsw> and it's bedtime ;)
[02:24] <smoser> indeed.
[02:24] <smoser> nn
[02:28] <smoser> rharper: https://code.launchpad.net/~mwhudson/livecd-rootfs/default-LANG-C.UTF-8/+merge/342144
[02:28] <smoser> \o/
[02:28] <smoser> i think that means we wont hit that generate any more.
[13:20] <smoser> do3meli: https://jenkins.ubuntu.com/server/job/cloud-init-ci/933/
[13:20] <smoser> and i am going to add you to the group that gets c-i run automatically for you.
[13:46] <do3meli> @smoser: thanks
[13:51] <do3meli> smoser: ci build was successfull. from my point of view good to merge now. if you feel the same you can set the status of the MR to approve ;-)
[13:55] <rharper> smoser: yay LANG!
[14:08] <smoser> do3meli: on its way.
[14:08] <smoser> thanks!
[14:08] <smoser> do3meli: you need more work now ?
[14:08] <smoser> :)
[14:25] <do3meli> haha :-) guess my company wants me to move on with other stuff now :-P
[14:25] <smoser> boooo
[14:27] <do3meli> well i will stay connected i guess ;-) maybe bit bug management or something like that
[14:27] <do3meli> well see
[14:30] <smoser> alright. then i wont set you up to get auto c-i just yet.
[16:07] <smoser>  https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/342214
[16:15] <blackboxsw> smoser: rharper filed the release 18.2 bug
[16:15] <blackboxsw> including it in a cloud-init release bump branch now
[16:16] <blackboxsw> smoser: let's include the dependency
[16:16] <blackboxsw> sound good?
[16:16] <blackboxsw> https://bugs.launchpad.net/cloud-init/+bug/1759318
[16:17] <smoser> eyah. in trunk it doesnt hurt anythin. and we want it in packaging.
[16:17] <smoser> so if you want to pull that, go ahead.
[16:17] <blackboxsw> yeah, figured almost zero risk
[16:17] <blackboxsw> landing your branch (after tox)
[16:20] <smoser> blackboxsw: i'm landing ubuntu branch change now.
[16:20] <blackboxsw> +1 smoser
[16:21] <blackboxsw> not touching it
[16:21] <smoser> done
[16:21] <blackboxsw> thx pulling
[16:33] <blackboxsw> smoser: are you landing https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/342215 ?
[16:33] <blackboxsw> wanted to pull that into 18.2
[16:35]  * blackboxsw misunderstood the 'I'm landing ubuntu branch change" comment
[16:36] <smoser> yeah, lets grab that.
[16:37] <smoser> blackboxsw: i'm landing that one now
[16:37] <blackboxsw> thanks ok will repropose 18.2 after that then
[17:03] <blackboxsw> hrm just dropped tip of master on a cent6 container. it didn't dedect DataSourceNoCloud and it's currently trying to hit Hetzner cloud... checking out what's up
[17:05] <smoser> hm.
[17:08]  * blackboxsw is thinking minimally something is up with the exception_cb
[17:08] <smoser> hm. hm.
[17:10] <smoser> blackboxsw: how do i reproduce that?
[17:10] <blackboxsw> 2018-03-27 17:00:29,960 - handlers.py[DEBUG]: finish: init-local/search-NoCloud: SUCCESS: no local data found from DataSourceNoCloud
[17:10] <blackboxsw> hmm, umm I expected that to find yes in the cent6 container. /me thinks..
[17:10] <blackboxsw> looking over ubuntu
[17:10] <smoser> blackboxsw: you're sure you have /var/lib/cloud/ stuff there ?
[17:11] <blackboxsw> [root@cloud-init-centos-2f4670b4 ~]# ls /var/lib/cloud/seed/
[17:11] <blackboxsw> [root@cloud-init-centos-2f4670b4 ~]#
[17:11] <smoser> the linuxcontainers.org provided images do not have metaata that says it should write that.
[17:11] <blackboxsw> hrm why not, thought that was baked into the lxc images
[17:11] <smoser> you would have to add the metadata to them to write it.
[17:12] <blackboxsw> ahh, ok, oops. thought I hadn't had to do that in the past... but maybe I just forgot a step (as I've only recently just run unittests on cent*)
[17:13] <blackboxsw> nevermind the crazy guy in the corner: PEBCAK
[17:15] <smoser> well, to run unit tests.
[17:15] <smoser>  ./tools/run-centos
[17:15] <smoser> ./tools/run-centos --rpm --srpm --unittest 6
[17:15] <blackboxsw> yeah that's what I kicked with -k -r
[17:15] <blackboxsw> it left me with the container that I then attempted to install the rpm on. neglecting to make sure the image had a seed
[17:16] <smoser> ah. right.
[17:18] <blackboxsw> ok created the seed on centos container, everything happy again with ds detection
[17:18] <blackboxsw> sorry for the noise
[17:19] <blackboxsw> though /me wants to test hetzner cloud again to make sure get_data bails quickly if !hetzner cloud
[17:20] <smoser> blackboxsw: it does not.
[17:21] <blackboxsw> ahh right timeout=2, sec_between=2, retries=30):
[17:21] <blackboxsw> ok
[17:21] <blackboxsw> so 2 mins of read_metadata attempts
[17:22] <blackboxsw> ok; as intended. I was uncertain if it was looping forever on my container. will check that it limits to 30 retries
[17:23] <blackboxsw> I see you landed the dhcp client. ... putting up cloud-init 18.2 in a min here
[17:24] <smoser> blackboxsw: i can fix hetzer
[17:24] <smoser> and i think maybe we should
[17:24] <blackboxsw> it's near last in our list of datasources, but I think we could throw up our hands a bit earlier
[17:24] <blackboxsw> to avoid making IBM wait :)
[17:25] <blackboxsw> smoser: ok on awaiting hetzner  fix for 18.2
[17:25] <smoser> http://paste.ubuntu.com/p/YvkFJCFyXk/
[17:26] <blackboxsw> checking ds-identify for comparison
[17:28] <blackboxsw> +1 system-manafacturer -> sys_vendor in sysfs
[17:28] <blackboxsw> ok
[17:30] <blackboxsw> s/hetzer/hetzner
[17:32] <blackboxsw> yeah 4 mins of Hetzner waiting for both read_metadata, read_userdata is too much waiting
[17:36] <smoser> blackboxsw: https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/342226
[17:39]  * smoser goes to fire up a hetzner cloud instance to just make sure i dont break it.
[17:45] <blackboxsw> smoser: +1, I'll let you merge it once you've hitup hetzner
[17:46] <blackboxsw> I
[17:46] <blackboxsw> I've approved the branch and tested on lxc without seed to see a quick fail
[17:46] <smoser> i put some info there that it was good.
[17:46] <smoser> now actually installing the deb and cloud-init-clean
[17:50] <blackboxsw> pylint and flake8 good on your branch
[17:53] <smoser> you can merge ?
[17:55] <blackboxsw> https://jenkins.ubuntu.com/server/view/cloud-init/job/cloud-init-ci/937/ waiting on ci, about 10 mins left looks like. but I think I can merge it when it's done.
[17:55] <blackboxsw> or I can merge now if you want as I expect a clean CI bill of health
[17:55] <smoser> oh. right.
[17:55] <smoser> i thought you were saying it ran
[17:55] <smoser> we wait on the all powerful jenkins
[17:56] <blackboxsw> pay no attention to the man behind the curtain
[17:56] <blackboxsw> yeah will wait and merge post-jenkins
[18:13]  * blackboxsw relocates
[18:52] <blackboxsw> smoser: rharper https://code.launchpad.net/~chad.smith/cloud-init/+git/cloud-init/+merge/342217 is ready for review
[18:52] <blackboxsw> I pushed a new 18.2 signed tag hopefully that's all we need.as git describe now tells me I'm on 18.2
[18:53] <blackboxsw> smoser's dhcp and hetzner fixes are landed
[19:18] <smoser> blackboxsw: woot.
[19:18] <smoser> ok.
[19:24] <rharper> citest noob;  so the ntp tests fail on my branch, if I ran citest via tox, where do I find the collected data, like cloud-init logs and the files the test collected?
[19:29] <rharper> smoser: for the ntp updates I think I need to a) update the ntp test-case match trunk w.r.t which client is to be installed; note this test changes depending on the release image; xenial, for example is going to always use timesyncd, but in bionic it would be chrony;  b) do we have a way to construct yaml on a per-release basis?  c) does ci-test run only on xenial or what other releases ?
[19:51] <blackboxsw> back
[19:51] <blackboxsw> rharper: ../results generally
[19:51] <rharper> that
[19:51] <rharper> that's in toplevel ?
[19:51] <rharper> when run via tox ?
[19:51] <blackboxsw> it's above your source tree root
[19:52] <blackboxsw> I *think* that's the default... checking
[19:52] <rharper> I think tox ends up doing tmpdir thingys ... I dunno
[19:53] <blackboxsw> might need --preserve-data
[19:53] <rharper> ok, and I wonder if that works under tox
[19:53] <rharper> well, I should just install the deps and run from tree getting it working and then test tox
[19:53] <blackboxsw> tox -e citest -- --preserve-data --results ../results
[19:53] <blackboxsw> or something like that... checking
[19:54] <blackboxsw> rharper .tox/citest/bin/python3 -m tests.cloud_tests run --os-name=bionic --platform=lxd --preserve-data --data-dir=../results --verbose -t modules/ntp
[19:55] <blackboxsw> and -t modules/ntp_servers   -t modules/ntp_pools
[19:56] <smoser> rharper: http://paste.ubuntu.com/p/629gZx94qs/
[19:57] <smoser> that is what i run
[19:57] <smoser> like:
[19:57] <smoser>   ../scripts/go-test lxc --test=tests/cloud_tests/testcases/modules/final_message.py
[19:57] <smoser> and it puts results into ./results.lxc.d/
[19:57] <smoser> you can change the 'xenial' around
[19:57] <smoser> c-i i think only runs with xenial
[19:57] <smoser> but we have other runners that run on test with different releases.
[19:58] <rharper> k
[19:59] <smoser> you should be able to check the release that is being used in the test case
[19:59] <smoser> i dont think we have per-release config
[19:59] <smoser> but you can make pre-release assertions
[20:09] <blackboxsw> ci looks done on 18.2 for cloud-init https://code.launchpad.net/~chad.smith/cloud-init/+git/cloud-init/+merge/342217
[20:12] <rharper> smoser: ok, I'm going to update the config to work with xenial, and branch out into other variants after that
[20:14] <rharper> tox-venv ?
[20:15] <rharper> need to pip install that I guess
[20:15]  * rharper relocates
[20:17] <smoser> rharper: tox-venv is ./tools/
[20:18] <smoser> tox-venv <name>
[20:18] <smoser> enters <name> and executes a command
[20:23] <smoser> the difference compared to 'tox -e' is that you can run any command and it doesnt do setup.py first (which is slow)
[20:24] <blackboxsw> incredibly slow
[20:25] <smoser> blackboxsw: where is the doc for release ?
[20:25] <smoser> https://hackmd.io/zbeM1cG-REq2NE3BvesqOw
[20:25] <blackboxsw> smoser: you mean rtd?
[20:25] <blackboxsw> ahh
[20:25] <smoser> we need to move that to somewhere i think
[20:26] <smoser> blackboxsw: do you want to do the rest ?
[20:26] <blackboxsw> +1, yeah, gist?
[20:26] <smoser> i pushed your tag
[20:27] <smoser> i can do curtin release then.
[20:27] <blackboxsw> +1 I'll do the rest
[20:27] <blackboxsw> on cloud-init
[20:36] <blackboxsw> smoser: rharper  can you see Create release here? https://launchpad.net/cloud-init/+milestone/18.2
[20:36] <rharper> looking
[20:36] <blackboxsw> I think I'm missing perms, as I can see that option on landscape projects
[20:37] <rharper> nope
[20:37] <blackboxsw> I hoped to see it under "Expected: "
[20:38] <blackboxsw> but that date is already set to today(maybe smoser already ticked it)
[20:38] <smoser> https://launchpad.net/cloud-init/+milestone/18.2 <--
[20:38] <blackboxsw> hrm nope:  cloud-init 18.2	2018-03-22	not yet released  from https://launchpad.net/cloud-init/trunk
[20:38] <smoser> thta was teh "due date"
[20:38] <smoser> now we have the release date
[20:40] <blackboxsw> hrm, ok that view changed for me now from what it used to be. probably clicked something I didn't have perms for I guess.
[20:43] <rharper> https://launchpad.net/cloud-init/+series   that has a reference to 2.0 and stackforge ...
[20:43] <rharper> we probably can nuke that
[20:43] <rharper> smoser: ^
[20:44] <smoser> gonzo
[20:47] <rharper> sweet
[20:50] <blackboxsw> smoser: I'm closing bugs now for 18.2 ok ?
[20:52] <smoser> yeah. i'm doing it now for curtin
[20:53] <smoser> stepping away from machine.
[20:53] <smoser> will check back in later tonight.
[20:53] <smoser> bugger
[20:53] <smoser> that stupid script really needs '--project'
[20:53] <smoser> i missed a s/Cloud-init/curtin/
[20:53] <smoser> oh well
[20:54] <blackboxsw> ohh right smoser  lp-bugs-released needs a --project-name
[20:54] <blackboxsw> will add it
[20:54] <rharper> hehe
[20:55] <blackboxsw> cloud-init bugs are marked fix-released
[20:55] <blackboxsw> missing tasks for: 1298921 1757176 1759307.... checking those out now
[20:57] <blackboxsw> I marked the cloud-init task fix released on https://bugs.launchpad.net/ubuntu/+source/cloud-init/+bug/1759307
[20:57]  * blackboxsw fires up a cloud-init MP for bionic
[20:59]  * rharper is iterating with ci-test script from smoser;  excellent 
[21:00] <rharper> having fun with system_config and user-data merging ...  now, I wonder what's going on; don't seem to be getting user-data ntp: dict merged when cc_ntp is running, so it's running in auto mode even when we specify a ntp_client in user-data
[21:01] <blackboxsw> 18.2 release publish MP into bionic https://code.launchpad.net/~chad.smith/cloud-init/+git/cloud-init/+merge/342239
[21:06]  * blackboxsw creates an SRU bug for xenial for 18.2
[21:07] <blackboxsw> xenial & artful rather
[21:09] <rharper> can confirm that the distro object is created before we cloudify, which reads the user-data, so distro itself doesn't have user-data; =(
[21:26] <smoser> blackboxsw: ok. that MP is on its way up... assuming it builds here.
[21:26] <smoser> i uploaded source tarball to https://launchpad.net/cloud-init/trunk/18.2
[21:26] <smoser> and now i'm out.
[21:26] <blackboxsw> thanks smoser see ya
[21:29]  * rharper refactors getting user-data config back int
[21:30] <rharper> and
[21:30]  * rharper relocates
[22:30] <blackboxsw> Xenial SRU for 18.2: https://code.launchpad.net/~chad.smith/cloud-init/+git/cloud-init/+merge/342248
[22:32] <blackboxsw> Artful SRU for 18.2: https://code.launchpad.net/~chad.smith/cloud-init/+git/cloud-init/+merge/342249
[22:32] <blackboxsw> ^ includes the debian/control changes for isc-dhcp-client
[22:57] <blackboxsw> ok just gen'd a trello card for cloud-init SRU https://trello.com/b/hFtWKUn3/daily-cloud-init-curtin
[22:57] <blackboxsw> oops
[22:57] <blackboxsw> https://trello.com/c/RZDKaWSv/719-cloud-init-sru-182-artful-xenial
[23:13] <rharper> ok, unbricked unittests after re-introducing usercfg to the get_ntp_client_info() path as distro object does not have user-data config
[23:38] <rharper> \o/ passing the updated ntp tests
[23:39] <rharper> now to add new client tests