[08:18] <fxpester> hi all, I have problem on xenial with configdrive, looks like  /tmp isn`t created yet but cloud-init tryes to use already
[08:19] <fxpester> so, mount failed
[08:20] <fxpester> cloud-init 0.7.9
[08:22] <fxpester> I`m sure I`ve seen this bug on launchpad, but can`t find it now
[08:45] <fxpester> hm, /tmp is present in image
[09:00] <fxpester> error is 'Stderr: mount: mount point /tmp/tmpmdrxpk3u does not exist' if I run `cloud-init init --local` everything is working correct
[09:02] <fxpester> finally found bug - https://bugs.launchpad.net/ubuntu/+source/cloud-init/+bug/1683974
[09:02] <ubot5> Ubuntu bug 1683974 in cloud-init (Ubuntu) "Cloud-init fails to mount datasource from iso disk sometimes" [Undecided,New]
[15:13] <robjo> smoser: Do I need to do another "merge proposal" in LP after I made the requested changes or do you get notified automatically after I pushed the changes?
[15:18] <smoser> robjo, i get email when you comment.
[15:19] <smoser> but i dont know if i get updates if you just git push or git push --force
[15:19] <nacc> smoser: i don't think you do, but i'm not 100% either
[15:19] <smoser> fwiw, i consider it always ok to say "smoser, why haven't you responded"
[15:19] <smoser> most of the time the answer is "busy"
[15:19] <smoser> but...
[15:19] <nacc> smoser: we use work-in-progress and needs-review state changes
[15:19] <smoser> it is ok to bother me.
[15:19] <smoser> nacc, yeah, i often times do that.
[15:19] <smoser> thanks
[15:20] <robjo> OK, so the suseIntegrate branch has been updated with the requested changes and is ready for another review
[15:20] <smoser> thanks. i'll take a read.
[15:23] <nacc> robjo: i do think it's a bug in LP that there isn't a notification for that, fwiw :) I think github notifies watchers
=== smoser changed the topic of #cloud-init to: reviews: https://code.launchpad.net/~cloud-init-dev/cloud-init/+git/cloud-init/+ref/master/+activereviews | next meeting Tue, 05 Sep 2017 16:30:00 +0000
[16:07] <smoser> blackboxsw, ^
[16:07] <blackboxsw> thanks smoser  +1
[16:23] <smoser> blackboxsw, i just took your ec2 branch and ran it on an instance and got ipv6. horay!
[16:23] <blackboxsw> superb smoser
[16:25] <smoser> 2017-08-29 16:19:04,367 - stages.py[INFO]: Applying network configuration from ds bringup=False: {'version': 1, 'config': [{'type': 'physical', 'name': 'eth0', 'subnets': [{'type': 'dhcp4'}, {'type': 'dhcp6'}], 'mac_address': '06:8d:72:65:39:7c'}]}
[16:25] <blackboxsw> I'm reworking the url_helper Oauthlib ImportError branch just a bit to make it testable.
[16:26] <blackboxsw> already landed larsks' initial branch making the import optional
[16:26] <blackboxsw> thx larsks
[16:56] <blackboxsw> smoser: are you waiting on a 2nd round of reviews for https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/329397?
[17:09] <smoser> blackboxsw i wasnt necessarily. just didnt see it as high priority , and then you talked about common datasources and things.
[17:09] <smoser> i dont think it really hurts anything to have it
[17:10] <smoser> also i didn't ever actually test it on an instance, so i wasn't going to pull it until i did so :)
[17:10] <smoser> i tested that 'read_md' worked, but not that the whole boot worked
[17:12] <blackboxsw> smoser: gotcha, that makes sense. I wasn't sure if that was something you were thinking of dropping ultimately because of the common datasource discussion as well. Yeah I do think that common datasource would solve some of that problem. It'll admitedly be a bit until we have common datasource methods landed (probably 2 branches to be written).
[17:16] <blackboxsw> I think your adding a main is good for the moment until we get around to a cohesive/standard base DataSource api definition for all datasources. This work also ties in a bit with the JSON storage/non-pickle datasource on disk.
[17:47] <smoser> blackboxsw, did you have other comments on that MP ?
[17:48] <smoser> i'll give a quick test and then comment and pull if you approve too
[17:51] <blackboxsw> smoser: just a nit added about allowing us to inject the testable metadata address instead of initializing to a global.
[17:52] <blackboxsw> and +1 there
[18:16] <smoser> blackboxsw, i'm going to add a 'log_time' around it too
[18:16] <smoser> (which ... had it already had the log time, we'd have probalby not ended up doign this)
[18:20] <smoser> blackboxsw, https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/329397
[18:23] <blackboxsw> +1 there when unit tests finish
[18:23] <blackboxsw> CI pass rather