[PATCH 2/4] nodewatcher: Output to stderr instead of log file

[mail at adrianschmutzler.de]

Hat zwar etwas gedauert, bis ich es verstanden hatte, aber finde ich sehr gut.

Dass die Subscripts so eh nach stderr schreiben, war mir gar nicht mehr bewusst.

Für stderr sollten wir dann aber noch ein "nodewatcher" Prefix in die debug() Funktion einbauen (im Patch 1/4).
Es wäre auch zu überlegen, ob man eine nodewatcher library Datei nimmt, dort die debug Funktion reinwirft und dann nur die (mini-)Library einbindet, damit man nicht immer alles zigmal ändern muss.

Bitte für diesen Patch noch die Einträge in https://github.com/FreifunkFranken/firmware/blob/master/src/packages/fff/fff-nodewatcher/files/etc/uci-defaults/93-fff-nodewatcher löschen.

Reviewed-by: Adrian Schmutzler <freifunk at adrianschmutzler.de>

OT: Ich würde sogar überlegen, ob man das DATA_FILE auch irgendwann wegwirft. Aber das ist nicht eilig.

Grüße

Adrian

> -----Original Message-----
> From: franken-dev [mailto:franken-dev-bounces at freifunk.net] On Behalf
> Of Fabian Bläse
> Sent: Sonntag, 19. April 2020 15:40
> To: franken-dev at freifunk.net
> Subject: [PATCH 2/4] nodewatcher: Output to stderr instead of log file
> 
> Maintaining a logfile manually is complicated and has no major improvements
> over just logging to stderr, because nodewatcher is no deamon and can be
> run manually for debugging purposes.
> 
> Also, the debug output from subscripts currently is not written to the log file
> anyway and the debug level is not used eiher.
> 
> Therefore, the file logging and debug level is removed from nodewatcher,
> which simplifies the code a bit.
> 
> Signed-off-by: Fabian Bläse <fabian at blaese.de>
> ---
>  .../files/usr/sbin/nodewatcher                | 26 ++-----------------
>  1 file changed, 2 insertions(+), 24 deletions(-)
> 
> diff --git a/src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher
> b/src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher
> index 8da7877..1226f49 100755
> --- a/src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher
> +++ b/src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher
> @@ -14,28 +14,10 @@ trap "lock -u \"$lockfile\"" INT TERM EXIT
> 
>  [ -s /etc/config/nodewatcher ] || exit 1
> 
> -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)
> 
> -if [ "$SCRIPT_ERROR_LEVEL" -gt "1" ]; then
> -	debug() {
> -		echo "$(date): $1" >> "$SCRIPT_LOGFILE"
> -	}
> -else
> -	debug() {
> -		:
> -	}
> -fi
> -
> -#This method checks if the log file has become too big and deletes the first X
> lines
> -delete_log() {
> -	if [ -f "$SCRIPT_LOGFILE" ]; then
> -		if [ "$(find "$SCRIPT_LOGFILE" -printf "%s")" -gt "6000" ]; then
> -			sed -i '1,60d' "$SCRIPT_LOGFILE"
> -			debug "Logfile has been made smaller"
> -		fi
> -	fi
> +debug() {
> +	(>&2 echo "$(date): $1")
>  }
> 
>  #This method generates the crawl data XML file that is being fetched by
> netmon @@ -60,10 +42,6 @@ crawl() {
> 
>  LANG=C
> 
> -#Prüft ob das logfile zu groß geworden ist -debug "Check logfile"
> -delete_log
> -
>  #Erzeugt die statusdaten
>  debug "Generate actual status data"
>  crawl
> --
> 2.26.0
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : openpgp-digital-signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 834 bytes
Beschreibung: nicht verfügbar
URL         : <https://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20200419/eec453a8/attachment.sig>