[PATCH 1/2] nodewatcher: split up

[Fabian Bläse]

Hey Tim,

super Sache, danke für die Arbeit!

Noch ein bisschen Senf inline:

On 08.09.19 17:18, Tim Niemeyer wrote:
> diff --git a/src/packages/fff/fff-batman-adv-legacy/files/usr/lib/nodewatcher.d/30-batman-adv.sh b/src/packages/fff/fff-batman-adv-legacy/files/usr/lib/nodewatcher.d/30-batman-adv.sh
> new file mode 100755
> index 0000000..c7fad80
> --- /dev/null
> +++ b/src/packages/fff/fff-batman-adv-legacy/files/usr/lib/nodewatcher.d/30-batman-adv.sh
> @@ -0,0 +1,63 @@
> +#!/bin/sh
> +# Netmon Nodewatcher (C) 2010-2012 Freifunk Oldenburg
> +# License; GPL v3
> +
> +debug() {
> +    (>&2 echo "$1")
Möchte man das $(date) vielleicht noch mit in diese Funktion nehmen?
Generell würde ich das vielleicht auch in die Commit Message schreiben, dass Funktionsnamen geändert wurden.

> diff --git a/src/packages/fff/fff-network/files/usr/lib/nodewatcher.d/20-interfaces.sh b/src/packages/fff/fff-network/files/usr/lib/nodewatcher.d/20-interfaces.sh
> new file mode 100755
> index 0000000..d999c9f
> --- /dev/null
> +++ b/src/packages/fff/fff-network/files/usr/lib/nodewatcher.d/20-interfaces.sh
> @@ -0,0 +1,74 @@
> +#!/bin/sh
> +# Netmon Nodewatcher (C) 2010-2012 Freifunk Oldenburg
> +# License; GPL v3
> +
> +IFACEBLACKLIST="lo ifb0"
> +IPWHITELIST="br-mesh"
Hat es einen Grund, warum die Listen nicht mehr in der uci-Konfiguration stehen?
Falls ja: -> Commitmessage

Ich fand die Konfigurierbarkeit eigentlich recht angenehm, so konnte man selbst entscheiden, ob man die IPs von Interfaces im Monitoring haben möchte.

> diff --git a/src/packages/fff/fff-network/files/usr/lib/nodewatcher.d/50-clients.sh b/src/packages/fff/fff-network/files/usr/lib/nodewatcher.d/50-clients.sh
> new file mode 100755
> index 0000000..426dd57
> --- /dev/null
> +++ b/src/packages/fff/fff-network/files/usr/lib/nodewatcher.d/50-clients.sh
> @@ -0,0 +1,25 @@
> +#!/bin/sh
> +# Netmon Nodewatcher (C) 2010-2012 Freifunk Oldenburg
> +# License; GPL v3
> +
> +MESH_INTERFACE=br-mesh
Siehe oben, nicht mehr konfigurierbar.

> diff --git a/src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher b/src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher
> index 90509ba..2137898 100755
> --- a/src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher
> +++ b/src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher
> @@ -12,28 +12,18 @@ if ! lock -n "$lockfile"; then
>  fi
>  trap "lock -u \"$lockfile\"" INT TERM EXIT
>  
> -SCRIPT_VERSION="53"
> -
>  #Get the configuration from the uci configuration file
>  #If it does not exists, then get it from a normal bash file with variables.
> -if [ -f /etc/config/nodewatcher ];then
> -    SCRIPT_ERROR_LEVEL=$(uci get nodewatcher. at script[0].error_level)
> -    SCRIPT_LOGFILE=$(uci get nodewatcher. at script[0].logfile)
> -    SCRIPT_DATA_FILE=$(uci get nodewatcher. at script[0].data_file)
> -    MESH_INTERFACE=$(uci get nodewatcher. at network[0].mesh_interface)
> -    IFACEBLACKLIST=$(uci get nodewatcher. at network[0].iface_blacklist)
> -    IPWHITELIST=$(uci get nodewatcher. at network[0].ip_whitelist)
> -    SCRIPT_STATUS_FILE=$(uci get nodewatcher. at script[0].status_text_file)
> -else
> -    . "$(dirname "$0")/nodewatcher_config"
> -fi
Es gab mal Bestrebungen den nodewatcher auch unter nicht-OpenWRT lauffähig zu machen.
Daher kommt vermutlich auch dieser Test.

Möchte man das weiter verfolgen? Gute Frage eigentlich, wie man das gut machen kann, da der Nodewatcher jetzt über mehrere Packages verteilt ist..

Ansonsten:
Reviewed-by: Fabian Bläse <fabian at blaese.de>

-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 833 bytes
Beschreibung: OpenPGP digital signature
URL         : <https://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20190908/c8d43b6d/attachment.sig>