net/dhcp: add udhcpc support#4190
Conversation
69c915c to
f517b90
Compare
Keep support for udhcpc (waiting for upstream PR: canonical/cloud-init#4190). Vultr patch has been merged upstream (canonical/cloud-init#2151).
The currently used dhcp client, dhclient, is coming from the unmaintained package, isc-dhcp-client (refer https://www.isc.org/dhcp/) which ended support in 2022. This change introduce support for the dhcp client, udhcpc, from the busybox project. Busybox advantages are that it is available across many distributions and comes with lightweight executables.
|
Hello! Thank you for this proposed change to cloud-init. This pull request is now marked as stale as it has not seen any activity in 14 days. If no activity occurs within the next 7 days, this pull request will automatically close. If you are waiting for code review and you are seeing this message, apologies! Please reply, tagging TheRealFalcon, and he will ensure that someone takes a look soon. (If the pull request is closed and you would like to continue working on it, please do tag TheRealFalcon to reopen it.) |
|
@jfroche Thanks for putting this together. Since there hasn't been any further activity I assume this is ready for review now?
|
|
I'll test this out, thanks. At first glance it looks like the generic implementation should work, but I don't see the datasource-specific methods implemented which means that this implementation might not work on all clouds. @jfroche Are there plans to implement those in a follow up PR? |
Yes I will make another PR for that. |
The currently used dhcp client, dhclient, is coming from the unmaintained package, isc-dhcp-client (refer https://www.isc.org/dhcp/) which ended support in 2022. This change introduce support for the dhcp client, udhcpc, from the busybox project. Busybox advantages are that it is available across many distributions and comes with lightweight executables.
Proposed Commit Message
Additional Context
Already discussed in #2125
Test Steps
Checklist: