fastd-2-uci and fix start/stop #219

rohammer · 2022-03-11T23:32:58+01:00

rohammer commented

2022-03-11 23:32:58 +01:00

Since fastd changed to procd our start/stop mechanism is broken.
This pr fixes the start/stop and by the way switches to uci config.

Since fastd changed to procd our start/stop mechanism is broken. This pr fixes the start/stop and by the way switches to uci config.

rohammer added 2 commits 2022-03-11 23:32:59 +01:00

893a2a5ec1 node: introduce variable "new_hoodfile"

This information can be evaluated by the vpn-skripts.

Signed-off-by: Robert Langhammer <rlanghammer@web.de>

653e78cc41 fff-fastd: switch to uci config and fix start/stop

Configure the fastd peers via uci.

Fix the start/stop.

fixes: #215

Signed-off-by: Robert Langhammer <rlanghammer@web.de>

fbl added this to the 20220405-beta milestone 2022-03-12 18:23:29 +01:00

fbl commented

2022-03-15 13:12:31 +01:00

Vom Wechsel zu uci (incl. commit) bin ich ehrlich gesagt nur eingeschränkt überzeugt, denn statt die Peers im RAM zu halten, werden diese jetzt persistent auf den Flash geschrieben. Das belastet nicht nur den Flash unnötig, sondern ändert potentiell auch das Verhalten der Geräte direkt nach einem Reboot.

Eigentlich würde ich bei node gerne sämtlichen dynamisch konfigurierten Zustand nicht-persistent im RAM halten, statt auf den Flash zu schreiben.

Zum restart mit new_hoodfile habe ich keine Meinung. Reload wäre mir lieber, Restart ist aber auch ok.

fastd start und stop unabhängig vom aktuellen Zustand zu machen, sollte mit procd einwandfrei funktionieren.

"Fixes:" in der commit-message groß schreiben :-)

Vom Wechsel zu uci (incl. commit) bin ich ehrlich gesagt nur eingeschränkt überzeugt, denn statt die Peers im RAM zu halten, werden diese jetzt persistent auf den Flash geschrieben. Das belastet nicht nur den Flash unnötig, sondern ändert potentiell auch das Verhalten der Geräte direkt nach einem Reboot. Eigentlich würde ich bei node gerne sämtlichen dynamisch konfigurierten Zustand nicht-persistent im RAM halten, statt auf den Flash zu schreiben. Zum restart mit new_hoodfile habe ich keine Meinung. Reload wäre mir lieber, Restart ist aber auch ok. fastd start und stop unabhängig vom aktuellen Zustand zu machen, sollte mit procd einwandfrei funktionieren. "Fixes:" in der commit-message groß schreiben :-)

rohammer commented

2022-03-15 14:55:41 +01:00

Der Flash ist auf jeden Fall ein Argument.
Eigentlich hatte ich vor das mit reload_config zu machen. Darum auch uci. Aber der ubus call lauft ins Leere. Kann also genauso gut unter /tmp bleiben.

Relad waere mir auch lieber, lifert aber nicht das gewuenschte Ergebnis. Ich probier das noch mal mit den peer-Dateien. Vielleicht macht da dann reload was es soll.

fastd start bemerkt dummerweise keine Aenderungen, wenn fastd schon an ist. Und ein restart moechte man auch nicht alle 5 Minuten haben.

Ich schau mir das nochmal an.

Der Flash ist auf jeden Fall ein Argument. Eigentlich hatte ich vor das mit reload_config zu machen. Darum auch uci. Aber der ubus call lauft ins Leere. Kann also genauso gut unter /tmp bleiben. Relad waere mir auch lieber, lifert aber nicht das gewuenschte Ergebnis. Ich probier das noch mal mit den peer-Dateien. Vielleicht macht da dann reload was es soll. fastd start bemerkt dummerweise keine Aenderungen, wenn fastd schon an ist. Und ein restart moechte man auch nicht alle 5 Minuten haben. Ich schau mir das nochmal an.

rohammer commented

2022-03-19 17:05:52 +01:00

Ich schliesse diesen pr. Siehe #221

rohammer closed this pull request

2022-03-19 17:05:58 +01:00