powersj | blackboxsw: if you are still around... in your review of the ec2 merge you asked me to add pyyaml to the tox ci env? | 00:33 |
---|---|---|
smoser | powersj: i think i did auto-op you. | 02:40 |
smoser | very confusing | 19:02 |
smoser | http://boto3.readthedocs.io/en/latest/reference/services/ec2.html | 19:02 |
smoser | look for 'UserData' there | 19:02 |
smoser | should you give that as base64 or not? | 19:02 |
powersj | smoser: I use https://boto3.readthedocs.io/en/latest/reference/services/ec2.html#EC2.ServiceResource.create_instances that function | 19:05 |
powersj | which says UserData='string', | 19:05 |
powersj | If you are using a command line tool, base64-encoding is performed for you, and you can load the text from a file. Otherwise, you must provide base64-encoded text. | 19:06 |
smoser | and then in bold | 19:06 |
powersj | and then right below.. | 19:06 |
powersj | yeah :) | 19:06 |
smoser | we should figure that out and pass None (or not pass the kwarg) | 19:12 |
smoser | rather than b'' | 19:12 |
blackboxsw | hrm not understanding why the diff isn't being created here https://code.launchpad.net/~chad.smith/cloud-init/+git/cloud-init/+merge/335468 | 19:20 |
blackboxsw | might resubmit a third time and avoid prerequisite branch on hogarth's branch as they don't really have to be dependent branches | 19:21 |
smoser | no ifrs | 19:21 |
smoser | no idea. maybe you have an oops in your inbox | 19:21 |
powersj | smoser: I'll look at the b'' shortly | 19:23 |
blackboxsw | ok resubmitted the simple fix (not dependent on any branch) | 19:54 |
blackboxsw | https://code.launchpad.net/~chad.smith/cloud-init/+git/cloud-init/+merge/335470 | 19:54 |
powersj | smoser: https://paste.ubuntu.com/26223182/ | 20:08 |
powersj | wiht **args of course | 20:11 |
smoser | powersj: yeah | 20:12 |
smoser | i just hit 'submit' on a review there. | 20:12 |
powersj | heh I took the t2.micro comment from your last review | 20:12 |
smoser | blackboxsw: responded | 20:15 |
blackboxsw | smoser: sure on not even bouncing if ifupdown not present, though FreeBSD uses ifconfig down|up. Shall we also check for that ? | 20:19 |
blackboxsw | or alternately, if util.is_FreeBSD()? | 20:19 |
blackboxsw | it makes things simple to avoid combing through datasource.cfg['hostname_bounce']['command'] for ifdown references | 20:21 |
blackboxsw | the only problem I see with not calling bounce_network_with_azure_hostname is that the bounce method actually might be the only thing calling set_hostname in the first place. Will check | 20:23 |
smoser | blackboxsw: hm... | 20:34 |
blackboxsw | yep, the bounce_network_with_azure_hostname won't actually allow cloud-init to observer #cloud-config\nhostname: mynewname declarations | 20:39 |
blackboxsw | on artful/bionic the vm is then left with the hostname it was originally created with in azure's UI/CLI | 20:40 |
blackboxsw | because no set-hostname is called | 20:40 |
smoser | ? | 20:41 |
smoser | confused. | 20:41 |
smoser | and that code is confusing | 20:42 |
blackboxsw | could show you in hangout. bionic vs xenial behavior, that bounce method is wrapped up in all the get/set_hostname from metadata/user-data calls in the temp_hostname context manager, if we skip it altogether we don't actualy 'see/observe' the user-data hostname provided | 20:42 |
blackboxsw | probably best if we don't hangout if you value the rest of your work day ;) | 20:44 |
smoser | blackboxsw: ok. hanoug | 20:52 |
blackboxsw | yeah, might be worth refactoring that code a bit since I'm touching it. Those two functions are tightly coupled and they probably shouldn't be. | 20:54 |
powersj | hmmm console log output isn't always there | 21:01 |
blackboxsw | smoser: Xmas list for me: | 21:23 |
blackboxsw | - https://code.launchpad.net/~sw37th/cloud-init/+git/cloud-init/+merge/335217 | 21:23 |
powersj | python3 only? | 21:23 |
blackboxsw | - https://code.launchpad.net/~rjschwei/cloud-init/+git/cloud-init/+merge/333772 (land as-is and followup with separate refactor branch of yours) | 21:24 |
blackboxsw | - and once I've updated this: https://code.launchpad.net/~chad.smith/cloud-init/+git/cloud-init/+merge/335470 | 21:25 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!