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

[Fabian Bläse]

Hey Adrian,

On 19.04.20 01:22, mail at adrianschmutzler.de wrote:
> 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.
Ouh, das klingt ja fies.
Wie gesagt: An diesem Dings würde ich jetzt nichts mehr ändern, das kann keiner nachvollziehen. Zumal die meisten meiner Anmerkungen eh neue "Features" sind, und daher in einem eigenen Patch eh besser aufgehoben sind.
Ich würde diese Wünsche aber gern noch vor der Beta drin haben, daher warte ich gerne auf deinen Server (außer du möchtest ohne Server trotzdem reviewen).

>> 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.
Was? Ne.
Der Timeout sollte eigentlich tatsächlich als Timeout funktioniert haben, getestet hab ichs aber nie..
Es gab dazu wohl mal etwas Diskussion mit Robert, die habe ich damals mangels Zeit nicht gelesen.

Aber wie gesagt: Extra Patches.

>>> 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).
Fixe ich beim applien.

Gruß
Fabian

-------------- 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/20200419/2a67cc4e/attachment.sig>