| 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:18 |
|---|---|---|
| fxpester | so, mount failed | 08:19 |
| fxpester | cloud-init 0.7.9 | 08:20 |
| fxpester | I`m sure I`ve seen this bug on launchpad, but can`t find it now | 08:22 |
| fxpester | hm, /tmp is present in image | 08:45 |
| fxpester | error is 'Stderr: mount: mount point /tmp/tmpmdrxpk3u does not exist' if I run `cloud-init init --local` everything is working correct | 09:00 |
| 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] | 09:02 |
| 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:13 |
| smoser | robjo, i get email when you comment. | 15:18 |
| 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:19 |
| 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:20 |
| 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 | 15:23 |
| === 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 | ||
| smoser | blackboxsw, ^ | 16:07 |
| blackboxsw | thanks smoser +1 | 16:07 |
| smoser | blackboxsw, i just took your ec2 branch and ran it on an instance and got ipv6. horay! | 16:23 |
| blackboxsw | superb smoser | 16:23 |
| 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:25 |
| blackboxsw | already landed larsks' initial branch making the import optional | 16:26 |
| blackboxsw | thx larsks | 16:26 |
| blackboxsw | smoser: are you waiting on a 2nd round of reviews for https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/329397? | 16:56 |
| 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:09 |
| 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:10 |
| 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:12 |
| 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:16 |
| smoser | blackboxsw, did you have other comments on that MP ? | 17:47 |
| smoser | i'll give a quick test and then comment and pull if you approve too | 17:48 |
| blackboxsw | smoser: just a nit added about allowing us to inject the testable metadata address instead of initializing to a global. | 17:51 |
| blackboxsw | and +1 there | 17:52 |
| 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:16 |
| smoser | blackboxsw, https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/329397 | 18:20 |
| blackboxsw | +1 there when unit tests finish | 18:23 |
| blackboxsw | CI pass rather | 18:23 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!