Message ID | 20180130131406.25550-1-rlanghammer@web.de |
---|---|
State | Superseded |
Headers | show |
diff --git a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood index 95a1bf4..f676233 100755 --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood @@ -305,8 +305,9 @@ if [ -s "$hoodfile" ]; then else echo "Address already set." fi - if ! ( ip -6 route show dev br-mesh | grep -q "fc00::" ) ; then - ip -6 route add fc00::/7 via fe80::1 dev br-mesh + if ! ( ip -6 route show table fff dev br-mesh | grep -q "fc00::" ) ; then + ip -6 route add fc00::/7 via fe80::1 dev br-mesh table fff + ip -6 rule add from all to fc00::/7 lookup fff echo "Set ULA route to br-mesh." else echo "Route already set." diff --git a/src/packages/fff/fff-network/files/etc/uci-defaults/55_add_table_fff b/src/packages/fff/fff-network/files/etc/uci-defaults/55_add_table_fff new file mode 100644 index 0000000..5dbf403 --- /dev/null +++ b/src/packages/fff/fff-network/files/etc/uci-defaults/55_add_table_fff @@ -0,0 +1,2 @@ +echo "# table fff" >> /etc/iproute2/rt_tables +echo "10 fff" >> /etc/iproute2/rt_tables
Hi Nur kurz vom Handy: Am 30. Januar 2018 14:14:06 MEZ schrieb Robert Langhammer <rlanghammer@web.de>: >Fixes #83 > >Signed-off-by: Robert Langhammer <rlanghammer@web.de> >--- >src/packages/fff/fff-hoods/files/usr/sbin/configurehood | >5 +++-- >src/packages/fff/fff-network/files/etc/uci-defaults/55_add_table_fff | >2 ++ > 2 files changed, 5 insertions(+), 2 deletions(-) >create mode 100644 >src/packages/fff/fff-network/files/etc/uci-defaults/55_add_table_fff > >diff --git a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood >b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood >index 95a1bf4..f676233 100755 >--- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood >+++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood >@@ -305,8 +305,9 @@ if [ -s "$hoodfile" ]; then > else > echo "Address already set." > fi >- if ! ( ip -6 route show dev br-mesh | grep -q "fc00::" ) ; then >- ip -6 route add fc00::/7 via fe80::1 dev br-mesh >+ if ! ( ip -6 route show table fff dev br-mesh | grep -q "fc00::" ) ; >then >+ ip -6 route add fc00::/7 via fe80::1 dev br-mesh table fff >+ ip -6 rule add from all to fc00::/7 lookup fff Du schaust im If nach route setzt dann aber hier auch die rule, gefällt mir irgendwie nicht. Allgemein will man das ganze vllt. einfach statisch in die /etc/config/network auslagern, eigentlich könnte man den ganzen fdff Kram auch gleich statisch dort mit reinschreiben, wir brauchen das eh immer immer immer und müssen es eigentlich nicht flexibel konfigurieren oder? Das einzige was flexibel sein muss sind die fd43 Adressen weil die sich pro Hood ändern. Glaube Fabian hat das schon mal irgendwann angeregt. mfg Christian > echo "Set ULA route to br-mesh." > else > echo "Route already set." >diff --git >a/src/packages/fff/fff-network/files/etc/uci-defaults/55_add_table_fff >b/src/packages/fff/fff-network/files/etc/uci-defaults/55_add_table_fff >new file mode 100644 >index 0000000..5dbf403 >--- /dev/null >+++ >b/src/packages/fff/fff-network/files/etc/uci-defaults/55_add_table_fff >@@ -0,0 +1,2 @@ >+echo "# table fff" >> /etc/iproute2/rt_tables >+echo "10 fff" >> /etc/iproute2/rt_tables
Hi, s.u. Am 30.01.2018 um 14:33 schrieb Christian Dresel: > Hi > > Nur kurz vom Handy: > > Am 30. Januar 2018 14:14:06 MEZ schrieb Robert Langhammer <rlanghammer@web.de>: >> Fixes #83 >> >> Signed-off-by: Robert Langhammer <rlanghammer@web.de> >> --- >> src/packages/fff/fff-hoods/files/usr/sbin/configurehood | >> 5 +++-- >> src/packages/fff/fff-network/files/etc/uci-defaults/55_add_table_fff | >> 2 ++ >> 2 files changed, 5 insertions(+), 2 deletions(-) >> create mode 100644 >> src/packages/fff/fff-network/files/etc/uci-defaults/55_add_table_fff >> >> diff --git a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood >> b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood >> index 95a1bf4..f676233 100755 >> --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood >> +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood >> @@ -305,8 +305,9 @@ if [ -s "$hoodfile" ]; then >> else >> echo "Address already set." >> fi >> - if ! ( ip -6 route show dev br-mesh | grep -q "fc00::" ) ; then >> - ip -6 route add fc00::/7 via fe80::1 dev br-mesh >> + if ! ( ip -6 route show table fff dev br-mesh | grep -q "fc00::" ) ; >> then >> + ip -6 route add fc00::/7 via fe80::1 dev br-mesh table fff >> + ip -6 rule add from all to fc00::/7 lookup fff > Du schaust im If nach route setzt dann aber hier auch die rule, gefällt mir irgendwie nicht. Route und Rule gehoeren schon zusammen, ohne Route braucht man auch keine Rule. Darum hier. > > Allgemein will man das ganze vllt. einfach statisch in die /etc/config/network auslagern, eigentlich könnte man den ganzen fdff Kram auch gleich statisch dort mit reinschreiben, wir brauchen das eh immer immer immer und müssen es eigentlich nicht flexibel konfigurieren oder? Das einzige was flexibel sein muss sind die fd43 Adressen weil die sich pro Hood ändern. Glaube Fabian hat das schon mal irgendwann angeregt. Das finde ich auch besser. Falls sich mal was aendern sollte muss man es so oder so rein patchen. Vielleicht komm ich heute Abend dazu. Gruesse Robert > > mfg > > Christian > > >> echo "Set ULA route to br-mesh." >> else >> echo "Route already set." >> diff --git >> a/src/packages/fff/fff-network/files/etc/uci-defaults/55_add_table_fff >> b/src/packages/fff/fff-network/files/etc/uci-defaults/55_add_table_fff >> new file mode 100644 >> index 0000000..5dbf403 >> --- /dev/null >> +++ >> b/src/packages/fff/fff-network/files/etc/uci-defaults/55_add_table_fff >> @@ -0,0 +1,2 @@ >> +echo "# table fff" >> /etc/iproute2/rt_tables >> +echo "10 fff" >> /etc/iproute2/rt_tables
Hallo, habe das Setzen der Route über die Tabelle manuell an einem betroffenen Router getestet und es funktioniert bei mir. Grüße Adrian > -----Original Message----- > From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf > Of robert > Sent: Dienstag, 30. Januar 2018 14:52 > To: franken-dev@freifunk.net > Subject: Re: [PATCH] fff-network: add routing table fff fff-hoods: use table > fff for fc00::/7 > > Hi, s.u. > > > Am 30.01.2018 um 14:33 schrieb Christian Dresel: > > Hi > > > > Nur kurz vom Handy: > > > > Am 30. Januar 2018 14:14:06 MEZ schrieb Robert Langhammer > <rlanghammer@web.de>: > >> Fixes #83 > >> > >> Signed-off-by: Robert Langhammer <rlanghammer@web.de> > >> --- > >> src/packages/fff/fff-hoods/files/usr/sbin/configurehood | > >> 5 +++-- > >> src/packages/fff/fff-network/files/etc/uci-defaults/55_add_table_fff > >> | > >> 2 ++ > >> 2 files changed, 5 insertions(+), 2 deletions(-) create mode 100644 > >> src/packages/fff/fff-network/files/etc/uci-defaults/55_add_table_fff > >> > >> diff --git a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > >> b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > >> index 95a1bf4..f676233 100755 > >> --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > >> +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > >> @@ -305,8 +305,9 @@ if [ -s "$hoodfile" ]; then > >> else > >> echo "Address already set." > >> fi > >> - if ! ( ip -6 route show dev br-mesh | grep -q "fc00::" ) ; then > >> - ip -6 route add fc00::/7 via fe80::1 dev br-mesh > >> + if ! ( ip -6 route show table fff dev br-mesh | grep -q "fc00::" ) > >> +; > >> then > >> + ip -6 route add fc00::/7 via fe80::1 dev br-mesh table > fff > >> + ip -6 rule add from all to fc00::/7 lookup fff > > Du schaust im If nach route setzt dann aber hier auch die rule, gefällt mir > irgendwie nicht. > Route und Rule gehoeren schon zusammen, ohne Route braucht man auch > keine Rule. Darum hier. > > > > Allgemein will man das ganze vllt. einfach statisch in die > /etc/config/network auslagern, eigentlich könnte man den ganzen fdff Kram > auch gleich statisch dort mit reinschreiben, wir brauchen das eh immer > immer immer und müssen es eigentlich nicht flexibel konfigurieren oder? > Das einzige was flexibel sein muss sind die fd43 Adressen weil die sich pro > Hood ändern. Glaube Fabian hat das schon mal irgendwann angeregt. > Das finde ich auch besser. Falls sich mal was aendern sollte muss man es so > oder so rein patchen. Vielleicht komm ich heute Abend dazu. > Gruesse > Robert > > > > mfg > > > > Christian > > > > > >> echo "Set ULA route to br-mesh." > >> else > >> echo "Route already set." > >> diff --git > >> a/src/packages/fff/fff-network/files/etc/uci-defaults/55_add_table_ff > >> f > >> b/src/packages/fff/fff-network/files/etc/uci-defaults/55_add_table_ff > >> f > >> new file mode 100644 > >> index 0000000..5dbf403 > >> --- /dev/null > >> +++ > >> b/src/packages/fff/fff-network/files/etc/uci-defaults/55_add_table_ff > >> f > >> @@ -0,0 +1,2 @@ > >> +echo "# table fff" >> /etc/iproute2/rt_tables > >> +echo "10 fff" >> /etc/iproute2/rt_tables > > -- > franken-dev mailing list > franken-dev@freifunk.net > http://lists.freifunk.net/mailman/listinfo/franken-dev-freifunk.net
Zwecks Christians Anmerkung: Mach einfach ein zweites if mit: if ! ( ip -6 rule show | grep -q "fc00::" ) ; then ip -6 rule add from all to fc00::/7 lookup fff echo "Set ULA Rule." else echo "Rule already set." fi Grüße Adrian > -----Original Message----- > From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf > Of robert > Sent: Dienstag, 30. Januar 2018 14:52 > To: franken-dev@freifunk.net > Subject: Re: [PATCH] fff-network: add routing table fff fff-hoods: use table > fff for fc00::/7 > > Hi, s.u. > > > Am 30.01.2018 um 14:33 schrieb Christian Dresel: > > Hi > > > > Nur kurz vom Handy: > > > > Am 30. Januar 2018 14:14:06 MEZ schrieb Robert Langhammer > <rlanghammer@web.de>: > >> Fixes #83 > >> > >> Signed-off-by: Robert Langhammer <rlanghammer@web.de> > >> --- > >> src/packages/fff/fff-hoods/files/usr/sbin/configurehood | > >> 5 +++-- > >> src/packages/fff/fff-network/files/etc/uci-defaults/55_add_table_fff > >> | > >> 2 ++ > >> 2 files changed, 5 insertions(+), 2 deletions(-) create mode 100644 > >> src/packages/fff/fff-network/files/etc/uci-defaults/55_add_table_fff > >> > >> diff --git a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > >> b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > >> index 95a1bf4..f676233 100755 > >> --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > >> +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > >> @@ -305,8 +305,9 @@ if [ -s "$hoodfile" ]; then > >> else > >> echo "Address already set." > >> fi > >> - if ! ( ip -6 route show dev br-mesh | grep -q "fc00::" ) ; then > >> - ip -6 route add fc00::/7 via fe80::1 dev br-mesh > >> + if ! ( ip -6 route show table fff dev br-mesh | grep -q "fc00::" ) > >> +; > >> then > >> + ip -6 route add fc00::/7 via fe80::1 dev br-mesh table > fff > >> + ip -6 rule add from all to fc00::/7 lookup fff > > Du schaust im If nach route setzt dann aber hier auch die rule, gefällt mir > irgendwie nicht. > Route und Rule gehoeren schon zusammen, ohne Route braucht man auch > keine Rule. Darum hier. > > > > Allgemein will man das ganze vllt. einfach statisch in die > /etc/config/network auslagern, eigentlich könnte man den ganzen fdff Kram > auch gleich statisch dort mit reinschreiben, wir brauchen das eh immer > immer immer und müssen es eigentlich nicht flexibel konfigurieren oder? > Das einzige was flexibel sein muss sind die fd43 Adressen weil die sich pro > Hood ändern. Glaube Fabian hat das schon mal irgendwann angeregt. > Das finde ich auch besser. Falls sich mal was aendern sollte muss man es so > oder so rein patchen. Vielleicht komm ich heute Abend dazu. > Gruesse > Robert > > > > mfg > > > > Christian > > > > > >> echo "Set ULA route to br-mesh." > >> else > >> echo "Route already set." > >> diff --git > >> a/src/packages/fff/fff-network/files/etc/uci-defaults/55_add_table_ff > >> f > >> b/src/packages/fff/fff-network/files/etc/uci-defaults/55_add_table_ff > >> f > >> new file mode 100644 > >> index 0000000..5dbf403 > >> --- /dev/null > >> +++ > >> b/src/packages/fff/fff-network/files/etc/uci-defaults/55_add_table_ff > >> f > >> @@ -0,0 +1,2 @@ > >> +echo "# table fff" >> /etc/iproute2/rt_tables > >> +echo "10 fff" >> /etc/iproute2/rt_tables > > -- > franken-dev mailing list > franken-dev@freifunk.net > http://lists.freifunk.net/mailman/listinfo/franken-dev-freifunk.net
Hallo nochmal, bei mir macht der Tab ganz am Schluss Probleme. Evtl. durch ein Leerzeichen "10 fff" ersetzen? Grüße Adrian > -----Original Message----- > From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf > Of Robert Langhammer > Sent: Dienstag, 30. Januar 2018 14:14 > To: franken-dev@freifunk.net > Subject: [PATCH] fff-network: add routing table fff fff-hoods: use table fff > for fc00::/7 > > Fixes #83 > > Signed-off-by: Robert Langhammer <rlanghammer@web.de> > --- > src/packages/fff/fff-hoods/files/usr/sbin/configurehood | 5 +++-- > src/packages/fff/fff-network/files/etc/uci-defaults/55_add_table_fff | 2 ++ > 2 files changed, 5 insertions(+), 2 deletions(-) create mode 100644 > src/packages/fff/fff-network/files/etc/uci-defaults/55_add_table_fff > > diff --git a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > index 95a1bf4..f676233 100755 > --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > @@ -305,8 +305,9 @@ if [ -s "$hoodfile" ]; then > else > echo "Address already set." > fi > - if ! ( ip -6 route show dev br-mesh | grep -q "fc00::" ) ; then > - ip -6 route add fc00::/7 via fe80::1 dev br-mesh > + if ! ( ip -6 route show table fff dev br-mesh | grep -q "fc00::" ) > ; then > + ip -6 route add fc00::/7 via fe80::1 dev br-mesh table > fff > + ip -6 rule add from all to fc00::/7 lookup fff > echo "Set ULA route to br-mesh." > else > echo "Route already set." > diff --git a/src/packages/fff/fff-network/files/etc/uci- > defaults/55_add_table_fff b/src/packages/fff/fff-network/files/etc/uci- > defaults/55_add_table_fff > new file mode 100644 > index 0000000..5dbf403 > --- /dev/null > +++ b/src/packages/fff/fff-network/files/etc/uci-defaults/55_add_table_f > +++ ff > @@ -0,0 +1,2 @@ > +echo "# table fff" >> /etc/iproute2/rt_tables > +echo "10 fff" >> /etc/iproute2/rt_tables > -- > 2.11.0 > > -- > franken-dev mailing list > franken-dev@freifunk.net > http://lists.freifunk.net/mailman/listinfo/franken-dev-freifunk.net
Moin Ich bin eigentlich gegen die rule, da sie a) die Komplexität erhöht b) die Übersicht verringert c) Ich bin mir nicht sicher, ob "from all" in wirklich _jedem_ Fall greift, der benötigt wird. Besser man arbeitet mit iif, was in dem Fall nicht geht, weil wir eigentlich _kein_ forwarding (routing) wollen! Letztlich müssen wir rausfinden, warum sich (einige?) Systeme "fehlerhaft" verhalten. Für mich sieht es aktuell so aus, dass wir irgendwas nicht verstanden haben und daher irgendwas falsch machen. Tim Am Dienstag, den 30.01.2018, 14:14 +0100 schrieb Robert Langhammer: > Fixes #83 > > Signed-off-by: Robert Langhammer <rlanghammer@web.de> > --- > src/packages/fff/fff- > hoods/files/usr/sbin/configurehood | 5 +++-- > src/packages/fff/fff-network/files/etc/uci-defaults/55_add_table_fff > | 2 ++ > 2 files changed, 5 insertions(+), 2 deletions(-) > create mode 100644 src/packages/fff/fff-network/files/etc/uci- > defaults/55_add_table_fff > > diff --git a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > index 95a1bf4..f676233 100755 > --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > @@ -305,8 +305,9 @@ if [ -s "$hoodfile" ]; then > else > echo "Address already set." > fi > - if ! ( ip -6 route show dev br-mesh | grep -q > "fc00::" ) ; then > - ip -6 route add fc00::/7 via fe80::1 dev br- > mesh > + if ! ( ip -6 route show table fff dev br-mesh | grep > -q "fc00::" ) ; then > + ip -6 route add fc00::/7 via fe80::1 dev br- > mesh table fff > + ip -6 rule add from all to fc00::/7 lookup > fff > echo "Set ULA route to br-mesh." > else > echo "Route already set." > diff --git a/src/packages/fff/fff-network/files/etc/uci- > defaults/55_add_table_fff b/src/packages/fff/fff- > network/files/etc/uci-defaults/55_add_table_fff > new file mode 100644 > index 0000000..5dbf403 > --- /dev/null > +++ b/src/packages/fff/fff-network/files/etc/uci- > defaults/55_add_table_fff > @@ -0,0 +1,2 @@ > +echo "# table fff" >> /etc/iproute2/rt_tables > +echo "10 fff" >> /etc/iproute2/rt_tables > -- > 2.11.0 >
Hi, hat jetzt etwas gedauert, war auch nicht so einfach. Tim, du hast da vollkommen recht, die table ist der falsche Ansatz. War eine hauruck Loesung. Der patch kann raus! Es funktioniert anders. Mehr dazu im mantis! Viele Gruesse Robert Am 31.01.2018 um 07:18 schrieb Tim Niemeyer: > Moin > > Ich bin eigentlich gegen die rule, da sie > a) die Komplexität erhöht > b) die Übersicht verringert > c) Ich bin mir nicht sicher, ob "from all" in wirklich _jedem_ Fall > greift, der benötigt wird. Besser man arbeitet mit iif, was in dem Fall > nicht geht, weil wir eigentlich _kein_ forwarding (routing) wollen! > > Letztlich müssen wir rausfinden, warum sich (einige?) Systeme > "fehlerhaft" verhalten. Für mich sieht es aktuell so aus, dass wir > irgendwas nicht verstanden haben und daher irgendwas falsch machen. > > Tim > > Am Dienstag, den 30.01.2018, 14:14 +0100 schrieb Robert Langhammer: >> Fixes #83 >> >> Signed-off-by: Robert Langhammer <rlanghammer@web.de> >> --- >> src/packages/fff/fff- >> hoods/files/usr/sbin/configurehood | 5 +++-- >> src/packages/fff/fff-network/files/etc/uci-defaults/55_add_table_fff >> | 2 ++ >> 2 files changed, 5 insertions(+), 2 deletions(-) >> create mode 100644 src/packages/fff/fff-network/files/etc/uci- >> defaults/55_add_table_fff >> >> diff --git a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood >> b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood >> index 95a1bf4..f676233 100755 >> --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood >> +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood >> @@ -305,8 +305,9 @@ if [ -s "$hoodfile" ]; then >> else >> echo "Address already set." >> fi >> - if ! ( ip -6 route show dev br-mesh | grep -q >> "fc00::" ) ; then >> - ip -6 route add fc00::/7 via fe80::1 dev br- >> mesh >> + if ! ( ip -6 route show table fff dev br-mesh | grep >> -q "fc00::" ) ; then >> + ip -6 route add fc00::/7 via fe80::1 dev br- >> mesh table fff >> + ip -6 rule add from all to fc00::/7 lookup >> fff >> echo "Set ULA route to br-mesh." >> else >> echo "Route already set." >> diff --git a/src/packages/fff/fff-network/files/etc/uci- >> defaults/55_add_table_fff b/src/packages/fff/fff- >> network/files/etc/uci-defaults/55_add_table_fff >> new file mode 100644 >> index 0000000..5dbf403 >> --- /dev/null >> +++ b/src/packages/fff/fff-network/files/etc/uci- >> defaults/55_add_table_fff >> @@ -0,0 +1,2 @@ >> +echo "# table fff" >> /etc/iproute2/rt_tables >> +echo "10 fff" >> /etc/iproute2/rt_tables >> -- >> 2.11.0
Fixes #83 Signed-off-by: Robert Langhammer <rlanghammer@web.de> --- src/packages/fff/fff-hoods/files/usr/sbin/configurehood | 5 +++-- src/packages/fff/fff-network/files/etc/uci-defaults/55_add_table_fff | 2 ++ 2 files changed, 5 insertions(+), 2 deletions(-) create mode 100644 src/packages/fff/fff-network/files/etc/uci-defaults/55_add_table_fff