Message ID | 20190423160908.28420-2-fabian@blaese.de |
---|---|
State | Accepted |
Headers | show |
diff --git a/src/packages/fff/fff-gateway/files/etc/gateway.d/10-meta b/src/packages/fff/fff-gateway/files/etc/gateway.d/10-meta new file mode 100644 index 0000000..3b8c9d2 --- /dev/null +++ b/src/packages/fff/fff-gateway/files/etc/gateway.d/10-meta @@ -0,0 +1,37 @@ +configure() { + ## set hostname + if name=$(uci -q get fff.system.hostname); then + uci -q set system.@system[0].hostname="$name" + else + echo "WARNING: No hostname set!" + fi + + ## set contact + if ! contact=$(uci -q get fff.system.contact); then + echo "WARNING: No contact set!" + fi + + ## set location + if ! lat=$(uci -q get fff.system.latitude) || ! long=$(uci -q get fff.system.longitude); then + echo "WARNING: No location set!" + fi + + ## hoodname + if hood=$(uci -q get fff.system.hoodname); then + echo "Setting hood name: $hood" + uci -q set "system.@system[0].hood=$hood" + else + echo "WARNING: No hoodname set!" + fi +} + +apply() { + uci commit system + uci commit fff + echo "$(uci get "system.@system[0].hostname")" > /proc/sys/kernel/hostname +} + +revert() { + uci revert system + uci revert fff +}
Hallo Fabian, da bist du mir ja weit entgegen gekommen. Trotzdem leider eine Anmerkung und eine Frage unten: > -----Original Message----- > From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf Of > Fabian Bläse > Sent: Dienstag, 23. April 2019 18:09 > To: franken-dev@freifunk.net > Subject: [PATCH v3 2/3] gateway.d: Add configuration script for meta > information > > This adds a configuration script for applying hostname, > coordinates, hoodname and contact address. > > Signed-off-by: Fabian Bläse <fabian@blaese.de> > --- > Changes in v3: > - Use meta information from /etc/config/fff > - rename 01-meta to 10-meta > --- > .../fff-gateway/files/etc/gateway.d/10-meta | 37 +++++++++++++++++++ > 1 file changed, 37 insertions(+) > create mode 100644 src/packages/fff/fff-gateway/files/etc/gateway.d/10- > meta > > diff --git a/src/packages/fff/fff-gateway/files/etc/gateway.d/10-meta > b/src/packages/fff/fff-gateway/files/etc/gateway.d/10-meta > new file mode 100644 > index 0000000..3b8c9d2 > --- /dev/null > +++ b/src/packages/fff/fff-gateway/files/etc/gateway.d/10-meta > @@ -0,0 +1,37 @@ > +configure() { > + ## set hostname > + if name=$(uci -q get fff.system.hostname); then > + uci -q set system.@system[0].hostname="$name" > + else > + echo "WARNING: No hostname set!" > + fi > + > + ## set contact > + if ! contact=$(uci -q get fff.system.contact); then Anmerkung: So wie du das jetzt nutzt, könnte man auf die Variablenzuweisung verzichten und einfach nur das uci get prüfen. Würde ich aber für nen Review auch so akzeptieren. > + echo "WARNING: No contact set!" > + fi > + > + ## set location > + if ! lat=$(uci -q get fff.system.latitude) || ! long=$(uci -q get > fff.system.longitude); then Selbe Anmerkung > + echo "WARNING: No location set!" > + fi > + > + ## hoodname > + if hood=$(uci -q get fff.system.hoodname); then Hmm, hostname und Location sind Eigenschaften des Routers/Gerät, aber der Hoodname ist eine Eigenschaft der Hood. Den hätte ich daher nicht nach /etc/config/fff getan. Der gehört meines Erachtens ins hoodfile. Da du ja aber meintest, du wolltest vom Hoodfile unabhängig sein, wäre mir hier dann /etc/config/gateway noch lieber. Am liebsten wäre mir aber, wenn das hier gar nicht steht, sondern man das irgendwo prüft, wo man Sachen macht, die mit der Hood zu tun haben. Wenn du das trotzdem hier machen willst, könnte ich mit /etc/config/gateway leben. Zur Not kann man das später ja auch nochmal rumschieben, wenn man das größere Bild besser sieht. Beste Grüße Adrian > + echo "Setting hood name: $hood" > + uci -q set "system.@system[0].hood=$hood" > + else > + echo "WARNING: No hoodname set!" > + fi > +} > + > +apply() { > + uci commit system > + uci commit fff > + echo "$(uci get "system.@system[0].hostname")" > > /proc/sys/kernel/hostname > +} > + > +revert() { > + uci revert system > + uci revert fff > +} > -- > 2.21.0
Hallo Adrian On 24.04.19 00:13, Adrian Schmutzler wrote: > Hallo Fabian, > > da bist du mir ja weit entgegen gekommen. War mir wie gesagt auch selbst ein Dorn im Auge.. >> +configure() { >> + ## set hostname >> + if name=$(uci -q get fff.system.hostname); then >> + uci -q set system.@system[0].hostname="$name" >> + else >> + echo "WARNING: No hostname set!" >> + fi >> + >> + ## set contact >> + if ! contact=$(uci -q get fff.system.contact); then > > Anmerkung: > So wie du das jetzt nutzt, könnte man auf die Variablenzuweisung verzichten und einfach nur das uci get prüfen. > Würde ich aber für nen Review auch so akzeptieren. Könnte man, ja. Schaden tuts jetzt aber auch nicht. >> + echo "WARNING: No location set!" >> + fi >> + >> + ## hoodname >> + if hood=$(uci -q get fff.system.hoodname); then > > Hmm, hostname und Location sind Eigenschaften des Routers/Gerät, aber der Hoodname ist eine Eigenschaft der Hood. Den hätte ich daher nicht nach /etc/config/fff getan. > Der gehört meines Erachtens ins hoodfile. Da du ja aber meintest, du wolltest vom Hoodfile unabhängig sein, wäre mir hier dann /etc/config/gateway noch lieber. Am liebsten wäre mir aber, wenn das hier gar nicht steht, sondern man das irgendwo prüft, wo man Sachen macht, die mit der Hood zu tun haben. > Wenn du das trotzdem hier machen willst, könnte ich mit /etc/config/gateway leben. Zur Not kann man das später ja auch nochmal rumschieben, wenn man das größere Bild besser sieht. Bisher weiß ich keinen besseren Platz. Das ist eigentlich auch nur für den Nodewatcher und das Monitoring. Lass das erstmal so übernehmen, bis zu einem ersten Release können wir das immer noch rumschieben, wie wir wollen. Gruß Fabian
Hallo Fabian, kann ich mit leben, hängt für mich aber das Etikett „bitte ändern“ dran. Reviewed-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> Grüße Adrian From: Fabian Bläse [mailto:fabian@blaese.de] Sent: Mittwoch, 24. April 2019 10:26 To: Adrian Schmutzler <mail@adrianschmutzler.de>; franken-dev@freifunk.net Subject: Re: [PATCH v3 2/3] gateway.d: Add configuration script for meta information Hallo Adrian On 24.04.19 00:13, Adrian Schmutzler wrote: > Hallo Fabian, > > da bist du mir ja weit entgegen gekommen. War mir wie gesagt auch selbst ein Dorn im Auge.. >> +configure() { >> + ## set hostname >> + if name=$(uci -q get fff.system.hostname); then >> + uci -q set system.@system[0].hostname="$name <mailto:system.@system[0].hostname=%22$name> " >> + else >> + echo "WARNING: No hostname set!" >> + fi >> + >> + ## set contact >> + if ! contact=$(uci -q get fff.system.contact); then > > Anmerkung: > So wie du das jetzt nutzt, könnte man auf die Variablenzuweisung verzichten und einfach nur das uci get prüfen. > Würde ich aber für nen Review auch so akzeptieren. Könnte man, ja. Schaden tuts jetzt aber auch nicht. >> + echo "WARNING: No location set!" >> + fi >> + >> + ## hoodname >> + if hood=$(uci -q get fff.system.hoodname); then > > Hmm, hostname und Location sind Eigenschaften des Routers/Gerät, aber der Hoodname ist eine Eigenschaft der Hood. Den hätte ich daher nicht nach /etc/config/fff getan. > Der gehört meines Erachtens ins hoodfile. Da du ja aber meintest, du wolltest vom Hoodfile unabhängig sein, wäre mir hier dann /etc/config/gateway noch lieber. Am liebsten wäre mir aber, wenn das hier gar nicht steht, sondern man das irgendwo prüft, wo man Sachen macht, die mit der Hood zu tun haben. > Wenn du das trotzdem hier machen willst, könnte ich mit /etc/config/gateway leben. Zur Not kann man das später ja auch nochmal rumschieben, wenn man das größere Bild besser sieht. Bisher weiß ich keinen besseren Platz. Das ist eigentlich auch nur für den Nodewatcher und das Monitoring. Lass das erstmal so übernehmen, bis zu einem ersten Release können wir das immer noch rumschieben, wie wir wollen. Gruß Fabian
Gibt es hierzu noch weitere Rückmeldungen? Ansonsten würde ich diesen Patch so wie er ist mit einem Review applien. Gruß Fabian On 23.04.19 18:09, Fabian Bläse wrote: > This adds a configuration script for applying hostname, > coordinates, hoodname and contact address. > > Signed-off-by: Fabian Bläse <fabian@blaese.de> > --- > Changes in v3: > - Use meta information from /etc/config/fff > - rename 01-meta to 10-meta > --- > .../fff-gateway/files/etc/gateway.d/10-meta | 37 +++++++++++++++++++ > 1 file changed, 37 insertions(+) > create mode 100644 src/packages/fff/fff-gateway/files/etc/gateway.d/10-meta > > diff --git a/src/packages/fff/fff-gateway/files/etc/gateway.d/10-meta b/src/packages/fff/fff-gateway/files/etc/gateway.d/10-meta > new file mode 100644 > index 0000000..3b8c9d2 > --- /dev/null > +++ b/src/packages/fff/fff-gateway/files/etc/gateway.d/10-meta > @@ -0,0 +1,37 @@ > +configure() { > + ## set hostname > + if name=$(uci -q get fff.system.hostname); then > + uci -q set system.@system[0].hostname="$name" > + else > + echo "WARNING: No hostname set!" > + fi > + > + ## set contact > + if ! contact=$(uci -q get fff.system.contact); then > + echo "WARNING: No contact set!" > + fi > + > + ## set location > + if ! lat=$(uci -q get fff.system.latitude) || ! long=$(uci -q get fff.system.longitude); then > + echo "WARNING: No location set!" > + fi > + > + ## hoodname > + if hood=$(uci -q get fff.system.hoodname); then > + echo "Setting hood name: $hood" > + uci -q set "system.@system[0].hood=$hood" > + else > + echo "WARNING: No hoodname set!" > + fi > +} > + > +apply() { > + uci commit system > + uci commit fff > + echo "$(uci get "system.@system[0].hostname")" > /proc/sys/kernel/hostname > +} > + > +revert() { > + uci revert system > + uci revert fff > +} >
Hi Am Dienstag, den 23.04.2019, 18:09 +0200 schrieb Fabian Bläse: > This adds a configuration script for applying hostname, > coordinates, hoodname and contact address. > > Signed-off-by: Fabian Bläse <fabian@blaese.de> > --- > Changes in v3: > - Use meta information from /etc/config/fff > - rename 01-meta to 10-meta > --- > .../fff-gateway/files/etc/gateway.d/10-meta | 37 > +++++++++++++++++++ > 1 file changed, 37 insertions(+) > create mode 100644 src/packages/fff/fff- > gateway/files/etc/gateway.d/10-meta > > diff --git a/src/packages/fff/fff-gateway/files/etc/gateway.d/10-meta > b/src/packages/fff/fff-gateway/files/etc/gateway.d/10-meta > new file mode 100644 > index 0000000..3b8c9d2 > --- /dev/null > +++ b/src/packages/fff/fff-gateway/files/etc/gateway.d/10-meta > @@ -0,0 +1,37 @@ > +configure() { > + ## set hostname > + if name=$(uci -q get fff.system.hostname); then > + uci -q set system.@system[0].hostname="$name" > + else > + echo "WARNING: No hostname set!" > + fi > + > + ## set contact > + if ! contact=$(uci -q get fff.system.contact); then > + echo "WARNING: No contact set!" > + fi > + > + ## set location > + if ! lat=$(uci -q get fff.system.latitude) || ! long=$(uci > -q get fff.system.longitude); then > + echo "WARNING: No location set!" > + fi > + > + ## hoodname > + if hood=$(uci -q get fff.system.hoodname); then > + echo "Setting hood name: $hood" > + uci -q set "system.@system[0].hood=$hood" > + else > + echo "WARNING: No hoodname set!" > + fi Ich verstehe nicht, wofür wir zu dieser Zeit ein hoodname brauchen. Auch im dritten Patch taucht das nicht auf. Hostname, Kontakt, Location ergeben für mich noch Sinn, da man diese vielleicht später mal in Richtung Monitoring kippen möchte. Aber hoodname? Hat ja eigentlich nix mit dem Router zu tun. Wenn da noch ein Patch in der Schublade schlummert, wo das dann benötigt wird, wäre es sauberer das auch erst später hinzuzufügen. Ich würde mich aber auch mit dieser unschönheit zufrieden geben, wenn das einfach so in den Gesamtplan passt. Wenn da allerdings nichts mehr kommt, dann sollte es mMn eher raus. Tim > +} > + > +apply() { > + uci commit system > + uci commit fff > + echo "$(uci get "system.@system[0].hostname")" > > /proc/sys/kernel/hostname > +} > + > +revert() { > + uci revert system > + uci revert fff > +}
Hallo Tim, On 08.05.19 21:28, Tim Niemeyer wrote: > Am Dienstag, den 23.04.2019, 18:09 +0200 schrieb Fabian Bläse: >> + ## hoodname >> + if hood=$(uci -q get fff.system.hoodname); then >> + echo "Setting hood name: $hood" >> + uci -q set "system.@system[0].hood=$hood" >> + else >> + echo "WARNING: No hoodname set!" >> + fi > Ich verstehe nicht, wofür wir zu dieser Zeit ein hoodname brauchen. > Auch im dritten Patch taucht das nicht auf. > > Hostname, Kontakt, Location ergeben für mich noch Sinn, da man diese > vielleicht später mal in Richtung Monitoring kippen möchte. Aber > hoodname? Hat ja eigentlich nix mit dem Router zu tun. Aus genau dem gleichen Grund ist da auch der Hoodname. Der Nodewatcher nimmt sich diesen von system.@system[0].hood, daher wird er dorthin kopiert. Was das Monitoring tut, wenn man keinen Hoodnamen mitsendet.. Weiß ich nicht. Da das aktuell aber der Mechanismus ist um Nodes, die zusammen gehören (APs, Router, ..) zusammenzufassen, ist das denke ich schon sinnvoll, das hier zu haben. Setzt man das nicht, gibts ja auch nur eine Warnung. Schlussendlich möchte man diesen Teil der Konfiguration vermutlich später mal (oder auch gleich?) in den Nodewatcher auslagern. Gruß Fabian
Am Mittwoch, den 08.05.2019, 21:35 +0200 schrieb Fabian Bläse: > Hallo Tim, > > On 08.05.19 21:28, Tim Niemeyer wrote: > > Am Dienstag, den 23.04.2019, 18:09 +0200 schrieb Fabian Bläse: > > > + ## hoodname > > > + if hood=$(uci -q get fff.system.hoodname); then > > > + echo "Setting hood name: $hood" > > > + uci -q set "system.@system[0].hood=$hood" > > > + else > > > + echo "WARNING: No hoodname set!" > > > + fi > > > > Ich verstehe nicht, wofür wir zu dieser Zeit ein hoodname brauchen. > > Auch im dritten Patch taucht das nicht auf. > > > > Hostname, Kontakt, Location ergeben für mich noch Sinn, da man > > diese > > vielleicht später mal in Richtung Monitoring kippen möchte. Aber > > hoodname? Hat ja eigentlich nix mit dem Router zu tun. > > Aus genau dem gleichen Grund ist da auch der Hoodname. > Der Nodewatcher nimmt sich diesen von system.@system[0].hood, daher > wird er dorthin kopiert. > > Was das Monitoring tut, wenn man keinen Hoodnamen mitsendet.. Weiß > ich nicht. > Da das aktuell aber der Mechanismus ist um Nodes, die zusammen > gehören (APs, Router, ..) zusammenzufassen, ist das denke ich schon > sinnvoll, das hier zu haben. > Setzt man das nicht, gibts ja auch nur eine Warnung. > > Schlussendlich möchte man diesen Teil der Konfiguration vermutlich > später mal (oder auch gleich?) in den Nodewatcher auslagern. Ah ok. Hm.. Glücklich bin ich damit zwar nicht, aber wir sollten das auf keinen Fall jetzt machen.. Soooo wichtig ist das halt nicht! Ich denke, wenn man sagt hoodname = monitoring_group, dann passt das schon in etwa. Geraderücken kann man das dann immer noch. Reviewed-by: Tim Niemeyer <tim@tn-x.org> Tim > > Gruß > Fabian >
applied.
This adds a configuration script for applying hostname, coordinates, hoodname and contact address. Signed-off-by: Fabian Bläse <fabian@blaese.de> --- Changes in v3: - Use meta information from /etc/config/fff - rename 01-meta to 10-meta --- .../fff-gateway/files/etc/gateway.d/10-meta | 37 +++++++++++++++++++ 1 file changed, 37 insertions(+) create mode 100644 src/packages/fff/fff-gateway/files/etc/gateway.d/10-meta