[PATCH 1/7] lldpd: fixed interface(s) parsing

Jo-Philipp Wich jo at mein.io
Tue Apr 2 16:12:01 PDT 2024


Hi,

> For interface type parameters, the man page documents patterns:
> ```
> *,!eth*,!!eth1
> 
> uses all interfaces, except interfaces starting with "eth",
> but including "eth1".
> ```

at some point, uci configuration was meant to provide a somewhat sane config 
abstraction over various damon specific native configurations, now I see the 
recurring trend to expose every last native config idiosyncrasy as-is in uci.

Is there really a need to support these weird micro formats in uci? The uci 
config for lldpd should deal with logical interface names and translate them 
into layer 2 ones as needed. People requiring complex, hand-tuned settings 
probably want to bypass uci entirly and simply start lldpd with a static 
native config file.

~ Jo



More information about the openwrt-devel mailing list