[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