[PATCH] Revert "Added an automatism which will retrive Router-Coordinates from the old Netmon system"

[Michael Fritscher]

Hi,

aus Mails habe ich da andere Infos (du hattest schonmal geschrieben,
dass sie abgeschalten wird, andere meinten, dass sie noch weiterlaufen wird)

Ich wusste bis gestern nichts von der autom. Datenübernahme und dachte,
dass ich das manuell übernehmen musste (ich wusste, dass da mal was
angedacht war, aber nicht, dass es (halbwegs) im Produktivbetrieb war/ist)

Das aktuelle Wiki schweigt sich komplett aus
(https://wiki.freifunk-franken.de/w/Update), und wenn man im IRC fragt
bekommt man unterschiedliche Antworten, "bin mir nicht sicher", oder
Berichte, dass es sehr unzuverlässig läuft.

Das hat zumindest bei mir nicht gerade dazu beigetragen frühzeitig zu
aktualisieren - auch weil ich schon damals keine wirklich guten
Antworten bekommen habe was passiert.

Naja, bei mir ging ja dann trotzdem alles gut, aber es wäre schön, wenn
diejenigen, die so eine tiefgreifende / brechende Änderung vornehmen
diese auch zumindest stichpunktartig dokumentieren, wenns produktiv wird
(z.B. ins Wiki). Im oben angesprochenen Dokument fehlt das Update zu
>0.5.2 nämlich komplett.

Auch das bei einem zukünftigen normalen sysupgrade die Koord etc.
übernommen wird (bei einem -n aber nicht, und früher galt die Regel
"mach lieber -n") ist eine wichtige Info.

Ich mein, ich krieg noch einiges mit und stecke ein wenig in der
Materie, aber vom "Wald und Wiesenbetreiber" kann nicht verlangt werden,
dass er in github reinschauen muss, um zu gucken, obs jetzt solche
Funktionen gibt. Und aktiv nachfragen traut der sich eher weniger.
Sondern lassens im Zweifel bis zum bitteren Ende laufen - tut ja noch.

Deswegen mein Vorschlag: Lasst das APi noch 2 Wochen laufen (läuft ja
grad noch) und schreibt ne Mail "Update ist jetzt noch 2 Wochen einfach
möglich (Daten werden übernommen, und selbst im Worst Case (Default
Hood) findet sich ne Lösung!), danach nur noch aufwendig und die Router
werden danach nicht mehr funktionieren.

Und das wir bei zukünftigen brechenden Änderungen ein wenig mehr
offizielle Doku erstellen und Migrationstests fahren, gerade wenn falls
es schiefgeht die Betreiber gröbere Probleme bekommen (Vor Ort Termin
oder Eingriff von jemanden mit Eingriff in die Default-Hood notwendig).
Die IRC-Diskussionen zeigen, dass da selbst heute noch viel Unklarheit
herrscht.

Viele Grüße,
Michael


P.S.
Bevor "mach doch" kommt: Eigentlich ist das meines Erachtens ne Aufgabe
von denen, die diese Änderungen vornehmen bzw. produktiv nehmen
(Einbinden von anderen konkret erlaubt ;) ). Außerdem stecke ich in
diesem Bereich nicht tief genug drinne und habe lokal auch keine
verbreitete HW (841/1043/4300) zum testen. Das ich persönlich von Anfang
an nicht besonders von überzeugt war trägt vielleicht auch ein wenig
dazu bei, ist an dieser Stelle aber auch nicht das Thema.

Am 12.03.2017 um 00:44 schrieb Tim Niemeyer:
> Hi
> 
> Am 11. März 2017 23:18:25 MEZ schrieb Michael Fritscher <michael at fritscher.net>:
>> hi
>>
>> sollte die api nicht noch bissle laenger gehen?
> 
> Ne, eigentlich nicht. Plan war gestern abzuschalten.
> 
> Tim
> 
> 
>> Michael
>>
>>> Jo, absolut.
>>>
>>> Reviewed-by: Tim Niemeyer <tim at tn-x.org>
>>>
>>> Tim
>>>
>>> Am 11. März 2017 11:59:14 MEZ schrieb Jan Kraus
>>> <mayosemmel at googlemail.com>:
>>>> This reverts commit d6f2f18e4980074e16910cffd92e6ac5e08a69ff.
>>>> The Netmon is gone, so this workaround is not needed any longer.
>>>>
>>>> Signed-off-by: Jan Kraus <mayosemmel at gmail.com>
>>>> ---
>>>> src/packages/fff/fff-sysupgrade/Makefile           |  2 +-
>>>> .../files/usr/lib/micron.d/fff-sysupgrade          |  1 -
>>>> .../files/usr/sbin/copy_netmon_coordinates.sh      | 69
>>>> ----------------------
>>>> 3 files changed, 1 insertion(+), 71 deletions(-)
>>>> delete mode 100644
>>>> src/packages/fff/fff-sysupgrade/files/usr/lib/micron.d/fff-sysupgrade
>>>> delete mode 100755
>>>> src/packages/fff/fff-sysupgrade/files/usr/sbin/copy_netmon_coordinates.sh
>>>>
>>>> diff --git a/src/packages/fff/fff-sysupgrade/Makefile
>>>> b/src/packages/fff/fff-sysupgrade/Makefile
>>>> index d671e0c..f6fb2cf 100644
>>>> --- a/src/packages/fff/fff-sysupgrade/Makefile
>>>> +++ b/src/packages/fff/fff-sysupgrade/Makefile
>>>> @@ -1,7 +1,7 @@
>>>> include $(TOPDIR)/rules.mk
>>>>
>>>> PKG_NAME:=fff-sysupgrade
>>>> -PKG_VERSION:=0.0.3
>>>> +PKG_VERSION:=0.0.4
>>>> PKG_RELEASE:=4
>>>>
>>>> PKG_BUILD_DIR:=$(BUILD_DIR)/fff-sysupgrade
>>>> diff --git
>>>> a/src/packages/fff/fff-sysupgrade/files/usr/lib/micron.d/fff-sysupgrade
>>>> b/src/packages/fff/fff-sysupgrade/files/usr/lib/micron.d/fff-sysupgrade
>>>> deleted file mode 100644
>>>> index 312972f..0000000
>>>> ---
>>>> a/src/packages/fff/fff-sysupgrade/files/usr/lib/micron.d/fff-sysupgrade
>>>> +++ /dev/null
>>>> @@ -1 +0,0 @@
>>>> -*/5 * * * * /usr/sbin/copy_netmon_coordinates.sh
>>>> \ No newline at end of file
>>>> diff --git
>>>> a/src/packages/fff/fff-sysupgrade/files/usr/sbin/copy_netmon_coordinates.sh
>>>> b/src/packages/fff/fff-sysupgrade/files/usr/sbin/copy_netmon_coordinates.sh
>>>> deleted file mode 100755
>>>> index 1d042e4..0000000
>>>> ---
>>>> a/src/packages/fff/fff-sysupgrade/files/usr/sbin/copy_netmon_coordinates.sh
>>>> +++ /dev/null
>>>> @@ -1,69 +0,0 @@
>>>> -#!/bin/sh
>>>> -
>>>> -delete_myself()
>>>> -{
>>>> -	echo "This Script will be deleted now!"
>>>> -	rm -f /usr/lib/micron.d/fff-sysupgrade
>>>> -	rm -f "$0"
>>>> -	exit 0
>>>> -}
>>>> -
>>>> -#Get Mac Address of br-mesh if already up
>>>> -if ! mac=$( cat /sys/class/net/br-mesh/address ); then
>>>> -	echo "Interface br-mesh is not available!"
>>>> -	exit 1
>>>> -fi
>>>> -
>>>> -#Check if Coordinates are already set
>>>> -if uci get system. at system[0].latitude && uci get
>>>> system. at system[0].longitude; then
>>>> -	echo "Coordinates are already set."
>>>> -	delete_myself
>>>> -fi
>>>> -
>>>> -#Get Router from Netmon Database
>>>> -xml_data=$( wget -q -O -
>>>> "http://fe80::ff:feee:1%br-mesh/api/rest/api.php?rquest=router&mac=${mac//:}"
>>>> 2>&1)
>>>> -
>>>> -if [ -z "$xml_data" ]; then
>>>> -	echo "xml_data is not set"
>>>> -	exit 1
>>>> -elif echo "$xml_data"|grep "can't connect to remote host" ;then
>>>> -	echo "The Netmon Server can't be reached."
>>>> -	exit 1
>>>> -elif echo "$xml_data"|grep "HTTP/1.1 404 Not Found" ;then
>>>> -	echo "This Router is not present in the Netmon Database."
>>>> -	delete_myself
>>>> -fi
>>>> -
>>>> -#Reduce XML_DATA to Router only
>>>> -xml_data=$( echo $xml_data |grep -o '<router>.*<\/router>' |sed -e
>>>> 's/<user>.*<\/user>//g' |sed -e 's/<chipset>.*<\/chipset>//g' |sed -e
>>>> 's/<chipset>.*<\/chipset>//g' )
>>>> -
>>>> -#Get needed Variables
>>>> -hostname=$( echo $xml_data |grep -o  '<hostname>.*<\/hostname>'|sed
>> -e
>>>> 's/<\/\?hostname>//g' )
>>>> -description=$( echo $xml_data |grep -o
>>>> '<description>.*<\/description>'|sed -e 's/<\/\?description>//g' )
>>>> -latitude=$( echo $xml_data |grep -o  '<latitude>.*<\/latitude>'|sed
>> -e
>>>> 's/<\/\?latitude>//g' )
>>>> -longitude=$( echo $xml_data |grep -o 
>> '<longitude>.*<\/longitude>'|sed
>>>> -e 's/<\/\?longitude>//g' )
>>>> -
>>>> -#Check for netmon default coordinates
>>>> -if [ "$latitude" -eq 0 ] && [ "$longitude" -eq 0 ]; then
>>>> -	echo "Unable to retrieve coordinates from Netmon."
>>>> -	echo "Maybe the coordinates are suppressed."
>>>> -	delete_myself
>>>> -elif [ -z "$latitude" ] || [ -z "$longitude" ]; then
>>>> -    echo "No coordinates, looks like an error"
>>>> -    exit 1
>>>> -fi
>>>> -
>>>> -uci set system. at system[0]=system
>>>> -echo "Setting hostname $hostname"
>>>> -uci set system. at system[0].hostname="$hostname"
>>>> -echo "Setting description $description"
>>>> -uci set system. at system[0].description="$description"
>>>> -echo "Setting latitude $latitude"
>>>> -uci set system. at system[0].latitude="$latitude"
>>>> -echo "Setting longitude $longitude"
>>>> -uci set system. at system[0].longitude="$longitude"
>>>> -uci commit
>>>> -
>>>> -echo "Coordinates are now copied from Netmon. Router will be
>>>> rebooted."
>>>> -reboot
>>>> -exit 0
>>>> \ No newline at end of file