*** tonyb_ is now known as tonyb | 08:57 | |
*** TheJulia_ is now known as TheJulia | 08:57 | |
*** mnasiadka_ is now known as mnasiadka | 08:57 | |
*** clarkb is now known as Guest5393 | 08:59 | |
*** ravlew is now known as Guest5411 | 09:00 | |
*** clarkb is now known as Guest5418 | 10:45 | |
*** Guest5418 is now known as clarkb | 15:03 | |
opendevreview | Julia Kreger proposed openstack/diskimage-builder master: Remove cloud-init when using simple-init https://review.opendev.org/c/openstack/diskimage-builder/+/899885 | 16:41 |
---|---|---|
opendevreview | Julia Kreger proposed openstack/diskimage-builder master: ConfigDrive: Disable cloud-init auto dhcp by default https://review.opendev.org/c/openstack/diskimage-builder/+/899886 | 16:41 |
TheJulia | I strongly suspect the two changes I just posted fix up my dib/glean sources of pain | 16:42 |
TheJulia | fwiw, I'm going to pull them into a CI job and just make sure all is well, but I've effectively done the same thing in a VM and forced config to be re-applyed | 16:43 |
clarkb | TheJulia: the first one makes sense to me but I think 899886 shouldn't be a default behavior | 16:45 |
clarkb | by default images should boot and do everything they can to actually boot so having a fallback to dhcp makes sense to me | 16:45 |
clarkb | a toggle to make that not the case for users that don't want the extra time spent waiting for dhcp would be fine though | 16:46 |
*** promethe- is now known as prometheanfire | 16:49 | |
*** prometheanfire is now known as Guest5452 | 16:49 | |
*** Guest5452 is now known as prometheanfire | 16:51 | |
TheJulia | I have mixed feelings on the latter, but there are two different behaviors noted in the network docs | 17:03 |
TheJulia | there is a full fallback which is dhcp by default, then there is the "do I go hunt for metadata sources" which is what it toggles | 17:03 |
TheJulia | I could kind of see the trap being someone thinking "I'll try everything" and the reality with cloud-init is you really can't do that | 17:04 |
TheJulia | clarkb: the key point is we're not toggling the full fallback, only the upfront attempt to dhcp, AIUI | 17:07 |
clarkb | I don't know enough about cloud init to know how to express it in its config but I think the ideal situation with config drive is to startup, read the metadata and attept to use the network configured there, if that fails or is non existent fallback to dhcp | 17:07 |
clarkb | oh interesting. So it should still attempt dhcp at the very end if all other attempts have failed? | 17:07 |
TheJulia | yeah, I'm not sure if you constrain down metadata if it would still get metadata, but basically yeah | 17:11 |
TheJulia | ... I almost made a separate element (actually, started it, and there was too much duplication, so I merged the idea in) | 17:12 |
TheJulia | the first patch works like a perfect charm | 19:11 |
TheJulia | Enough, it has helped reproduce another bug I've been chasing, so that first before the second patch | 19:11 |
clarkb | I'll review that first one shortly. My main concern is that the pacakge is called cloud-init on all the platforms (I'll check) | 19:58 |
TheJulia | some spot checking seems that it is oddly consistent | 20:00 |
TheJulia | but I only checked a hand-full | 20:00 |
TheJulia | well, half handfull | 20:00 |
clarkb | I checked ubuntu, debian, opensuse, centos, and fedora. Its all cloud-init there. The only weird one is ubuntu kubic doesn't appear to have the package at all but the rest of the ubuntu releases do | 20:05 |
clarkb | so I think that change is fine | 20:05 |
TheJulia | clarkb: that seems... odd w/r/t kubic | 21:36 |
clarkb | TheJulia: might be a bug in the ubuntu package search sytem | 21:55 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!