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