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

Fabian Bläse fabian at blaese.de
Mi Jan 9 23:41:05 CET 2019


Hallo Adrian,

eine knappe Mehrheit finde ich bei so einer grundlegenden Änderung für etwas dürftig, da würde ich mal nichts überstürzen.

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.

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 adrianschmutzler.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 blaese.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: OpenPGP digital signature
URL         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20190109/897e5a7d/attachment.sig>


Mehr Informationen über die Mailingliste franken-dev