| resmo | hi | 12:33 |
|---|---|---|
| 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:34 |
| resmo | ok, found the necessary infos here https://github.com/number5/cloud-init/blob/master/doc/status.txt | 12:56 |
| resmo | Thanks ;) | 12:56 |
| 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:22 |
| smoser | do3meli: rebase to master. | 13:23 |
| smoser | (that is fixed in trunk already) | 13:23 |
| smoser | make sense ? | 13:23 |
| do3meli | allright, let me do that | 13:24 |
| do3meli | smoser: rebase done and pushed | 13:34 |
| smoser | do3meli: running https://jenkins.ubuntu.com/server/view/cloud-init/job/cloud-init-ci/929/console | 13:38 |
| do3meli | thx | 13:49 |
| smoser | do3meli: curious... have you tested this with linux ? | 14:17 |
| smoser | or are you just adding support for zfs resize of freebsd | 14:18 |
| do3meli | i am mainly focusing on freebsd | 14:18 |
| do3meli | i have no zfs on linux to test | 14:19 |
| do3meli | with root on zpool | 14:19 |
| smoser | right. | 14:22 |
| 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:24 |
| smoser | do3meli: i responded. | 14:32 |
| 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:33 |
| do3meli | thanks for the feedback. will go over it later and let you know my thoughts | 14:34 |
| do3meli | smoser: what you think about the adjusted commit message? | 14:55 |
| smoser | do3meli: updated a bit | 15:03 |
| 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:04 |
| do3meli | smoser: should i try add some tests with the outputs you mentioned from curtin' | 15:06 |
| 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:11 |
| heatfanjohn | http://cloud-init.org is returning 502 Gateway timeout errors | 15:23 |
| heatfanjohn | Sorry that 502 Bad gateway errors | 15:23 |
| nacc | heatfanjohn: are you looking for docs? (agree on the 502) | 15:28 |
| smoser | heatfanjohn: thanks | 16:22 |
| 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:17 |
| rharper | blackboxsw: I thought it was used to extract the client name (which differs from the installed package) | 19:48 |
| rharper | but, that may have been refactored out, looks droppable right now | 19:49 |
| 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:04 |
| 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:49 |
| blackboxsw | rharper: I also tweaked the unittest to use temp_utils.mkstemp instead of NamedTemporaryFile | 20:50 |
| blackboxsw | I'm going to get a couple of deployments going on this to kick the tires | 20:51 |
| 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:52 |
| smoser | i think i'm going to just upload that too. | 20:53 |
| smoser | just to have that change into ubuntu now. | 20:54 |
| blackboxsw | silly revert pused up https://code.launchpad.net/~chad.smith/cloud-init/+git/cloud-init/+merge/342154 | 22:33 |
| blackboxsw | pushed even | 22:33 |
| 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 | 22:34 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!