| * blackboxsw needs to add an integration test for restarting cloudinit.service on upgrade path | 00:20 | |
| rharper | 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:28 |
|---|---|---|
| blackboxsw | yeah I know, I'll add an upgrade cli invocation integration test with the fix for this rharper | 01:31 |
| blackboxsw | so we don't get hit by any other upgrade obj.pkl isms on new class attrs | 01:31 |
| blackboxsw | was thinking about the following http://paste.ubuntu.com/26544332/ will test it out | 01:44 |
| smoser | blackboxsw: thanks for finding :-( | 01:46 |
| smoser | blackboxsw: https://bugs.launchpad.net/cloud-init/+bug/1735331 | 01:58 |
| ubot5 | Ubuntu bug 1735331 in cloud-init "ec2: zesty tempfile sandbox dhclient.pid file can't be created" [High,Fix released] | 01:58 |
| smoser | a comment there. i asked for logs. | 01:58 |
| blackboxsw | https://bugs.launchpad.net/ubuntu/+source/cloud-init/+bug/1748354 | 03:53 |
| ubot5 | Ubuntu bug 1748354 in cloud-init (Ubuntu) "ec2: systemctl restart cloud-init after upgrade 17.1.46 -> 17.2.30" [High,Triaged] | 03:53 |
| amitprakash | 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 |
| amitprakash | any ideas why the path is broken for cloud-init | 07:29 |
| kutasek | 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? | 08:58 |
| kutasek | e.g I wish to be able to do something like: "hostname: abc-dev-$INSTANCE_ID.domain.com" | 09:08 |
| jchevalier | HEllo, modules-config and modules-final won't start, but works fine manually | 09:34 |
| jchevalier | How can I debug this? Is there a configuration somewhere saying which steps must run? | 09:35 |
| PsyRabbit | s as-admin-p2.central.infra.maz | 13:38 |
| PsyRabbit | Sorry, wrong window :/ | 13:38 |
| blackboxsw | 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:05 |
| kutasek | blackboxsw: I can put that template directly in the user-data of my aws instance? | 14:06 |
| blackboxsw | correct kutasek once I get that branch touched up and landed (it would make it into bionic images until we SRU it) | 14:06 |
| blackboxsw | SRU (stable release update into xenial and artful) | 14:06 |
| blackboxsw | yeah you'd just have to provied the ## template: jinja2 header above the #cloud-config you provide anyway | 14:07 |
| blackboxsw | sorry ## template: jinja I mean | 14:07 |
| kutasek | blackboxsw: Thanks for the info, one more question will it support aws meta-data variable out of the box? | 14:08 |
| blackboxsw | kutasek: it pulls anything from /run/cloud-init/instance-data.json ... leeme get you an ec2 example | 14:09 |
| blackboxsw | https://paste.ubuntu.com/26368359/ | 14:10 |
| kutasek | 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:11 |
| blackboxsw | v1 is our stadardized/generalized fields across all clouds. we will be adding more | 14:13 |
| blackboxsw | the rest of the top level keys are cloud specific | 14:14 |
| kutasek | awesome thanks a lot, can't wait for it, gonna have to use bash in meantime :D | 14:14 |
| blackboxsw | 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:31 |
| kutasek | blackboxsw: I am not able to change the hostname with runcmd for some reasons (need to update it on each boot) | 14:50 |
| === simpoir is now known as simpoir|lunch | ||
| === simpoir|lunch is now known as simpoir | ||
| smoser | mgerdts: around ? | 18:40 |
| mgerdts | yep | 18:41 |
| smoser | http://paste.ubuntu.com/26547352/ | 18:41 |
| smoser | thatfixes your 2.6 issues | 18:42 |
| mgerdts | great, thanks! | 18:42 |
| blackboxsw | smoser: https://code.launchpad.net/~chad.smith/cloud-init/+git/cloud-init/+merge/337475 | 19:08 |
| blackboxsw | 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:10 |
| smoser | i still dont know what we do about python2 | 19:40 |
| smoser | blackboxsw: looking to fix. fixed the flake, but the test is failing for me. | 19:54 |
| blackboxsw | just saw my latest change busted the tst | 19:54 |
| blackboxsw | pushed | 19:54 |
| blackboxsw | back in 1- | 19:54 |
| blackboxsw | back in 10 | 19:54 |
| smoser | oh. ok. | 19:54 |
| smoser | blackboxsw: still no :-( | 20:01 |
| smoser | you needc http://paste.ubuntu.com/26547563/ for flake | 20:01 |
| smoser | but i still get the test failure. | 20:01 |
| smoser | http://paste.ubuntu.com/26547572/ | 20:06 |
| blackboxsw | ok looking for real | 20:10 |
| smoser | its not getting registered correctly | 20:11 |
| blackboxsw | was a complex dict registration on httppretty | 20:28 |
| blackboxsw | fixing now to handle children | 20:28 |
| blackboxsw | 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 |
| blackboxsw | without all the nested children routes | 20:29 |
| blackboxsw | fixing that now | 20:29 |
| smoser | hm. | 20:30 |
| blackboxsw | the unit tests right now fall over when trying to materialize dynamic/instance-identity/document url | 20:30 |
| blackboxsw | which isn't registered via httpretty properly | 20:30 |
| smoser | i'm missing nit. | 20:35 |
| blackboxsw | 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 |
| blackboxsw | ok | 20:42 |
| smoser | blackboxsw: http://paste.ubuntu.com/=nNgvNtJJCD/ | 20:56 |
| smoser | i'm guessing you have that now though | 20:56 |
| smoser | blackboxsw: i gotta walk away for 30-45 minutes. i'll return though and will ack your MP assuming c-i does. | 20:59 |
| smoser | my only comment is that the @property is tempting (for self.identity) | 20:59 |
| smoser | blackboxsw: you there ? | 21:44 |
| smoser | did you open a bug for that ? | 21:44 |
| blackboxsw | smoser: yep, pushed changes and finding the bug | 21:58 |
| blackboxsw | I'm sparsely here, need to mark a sick day today | 21:58 |
| smoser | blackboxsw: i merged | 22:01 |
| blackboxsw | 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 |
| ubot5 | 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 |
| blackboxsw | ok thanks | 22:02 |
| blackboxsw | a lot | 22:02 |
| smoser | and i'm going to push overwrite | 22:02 |
| smoser | i can upload to bionic | 22:02 |
| blackboxsw | thanks, pretty much absent today. | 22:03 |
| smoser | on its way to bionic. | 22:04 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!