[00:20] * blackboxsw needs to add an integration test for restarting cloudinit.service on upgrade path [01:28] blackboxsw: bummer, we all missed it: https://code.launchpad.net/~ajorgens/cloud-init/+git/cloud-init/+merge/329657 in the review +1 on integration test [01:31] yeah I know, I'll add an upgrade cli invocation integration test with the fix for this rharper [01:31] so we don't get hit by any other upgrade obj.pkl isms on new class attrs [01:44] was thinking about the following http://paste.ubuntu.com/26544332/ will test it out [01:46] blackboxsw: thanks for finding :-( [01:58] blackboxsw: https://bugs.launchpad.net/cloud-init/+bug/1735331 [01:58] Ubuntu bug 1735331 in cloud-init "ec2: zesty tempfile sandbox dhclient.pid file can't be created" [High,Fix released] [01:58] a comment there. i asked for logs. [03:53] https://bugs.launchpad.net/ubuntu/+source/cloud-init/+bug/1748354 [03:53] Ubuntu bug 1748354 in cloud-init (Ubuntu) "ec2: systemctl restart cloud-init after upgrade 17.1.46 -> 17.2.30" [High,Triaged] [07:29] Can someone please help me make sense of why cloud-init is failing? https://gist.github.com/291e61880cfdc6c9e1bc46b243d47760 I've verified that /bin/ip exists. Modifying net/*.py with abspath to binaries allows it to find ip/resize2fs etc commands [07:29] any ideas why the path is broken for cloud-init [08:58] hello, I am trying to set a hostname for an AWS Ami on reboot with the instance id in the name. Is it possible to substiture some variable in the hostname param? [09:08] e.g I wish to be able to do something like: "hostname: abc-dev-$INSTANCE_ID.domain.com" [09:34] HEllo, modules-config and modules-final won't start, but works fine manually [09:35] How can I debug this? Is there a configuration somewhere saying which steps must run? [13:38] s as-admin-p2.central.infra.maz [13:38] Sorry, wrong window :/ [14:05] kutasek: watch for this branch to land https://code.launchpad.net/~chad.smith/cloud-init/+git/cloud-init/+merge/335290 you can then provide something like this https://pastebin.ubuntu.com/26546614/ [14:06] blackboxsw: I can put that template directly in the user-data of my aws instance? [14:06] correct kutasek once I get that branch touched up and landed (it would make it into bionic images until we SRU it) [14:06] SRU (stable release update into xenial and artful) [14:07] yeah you'd just have to provied the ## template: jinja2 header above the #cloud-config you provide anyway [14:07] sorry ## template: jinja I mean [14:08] blackboxsw: Thanks for the info, one more question will it support aws meta-data variable out of the box? [14:09] kutasek: it pulls anything from /run/cloud-init/instance-data.json ... leeme get you an ec2 example [14:10] https://paste.ubuntu.com/26368359/ [14:11] great, and about that v1. prefix, will there be more ? is it linked to the aws meta-data api date they use for versioning? [14:13] v1 is our stadardized/generalized fields across all clouds. we will be adding more [14:14] the rest of the top level keys are cloud specific [14:14] awesome thanks a lot, can't wait for it, gonna have to use bash in meantime :D [14:31] kutasek: yeah I bet with a runcmd section containing - cat /run/cloud-init/instance-data.json | jq -r '.v1.instance-id' you might get pretty far [14:50] blackboxsw: I am not able to change the hostname with runcmd for some reasons (need to update it on each boot) === simpoir is now known as simpoir|lunch === simpoir|lunch is now known as simpoir [18:40] mgerdts: around ? [18:41] yep [18:41] http://paste.ubuntu.com/26547352/ [18:42] thatfixes your 2.6 issues [18:42] great, thanks! [19:08] smoser: https://code.launchpad.net/~chad.smith/cloud-init/+git/cloud-init/+merge/337475 [19:10] i;m afk a while so if it needs desc commit message work to land can't get to it for an hour pr so [19:40] i still dont know what we do about python2 [19:54] blackboxsw: looking to fix. fixed the flake, but the test is failing for me. [19:54] just saw my latest change busted the tst [19:54] pushed [19:54] back in 1- [19:54] back in 10 [19:54] oh. ok. [20:01] blackboxsw: still no :-( [20:01] you needc http://paste.ubuntu.com/26547563/ for flake [20:01] but i still get the test failure. [20:06] http://paste.ubuntu.com/26547572/ [20:10] ok looking for real [20:11] its not getting registered correctly [20:28] was a complex dict registration on httppretty [20:28] fixing now to handle children [20:29] I was originally testing with just the specific instanceId mocked and tried changing the test to mock the whole nested instance-identity document [20:29] without all the nested children routes [20:29] fixing that now [20:30] hm. [20:30] the unit tests right now fall over when trying to materialize dynamic/instance-identity/document url [20:30] which isn't registered via httpretty properly [20:35] i'm missing nit. [20:42] ohh the problem is in our registration_helper we can't actually return a dictionary if provided, we actually register separate URIs for each key-value [20:42] ok [20:56] blackboxsw: http://paste.ubuntu.com/=nNgvNtJJCD/ [20:56] i'm guessing you have that now though [20:59] blackboxsw: i gotta walk away for 30-45 minutes. i'll return though and will ack your MP assuming c-i does. [20:59] my only comment is that the @property is tempting (for self.identity) [21:44] blackboxsw: you there ? [21:44] did you open a bug for that ? [21:58] smoser: yep, pushed changes and finding the bug [21:58] I'm sparsely here, need to mark a sick day today [22:01] blackboxsw: i merged [22:02] ok thanks was just writing the commit message over with real content and the https://bugs.launchpad.net/ubuntu/+source/cloud-init/+bug/1748354 [22:02] Ubuntu bug 1748354 in cloud-init (Ubuntu) "ec2: systemctl restart cloud-init after upgrade 17.1.46 -> 17.2.30" [High,In progress] [22:02] ok thanks [22:02] a lot [22:02] and i'm going to push overwrite [22:02] i can upload to bionic [22:03] thanks, pretty much absent today. [22:04] on its way to bionic.