[RFC PATCH] Add package fff-vxlan-node

[Robert Langhammer]

Hi, s. inline

Am 08.08.20 um 02:02 schrieb Adrian Schmutzler:
> Hallo,
>
> nur zwei Einzelkommentare:
>
>> +vxlan_clear() {
>> +	uci set network.vx0.disabled="1"
>> +	while uci delete network. at vxlan_peer[0]; do :; done }
> Gefühlt und für die Konsistenz wäre mir hier ein config_foreach lieber, auch wenn man dann ne Funktion braucht.
Ich bin ein Fan von kleinen Einzeilern und möglichst viel shell-builtin
wenn möglich. Darum so.
>
>> +
>> +vxlan_addpeer() {
>> +	json_get_var vid vid
>> +	json_get_var address address
>> +	address="$( ping6 -w1 -c1 "$address" | awk -F '[()]' '{print $2; exit}' )"
>> +	[ -z $address ] && return
>> +	uci set network.vx0.vid="$vid"
>> +	uci set network.vx0.peer6addr="$address"
>> +	uci set network.vx0.disabled="0"
>> +
>> +	uci add network vxlan_peer
>> +	uci set network. at vxlan_peer[-1].vxlan="vx0"
>> +	uci set network. at vxlan_peer[-1].dst="$address"
>> +}
>> +
>> +vxlan_start_stop() {
>> +	uci commit network
>> +	reload_config
> Hier bin ich mir etwas unschlüssig, ob man mitten "in" configurehood noch ein reload_config machen muss/möchte, oder ob man das vll. besser einmal am Schluss macht. Ordentlicher ist es natürlich so, aber dann haben wir irgendwann vier reload_config während einem configurehood run ...

Ich finde es muss hier hin. Sonst bekommen wir eine kaum sichtbare
Abhängigkeit zu configurehood. Und müssen darauf hoffen, dass da nochmal
ein reload_config läuft.

Viele Grüße
Robert

>
> Grüße
>
> Adrian
>
>> +}
>> +
>> --
>> 2.20.1

-------------- 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/20200808/51fd5606/attachment.sig>