[PATCH 1/5] nodewatcher: split into nodewatcher.d scripts for individual task

[mail at adrianschmutzler.de]

Hallo Fabian,

> Ich hätte $(date) noch mit in die debug Funktion mit rein genommen. 

Kann man machen. Würde ich aber auch separat tun, da hier ja die alte Funktion kopiert wird.

> Unbedingt würde ich auch den Exitstatus der Sub-Skripte prüfen und ggf. abbrechen. 
> Außerdem ist mit deiner Version der timeout weg gefallen, das fand ich eigentlich gar nicht schlecht.. 

Den Timeout mochte ich nicht, da es ja effektiv kein Timeout war, sondern eine fixe Wartezeit für jedes Subskript. Ich halte nichts davon, für jedes Subskript zwei Minuten zu warten, sodass dann der ganze nodewatcher 20-30 Sekunden dauert (im Moment). Und beim Exitstatus wurde es dann noch kompliziert.
So oder so waren aber ja der Exitstatus und der "Timeout" im Prinzip Zusatzfeatures, während meine Version aus einem Skript eine Aneinanderreihung von vielen Einzelskripten macht.
Ich würde das also separat diskutieren, aber es ist in meinen Augen keine Notwendigkeit für die Aufteilung an sich.

> In den Sub-Skripten werden Debugausgaben auf stderr ausgegeben, im Hauptskript in eine Datei geschreiben, deren Länge irgendwie hingepfriemelt wird.
> Ich bin hier ja fast dafür, alles auf stderr auszugeben und die Datei fallen zu lassen. Man könnte aber auch stderr der Subskripte mit in die Datei stecken.

Das habe ich ehrlich gesagt auch gar nicht durchdacht, sondern einfach nur als gegeben betrachtet. Das sollte man in der Tat mal diskutieren, ich habe hier überhaupt keine Meinung, was sinnvoll ist.
Auf jeden Fall ist die momentane Implementierung ziemlich verwirrend.

> Da wir uns an diesem Patch aber noch ewig aufhalten würden, bis jeder absolut zufrieden ist, würde ich diesen Patch so akzeptieren und dann lieber hinterher noch Dinge fixen.

Ich kriege frühestens am Dienstag ein neues Netzteil, und dann werde ich erstmal kucken, was an meinem Server noch geht. Bis dahin werde ich nichts ändern, schicken oder applien.
Falls du ein Release machen willst, also bitte selbst einwerfen.

> > Note that since the nodewatcher.d scripts are evaluated by using there 
> "using their"? 

In der Tat. Einer meiner häufigsten Typos im Englischen (neben it's<>its).

Grüße

Adrian 
-------------- 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/64ac37e5/attachment.sig>