Browse Source

mini_snmpd: fix initscript to use l3_device

for working out the interface name
Working:
ubus -S call network.interface dump|jsonfilter -e "@.interface[@.interface=\"wan\"].l3_device"
Broken:
ubus -S call network.interface dump|jsonfilter -e "@.interface[@.interface=\"wan\"].device"

Fix run tested:
root@wifi:/overlay/upper# ps |grep mini_snmpd
 1404 root       980 S    /usr/bin/mini_snmpd -n -c public -L Undisclosed -C VGB <admin@victimsofgaybullying.com> -t 1 -a -d /overlay,/tmp -i br-lan,pppoe-w

Before it wasn't using the pppoe interface it was using the parent
interface eth0 twice. Small 1 line fix. Merge at your convenience.

Signed-off-by: Luke McKee <hojuruku@gmail.com>
lilik-openwrt-22.03
Luke McKee 8 years ago
parent
commit
e55da65b14
2 changed files with 2 additions and 2 deletions
  1. +1
    -1
      net/mini_snmpd/Makefile
  2. +1
    -1
      net/mini_snmpd/files/mini_snmpd.init

+ 1
- 1
net/mini_snmpd/Makefile View File

@ -9,7 +9,7 @@ include $(TOPDIR)/rules.mk
PKG_NAME:=mini_snmpd
PKG_VERSION:=1.4-rc1
PKG_RELEASE:=1
PKG_RELEASE:=2
PKG_MAINTAINER:=Luke McKee <hojuruku@gmail.com>
PKG_LICENSE:=GPL-2.0
PKG_LICENSE_FILES:=COPYING


+ 1
- 1
net/mini_snmpd/files/mini_snmpd.init View File

@ -71,7 +71,7 @@ append_interface() {
local name="$1" netdev netdev_count
[ -z $netdev_count ] && netdev_count=0
# for the purposes of snmp monitoring it doesn't need to be up, it just needs to exist in /proc/net/dev
netdev=$(ubus -S call network.interface dump|jsonfilter -e "@.interface[@.interface=\"$name\"].device")
netdev=$(ubus -S call network.interface dump|jsonfilter -e "@.interface[@.interface=\"$name\"].l3_device")
if [ -n "$netdev" ] && grep -qF "$netdev" /proc/net/dev ]; then
[ $netdev_count -ge 4 ] && {
_err "$cfg: too many network interfaces configured, ignoring $name"


Loading…
Cancel
Save