packages/net/nut/files, branch master

nut: split interface_triggers function

2026-07-15T07:34:39Z

Using add_interface_triggers and check_interface_up is a better separation of concerns and the code has little in common (no duplication of code concerns). While we are at it make both nut-monitor and nut-server failure of add_interface_triggers non-fatal with a logged error message. Signed-off-by: Daniel F. Dickinson

nut: add more common validators and add more validation

2026-07-15T07:34:39Z

Factor out some common validation code into some validator functions, and add more UCI config validation. Signed-off-by: Daniel F. Dickinson

nut: simplify/improve consistency for RUNAS and STATEPATH

2026-07-15T07:34:39Z

We claimed we only supported a single upsd section named 'upsd' and a single uspmon section named 'upsmon', but when determining RUNAS and STATEPATH we were using overly-complex functions as we were attempting to support multiple upsd and upsmon sections. Simplify the logic by adhering to our stated support. While were are at it we make RUNAS handling more consistent between nut-server (upsd) and nut-monitor (upsmon). We no longer try to limp along if RUNAS is not set by find_runas; to do this properly would require over-complicated logic when setting file and directory permissions and ownership. Signed-off-by: Daniel F. Dickinson

nut: minor fixes found via automated code review

2026-07-15T07:34:39Z

* Fixes a missing word in a comment * Fixes inconsistent indentation in sample configurations * Drops redundant default assignment in find_statepath() Signed-off-by: Daniel F. Dickinson

nut: do not trigger on interface activity by default

2026-07-15T07:34:39Z

The defaults were triggiering on any activity on the network, which caused excessive restarts. So default to no interface trigger, with 'all' (or specific interfaces) as an option. See https://github.com/openwrt/packages/pull/29607#discussion_r3463553192 Signed-off-by: Daniel F. Dickinson

nut: fix hard-coded trigger scripts for interface triggers

2026-07-15T07:34:39Z

The trigger script and instance to reload should depend on which initscript is calling interface_triggers. It wasn't, so we fix that. See https://github.com/openwrt/packages/pull/29607#discussion_r3463544231 Signed-off-by: Daniel F. Dickinson

nut: remove driverpath configuration

2026-07-15T07:34:39Z

While driverpath is could in theory be configurable, this was not implemented in the OpenWrt NUT package, and there is not need for it. Therefore remove the sample configuration for it, and setting that configuration in the upsd.conf file. See https://github.com/openwrt/packages/pull/29607#discussion_r3463526037 Signed-off-by: Daniel F. Dickinson

nut: fix erroneous validation of STATEPATH

2026-07-15T07:34:39Z

We should fallback to the default if one is not specified. In addition find_statepath is the wrong place to check for its existence - it should be created by the initscript if it does not exist. See https://github.com/openwrt/packages/pull/29607#discussion_r3462976647 Signed-off-by: Daniel F. Dickinson

nut: use better usbhid-ups config example

2026-07-15T07:34:39Z

The previous configuration did not make clear the need for productid and vendorid options. See https://github.com/openwrt/packages/pull/29607#discussion_r3462906851 Signed-off-by: Daniel F. Dickinson

nut: rewrite scripts

2026-07-02T04:43:08Z

The scripts were a mess. Attempting even a simple update caused many Copilot complaints. So we rewrite the scripts to be cleaner and resolve the issues found by automated code review (such as Copilot). Made extensive use of Qwen3.6-27B, LATE, and llama.ccp for local AI code reviews during development. In the process we deduplicate the nut-server and nut-monitor initscripts and split them into several files, for easier automatic and human review. Incorporates and supersedes: #28308 Should supersede #21014 Closes: #28298 Signed-off-by: Daniel F. Dickinson