[12:33] hi [12:34] I looked for a way to determine if cloud-init has finished to configure the instance. What is the right way to do so? [12:56] ok, found the necessary infos here https://github.com/number5/cloud-init/blob/master/doc/status.txt [12:56] Thanks ;) [13:22] smoser: thanks for the CI run. the error that got thrown does not seems to be related to my branch. am i right with that? [13:22] error given in https://jenkins.ubuntu.com/server/job/cloud-init-ci/928/console [13:22] cloudinit/config/cc_puppet.py:143: [W1505(deprecated-method), handle] Using deprecated method readfp() [13:23] do3meli: rebase to master. [13:23] (that is fixed in trunk already) [13:23] make sense ? [13:24] allright, let me do that [13:34] smoser: rebase done and pushed [13:38] do3meli: running https://jenkins.ubuntu.com/server/view/cloud-init/job/cloud-init-ci/929/console [13:49] thx [14:17] do3meli: curious... have you tested this with linux ? [14:18] or are you just adding support for zfs resize of freebsd [14:18] i am mainly focusing on freebsd [14:19] i have no zfs on linux to test [14:19] with root on zpool [14:22] right. [14:24] do3meli i'm reviewing. i'll just put comments tthere. [14:24] but will submit hopefully in 5 minutes [14:24] perfect, thanks [14:32] do3meli: i responded. [14:33] i think that the end result is just "update the message" [14:33] but i'd like for you to think through the "safe for linux" thought path also. [14:34] thanks for the feedback. will go over it later and let you know my thoughts [14:55] smoser: what you think about the adjusted commit message? [15:03] do3meli: updated a bit [15:04] perfect, thx [15:04] btw added a cloud_init_facts ansible module https://github.com/ansible/ansible/pull/37932 [15:04] i think through the other stuff you mentioned now. [15:06] smoser: should i try add some tests with the outputs you mentioned from curtin' [15:11] do3meli: if you are willing to do that, that might be nice. [15:11] but actually.. [15:11] the files used for linux are /proc/1/mountinfo [15:11] which differs in format from /proc/1/mounts [15:11] er... [15:11] /proc/mounts [15:11] which is what we're using incloud-init [15:11] so the info isn't really there. [15:23] http://cloud-init.org is returning 502 Gateway timeout errors [15:23] Sorry that 502 Bad gateway errors [15:28] heatfanjohn: are you looking for docs? (agree on the 502) [16:22] heatfanjohn: thanks [19:17] rharper: in your chrony branch, what's the purpose of the 'name' key in each ntp client config dictionary? [19:17] I don't think I see it used anywhere? [19:48] blackboxsw: I thought it was used to extract the client name (which differs from the installed package) [19:49] but, that may have been refactored out, looks droppable right now [20:04] rharper: regarding ConditionFileExists evaluation time [20:04] http://paste.ubuntu.com/p/7vhSWgPKfc/ [20:04] that indicates that it is at unit start time. [20:04] yeah [20:49] rharper: last volley of comments and schema pastebin @ https://code.launchpad.net/~raharper/cloud-init/+git/cloud-init/+merge/339438 [20:49] blackboxsw: thanks, fixing up your first round [20:50] rharper: I also tweaked the unittest to use temp_utils.mkstemp instead of NamedTemporaryFile [20:51] I'm going to get a couple of deployments going on this to kick the tires [20:52] hey. i have to be headed out. i'll check back in for you all in probably 3.5 hours. [20:52] k [20:52] blackboxsw: interesting, ok [20:52] i just pushed https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/342005 [20:53] i think i'm going to just upload that too. [20:54] just to have that change into ubuntu now. [22:33] silly revert pused up https://code.launchpad.net/~chad.smith/cloud-init/+git/cloud-init/+merge/342154 [22:33] pushed even [22:34] rharper: I'll checkout your updates thx. I think I already made the change to temp_utils and unit test mock in my pastebin