packages/net/nut/files/nut-service.sh.functions, 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: 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: 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: 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