[PATCH 1/2] nodewatcher: split up

[robert]

HI,

was machen wir denn jetzt damit?

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.

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.

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

Viele Grüße
Robert

Am 10.10.19 um 12:46 schrieb Adrian Schmutzler:
> Hallo nochmal,
>
> im Prinzip ist die ganze run_with_timeout Funktion ja ein Zusatzfeature.
>
> Spricht etwas dagegen, erstmal die Aufteilung des nodewatchers umzusetzen und dabei die nodewatcher.d Dateien einfach direkt auszuführen?
> Dies würde im Wesentlichen das Gleiche tun wie der nodewatcher jetzt. Dann könnte man den großen Teil des Patches relativ schnell einbauen und würde danach nur noch über die Ausgestaltung der Funktion run_with_timeout diskutieren.
>
> Man würde dann zunächst einfach
>
> tmp=$("$f")
> if [ -n "$f" ]; then
>    ...
>
> verwenden.
>
> Man müsste dann nur noch:
> - klären, ob man 20-interfaces.sh nach network oder wireless tut
> - Die Commit-Message schön machen
> - Sich bezüglich uci vs. hardcode aus Fabian E-Mail einigen (mMn könnte man das zunächst wie im Patch lassen und danach ggf. wieder in einem separaten Patch uci einbauen/diskutieren)
> - Rebasen
>
> @Tim: Hast du Lust, das nochmal zu rebasen/die Commit Message zu schreiben oder soll ich das machen.
>
> Grüße
>
> Adrian
>

-------------- 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/20191019/e1a066a4/attachment.sig>