[RFC PATCH] fff-batman-adv: Disable batman gw-selection

So Jan 13 13:01:27 CET 2019

Moin Fabian

Am Dienstag, den 11.12.2018, 15:31 +0100 schrieb Fabian Bläse:
> Hat sich der Rest, der sich negativ zu diesem Patch geäußert hat, mal
> überlegt, ob das verzögern von DHCP Offers auch eine denkbare
> Alternative wäre? Siehe letzte Mail von mir.

Ich hatte mich negativ geäußert, weil ich das System noch ohne GW-
Selection kenne und das war alles andere als toll.

Deine Argumente habe ich mMn alle ausser Kraft gesetzt. Mein Vorschlag,
das Thema mit der Refresh rate auf andere Weise zu verbessern wurde
nicht weiter verfolgt, geschweige denn kommentiert.

Du hast noch keine Alternative gezeigt. Der Vorschlag es _irgendwie_
mit TC zu machen klingt erstmal unausgegohren und letztlich riecht das 
auch total nach pfusch. Wer weiß, welche Probleme dabei auftreten.

So lange es keine bessere Alternative gibt würde ich das auf keinen
Fall einfach weg nehmen. Wenn es eine gibt, dann kann man überlegen ob
man umsteigt.

Letztlich ist mir das aber total egal, ob das in den zentral
verwalteten Hoods drin ist oder nicht.

Tim

> Gruß
> Fabian
> 
> On 02.01.19 10:07, mail at adrianschmutzler.de wrote:
> > Hallo,
> > 
> >  
> > 
> > sieht so aus, als hätten wir 2 zu 1 für den Patch.
> > 
> >  
> > 
> > Dazu kommt Tims Kommentar, der eher gegen den Patch war sowie meine
> > Tendenz den Patch zu verfolgen.
> > 
> >  
> > 
> > Entsprechend ist die (knappe) Mehrheit dafür und Fabian kann eine
> > „richtige“ Version schicken.
> > 
> >  
> > 
> > Dann bleibt auch noch ein bisschen Zeit für Vetos.
> > 
> >  
> > 
> > Beste Grüße
> > 
> >  
> > 
> > Adrian
> > 
> >  
> > 
> > *From:*Fabian Bläse [mailto:fabian at blaese.de]
> > *Sent:* Sonntag, 30. Dezember 2018 17:55
> > *To:* mail at adrianschmutzler.de; franken-dev at freifunk.net
> > *Subject:* Re: [RFC PATCH] fff-batman-adv: Disable batman gw-
> > selection
> > 
> >  
> > 
> > Ich bin dafür es rauszunehmen. Wenn man wirklich einen DHCP über
> > den anderen priorisieren möchte, könnte man das mit vermutlich
> > weniger fehleranfälligen tc Regeln machen. Manche DHCP Server
> > können die Offers glaube ich auch von sich aus verzögern.
> > 
> > Dann spart man sich sowohl das regelmäßig auf den Routern laufende
> > Skript, als auch das geskripte, ob noch Leases frei sind und
> > entsprechendes Deaktivieren des gateway-announcements, falls nicht.
> > 
> > Dann müsste man die IP-Netze auch nicht mehr zwingend doppelt so
> > groß machen, wie maximal Clients zu erwarten sind.
> > 
> > Gruß
> > Fabian
> > 
> > On 29.12.18 15:23, mail at adrianschmutzler.de <mailto:mail at adrianschm
> > utzler.de> wrote:
> > > Hallo zusammen,
> > > 
> > > die Diskussion zu diesem Patch ist glaube ich soweit durch, ich
> > > würde es für sinnvoll erachten, wenn nun jeder kurz mitteilt, ob
> > > er dafür ist, diesen Patch weiter zu verfolgen oder lieber beim
> > > bestehenden System bleibt.
> > > 
> > > Grüße
> > > 
> > > Adrian
> > > 
> > > > -----Original Message-----
> > > > From: franken-dev [mailto:franken-dev-bounces at freifunk.net] On
> > > > Behalf
> > > > Of Fabian Bläse
> > > > Sent: Sonntag, 9. Dezember 2018 16:07
> > > > To: franken-dev at freifunk.net <mailto:franken-dev at freifunk.net>
> > > > Subject: [RFC PATCH] fff-batman-adv: Disable batman gw-
> > > > selection
> > > > 
> > > > For our centralized setup, batmans gateway selection makes way
> > > > more
> > > > problems than it solves for various reasons.
> > > > 
> > > > Mainly a broken DHCP server is not recognized by it, therefore
> > > > nodes might
> > > > select a gateway with a broken dhcp server.
> > > > Routers have to run a cronjob every minute to reevaluate
> > > > gateway metrics
> > > > because of weird refresh behaviour with specific client modes.
> > > > 
> > > > Also, gateway selection violates the OSI model by tampering
> > > > with protocols
> > > > on a different layer.
> > > > 
> > > > When disabling it, every DHCP Server will reply to a clients
> > > > request and the
> > > > client decides which offer it is going to use. Typically the
> > > > first response is
> > > > used.
> > > > 
> > > > Signed-off-by: Fabian Bläse <fabian at blaese.de <mailto:fabian at bl
> > > > aese.de>>
> > > > ---
> > > >   .../fff/fff-batman-adv/files/etc/uci-defaults/93-fff-batman-
> > > > adv | 2 --
> > > >   .../fff/fff-batman-adv/files/usr/lib/micron.d/fff-batman-
> > > > adv    | 1 -
> > > >   2 files changed, 3 deletions(-)
> > > >   delete mode 100644 src/packages/fff/fff-batman-
> > > > adv/files/usr/lib/micron.d/fff-batman-adv
> > > > 
> > > > diff --git a/src/packages/fff/fff-batman-adv/files/etc/uci-
> > > > defaults/93-fff-
> > > > batman-adv b/src/packages/fff/fff-batman-adv/files/etc/uci-
> > > > defaults/93-fff-
> > > > batman-adv
> > > > index f312c49..ad522b5 100644
> > > > --- a/src/packages/fff/fff-batman-adv/files/etc/uci-
> > > > defaults/93-fff-batman-
> > > > adv
> > > > +++ b/src/packages/fff/fff-batman-adv/files/etc/uci-
> > > > defaults/93-fff-batm
> > > > +++ an-adv
> > > > @@ -3,8 +3,6 @@
> > > >   uci batch <<EOF
> > > >     delete batman-adv.bat0
> > > >     set batman-adv.bat0=mesh
> > > > -  set batman-adv.bat0.gw_mode='client'
> > > > -  set batman-adv.bat0.gw_sel_class='1'
> > > >     set batman-adv.bat0.bridge_loop_avoidance='0'
> > > >     set batman-adv.bat0.network_coding='0'
> > > >     set batman-adv.bat0.aggregated_ogms='1'
> > > > diff --git a/src/packages/fff/fff-batman-
> > > > adv/files/usr/lib/micron.d/fff-
> > > > batman-adv b/src/packages/fff/fff-batman-
> > > > adv/files/usr/lib/micron.d/fff-
> > > > batman-adv
> > > > deleted file mode 100644
> > > > index 21c857b..0000000
> > > > --- a/src/packages/fff/fff-batman-
> > > > adv/files/usr/lib/micron.d/fff-batman-adv
> > > > +++ /dev/null
> > > > @@ -1 +0,0 @@
> > > > -*/1 * * * * /usr/sbin/batctl gw off; sleep 1; /usr/sbin/batctl
> > > > gw client
> > > > --
> > > > 2.19.2
> > 
> >  
> > 
> 
> 
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 833 bytes
Beschreibung: This is a digitally signed message part
URL         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20190113/229f4a46/attachment.sig>