| powersj | blackboxsw: your ci run hung it appears. I just relaunched it | 15:36 |
|---|---|---|
| blackboxsw | thanks powersj . yeah I just double checked to make sure i didn't leave a pdb.set_trace() anywhere | 15:43 |
| blackboxsw | running tox locally seems to run fine. | 15:43 |
| powersj | well remember CI is much more than tox ;) | 15:44 |
| dgarstang | So, it's been a couple of months since I was here. At the time, cloud-init didn't seem to support fs_setup on CentOS. Seems that's still the case | 17:02 |
| dgarstang | Sure wish you guys would fix that | 17:10 |
| smoser | dgarstang, it is probably much closer. | 17:11 |
| smoser | and we have trunk rpms that you can test | 17:12 |
| smoser | but i dont think we've explicitly fixed it | 17:12 |
| blackboxsw | dgarstang: is there a bug open about this? I see we've got a released bug https://bugs.launchpad.net/cloud-init/+bug/1687712 that helped a bit | 17:12 |
| ubot5 | Ubuntu bug 1687712 in cloud-init "cc_disk_setup: fs_setup with cmd doesn't work" [Medium,Confirmed] | 17:12 |
| dgarstang | smoser: last time I tried that it broke my instance, it wouldn't bot | 17:12 |
| smoser | https://copr.fedorainfracloud.org/coprs/g/cloud-init/cloud-init-dev/ | 17:12 |
| dgarstang | boot even | 17:12 |
| dgarstang | I also need this to work with CentOS 6, not just 7 | 17:13 |
| smoser | rpms there for both | 17:13 |
| smoser | but probalby not fixed. :-( | 17:13 |
| dgarstang | :( | 17:14 |
| gbenhaim | Hi All, not sure if this is the correct place to ask, but I'll try. I wanted to know if "merging user-data sections" is supported when using "NoCloud" datasource ? | 17:30 |
| blackboxsw | smoser: approved https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/327532 | 17:53 |
| smoser | blackboxsw, had some questions https://code.launchpad.net/~chad.smith/cloud-init/+git/cloud-init/+merge/327827 | 18:13 |
| blackboxsw | smoser: per your review there, you mentioned we want EphemeralIPv4Network to validate and setup networking to match the requested ip addr. Do we have existing funcitons in net somewhere that look at the existing interfaces IP? I found netdev_info in cloudinit/netinfo.py. Any other utility function I should be using? | 19:41 |
| blackboxsw | 19:41 | |
| blackboxsw | I didn't want to rewrite something if we already have a utility function | 19:42 |
| smoser | blackboxsw, i dont have anything :-( | 19:42 |
| smoser | i'm open to comments like "can't we do that later". | 19:43 |
| smoser | blackboxsw, a low cost/tech path might be this | 19:45 |
| smoser | root@z1:~# ip -family inet addr add 192.168.1.2/27 dev eth0 | 19:45 |
| smoser | root@z1:~# ip -family inet addr add 192.168.1.2/27 dev eth0 | 19:45 |
| smoser | RTNETLINK answers: File exists | 19:45 |
| smoser | root@z1:~# echo $? | 19:45 |
| smoser | 2 | 19:45 |
| smoser | so if its already present then it will exit 2 and 'File exists'. (if we pay attention to stderr/stdout then we need to LANG=C) | 19:46 |
| blackboxsw | yeah I was leaning toward trying to use ip cmds instead of ifconfig (as netdev_info uses) just to be more forward looking | 19:46 |
| smoser | for sure use ip | 19:46 |
| blackboxsw | smoser: yeah I think we can do something simple for this branch. No need to put off content as tech debt if it's a sensible approach and easy implementation. | 19:49 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!