/srv/irclogs.ubuntu.com/2020/01/13/#cloud-init.txt

meenaturova: what do your logs say?08:33
turovameena it looks like it loads a network-config file from the mounted cloudinit drive that has correct values, but those never get to their destination12:50
turovait reads from /etc/cloud/cloud.cfg.d/50-curtin-networking.cfg, finds that it has no interfaces to rename (maybe because I already did that) and then just writes the blank config over12:52
meenaiiiiiinteresting.13:28
turovait supposedly works with the premade proxmox OS templates, so I guess I'll just pull one of those down and see what the differences are when I have some time13:41
meena*nod *nod *nod14:06
* meena has never worked with proxmox14:06
otuboOdd_Bloke: Hi! I've already signed the CLA, can you take a quick look at https://github.com/canonical/cloud-init/pull/70 whenever you have time? Should I update the branch?15:29
robjono assigned reviewer yet for https://github.com/canonical/cloud-init/pull/162 can we please get some eyes on this? thanks16:23
blackboxswrobjo: I'll give first pass today16:24
blackboxswthanks16:24
meenarobjo: i have read your email to the list… and i like it.16:26
meenarobjo: do you think you could outline what a refactoring would look like in my document?16:27
robjomeena: I think the step before such an outline is to get conceptual agreement, i.e. rharper blackboxsw, smoser and others would have to agree that this rather large change is the way forward w.r.t. network config writing16:29
meenarobjo: i meant… in words.16:29
robjoWell it would basically the same as what I stated in e-mail. I thought about whether I wanted to comment in the doc or via e-mail and then decided to go with the e-mail approach16:30
meenarobjo: aye.16:33
meenarobjo: so, my main issue re writing network config is that right now, a lot of code for reading and *directly* manipulating (Ephemeral*) network only works on linux.16:34
meenaso Goneri created a cool patch for FreeBSD, but it only works for DataSources that don't rely on the network.16:34
robjoI get that. The question for me becomes do we want to add another way of handling it on the side YOUR_FAVORITE_BSD rendereror are we better of to stick this stuff with the distro implementation16:37
Goneriyes, we should have a method in distros/freebsd.py to set-up a temporary network.16:38
robjobased on the way things look ATM I think there is a good argument to be made that the network config rendering belongs with the distribution implementation in some way, i.e. the distro implementation knows how it should render the network config rather than the network config is generic and considers distribution needs as a side effect16:38
Gonerion BSD, Yes. On Linux, it's different.16:40
Gonerithis is the reason why the NetBSD and OpenBSD PR come with their own net renderer too.16:40
robjoThe temporary network setup, while it works, is not linux generic either. It depends on the dhcp-client package which is no longer supported in SUSE16:40
meenai believe i did mention that somewhere…16:41
meenabut it might have just  been IRC, so: lost in noise16:41
GoneriNaive question, I'm under the impression the Ephemeral network thing is just here to handle some corner-cases. Is it really the case?16:42
robjoA generic way for the temporary network setup is to have Python code in cloud-init that sends a basic dhcp4 and or dhcp6 request16:42
robjosmoser: was exploring that at som epoint16:42
robjothis would make the temporary network setup generic and should really be the first step16:43
meenai've looked at dhcp implementations in python at one point, and i didn't like what i saw.16:50
meenait would be cool if the OS had standard network library, which exposed dhcp that we could wrap in ctypes and call16:51
Goneriwell, we should port NetworkManager on the BSD :-)16:54
blackboxswotubo: thanks! I think you probably need to use util.del_file() https://github.com/canonical/cloud-init/pull/70/files#r365882842  and I just added a couple of minor nits in review as well.16:55
blackboxswotubo: other than the nits, it looks good to go16:55
blackboxswand we can merge that today when you get a chance to resolve or respond/reject comments.16:56
meenaGoneri: when i suggested we port netplan, you were against it!16:56
GoneriI was joking in both cases, the real solution being systemd-networkd :-)16:59
smosermeena: python-scapy was the only thing that I thought could relaly get us there.17:26
meenasmoser: scapy looks powerful… dangerously so.18:21
smosermeena: well.. doing a dhcp client really *should* be easy.18:26
smoserbut, almost hilariously, all example dhcp clients in python assume there is already an ip address on the interface.18:26
smoserfrom memory... they do not use raw sockets and would need to to actually do a dhcp request on a nic that did not have an ip yet.18:27
meenasoooooo, that's why it needs libpcap?18:29
meenaokay, so, how do we get buy-in? i think maybe i'll be able to attend tomorrow's meeting and try to "sell" robjo's idea?20:11
robjoWell the ISC reference implementation for DHCP is Mozilla Public License 2.0 thus we cannot subsume it and wrap a Python interface around it. I am afraid if we are going to be independent of any external tool for the temporary network setup someone has to sit down and write the code in Python and send a PR20:19
robjoThat would resolve the issue on *BSD for the temporary network setup, and would resolve the issue on SUSE w.r.t. depending on an unsupported package20:20
robjoAfter that the second step is to decide on the renderer approach20:21
robjoFrom my perspective there is buy in for step 1 already20:21
rharperreplacing the EphemeralDHCP with a scapy (or something else) aka- cloud-init-dhclient is definitely something we're interested in; it allows us to drop the dhclient package from newer images, and sounds like it can be more portable to non-linux distros as well20:25
rharperfor renderers and net module refactoring; I'm certainly on board with the goal of enhacing cross platform and cross distro support for the cloudinit.net module and additional renderers (and robjo distro variant support within a renderer class)20:27
meenai feel like a giant but should be coming out of rharper any second now20:47
meenagon… damn.21:01

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!