[RFC PATCH] Add package fff-vxlan-node

Adrian Schmutzler mail at adrianschmutzler.de
Sa Aug 8 02:02:10 CEST 2020


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.

> +
> +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 ...

Grüße

Adrian

> +}
> +
> --
> 2.20.1
-------------- 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/20200808/0e20dc4b/attachment.sig>


Mehr Informationen über die Mailingliste franken-dev