[PATCH 1/2] nodewatcher: split up

[Adrian Schmutzler]

Hallo Robert,

> run_with_timeout: also ich finde die 2 Sekunden nicht schlimm. Der 
> Timeout "schützt" das Sysytem vor einer hängenden Funktion. Das ist 
> schon ein gewisser Vorteil. 

Wir haben ja dann 6 oder 7 Mal 2 Sekunden.

Zudem hat dieses Konzept den Nachteil, dass man bei den schnellen nodewatcher.d Skripten zwei Sekunden warten muss, während bei den langsamen ggf. 2 Sekunden zu kurz sind, und diese öfters kaputt gehen.

Hätte die Lösung "funktioniert", sodass die Wartezeit nicht immer läuft, hätte ich eine längere Zeit in Richtung 5 Sekunden vorgeschlagen.

Das Hängen des Ganzen könnte man auch einfach implementieren, indem man die Skripte "normal" called oder sourced, und dann einfach für den ganzen nodewatcher ein timeout von z.B. 30 Sekunden setzt.

Ich bin ohnehin dringend dafür, dass bei Scheitern eines nodewatcher.d Teils der ganze Datensatz verworfen wird, wodurch ein spezifisches Auswerten des Erfolgs der Teilskripte hinfällig wird.

> uci: Es wäre zwar schön einen Plattform unabhängigen nodewatcher zu 
> haben, ich glaube allerdings, dass sich keiner finden wird das hin 
> bastelt. Ist auch nicht ganz trivial. Ich würde vorschlagen, hier eine 
> Openwrt Variante mit uci zu machen. Ein "Gerippe", das man sich dann auf 
> einer anderen Kiste anpasst, könnte man zusätzlich noch irgendwie anbieten. 

Das deckt sich glaube ich mit meiner zuvor geäußerten Meinung.

> 20-interfaces.sh: Spielt das überhaupt eine Rolle? 

Nur für ein sauberes Trennen der dependencies. Praktisch relevant ist das erstmal nicht, deshalb kann man das auch erstmal nicht anfassen. Sauber trennen kann man es mit der aktuellen xml Struktur ohnehin nicht, man muss sich also für die weniger falsche Lösung entscheiden.

Generell noch:

Je mehr ich darüber nachdenke, desto mehr bin dafür, den Umbau erst NACH dem Release einzubauen. Einen nodewatcher-Umbau will ich ausführlich getestet haben, bevor er in eine Firmware hineinkommt, die dann vll. wieder ein Jahr lang ohne Nachfolger rumfliegt. Ich bin sehr für diesen Umbau, aber ich habe auch keine Lust, danach ggf. lang am Monitoring nachsitzen zu müssen.

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/20191021/774d11df/attachment.sig>