fff-babeld: Clarify warning about missing ipv4 peering address #75

fbl · 2021-01-11T12:10:03+01:00

fbl commented

2021-01-11 12:10:03 +01:00

When neither peer_ip nor ipaddr are set, no ipv4 address for peering
interfaces is available. Therefore, no IPv4 routes can be advertised.

Other than that, a configuration like this is perfectly valid and
configuration is already continued. Therefore, the "FATAL" message might
be misleading so it is replaced with "WARNING" and clarified slightly.

Signed-off-by: Fabian Bläse fabian@blaese.de

When neither peer_ip nor ipaddr are set, no ipv4 address for peering interfaces is available. Therefore, no IPv4 routes can be advertised. Other than that, a configuration like this is perfectly valid and configuration is already continued. Therefore, the "FATAL" message might be misleading so it is replaced with "WARNING" and clarified slightly. Signed-off-by: Fabian Bläse <fabian@blaese.de>

fbl added this to the 20210211-beta milestone 2021-01-11 12:10:06 +01:00

fbl added the

layer3

trivial

labels 2021-01-11 12:11:06 +01:00

fbl force-pushed peerip from d1bf5fef27 to f1137cafa5

2021-01-11 12:12:31 +01:00

Compare

fbl requested review from jkimmel 2021-01-11 12:13:47 +01:00

jkimmel reviewed 2021-01-11 12:43:58 +01:00

src/packages/fff/fff-babeld/files/lib/functions/fff/babel

						
				@ -41,3 +41,3 @@

						uci add_list "$option"=$(echo $ipaddr | cut -d / -f1)

					else

						echo "FATAL: Neither peer_ip nor ipaddr set! No peering ipv4 set!"

						echo "WARNING: Neither peer_ip nor ipaddr set! IPv4 routing is not possible."

jkimmel commented

2021-01-11 12:43:58 +01:00

Ich glaube "Native IPv4 routing is not possible" wäre noch ein ticken bisser, oder?
Vielleicht waere auch ein Hinweis gut, dass falls das Absicht ist, man eventuell an den DNS64 denken soll?

Ich glaube "Native IPv4 routing is not possible" wäre noch ein ticken bisser, oder? Vielleicht waere auch ein Hinweis gut, dass falls das Absicht ist, man eventuell an den DNS64 denken soll?

fbl commented

2021-01-11 12:55:28 +01:00

"WARNING: Neither peer_ip nor ipaddr set! Native IPv4 routing is not possible. Using a DNS64-capable DNS server is recommended."

Allerdings bin ich mir bei dem Hinweis nicht sicher, ob ich das (aktuell!) wirklich da haben will. Wenn man so ein Setup aktuell baut, dann weiß man entweder, was man tut, oder man sollte es (aktuell) vermeiden..

"WARNING: Neither peer_ip nor ipaddr set! Native IPv4 routing is not possible. Using a DNS64-capable DNS server is recommended." Allerdings bin ich mir bei dem Hinweis nicht sicher, ob ich das (aktuell!) wirklich da haben will. Wenn man so ein Setup aktuell baut, dann weiß man entweder, was man tut, oder man sollte es (aktuell) vermeiden..

jkimmel commented

2021-01-11 13:29:04 +01:00

Also das mit Native... find ich weiterhin gut. Formulierung fuer den DNS64 find ich auch gelungen.

Aber ja, eigentlich hast du recht, dass das momentan nur wirklich passiert, wenn man das mit Absicht macht und dann ists klar wegen DNS64.

Ich bin auch ohne dem letzten Satz einverstanden. Vielleicht hat jemanden anderes noch eine Meinung dazu.

Also das mit `Native...` find ich weiterhin gut. Formulierung fuer den `DNS64` find ich auch gelungen. Aber ja, eigentlich hast du recht, dass das momentan nur wirklich passiert, wenn man das mit Absicht macht und dann ists klar wegen `DNS64`. Ich bin auch ohne dem letzten Satz einverstanden. Vielleicht hat jemanden anderes noch eine Meinung dazu.

jkimmel marked this conversation as resolved

jkimmel approved these changes 2021-01-11 13:29:29 +01:00

jkimmel commented

2021-01-11 13:42:42 +01:00

Reviewed-by: Johannes Kimmel <fff@bareminimum.eu>

`Reviewed-by: Johannes Kimmel <fff@bareminimum.eu>`

adschm added the

packages/fff

label 2021-01-13 21:50:56 +01:00

adschm commented

2021-01-13 21:51:03 +01:00

Wird gleich gemergt.