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

Submitted by Jan Kraus on March 11, 2017, 10:59 a.m.

Details

Message ID 1489229954-4220-1-git-send-email-mayosemmel@gmail.com
State Accepted
Headers show

Commit Message

Jan Kraus March 11, 2017, 10:59 a.m.
This reverts commit d6f2f18e4980074e16910cffd92e6ac5e08a69ff.
The Netmon is gone, so this workaround is not needed any longer.

Signed-off-by: Jan Kraus <mayosemmel@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

Patch hide | download patch | download mbox

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.@system[0].latitude && uci get system.@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.@system[0]=system
-echo "Setting hostname $hostname"
-uci set system.@system[0].hostname="$hostname"
-echo "Setting description $description"
-uci set system.@system[0].description="$description"
-echo "Setting latitude $latitude"
-uci set system.@system[0].latitude="$latitude"
-echo "Setting longitude $longitude"
-uci set system.@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

Comments

Tim Niemeyer March 11, 2017, 5:21 p.m.
Jo, absolut.

Reviewed-by: Tim Niemeyer <tim@tn-x.org>

Tim

Am 11. März 2017 11:59:14 MEZ schrieb Jan Kraus <mayosemmel@googlemail.com>:
>This reverts commit d6f2f18e4980074e16910cffd92e6ac5e08a69ff.
>The Netmon is gone, so this workaround is not needed any longer.
>
>Signed-off-by: Jan Kraus <mayosemmel@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.@system[0].latitude && uci get
>system.@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.@system[0]=system
>-echo "Setting hostname $hostname"
>-uci set system.@system[0].hostname="$hostname"
>-echo "Setting description $description"
>-uci set system.@system[0].description="$description"
>-echo "Setting latitude $latitude"
>-uci set system.@system[0].latitude="$latitude"
>-echo "Setting longitude $longitude"
>-uci set system.@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
Michael Fritscher March 11, 2017, 10:18 p.m.
hi

sollte die api nicht noch bissle laenger gehen?

Michael

> Jo, absolut.
>
> Reviewed-by: Tim Niemeyer <tim@tn-x.org>
>
> Tim
>
> Am 11. März 2017 11:59:14 MEZ schrieb Jan Kraus
> <mayosemmel@googlemail.com>:
>>This reverts commit d6f2f18e4980074e16910cffd92e6ac5e08a69ff.
>>The Netmon is gone, so this workaround is not needed any longer.
>>
>>Signed-off-by: Jan Kraus <mayosemmel@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.@system[0].latitude && uci get
>>system.@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.@system[0]=system
>>-echo "Setting hostname $hostname"
>>-uci set system.@system[0].hostname="$hostname"
>>-echo "Setting description $description"
>>-uci set system.@system[0].description="$description"
>>-echo "Setting latitude $latitude"
>>-uci set system.@system[0].latitude="$latitude"
>>-echo "Setting longitude $longitude"
>>-uci set system.@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
Tim Niemeyer March 11, 2017, 11:44 p.m.
Hi

Am 11. März 2017 23:18:25 MEZ schrieb Michael Fritscher <michael@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@tn-x.org>
>>
>> Tim
>>
>> Am 11. März 2017 11:59:14 MEZ schrieb Jan Kraus
>> <mayosemmel@googlemail.com>:
>>>This reverts commit d6f2f18e4980074e16910cffd92e6ac5e08a69ff.
>>>The Netmon is gone, so this workaround is not needed any longer.
>>>
>>>Signed-off-by: Jan Kraus <mayosemmel@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.@system[0].latitude && uci get
>>>system.@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.@system[0]=system
>>>-echo "Setting hostname $hostname"
>>>-uci set system.@system[0].hostname="$hostname"
>>>-echo "Setting description $description"
>>>-uci set system.@system[0].description="$description"
>>>-echo "Setting latitude $latitude"
>>>-uci set system.@system[0].latitude="$latitude"
>>>-echo "Setting longitude $longitude"
>>>-uci set system.@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
Michael Fritscher March 12, 2017, 9:21 a.m.
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@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@tn-x.org>
>>>
>>> Tim
>>>
>>> Am 11. März 2017 11:59:14 MEZ schrieb Jan Kraus
>>> <mayosemmel@googlemail.com>:
>>>> This reverts commit d6f2f18e4980074e16910cffd92e6ac5e08a69ff.
>>>> The Netmon is gone, so this workaround is not needed any longer.
>>>>
>>>> Signed-off-by: Jan Kraus <mayosemmel@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.@system[0].latitude && uci get
>>>> system.@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.@system[0]=system
>>>> -echo "Setting hostname $hostname"
>>>> -uci set system.@system[0].hostname="$hostname"
>>>> -echo "Setting description $description"
>>>> -uci set system.@system[0].description="$description"
>>>> -echo "Setting latitude $latitude"
>>>> -uci set system.@system[0].latitude="$latitude"
>>>> -echo "Setting longitude $longitude"
>>>> -uci set system.@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
Steffen Pankratz April 6, 2017, 7:21 p.m.
On 03/11/2017 11:59 AM, Jan Kraus wrote:

Hi Jan

> This reverts commit d6f2f18e4980074e16910cffd92e6ac5e08a69ff.
> The Netmon is gone, so this workaround is not needed any longer.

Leider nicht ganz :(
Wenn man sich
https://github.com/FreifunkFranken/firmware/commit/d6f2f18e4980074e16910cffd92e6ac5e08a69ff
anschaut
wurde da micrond als Abhaengigkeit hinzugefuegt, welche durch diesen
Patch hier nicht entfernt wird.
Aus meiner Sicht braucht man diese Abhaengigkeit nicht mehr, wie siehst
du es?


Gruss
-Steffen
Tobias Klaus May 29, 2017, 8:50 p.m.
Hey,

Steffen hat recht, wir die Abhängigkeit sollte nicht mehr benötigt werden.

@Jan: schickst du den Patch nochmal, oder soll ich ihn beim einspielen 
anpassen?

Viele Grüeß
Am Donnerstag, 6. April 2017, 21:21:59 CEST schrieb Steffen Pankratz:
> On 03/11/2017 11:59 AM, Jan Kraus wrote:
> 
> Hi Jan
> 
> > This reverts commit d6f2f18e4980074e16910cffd92e6ac5e08a69ff.
> > The Netmon is gone, so this workaround is not needed any longer.
> 
> Leider nicht ganz :(
> Wenn man sich
> https://github.com/FreifunkFranken/firmware/commit/d6f2f18e4980074e16910cffd
> 92e6ac5e08a69ff anschaut
> wurde da micrond als Abhaengigkeit hinzugefuegt, welche durch diesen
> Patch hier nicht entfernt wird.
> Aus meiner Sicht braucht man diese Abhaengigkeit nicht mehr, wie siehst
> du es?
> 
> 
> Gruss
> -Steffen
Adrian Schmutzler June 1, 2017, 11:30 a.m.
Hallo,

Tested-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>

Firmware mit Patch installiert auf WR841N v10 (nur Mesh), CPE210 (nur Mesh)
und AR150 (Mesh und WAN).

Grüße

Adrian

-----Original Message-----
From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf Of
Tobias Klaus
Sent: Montag, 29. Mai 2017 22:50
To: franken-dev@freifunk.net; Jan Kraus <mayosemmel@googlemail.com>
Subject: Re: [PATCH] Revert "Added an automatism which will retrive
Router-Coordinates from the old Netmon system"

Hey,

Steffen hat recht, wir die Abhängigkeit sollte nicht mehr benötigt werden.

@Jan: schickst du den Patch nochmal, oder soll ich ihn beim einspielen
anpassen?

Viele Grüeß
Am Donnerstag, 6. April 2017, 21:21:59 CEST schrieb Steffen Pankratz:
> On 03/11/2017 11:59 AM, Jan Kraus wrote:
>
> Hi Jan
>
> > This reverts commit d6f2f18e4980074e16910cffd92e6ac5e08a69ff.
> > The Netmon is gone, so this workaround is not needed any longer.
>
> Leider nicht ganz :(
> Wenn man sich
>
https://github.com/FreifunkFranken/firmware/commit/d6f2f18e4980074e16910cf
fd
> 92e6ac5e08a69ff anschaut
> wurde da micrond als Abhaengigkeit hinzugefuegt, welche durch diesen 
> Patch hier nicht entfernt wird.
> Aus meiner Sicht braucht man diese Abhaengigkeit nicht mehr, wie 
> siehst du es?
>
>
> Gruss
> -Steffen


--
franken-dev mailing list
franken-dev@freifunk.net
http://lists.freifunk.net/mailman/listinfo/franken-dev-freifunk.net
Tim Niemeyer July 2, 2017, 4:12 p.m.
Am Montag, den 29.05.2017, 22:50 +0200 schrieb Tobias Klaus:
> Hey,
> 
> Steffen hat recht, wir die Abhängigkeit sollte nicht mehr benötigt werden.
> 
> @Jan: schickst du den Patch nochmal, oder soll ich ihn beim einspielen 
> anpassen?
Ich hab das jetzt beim applien gefixt. War hin und her gerissen, ob ich
den Patch dann nochmal als v2 über die Liste kippe, aber jetzt haben wir
eine kleine Baustelle weniger.

Tim

> 
> Viele Grüeß
> Am Donnerstag, 6. April 2017, 21:21:59 CEST schrieb Steffen Pankratz:
> > On 03/11/2017 11:59 AM, Jan Kraus wrote:
> > 
> > Hi Jan
> > 
> > > This reverts commit d6f2f18e4980074e16910cffd92e6ac5e08a69ff.
> > > The Netmon is gone, so this workaround is not needed any longer.
> > 
> > Leider nicht ganz :(
> > Wenn man sich
> > https://github.com/FreifunkFranken/firmware/commit/d6f2f18e4980074e16910cffd
> > 92e6ac5e08a69ff anschaut
> > wurde da micrond als Abhaengigkeit hinzugefuegt, welche durch diesen
> > Patch hier nicht entfernt wird.
> > Aus meiner Sicht braucht man diese Abhaengigkeit nicht mehr, wie siehst
> > du es?
> > 
> > 
> > Gruss
> > -Steffen
> 
>