| Message ID | 20190128100724.46661-1-fabian@blaese.de |
|---|---|
| State | Accepted |
| Headers | show |
diff --git a/buildscript b/buildscript index f8d435c..b85ff2a 100755 --- a/buildscript +++ b/buildscript @@ -40,7 +40,7 @@ ROUTING=(routing https://git.openwrt.org/feed/routing.git 1b9d1c419f0ecefda51922a7845ab2183d6acd76 "0002-Add-batman-adv-patch-to-remove-gw-mode-switch-messag.patch") # openwrt-18.06 from 2018-06-13 -ROUTING_PKGS="kmod-batman-adv batctl alfred" +ROUTING_PKGS="kmod-batman-adv batctl alfred babeld" FFF=(fff) FFF_PKGS="-a" diff --git a/src/packages/fff/fff-babeld/Makefile b/src/packages/fff/fff-babeld/Makefile new file mode 100644 index 0000000..c53841b --- /dev/null +++ b/src/packages/fff/fff-babeld/Makefile @@ -0,0 +1,40 @@ +include $(TOPDIR)/rules.mk + +PKG_NAME:=fff-babeld +PKG_VERSION:=1 +PKG_RELEASE:=1 + +PKG_BUILD_DIR:=$(BUILD_DIR)/fff-babeld + +include $(INCLUDE_DIR)/package.mk + +define Package/fff-babeld + SECTION:=base + CATEGORY:=Freifunk + TITLE:=Freifunk-Franken babeld configuration example + URL:=http://www.freifunk-franken.de + DEPENDS:=+babeld +endef + +define Package/fff-babeld/description + This is the Freifunk Franken Firmware babeld package. + This package provides an example babeld configuration. +endef + +define Build/Prepare + echo "all: " > $(PKG_BUILD_DIR)/Makefile +endef + +define Build/Configure + # nothing +endef + +define Build/Compile + # nothing +endef + +define Package/fff-babeld/install + $(CP) ./files/* $(1)/ +endef + +$(eval $(call BuildPackage,fff-babeld)) diff --git a/src/packages/fff/fff-babeld/files/etc/config/babeld b/src/packages/fff/fff-babeld/files/etc/config/babeld new file mode 100644 index 0000000..a62788a --- /dev/null +++ b/src/packages/fff/fff-babeld/files/etc/config/babeld @@ -0,0 +1,40 @@ +config general + option export_table '10' + option import_table '10' + option first_table_number '100' + option local_port '33123' + +config interface + option type 'wired' + +config filter + option type 'redistribute' + option local 'true' + option ip '10.50.0.0/16' + +config filter + option type 'redistribute' + option local 'true' + option ip '10.83.0.0/16' + +config filter + option type 'redistribute' + option local 'true' + option ip 'fd43:5602:29bd::/48' + +config filter + option type 'redistribute' + option local 'true' + option action 'deny' + +config filter + option type 'redistribute' + option ip '10.50.0.0/16' + +config filter + option type 'redistribute' + option ip '10.83.0.0/16' + +config filter + option type 'redistribute' + option ip 'fd43:5602:29bd::/48' diff --git a/src/packages/fff/fff-babeld/files/etc/uci-defaults/01-iproute b/src/packages/fff/fff-babeld/files/etc/uci-defaults/01-iproute new file mode 100644 index 0000000..93d0574 --- /dev/null +++ b/src/packages/fff/fff-babeld/files/etc/uci-defaults/01-iproute @@ -0,0 +1,3 @@ +#!/bin/sh + +echo "10 fff" >> /etc/iproute2/rt_tables diff --git a/src/packages/fff/fff-babeld/files/etc/uci-defaults/02-network-rules b/src/packages/fff/fff-babeld/files/etc/uci-defaults/02-network-rules new file mode 100644 index 0000000..094b04f --- /dev/null +++ b/src/packages/fff/fff-babeld/files/etc/uci-defaults/02-network-rules @@ -0,0 +1,47 @@ +#!/bin/sh + +uci batch <<EOF + delete batman-adv.bat0 + add network rule + set network.@rule[0].src='10.50.0.0/16' + set network.@rule[0].lookup='10' + set network.@rule[0].priority='20' + add network rule + set network.@rule[1]=rule + set network.@rule[1].dest='10.50.0.0/16' + set network.@rule[1].lookup='10' + set network.@rule[1].priority='20' + add network rule + set network.@rule[2].src='10.83.0.0/16' + set network.@rule[2].lookup='10' + set network.@rule[2].priority='20' + add network rule + set network.@rule[3]=rule + set network.@rule[3].dest='10.83.0.0/16' + set network.@rule[3].lookup='10' + set network.@rule[3].priority='20' + add network rule + set network.@rule[4]=rule + set network.@rule[4].in='mesh' + set network.@rule[4].lookup='10' + set network.@rule[4].priority='31' + add network rule6 + set network.@rule6[0]=rule6 + set network.@rule6[0].src='fc00::/7' + set network.@rule6[0].lookup='10' + set network.@rule6[0].priority='20' + add network rule6 + set network.@rule6[1]=rule6 + set network.@rule6[1].dest='fc00::/7' + set network.@rule6[1].lookup='10' + set network.@rule6[1].priority='20' + add network rule6 + set network.@rule6[2]=rule6 + set network.@rule6[2].in='mesh' + set network.@rule6[2].lookup='10' + set network.@rule6[2].priority='31' +EOF + +uci commit + +exit 0
Hallo Fabian, gefällt mir schon gut, ein paar Kleinigkeiten unten. > -----Original Message----- > From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf Of > Fabian Bläse > Sent: Montag, 28. Januar 2019 11:07 > To: franken-dev@freifunk.net > Subject: [PATCH] packages/fff: add new fff-babeld package > > From: Tim Niemeyer <tim@tn-x.org> > > Signed-off-by: Tim Niemeyer <tim@tn-x.org> > Signed-off-by: Fabian Bläse <fabian@blaese.de> > - Convert configuration to uci > - Split IPv4 network rules to fff specific subnets > - Add IPv6 ULA network rules > - Change PKG_VERSION to single number versioning > --- > buildscript | 2 +- > src/packages/fff/fff-babeld/Makefile | 40 ++++++++++++++++ > .../fff/fff-babeld/files/etc/config/babeld | 40 ++++++++++++++++ > .../files/etc/uci-defaults/01-iproute | 3 ++ > .../files/etc/uci-defaults/02-network-rules | 47 +++++++++++++++++++ > 5 files changed, 131 insertions(+), 1 deletion(-) > create mode 100644 src/packages/fff/fff-babeld/Makefile > create mode 100644 src/packages/fff/fff-babeld/files/etc/config/babeld > create mode 100644 src/packages/fff/fff-babeld/files/etc/uci-defaults/01- > iproute > create mode 100644 src/packages/fff/fff-babeld/files/etc/uci-defaults/02- > network-rules > > diff --git a/buildscript b/buildscript > index f8d435c..b85ff2a 100755 > --- a/buildscript > +++ b/buildscript > @@ -40,7 +40,7 @@ ROUTING=(routing > https://git.openwrt.org/feed/routing.git > 1b9d1c419f0ecefda51922a7845ab2183d6acd76 > "0002-Add-batman-adv-patch-to-remove-gw-mode-switch- > messag.patch") # openwrt-18.06 from 2018-06-13 > -ROUTING_PKGS="kmod-batman-adv batctl alfred" > +ROUTING_PKGS="kmod-batman-adv batctl alfred babeld" > > FFF=(fff) > FFF_PKGS="-a" > diff --git a/src/packages/fff/fff-babeld/Makefile b/src/packages/fff/fff- > babeld/Makefile > new file mode 100644 > index 0000000..c53841b > --- /dev/null > +++ b/src/packages/fff/fff-babeld/Makefile > @@ -0,0 +1,40 @@ > +include $(TOPDIR)/rules.mk > + > +PKG_NAME:=fff-babeld > +PKG_VERSION:=1 > +PKG_RELEASE:=1 > + > +PKG_BUILD_DIR:=$(BUILD_DIR)/fff-babeld > + > +include $(INCLUDE_DIR)/package.mk > + > +define Package/fff-babeld > + SECTION:=base > + CATEGORY:=Freifunk > + TITLE:=Freifunk-Franken babeld configuration example > + URL:=http://www.freifunk-franken.de > + DEPENDS:=+babeld > +endef > + > +define Package/fff-babeld/description > + This is the Freifunk Franken Firmware babeld package. > + This package provides an example babeld configuration. > +endef > + > +define Build/Prepare > + echo "all: " > $(PKG_BUILD_DIR)/Makefile > +endef > + > +define Build/Configure > + # nothing > +endef > + > +define Build/Compile > + # nothing > +endef > + > +define Package/fff-babeld/install > + $(CP) ./files/* $(1)/ > +endef > + > +$(eval $(call BuildPackage,fff-babeld)) > diff --git a/src/packages/fff/fff-babeld/files/etc/config/babeld > b/src/packages/fff/fff-babeld/files/etc/config/babeld > new file mode 100644 > index 0000000..a62788a > --- /dev/null > +++ b/src/packages/fff/fff-babeld/files/etc/config/babeld > @@ -0,0 +1,40 @@ > +config general > + option export_table '10' > + option import_table '10' > + option first_table_number '100' > + option local_port '33123' > + > +config interface > + option type 'wired' > + > +config filter > + option type 'redistribute' > + option local 'true' > + option ip '10.50.0.0/16' > + > +config filter > + option type 'redistribute' > + option local 'true' > + option ip '10.83.0.0/16' > + > +config filter > + option type 'redistribute' > + option local 'true' > + option ip 'fd43:5602:29bd::/48' Hier nehmen wird das /48, was ich gut finde. Unten bei den network Rules nehmen wird das /7. Ist das Absicht? > + > +config filter > + option type 'redistribute' > + option local 'true' > + option action 'deny' > + > +config filter > + option type 'redistribute' > + option ip '10.50.0.0/16' > + > +config filter > + option type 'redistribute' > + option ip '10.83.0.0/16' > + > +config filter > + option type 'redistribute' > + option ip 'fd43:5602:29bd::/48' > diff --git a/src/packages/fff/fff-babeld/files/etc/uci-defaults/01-iproute > b/src/packages/fff/fff-babeld/files/etc/uci-defaults/01-iproute > new file mode 100644 > index 0000000..93d0574 > --- /dev/null > +++ b/src/packages/fff/fff-babeld/files/etc/uci-defaults/01-iproute > @@ -0,0 +1,3 @@ > +#!/bin/sh > + > +echo "10 fff" >> /etc/iproute2/rt_tables > diff --git a/src/packages/fff/fff-babeld/files/etc/uci-defaults/02-network-rules > b/src/packages/fff/fff-babeld/files/etc/uci-defaults/02-network-rules > new file mode 100644 > index 0000000..094b04f > --- /dev/null > +++ b/src/packages/fff/fff-babeld/files/etc/uci-defaults/02-network-rules > @@ -0,0 +1,47 @@ > +#!/bin/sh > + > +uci batch <<EOF > + delete batman-adv.bat0 Das gehört hier nicht hin? In fff-batman-adv machen wir das (danach) eh nochmal neu. > + add network rule > + set network.@rule[0].src='10.50.0.0/16' > + set network.@rule[0].lookup='10' > + set network.@rule[0].priority='20' > + add network rule > + set network.@rule[1]=rule > + set network.@rule[1].dest='10.50.0.0/16' > + set network.@rule[1].lookup='10' > + set network.@rule[1].priority='20' > + add network rule > + set network.@rule[2].src='10.83.0.0/16' > + set network.@rule[2].lookup='10' > + set network.@rule[2].priority='20' > + add network rule > + set network.@rule[3]=rule > + set network.@rule[3].dest='10.83.0.0/16' > + set network.@rule[3].lookup='10' > + set network.@rule[3].priority='20' > + add network rule > + set network.@rule[4]=rule > + set network.@rule[4].in='mesh' > + set network.@rule[4].lookup='10' > + set network.@rule[4].priority='31' > + add network rule6 > + set network.@rule6[0]=rule6 > + set network.@rule6[0].src='fc00::/7' Hier wäre mir 'fd43:5602:29bd::/48' lieber, oder verstehe ich hier was falsch? > + set network.@rule6[0].lookup='10' > + set network.@rule6[0].priority='20' > + add network rule6 > + set network.@rule6[1]=rule6 > + set network.@rule6[1].dest='fc00::/7' Hier auch. Grüße Adrian > + set network.@rule6[1].lookup='10' > + set network.@rule6[1].priority='20' > + add network rule6 > + set network.@rule6[2]=rule6 > + set network.@rule6[2].in='mesh' > + set network.@rule6[2].lookup='10' > + set network.@rule6[2].priority='31' > +EOF > + > +uci commit > + > +exit 0 > -- > 2.20.1
hi kleine Rückfrage inline, sonst siehts gut aus, wenn du jetzt keine total haarsträubende Geschichte bei dieser einen Zeile bringst: Reviewed-by: Christian Dresel <fff@chrisi01.de> On 28.01.19 11:07, Fabian Bläse wrote: > From: Tim Niemeyer <tim@tn-x.org> > > Signed-off-by: Tim Niemeyer <tim@tn-x.org> > Signed-off-by: Fabian Bläse <fabian@blaese.de> > - Convert configuration to uci > - Split IPv4 network rules to fff specific subnets > - Add IPv6 ULA network rules > - Change PKG_VERSION to single number versioning > --- > buildscript | 2 +- > src/packages/fff/fff-babeld/Makefile | 40 ++++++++++++++++ > .../fff/fff-babeld/files/etc/config/babeld | 40 ++++++++++++++++ > .../files/etc/uci-defaults/01-iproute | 3 ++ > .../files/etc/uci-defaults/02-network-rules | 47 +++++++++++++++++++ > 5 files changed, 131 insertions(+), 1 deletion(-) > create mode 100644 src/packages/fff/fff-babeld/Makefile > create mode 100644 src/packages/fff/fff-babeld/files/etc/config/babeld > create mode 100644 src/packages/fff/fff-babeld/files/etc/uci-defaults/01-iproute > create mode 100644 src/packages/fff/fff-babeld/files/etc/uci-defaults/02-network-rules > > diff --git a/buildscript b/buildscript > index f8d435c..b85ff2a 100755 > --- a/buildscript > +++ b/buildscript > @@ -40,7 +40,7 @@ ROUTING=(routing > https://git.openwrt.org/feed/routing.git > 1b9d1c419f0ecefda51922a7845ab2183d6acd76 > "0002-Add-batman-adv-patch-to-remove-gw-mode-switch-messag.patch") # openwrt-18.06 from 2018-06-13 > -ROUTING_PKGS="kmod-batman-adv batctl alfred" > +ROUTING_PKGS="kmod-batman-adv batctl alfred babeld" > > FFF=(fff) > FFF_PKGS="-a" > diff --git a/src/packages/fff/fff-babeld/Makefile b/src/packages/fff/fff-babeld/Makefile > new file mode 100644 > index 0000000..c53841b > --- /dev/null > +++ b/src/packages/fff/fff-babeld/Makefile > @@ -0,0 +1,40 @@ > +include $(TOPDIR)/rules.mk > + > +PKG_NAME:=fff-babeld > +PKG_VERSION:=1 > +PKG_RELEASE:=1 > + > +PKG_BUILD_DIR:=$(BUILD_DIR)/fff-babeld > + > +include $(INCLUDE_DIR)/package.mk > + > +define Package/fff-babeld > + SECTION:=base > + CATEGORY:=Freifunk > + TITLE:=Freifunk-Franken babeld configuration example > + URL:=http://www.freifunk-franken.de > + DEPENDS:=+babeld > +endef > + > +define Package/fff-babeld/description > + This is the Freifunk Franken Firmware babeld package. > + This package provides an example babeld configuration. > +endef > + > +define Build/Prepare > + echo "all: " > $(PKG_BUILD_DIR)/Makefile > +endef > + > +define Build/Configure > + # nothing > +endef > + > +define Build/Compile > + # nothing > +endef > + > +define Package/fff-babeld/install > + $(CP) ./files/* $(1)/ > +endef > + > +$(eval $(call BuildPackage,fff-babeld)) > diff --git a/src/packages/fff/fff-babeld/files/etc/config/babeld b/src/packages/fff/fff-babeld/files/etc/config/babeld > new file mode 100644 > index 0000000..a62788a > --- /dev/null > +++ b/src/packages/fff/fff-babeld/files/etc/config/babeld > @@ -0,0 +1,40 @@ > +config general > + option export_table '10' > + option import_table '10' > + option first_table_number '100' > + option local_port '33123' > + > +config interface > + option type 'wired' > + > +config filter > + option type 'redistribute' > + option local 'true' > + option ip '10.50.0.0/16' > + > +config filter > + option type 'redistribute' > + option local 'true' > + option ip '10.83.0.0/16' > + > +config filter > + option type 'redistribute' > + option local 'true' > + option ip 'fd43:5602:29bd::/48' > + > +config filter > + option type 'redistribute' > + option local 'true' > + option action 'deny' > + > +config filter > + option type 'redistribute' > + option ip '10.50.0.0/16' > + > +config filter > + option type 'redistribute' > + option ip '10.83.0.0/16' > + > +config filter > + option type 'redistribute' > + option ip 'fd43:5602:29bd::/48' > diff --git a/src/packages/fff/fff-babeld/files/etc/uci-defaults/01-iproute b/src/packages/fff/fff-babeld/files/etc/uci-defaults/01-iproute > new file mode 100644 > index 0000000..93d0574 > --- /dev/null > +++ b/src/packages/fff/fff-babeld/files/etc/uci-defaults/01-iproute > @@ -0,0 +1,3 @@ > +#!/bin/sh > + > +echo "10 fff" >> /etc/iproute2/rt_tables > diff --git a/src/packages/fff/fff-babeld/files/etc/uci-defaults/02-network-rules b/src/packages/fff/fff-babeld/files/etc/uci-defaults/02-network-rules > new file mode 100644 > index 0000000..094b04f > --- /dev/null > +++ b/src/packages/fff/fff-babeld/files/etc/uci-defaults/02-network-rules > @@ -0,0 +1,47 @@ > +#!/bin/sh > + > +uci batch <<EOF > + delete batman-adv.bat0 was wird da genau entfernt und warum? Gruß Christian > + add network rule > + set network.@rule[0].src='10.50.0.0/16' > + set network.@rule[0].lookup='10' > + set network.@rule[0].priority='20' > + add network rule > + set network.@rule[1]=rule > + set network.@rule[1].dest='10.50.0.0/16' > + set network.@rule[1].lookup='10' > + set network.@rule[1].priority='20' > + add network rule > + set network.@rule[2].src='10.83.0.0/16' > + set network.@rule[2].lookup='10' > + set network.@rule[2].priority='20' > + add network rule > + set network.@rule[3]=rule > + set network.@rule[3].dest='10.83.0.0/16' > + set network.@rule[3].lookup='10' > + set network.@rule[3].priority='20' > + add network rule > + set network.@rule[4]=rule > + set network.@rule[4].in='mesh' > + set network.@rule[4].lookup='10' > + set network.@rule[4].priority='31' > + add network rule6 > + set network.@rule6[0]=rule6 > + set network.@rule6[0].src='fc00::/7' > + set network.@rule6[0].lookup='10' > + set network.@rule6[0].priority='20' > + add network rule6 > + set network.@rule6[1]=rule6 > + set network.@rule6[1].dest='fc00::/7' > + set network.@rule6[1].lookup='10' > + set network.@rule6[1].priority='20' > + add network rule6 > + set network.@rule6[2]=rule6 > + set network.@rule6[2].in='mesh' > + set network.@rule6[2].lookup='10' > + set network.@rule6[2].priority='31' > +EOF > + > +uci commit > + > +exit 0
Hallo Adrian, On 28.01.19 11:29, Adrian Schmutzler wrote: >> --- /dev/null >> +++ b/src/packages/fff/fff-babeld/files/etc/config/babeld >> @@ -0,0 +1,40 @@ >> +config general >> + option export_table '10' >> + option import_table '10' >> + option first_table_number '100' >> + option local_port '33123' >> + >> +config interface >> + option type 'wired' >> + >> +config filter >> + option type 'redistribute' >> + option local 'true' >> + option ip '10.50.0.0/16' >> + >> +config filter >> + option type 'redistribute' >> + option local 'true' >> + option ip '10.83.0.0/16' >> + >> +config filter >> + option type 'redistribute' >> + option local 'true' >> + option ip 'fd43:5602:29bd::/48' > > Hier nehmen wird das /48, was ich gut finde. Unten bei den network Rules nehmen wird das /7. Ist das Absicht? Hier wählen wir aus, welche Routen tatsächlich vom Router selbst announced werden dürfen, weiter unten legen wir rules fest, welche Netze wo erreichbar sind. Die redistribute Filter sollten aus diesem Grund so klein wie möglich gehalten werden. Zu den rules gehört eigentlich das komplette 10.0.0.0/8 (icvpn) und ggf. auch noch 172.20.0.0/14 (dn42), was ja alles innerhalb des Freifunk Franken Netzes geroutet wird. Daher auch fc00::/7. Da bei IPv4 aber die Internetverbindung für Tunnel häufig auch Private Adressen verwendet und ich aktuell nicht so genau sagen kann, was für lustige Effekte das hat, wenn sich die Netze überlappen, habe ich es für IPv4 bei den rules erstmal dabei gelassen, nur die FFF /16-er Netze zu verwenden. Bei IPv6 ist das nicht so dramatisch, da für Internet sowieso öffentliche Adressen verfügbar sind und Tunnel dann dennoch klappen. Generell ist das für Routing aber auch gar nicht so dramatisch, da für sämtliche Peering-Interfaces sowieso eine iif-rule hinzugefügt werden sollte, was dann sowieso dazu führt, dass in die fff-Table geguckt wird. Nur für Pakete, die der Router selbst sendet, ist das ein Problem. Das ganze müsste man sich irgendwann nochmal etwas genauer angucken, es könnte ja durchaus auch sein, dass jemand eins der /16-er Netze Zuhause verwendet. >> --- /dev/null >> +++ b/src/packages/fff/fff-babeld/files/etc/uci-defaults/02-network-rules >> @@ -0,0 +1,47 @@ >> +#!/bin/sh >> + >> +uci batch <<EOF >> + delete batman-adv.bat0 > > Das gehört hier nicht hin? In fff-batman-adv machen wir das (danach) eh nochmal neu. Oh. Nein, das gehört hier nicht hin. Das ist da wohl irgendwie reingerutscht. Kommt weg. Gruß Fabian
Hallo Christian, On 28.01.19 11:32, Christian Dresel wrote: >> --- /dev/null >> +++ b/src/packages/fff/fff-babeld/files/etc/uci-defaults/02-network-rules >> @@ -0,0 +1,47 @@ >> +#!/bin/sh >> + >> +uci batch <<EOF >> + delete batman-adv.bat0 > > was wird da genau entfernt und warum? Dieses blöde Batman.. Überall taucht es aus dem nichts auf und macht Probleme.. ;-) Gehört hier natürlich nicht hin, kommt weg. Gruß Fabian
Hallo Fabian, ich verwende z.B. zu Hause ein Subnet aus fc00::/7, dass über den WAN eines dezentralen Gateways dann auch dessen eth0.2 eine entsprechende Adresse zuweist etc. Tut sich das dann mit dem fc00::/7 bei den rules weh? (Das was du für die IPv4 /16 meintest: „es könnte ja durchaus auch sein, dass jemand eins der /16-er Netze Zuhause verwendet“)? Grüße Adrian From: Fabian Bläse [mailto:fabian@blaese.de] Sent: Montag, 28. Januar 2019 11:38 To: Adrian Schmutzler <mail@adrianschmutzler.de>; franken-dev@freifunk.net Subject: Re: [PATCH] packages/fff: add new fff-babeld package Hallo Adrian, On 28.01.19 11:29, Adrian Schmutzler wrote: >> --- /dev/null >> +++ b/src/packages/fff/fff-babeld/files/etc/config/babeld >> @@ -0,0 +1,40 @@ >> +config general >> + option export_table '10' >> + option import_table '10' >> + option first_table_number '100' >> + option local_port '33123' >> + >> +config interface >> + option type 'wired' >> + >> +config filter >> + option type 'redistribute' >> + option local 'true' >> + option ip '10.50.0.0/16' >> + >> +config filter >> + option type 'redistribute' >> + option local 'true' >> + option ip '10.83.0.0/16' >> + >> +config filter >> + option type 'redistribute' >> + option local 'true' >> + option ip 'fd43:5602:29bd::/48' > > Hier nehmen wird das /48, was ich gut finde. Unten bei den network Rules nehmen wird das /7. Ist das Absicht? Hier wählen wir aus, welche Routen tatsächlich vom Router selbst announced werden dürfen, weiter unten legen wir rules fest, welche Netze wo erreichbar sind. Die redistribute Filter sollten aus diesem Grund so klein wie möglich gehalten werden. Zu den rules gehört eigentlich das komplette 10.0.0.0/8 (icvpn) und ggf. auch noch 172.20.0.0/14 (dn42), was ja alles innerhalb des Freifunk Franken Netzes geroutet wird. Daher auch fc00::/7. Da bei IPv4 aber die Internetverbindung für Tunnel häufig auch Private Adressen verwendet und ich aktuell nicht so genau sagen kann, was für lustige Effekte das hat, wenn sich die Netze überlappen, habe ich es für IPv4 bei den rules erstmal dabei gelassen, nur die FFF /16-er Netze zu verwenden. Bei IPv6 ist das nicht so dramatisch, da für Internet sowieso öffentliche Adressen verfügbar sind und Tunnel dann dennoch klappen. Generell ist das für Routing aber auch gar nicht so dramatisch, da für sämtliche Peering-Interfaces sowieso eine iif-rule hinzugefügt werden sollte, was dann sowieso dazu führt, dass in die fff-Table geguckt wird. Nur für Pakete, die der Router selbst sendet, ist das ein Problem. Das ganze müsste man sich irgendwann nochmal etwas genauer angucken, es könnte ja durchaus auch sein, dass jemand eins der /16-er Netze Zuhause verwendet. >> --- /dev/null >> +++ b/src/packages/fff/fff-babeld/files/etc/uci-defaults/02-network-rules >> @@ -0,0 +1,47 @@ >> +#!/bin/sh >> + >> +uci batch <<EOF >> + delete batman-adv.bat0 > > Das gehört hier nicht hin? In fff-batman-adv machen wir das (danach) eh nochmal neu. Oh. Nein, das gehört hier nicht hin. Das ist da wohl irgendwie reingerutscht. Kommt weg. Gruß Fabian
Es würde sich wohl genauso verhalten, ja. Vermutlich ist dann für den Router dein Subnetz aus fc00::/7 nicht erreichbar, wenn es auch im babel announced wird. Da die fc00::/7 Netze aber sowieso generiert werden müssen ist die Wahrscheinlichkeit einer Überlappung ziemlich gering, solange niemand eine nicht-source-specific default Route announcned. Das tut aber nicht (direkt) weh, weil für Tunnel ja öffentliche Adressen gebraucht und verwendet werden. Und die fallen nicht in fc00::/7. Insgesamt ein Problem, dem man nochmal nachgehen sollte. Vermutlich lässt sich das aber gar nicht so richtig sauber lösen.. Gruß Fabian On 28.01.19 11:45, Adrian Schmutzler wrote: > Hallo Fabian, > > > > ich verwende z.B. zu Hause ein Subnet aus fc00::/7, dass über den WAN eines dezentralen Gateways dann auch dessen eth0.2 eine entsprechende Adresse zuweist etc. > > Tut sich das dann mit dem fc00::/7 bei den rules weh? (Das was du für die IPv4 /16 meintest: „es könnte ja durchaus auch sein, dass jemand eins der /16-er Netze Zuhause verwendet“)? > > > > Grüße > > > > Adrian
Hallo, „nicht erreichbar, wenn es auch im babel announced wird.“ Okay, damit ist für mich die Sache erledigt, wie du bereits geschrieben hast ist das aufgrund der Wahrscheinlichkeit erstmal tolerabel. Reviewed-by: Adrian Schmutzler freifunk@adrianschmutzler.de <mailto:freifunk@adrianschmutzler.de> Lediglich die eine Zeile mit dem batman-adv muss noch raus, entweder du schickst nochmal nen Patch (kannst den Review mit ranhängen) oder wir machen das beim Applien mit raus (letzteres ist wohl einfacher). Grüße Adrian From: Fabian Bläse [mailto:fabian@blaese.de] Sent: Montag, 28. Januar 2019 11:51 To: Adrian Schmutzler <mail@adrianschmutzler.de>; franken-dev@freifunk.net Subject: Re: [PATCH] packages/fff: add new fff-babeld package Es würde sich wohl genauso verhalten, ja. Vermutlich ist dann für den Router dein Subnetz aus fc00::/7 nicht erreichbar, wenn es auch im babel announced wird. Da die fc00::/7 Netze aber sowieso generiert werden müssen ist die Wahrscheinlichkeit einer Überlappung ziemlich gering, solange niemand eine nicht-source-specific default Route announcned. Das tut aber nicht (direkt) weh, weil für Tunnel ja öffentliche Adressen gebraucht und verwendet werden. Und die fallen nicht in fc00::/7. Insgesamt ein Problem, dem man nochmal nachgehen sollte. Vermutlich lässt sich das aber gar nicht so richtig sauber lösen.. Gruß Fabian On 28.01.19 11:45, Adrian Schmutzler wrote: > Hallo Fabian, > > > > ich verwende z.B. zu Hause ein Subnet aus fc00::/7, dass über den WAN eines dezentralen Gateways dann auch dessen eth0.2 eine entsprechende Adresse zuweist etc. > > Tut sich das dann mit dem fc00::/7 bei den rules weh? (Das was du für die IPv4 /16 meintest: „es könnte ja durchaus auch sein, dass jemand eins der /16-er Netze Zuhause verwendet“)? > > > > Grüße > > > > Adrian
Hallo, nachdem zwei Reviews da sind und wir uns zwecks batman-adv alle einig waren, habe ich es jetzt doch gleich applied. Grüße Adrian > -----Original Message----- > From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf Of > Fabian Bläse > Sent: Montag, 28. Januar 2019 11:07 > To: franken-dev@freifunk.net > Subject: [PATCH] packages/fff: add new fff-babeld package > > From: Tim Niemeyer <tim@tn-x.org> > > Signed-off-by: Tim Niemeyer <tim@tn-x.org> > Signed-off-by: Fabian Bläse <fabian@blaese.de> > - Convert configuration to uci > - Split IPv4 network rules to fff specific subnets > - Add IPv6 ULA network rules > - Change PKG_VERSION to single number versioning > --- > buildscript | 2 +- > src/packages/fff/fff-babeld/Makefile | 40 ++++++++++++++++ > .../fff/fff-babeld/files/etc/config/babeld | 40 ++++++++++++++++ > .../files/etc/uci-defaults/01-iproute | 3 ++ > .../files/etc/uci-defaults/02-network-rules | 47 +++++++++++++++++++ > 5 files changed, 131 insertions(+), 1 deletion(-) > create mode 100644 src/packages/fff/fff-babeld/Makefile > create mode 100644 src/packages/fff/fff-babeld/files/etc/config/babeld > create mode 100644 src/packages/fff/fff-babeld/files/etc/uci-defaults/01- > iproute > create mode 100644 src/packages/fff/fff-babeld/files/etc/uci-defaults/02- > network-rules > > diff --git a/buildscript b/buildscript > index f8d435c..b85ff2a 100755 > --- a/buildscript > +++ b/buildscript > @@ -40,7 +40,7 @@ ROUTING=(routing > https://git.openwrt.org/feed/routing.git > 1b9d1c419f0ecefda51922a7845ab2183d6acd76 > "0002-Add-batman-adv-patch-to-remove-gw-mode-switch- > messag.patch") # openwrt-18.06 from 2018-06-13 > -ROUTING_PKGS="kmod-batman-adv batctl alfred" > +ROUTING_PKGS="kmod-batman-adv batctl alfred babeld" > > FFF=(fff) > FFF_PKGS="-a" > diff --git a/src/packages/fff/fff-babeld/Makefile b/src/packages/fff/fff- > babeld/Makefile > new file mode 100644 > index 0000000..c53841b > --- /dev/null > +++ b/src/packages/fff/fff-babeld/Makefile > @@ -0,0 +1,40 @@ > +include $(TOPDIR)/rules.mk > + > +PKG_NAME:=fff-babeld > +PKG_VERSION:=1 > +PKG_RELEASE:=1 > + > +PKG_BUILD_DIR:=$(BUILD_DIR)/fff-babeld > + > +include $(INCLUDE_DIR)/package.mk > + > +define Package/fff-babeld > + SECTION:=base > + CATEGORY:=Freifunk > + TITLE:=Freifunk-Franken babeld configuration example > + URL:=http://www.freifunk-franken.de > + DEPENDS:=+babeld > +endef > + > +define Package/fff-babeld/description > + This is the Freifunk Franken Firmware babeld package. > + This package provides an example babeld configuration. > +endef > + > +define Build/Prepare > + echo "all: " > $(PKG_BUILD_DIR)/Makefile > +endef > + > +define Build/Configure > + # nothing > +endef > + > +define Build/Compile > + # nothing > +endef > + > +define Package/fff-babeld/install > + $(CP) ./files/* $(1)/ > +endef > + > +$(eval $(call BuildPackage,fff-babeld)) > diff --git a/src/packages/fff/fff-babeld/files/etc/config/babeld > b/src/packages/fff/fff-babeld/files/etc/config/babeld > new file mode 100644 > index 0000000..a62788a > --- /dev/null > +++ b/src/packages/fff/fff-babeld/files/etc/config/babeld > @@ -0,0 +1,40 @@ > +config general > + option export_table '10' > + option import_table '10' > + option first_table_number '100' > + option local_port '33123' > + > +config interface > + option type 'wired' > + > +config filter > + option type 'redistribute' > + option local 'true' > + option ip '10.50.0.0/16' > + > +config filter > + option type 'redistribute' > + option local 'true' > + option ip '10.83.0.0/16' > + > +config filter > + option type 'redistribute' > + option local 'true' > + option ip 'fd43:5602:29bd::/48' > + > +config filter > + option type 'redistribute' > + option local 'true' > + option action 'deny' > + > +config filter > + option type 'redistribute' > + option ip '10.50.0.0/16' > + > +config filter > + option type 'redistribute' > + option ip '10.83.0.0/16' > + > +config filter > + option type 'redistribute' > + option ip 'fd43:5602:29bd::/48' > diff --git a/src/packages/fff/fff-babeld/files/etc/uci-defaults/01-iproute > b/src/packages/fff/fff-babeld/files/etc/uci-defaults/01-iproute > new file mode 100644 > index 0000000..93d0574 > --- /dev/null > +++ b/src/packages/fff/fff-babeld/files/etc/uci-defaults/01-iproute > @@ -0,0 +1,3 @@ > +#!/bin/sh > + > +echo "10 fff" >> /etc/iproute2/rt_tables > diff --git a/src/packages/fff/fff-babeld/files/etc/uci-defaults/02-network-rules > b/src/packages/fff/fff-babeld/files/etc/uci-defaults/02-network-rules > new file mode 100644 > index 0000000..094b04f > --- /dev/null > +++ b/src/packages/fff/fff-babeld/files/etc/uci-defaults/02-network-rules > @@ -0,0 +1,47 @@ > +#!/bin/sh > + > +uci batch <<EOF > + delete batman-adv.bat0 > + add network rule > + set network.@rule[0].src='10.50.0.0/16' > + set network.@rule[0].lookup='10' > + set network.@rule[0].priority='20' > + add network rule > + set network.@rule[1]=rule > + set network.@rule[1].dest='10.50.0.0/16' > + set network.@rule[1].lookup='10' > + set network.@rule[1].priority='20' > + add network rule > + set network.@rule[2].src='10.83.0.0/16' > + set network.@rule[2].lookup='10' > + set network.@rule[2].priority='20' > + add network rule > + set network.@rule[3]=rule > + set network.@rule[3].dest='10.83.0.0/16' > + set network.@rule[3].lookup='10' > + set network.@rule[3].priority='20' > + add network rule > + set network.@rule[4]=rule > + set network.@rule[4].in='mesh' > + set network.@rule[4].lookup='10' > + set network.@rule[4].priority='31' > + add network rule6 > + set network.@rule6[0]=rule6 > + set network.@rule6[0].src='fc00::/7' > + set network.@rule6[0].lookup='10' > + set network.@rule6[0].priority='20' > + add network rule6 > + set network.@rule6[1]=rule6 > + set network.@rule6[1].dest='fc00::/7' > + set network.@rule6[1].lookup='10' > + set network.@rule6[1].priority='20' > + add network rule6 > + set network.@rule6[2]=rule6 > + set network.@rule6[2].in='mesh' > + set network.@rule6[2].lookup='10' > + set network.@rule6[2].priority='31' > +EOF > + > +uci commit > + > +exit 0 > -- > 2.20.1
Hallo, zu diesem alten Patch eine Frage unten. > -----Original Message----- > From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf Of > Fabian Bläse > Sent: Montag, 28. Januar 2019 11:07 > To: franken-dev@freifunk.net > Subject: [PATCH] packages/fff: add new fff-babeld package > > From: Tim Niemeyer <tim@tn-x.org> > > Signed-off-by: Tim Niemeyer <tim@tn-x.org> > Signed-off-by: Fabian Bläse <fabian@blaese.de> > - Convert configuration to uci > - Split IPv4 network rules to fff specific subnets > - Add IPv6 ULA network rules > - Change PKG_VERSION to single number versioning > --- > buildscript | 2 +- > src/packages/fff/fff-babeld/Makefile | 40 ++++++++++++++++ > .../fff/fff-babeld/files/etc/config/babeld | 40 ++++++++++++++++ > .../files/etc/uci-defaults/01-iproute | 3 ++ > .../files/etc/uci-defaults/02-network-rules | 47 +++++++++++++++++++ > 5 files changed, 131 insertions(+), 1 deletion(-) > create mode 100644 src/packages/fff/fff-babeld/Makefile > create mode 100644 src/packages/fff/fff-babeld/files/etc/config/babeld > create mode 100644 src/packages/fff/fff-babeld/files/etc/uci-defaults/01- > iproute > create mode 100644 src/packages/fff/fff-babeld/files/etc/uci-defaults/02- > network-rules > > diff --git a/buildscript b/buildscript > index f8d435c..b85ff2a 100755 > --- a/buildscript > +++ b/buildscript > @@ -40,7 +40,7 @@ ROUTING=(routing > https://git.openwrt.org/feed/routing.git > 1b9d1c419f0ecefda51922a7845ab2183d6acd76 > "0002-Add-batman-adv-patch-to-remove-gw-mode-switch- > messag.patch") # openwrt-18.06 from 2018-06-13 > -ROUTING_PKGS="kmod-batman-adv batctl alfred" > +ROUTING_PKGS="kmod-batman-adv batctl alfred babeld" > > FFF=(fff) > FFF_PKGS="-a" > diff --git a/src/packages/fff/fff-babeld/Makefile b/src/packages/fff/fff- > babeld/Makefile > new file mode 100644 > index 0000000..c53841b > --- /dev/null > +++ b/src/packages/fff/fff-babeld/Makefile > @@ -0,0 +1,40 @@ > +include $(TOPDIR)/rules.mk > + > +PKG_NAME:=fff-babeld > +PKG_VERSION:=1 > +PKG_RELEASE:=1 > + > +PKG_BUILD_DIR:=$(BUILD_DIR)/fff-babeld > + > +include $(INCLUDE_DIR)/package.mk > + > +define Package/fff-babeld > + SECTION:=base > + CATEGORY:=Freifunk > + TITLE:=Freifunk-Franken babeld configuration example > + URL:=http://www.freifunk-franken.de > + DEPENDS:=+babeld > +endef > + > +define Package/fff-babeld/description > + This is the Freifunk Franken Firmware babeld package. > + This package provides an example babeld configuration. > +endef > + > +define Build/Prepare > + echo "all: " > $(PKG_BUILD_DIR)/Makefile > +endef > + > +define Build/Configure > + # nothing > +endef > + > +define Build/Compile > + # nothing > +endef > + > +define Package/fff-babeld/install > + $(CP) ./files/* $(1)/ > +endef > + > +$(eval $(call BuildPackage,fff-babeld)) > diff --git a/src/packages/fff/fff-babeld/files/etc/config/babeld > b/src/packages/fff/fff-babeld/files/etc/config/babeld > new file mode 100644 > index 0000000..a62788a > --- /dev/null > +++ b/src/packages/fff/fff-babeld/files/etc/config/babeld > @@ -0,0 +1,40 @@ > +config general > + option export_table '10' > + option import_table '10' > + option first_table_number '100' > + option local_port '33123' > + > +config interface > + option type 'wired' Hat dieses "halbe" Interface einen Sinn? Vor configurehood kommt babeld eh nicht hoch, sondern beschwert sich darüber, dass es kein Interface gibt. (zumindest bei mir) Grüße Adrian > + > +config filter > + option type 'redistribute' > + option local 'true' > + option ip '10.50.0.0/16' > + > +config filter > + option type 'redistribute' > + option local 'true' > + option ip '10.83.0.0/16' > + > +config filter > + option type 'redistribute' > + option local 'true' > + option ip 'fd43:5602:29bd::/48' > + > +config filter > + option type 'redistribute' > + option local 'true' > + option action 'deny' > + > +config filter > + option type 'redistribute' > + option ip '10.50.0.0/16' > + > +config filter > + option type 'redistribute' > + option ip '10.83.0.0/16' > + > +config filter > + option type 'redistribute' > + option ip 'fd43:5602:29bd::/48' > diff --git a/src/packages/fff/fff-babeld/files/etc/uci-defaults/01-iproute > b/src/packages/fff/fff-babeld/files/etc/uci-defaults/01-iproute > new file mode 100644 > index 0000000..93d0574 > --- /dev/null > +++ b/src/packages/fff/fff-babeld/files/etc/uci-defaults/01-iproute > @@ -0,0 +1,3 @@ > +#!/bin/sh > + > +echo "10 fff" >> /etc/iproute2/rt_tables > diff --git a/src/packages/fff/fff-babeld/files/etc/uci-defaults/02-network-rules > b/src/packages/fff/fff-babeld/files/etc/uci-defaults/02-network-rules > new file mode 100644 > index 0000000..094b04f > --- /dev/null > +++ b/src/packages/fff/fff-babeld/files/etc/uci-defaults/02-network-rules > @@ -0,0 +1,47 @@ > +#!/bin/sh > + > +uci batch <<EOF > + delete batman-adv.bat0 > + add network rule > + set network.@rule[0].src='10.50.0.0/16' > + set network.@rule[0].lookup='10' > + set network.@rule[0].priority='20' > + add network rule > + set network.@rule[1]=rule > + set network.@rule[1].dest='10.50.0.0/16' > + set network.@rule[1].lookup='10' > + set network.@rule[1].priority='20' > + add network rule > + set network.@rule[2].src='10.83.0.0/16' > + set network.@rule[2].lookup='10' > + set network.@rule[2].priority='20' > + add network rule > + set network.@rule[3]=rule > + set network.@rule[3].dest='10.83.0.0/16' > + set network.@rule[3].lookup='10' > + set network.@rule[3].priority='20' > + add network rule > + set network.@rule[4]=rule > + set network.@rule[4].in='mesh' > + set network.@rule[4].lookup='10' > + set network.@rule[4].priority='31' > + add network rule6 > + set network.@rule6[0]=rule6 > + set network.@rule6[0].src='fc00::/7' > + set network.@rule6[0].lookup='10' > + set network.@rule6[0].priority='20' > + add network rule6 > + set network.@rule6[1]=rule6 > + set network.@rule6[1].dest='fc00::/7' > + set network.@rule6[1].lookup='10' > + set network.@rule6[1].priority='20' > + add network rule6 > + set network.@rule6[2]=rule6 > + set network.@rule6[2].in='mesh' > + set network.@rule6[2].lookup='10' > + set network.@rule6[2].priority='31' > +EOF > + > +uci commit > + > +exit 0 > -- > 2.20.1
Hallo Adrian, On 09.05.19 12:29, Adrian Schmutzler wrote:>> +config interface >> + option type 'wired' > > Hat dieses "halbe" Interface einen Sinn? > Vor configurehood kommt babeld eh nicht hoch, sondern beschwert sich darüber, dass es kein Interface gibt. (zumindest bei mir)jo, hat es. Wenn kein Name angegeben wird, entsteht daraus die babeld "default" Konfiguration, also hier dann die Zeile: default type wired Gruß Fabian