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

This patch adds json to the mqtt monitoring package. #216

Open
McUles wants to merge 6 commits from McUles/firmware:master into master
pull from: McUles/firmware:master
merge into: freifunk-franken:master
Conversation 7 Commits 6 Files Changed 11 +1 -1
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit e11cb2d012 - Show all commits

2
src/packages/fff/fff-mqtt-monitoring/files/usr/sbin/json-mqtt-proxy
View File

@ -3,5 +3,5 @@ MACADDR=$(cat /sys/class/net/br-client/address | /bin/sed 's/://g')
SCRIPT_DATA_FILE=$(uci get nodewatcher.@script[0].data_file).json
if [ -n "$(uci -q get fff.mqtt.server)" ] ; then
mosquitto_pub -h "$(uci get fff.mqtt.server)" -p 1883 -t /monitoring/v2/$MACADDR -m "$SCRIPT_DATA_FILE"
mosquitto_pub -h "$(uci get fff.mqtt.server)" -p 1883 -t /monitoring/v2/$MACADDR -m "$(cat $SCRIPT_DATA_FILE)"
jkimmel commented 2022-03-07 11:27:34 +01:00
Outdated
Review
Copy Link

Das -m "$(cat $SCRIPT_DATA_FILE)" ist vermutlich nicht so geschickt. Irgendwann ist nämlich auch mal der Puffer für Kommandozeile voll :)

Meine lokale man page sagt:

       -f, --file
           Send the contents of a file as the message.

Kann das das openwrt mosquitto_pub vielleicht auch?
Alternativ gibt es auch noch:

       -s, --stdin-file
           Send a message read from stdin, sending the entire content as a
           single message.

Damit könnte man sich das zwischenspeichern in einer Datei eventuell komplett sparen.

Das `-m "$(cat $SCRIPT_DATA_FILE)"` ist vermutlich nicht so geschickt. Irgendwann ist nämlich auch mal der Puffer für Kommandozeile voll :) Meine lokale man page sagt: ``` -f, --file Send the contents of a file as the message. ``` Kann das das openwrt `mosquitto_pub` vielleicht auch? Alternativ gibt es auch noch: ``` -s, --stdin-file Send a message read from stdin, sending the entire content as a single message. ``` Damit könnte man sich das zwischenspeichern in einer Datei eventuell komplett sparen.
McUles commented 2022-03-07 11:34:40 +01:00
Outdated
Review
Copy Link

Die Datei direkt übergeben mit -f funktioniert.
Einsparen der Datei wird nicht so einfach funktionieren weil die einzelnen Scripts die Datei öffnen und ergänzen.
Hier werden auch in vorherige Nodes Daten eingefügt.
Wenn es hier noch eine schlauere Idee gibt, gerne.

Die Datei direkt übergeben mit -f funktioniert. Einsparen der Datei wird nicht so einfach funktionieren weil die einzelnen Scripts die Datei öffnen und ergänzen. Hier werden auch in vorherige Nodes Daten eingefügt. Wenn es hier noch eine schlauere Idee gibt, gerne.
ChristianD commented 2022-03-07 14:59:56 +01:00
Outdated
Review
Copy Link

Ich würde die Datei gerne behalten. Es kann ja sein das jemand (also irgendein Script o.ä.) anders auch an die json ran will und was damit tun will und nicht nur der mqtt. Wir sollten Nodewatcher-json und mqtt mMn getrennt halten.

Eventuell sollte man daraus sogar 2 Patches machen:

  1. das ganze json-nodewatcher Zeug => was am Ende die Datei nach /tmp/irgendwas legt
  2. nodewatcherjson-to-mqtt-proxy => was die Datei /tmp/irgendwas holt und ins mqtt stopft.

(Name frei erfunden, gerne anpassen ;))
Im Moment sind die Patches ja eh ein ziemliches durcheinander und sollten noch sortiert werden.

Mittelfristig müsste man dann überlegen ob man dem Monitoring wieder ein API verpasst wo man die json hinwirft oder ob das Monitoring auch einfach auf den mqtt zugreift und sich dort die Daten holt, mir wäre letztes deutlich lieber weil man dann weniger in der Firmware pflegen muss.

Ich würde die Datei gerne behalten. Es kann ja sein das jemand (also irgendein Script o.ä.) anders auch an die json ran will und was damit tun will und nicht nur der mqtt. Wir sollten Nodewatcher-json und mqtt mMn getrennt halten. Eventuell sollte man daraus sogar 2 Patches machen: 1) das ganze json-nodewatcher Zeug => was am Ende die Datei nach /tmp/irgendwas legt 2) nodewatcherjson-to-mqtt-proxy => was die Datei /tmp/irgendwas holt und ins mqtt stopft. (Name frei erfunden, gerne anpassen ;)) Im Moment sind die Patches ja eh ein ziemliches durcheinander und sollten noch sortiert werden. Mittelfristig müsste man dann überlegen ob man dem Monitoring wieder ein API verpasst wo man die json hinwirft oder ob das Monitoring auch einfach auf den mqtt zugreift und sich dort die Daten holt, mir wäre letztes deutlich lieber weil man dann weniger in der Firmware pflegen muss.
fi