luci icon indicating copy to clipboard operation
luci copied to clipboard

luci-mod-network static leases not bound to specific instance

Open professor-jonny opened this issue 2 years ago • 4 comments

When configuting a static lease from luci it does not bind it to a specific dnsmasq instance.

Manual editing of the config files is required to acomplish this.

professor-jonny avatar May 08 '23 21:05 professor-jonny

As I believe multiple dnsmasq instances are not supported well in luci and there is various other issues.

professor-jonny avatar Aug 09 '23 12:08 professor-jonny

Assigning static leases to instances was added in https://github.com/openwrt/luci/commit/f73abffa2aa64a91e6c16ddae7966a5c35e73fe1 in snapshots, but the implementation seems broken because the instance value should be the name of the config dnsmasq section and not an index number.

Pinging @systemcrash or @jow- to validate my findings. Discussed in the forum recently.

dave14305 avatar Sep 13 '23 19:09 dave14305

The GUI additions in f73abffa2aa64a91e6c16ddae7966a5c35e73fe1 ask luci to poll for the available sections, so those are available to choose from if they exist. I think other sections only appear in manually edited configs anyhow, since the GUI isn't yet set up for multiple dnsmasq instances.

systemcrash avatar Sep 13 '23 19:09 systemcrash

@dave14305 they seem bound to the correct instance by config in my setup as, I can create static leases for the same mac addresses in different lan segments without issue it is just that luci reports them as the name of the first instance in the DHCP config file.

professor-jonny avatar Sep 13 '23 21:09 professor-jonny