freifunk-franken
/
firmware
9
7
Fork 13
Code Issues 48 Pull Requests 7 Releases 17 Wiki Activity

fff-layer3: Add latency to nodewatcher #193

Closed
ChristianD wants to merge 1 commits from ChristianD/firmware:nodewatcherlatency into master
pull from: ChristianD/firmware:nodewatcherlatency
merge into: freifunk-franken:master
Conversation 11 Commits 1 Files Changed 1 +19
1 changed files with 19 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
src/packages/fff/fff-layer3/files/usr/lib/nodewatcher.d/90-latency.sh Executable file
View File

@ -0,0 +1,19 @@
#!/bin/sh
ipv4dest=$(uci -q get fff.latency.ipv4)
ipv6dest=$(uci -q get fff.latency.ipv6)
rohammer commented 2022-03-11 14:33:47 +01:00
Review
Copy Link

Es gibt doch ein /etc/config/nodewatcher. Dort wäre das doch besser aufgehoben.

In fff ist das nicht so schön.

Robert

Es gibt doch ein /etc/config/nodewatcher. Dort wäre das doch besser aufgehoben. In fff ist das nicht so schön. Robert
ChristianD commented 2022-03-11 15:16:29 +01:00
Review
Copy Link

Prinzipiell hast du recht,

Prinzipiell will ich aber eigentlich alles, was ein User konfiguriert mal auf /etc/config/fff zusammen gefasst haben, siehe auch hier:

#118 (comment)

wir sollten uns da mal auf einen Konsens einigen sonst stolpern wir da ständig drüber ;)

Prinzipiell hast du recht, Prinzipiell will ich aber eigentlich alles, was ein User konfiguriert mal auf /etc/config/fff zusammen gefasst haben, siehe auch hier: https://git.freifunk-franken.de/freifunk-franken/firmware/issues/118#issuecomment-2558 wir sollten uns da mal auf einen Konsens einigen sonst stolpern wir da ständig drüber ;)
fbl commented 2022-03-11 15:24:35 +01:00
Review
Copy Link

/etc/config/nodewatcher ist aktuell nicht update-sicher.

/etc/config/nodewatcher ist aktuell nicht update-sicher.
rohammer commented 2022-03-11 15:34:55 +01:00
Review
Copy Link

Ich waere dafuer das zu aendern. Hatte da mal interfaces drin blacklisted. Nervt schon, wenn das jedesmal weg ist.

Ich waere dafuer das zu aendern. Hatte da mal interfaces drin blacklisted. Nervt schon, wenn das jedesmal weg ist.
fbl commented 2022-03-11 17:27:43 +01:00
Review
Copy Link

Dafür müssten wir allerdings erstmal sämtliche Parameter dort entfernen, die nicht dafür vorgesehen sind, persistent zu sein (mesh_interface, die vorgegebenen Interfaces für black- und whitelist, status_text_file und data_file).

Dafür müssten wir allerdings erstmal sämtliche Parameter dort entfernen, die nicht dafür vorgesehen sind, persistent zu sein (mesh_interface, die vorgegebenen Interfaces für black- und whitelist, status_text_file und data_file).
ChristianD commented 2022-03-11 18:11:47 +01:00
Review
Copy Link

#118 (comment)

ich verweise nochmal darauf. Wollen wir jetzt vom User einstellbare config an einer Stelle haben oder nicht? Das ist mMn erstmal die grundlegende Frage.

https://git.freifunk-franken.de/freifunk-franken/firmware/issues/118#issuecomment-2558 ich verweise nochmal darauf. Wollen wir jetzt vom User einstellbare config an einer Stelle haben oder nicht? Das ist mMn erstmal die grundlegende Frage.
fbl commented 2022-03-15 13:15:18 +01:00
Review
Copy Link

Auch wenn dies später möglicherweise eine Migration erforderlich macht: Ich würde das aktuell mal zurückstellen und erstmal in config/fff liegen lassen. Wie wir das langfristig bauen wollen, können wir in #118 weiter verfolgen.

Auch wenn dies später möglicherweise eine Migration erforderlich macht: Ich würde das aktuell mal zurückstellen und erstmal in config/fff liegen lassen. Wie wir das langfristig bauen wollen, können wir in #118 weiter verfolgen.
if [ -n "$ipv4dest" ] || [ -n "$ipv6dest" ] ; then
printf "<latency>"
if [ -n "$ipv4dest" ] ; then
ipv4latency=$(ping -qc3 -4 $ipv4dest 2>&1 | awk -F'/' 'END{ print (/^round-trip/? $4:"0") }')
printf "<ipv4latency>$ipv4latency</ipv4latency><ipv4dest>$ipv4dest</ipv4dest>"
fi
if [ -n "$ipv6dest" ] ; then
ipv6latency=$(ping -qc3 -6 $ipv6dest 2>&1 | awk -F'/' 'END{ print (/^round-trip/? $4:"0") }')
printf "<ipv6latency>$ipv6latency</ipv6latency><ipv6dest>$ipv6dest</ipv6dest>"
fi
printf "</latency>"
fi
exit 0