[systemd-devel] [PATCH] networkd: update TODO

[Tom Gundersen]

On Mon, Apr 28, 2014 at 12:00 AM, Umut Tezduyar Lindskog
<umut at tezduyar.com> wrote:
> On Sun, Apr 27, 2014 at 11:53 PM, Tom Gundersen <teg at jklm.no> wrote:
>> On Sun, Apr 27, 2014 at 11:45 PM, Umut Tezduyar Lindskog
>> <umut.tezduyar at axis.com> wrote:
>>> ---
>>>  TODO |    5 ++---
>>>  1 file changed, 2 insertions(+), 3 deletions(-)
>>>
>>> diff --git a/TODO b/TODO
>>> index 1963af3..6ec9515 100644
>>> --- a/TODO
>>> +++ b/TODO
>>> @@ -681,14 +681,13 @@ Features:
>>>     - add support for multi-part messages (simply call the same callback repeatedly?)
>>>
>>>  * networkd:
>>> -   - change from udev to rtnl being the main events we listen to devices on, udev will then simply
>>> -     be used for synchronization
>>>     - add more keys to [Route] and [Address] sections
>>>     - add support for more DHCPv4 options (and, longer term, other kinds of dynamic config)
>>>     - add proper initrd support (in particular generate .network/.link files based on /proc/cmdline)
>>>     - add reduced [Link] support to .network files
>>> -   - add IPv4LL tests (inspire by DHCP)
>>>     - add Scope= parsing option for [Network]
>>> +   - properly handle routerless dhcp leases
>>
>> Hm, I forgot, what do you have in mind to do here?
>
> If DHCP doesn't send the router option in the lease, we still set the
> router as 0.0.0.0. We need to fix this with:
> a) Understand the case when lease doesn't have router option. Return -ENOENT etc
> b) networkd thinks it is not OK to set up DHCP without lease. Just
> don't set the routes if there is no router information in the lease.


Yeah, sounds good.

-t