[PATCH 2/4] fff-hoods: move start of configurehood from rc.local to init.d

Adrian Schmutzler mail at adrianschmutzler.de
Fr Apr 12 12:55:18 CEST 2019


Hallo Robert,

da ich gerade versuche, den Start umzubauen, bin ich wieder auf deinen Patchvorschlag gestoßen.

Tatsächlich wäre es wohl gar nicht dumm, den Start von configurehood einfach auf START=30 oder START=40 vorzulegen, und dann sind wir /tmp/started los.

Allerdings habe ich beim Durchgucken der Dateien festgestellt, dass /etc/init.d/fastd ein START=95 enthält. Weiß jemand, ob das relevant wäre, so wie wir fastd im Moment benutzen?

Grüße

Adrian

> -----Original Message-----
> From: franken-dev [mailto:franken-dev-bounces at freifunk.net] On Behalf Of
> robert
> Sent: Donnerstag, 27. September 2018 23:22
> To: franken-dev at freifunk.net
> Subject: Re: [PATCH 2/4] fff-hoods: move start of configurehood from rc.local to
> init.d
> 
> Hallo Adrian,
> 
> alles gut, ich hatte die letzten Wochen auch keine Zeit irgend was zu
> machen.
> 
> Der ganze Systemstart ist irgendwie nicht zufriedenstellend. Das sieht
> man auch an den Beitraegen zu dem Patchset. Ich moechte mir das alles
> nochmal anschauen und Ideen sammeln, wenn ich wieder mehr Zeit habe.
> 
> Im Moment finde ich die Variante alles als Dienst zu definieren recht
> vielversprechend. Habs aber noch nicht zu Ende gedacht. Und der procd
> birgt auch noch Geheimnisse.
> 
> Störend ist auch, dass wir um OpenWrt herum bauen. Ich vermute aber,
> dass der Aufwand ein eigenes Startkonzept zu pflegen größer ist.
> 
> Robert
> 
> 
> 
> Am 27.09.2018 um 22:21 schrieb mail at adrianschmutzler.de:
> > Hallo Robert,
> >
> > nur das du nicht denkst, du wirst ignoriert:
> >
> > Ich finde das Patchset interessant, und wenn ich irgendwann mal Zeit habe,
> > werde ich den ganzen Spaß mal durchtesten.
> >
> > Im Moment ist es aber einfach schwierig.
> >
> > Grüße
> >
> > Adrian
> >
> >> -----Original Message-----
> >> From: franken-dev [mailto:franken-dev-bounces at freifunk.net] On Behalf
> >> Of robert
> >> Sent: Sonntag, 16. September 2018 00:39
> >> To: franken-dev at freifunk.net
> >> Subject: Re: [PATCH 2/4] fff-hoods: move start of configurehood from
> >> rc.local to init.d
> >>
> >> Hallo,
> >>
> >> Am 15.09.2018 um 13:52 schrieb mail at adrianschmutzler.de:
> >>> Hallo Robert,
> >>>
> >>> siehe unten.
> >>>
> >>>> -----Original Message-----
> >>>> From: franken-dev [mailto:franken-dev-bounces at freifunk.net] On
> >> Behalf
> >>>> Of robert
> >>>> Sent: Samstag, 15. September 2018 09:02
> >>>> To: franken-dev at freifunk.net
> >>>> Subject: Re: [PATCH 2/4] fff-hoods: move start of configurehood from
> >>>> rc.local to init.d
> >>>>
> >>>> Hi Adrian,
> >>>>
> >>>>
> >>>> Am 14.09.2018 um 23:44 schrieb mail at adrianschmutzler.de:
> >>>>> Hallo Robert,
> >>>>>
> >>>>> der micrond startet mit START=50.
> >>>>>
> >>>>> Wenn wir configurehood erst danach ausführen, können wir nicht auf
> >>>>> /tmp/started verzichten.
> >>>> Meinst du wegen Nodewatcher? configurehood schuetzt sich selbst.
> >>> Du hast Recht, aufgrund des frühen Starts von configurenetwork und des
> >> einmaligen Ausführens müsste es so gehen.
> >>> Es würde dann Sinn machen, die gleiche Sperre für doppeltes Ausführen
> >> auch in den Nodewatcher einzubauen!
> >> Für den Nodewatcher sehe ich die Notwendigkeit nicht. Der wird nur via
> >> cron
> >> gestartet. Sollte eigentlich nie doppelt laufen. Wenn doch noch eine
> >> Instanz
> >> vorhanden ist, liegt der Fehler wo anders. Das Einzige, was passieren
> >> kann,
> >> dass beim Booten der Nodewatcher zu früh startet. Ich denke aber nicht,
> >> dass das etwas kaputt macht. Ich bin noch am Überlegen, ob es Sinn macht,
> >> den Nodewatcher als Daemon laufen zu lassen, von procd überwacht. Man
> >> könnte auch einfach das sleep im cronjob austauschen ( random 300 329 )
> >>>>> Ich weiß nicht, ob wir alles nötige haben, um configurehood vor
> >>>>> START=50 auszuführen.
> >>>>>
> >>>>> Und wir hätten wieder das Problem, dass dann ggf. mehrere Minuten
> >>>>> lang die LEDs nicht gehen, weil configurehood beim random hängt.
> >>>> Welches random? Wenn /init.d/configurehood startet gibt es nur 2
> >>>> Moeglichkeiten. Es laeuft an, oder es laeuft schon.
> >>> Wir hatten/haben das Problem, dass der Teil mit dem vpn-select mehrere
> >> Minuten braucht (ich glaube da ging es um das Generieren des Keys oder
> >> so,
> >> was etwas gebraucht hat wegen Entropie). Die LEDs werden mit START=96
> >> gestartet. D.h. wenn das Skript lange braucht, dann sind solange die LEDs
> >> nicht verfügbar. Ich habe aus diesem Grund damals vorgeschlagen (und
> >> verwende das auch so), dass man configurehood mit START=98 startet:
> >> Soweit ich das sehe, passiert da nicht viel. Die LEDs gehen an. Darum
> >> wäre
> >> mir das egal ob 95 oder 98.
> >> Wenn configurehood wirklich zu lange braucht, sollten wir uns das noch
> >> mal
> >> anschauen. Am Generieren des fastd-keys sollte es nicht liegen. Da gab es
> >> einen Patch:  https://pw.freifunk-franken.de/patch/687/ Und das passiert
> >> nur beim Ersten Boot nach dem Flashen.
> >>> https://pw.freifunk-franken.de/patch/764/
> >>>
> >>> Das ist jetzt aber eigtl. kein Problem deines Patches, das war vorher
> >>> auch
> >> schon so. Ich fände aber ein START=98 hier besser.
> >>> Gelegentlich wird dann halt das configurehood mal vom micrond mit
> >> START=50 gestartet werden. Keine Ahnung, ob bei 50 schon alles
> >> funktioniert. Das könnte man aber einfach testen, vll. mach ich mir bei
> >> Gelegenheit mal die Arbeit.
> >> Bei START=50 sollte alles schon da sein, was configurehood braucht. Das
> >> init-
> >> Skript könnte man eigentlich auch weg lassen. Es ist ja nur dazu da, dass
> >> es
> >> nicht so lange dauert, bis configurehood startet.
> >>
> >> Robert
> >>> Essenz: Die Kommentare von mir diesbezügilch aus der vorigen Mail sind
> >> erstmal hinfällig. Das mit rc.local fände ich nach wie vor wichtig, das
> >> mit
> >> START=98 ist ein Vorschlag.
> >>> Grüße
> >>>
> >>> Adrian
> >>>
> >>>>> Auch hier fehlt das Entfernen aus rc.local.
> >>>>>
> >>>>> Grüße
> >>>>>
> >>>>> Adrian
> >>>>>
> >>>>>> -----Original Message-----
> >>>>>> From: franken-dev [mailto:franken-dev-bounces at freifunk.net] On
> >>>> Behalf
> >>>>>> Of Robert Langhammer
> >>>>>> Sent: Montag, 10. September 2018 21:48
> >>>>>> To: franken-dev at freifunk.net
> >>>>>> Subject: [PATCH 2/4] fff-hoods: move start of configurehood from
> >>>>>> rc.local
> >>>>> to
> >>>>>> init.d
> >>>>>>
> >>>>>> and prevent running configurehood multiple times
> >>>>>>
> >>>>>> Signed-off-by: Robert Langhammer <rlanghammer at web.de>
> >>>>>> ---
> >>>>>>  src/packages/fff/fff-hoods/files/etc/init.d/configurehood   | 9
> >>>> +++++++++
> >>>>>>  src/packages/fff/fff-hoods/files/usr/lib/micron.d/fff-hoods | 2 +-
> >>>>>>  src/packages/fff/fff-hoods/files/usr/sbin/configurehood     | 4 ++++
> >>>>>>  3 files changed, 14 insertions(+), 1 deletion(-)  create mode
> >>>>>> 100755 src/packages/fff/fff-hoods/files/etc/init.d/configurehood
> >>>>>>
> >>>>>> diff --git
> >>>>>> a/src/packages/fff/fff-hoods/files/etc/init.d/configurehood
> >>>>>> b/src/packages/fff/fff-hoods/files/etc/init.d/configurehood
> >>>>>> new file mode 100755
> >>>>>> index 0000000..61fe735
> >>>>>> --- /dev/null
> >>>>>> +++ b/src/packages/fff/fff-hoods/files/etc/init.d/configurehood
> >>>>>> @@ -0,0 +1,9 @@
> >>>>>> +#!/bin/sh /etc/rc.common
> >>>>>> +
> >>>>>> +START=95
> >>>>>> +
> >>>>>> +boot()
> >>>>>> +{
> >>>>>> +    /usr/sbin/configurehood
> >>>>>> +}
> >>>>>> +
> >>>>>> diff --git
> >>>>>> a/src/packages/fff/fff-hoods/files/usr/lib/micron.d/fff-hoods
> >>>>>> b/src/packages/fff/fff-hoods/files/usr/lib/micron.d/fff-hoods
> >>>>>> index 39e800e..ca8d798 100644
> >>>>>> --- a/src/packages/fff/fff-hoods/files/usr/lib/micron.d/fff-hoods
> >>>>>> +++ b/src/packages/fff/fff-hoods/files/usr/lib/micron.d/fff-hoods
> >>>>>> @@ -1 +1 @@
> >>>>>> -*/5 * * * * [ -f /tmp/started ] && /usr/sbin/configurehood
> >>>>>> +*/5 * * * * /usr/sbin/configurehood
> >>>>>> diff --git
> >>>>>> a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> >>>>>> b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> >>>>>> index 47e228b..7a330d9 100755
> >>>>>> --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> >>>>>> +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> >>>>>> @@ -1,5 +1,9 @@
> >>>>>>  #!/bin/sh
> >>>>>>
> >>>>>> +# prevent multiple runs of the script
> >>>>>> +
> >>>>>> +[ $( pidof $( basename $0 )  | wc -w ) -gt 2 ] && exit
> >>>>>> +
> >>>>>>  . /usr/share/libubox/jshn.sh
> >>>>>>  . /lib/functions/fff/keyxchange
> >>>>>>  . /lib/functions/fff/network
> >>>>>> --
> >>>>>> 2.11.0
> >
> 
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : nicht verfügbar
Dateityp    : application/pgp-signature
Dateigröße  : 834 bytes
Beschreibung: nicht verfügbar
URL         : <https://{'listname': 'franken-dev-freifunk.net', 'hostname': 'lists.freifunk.net'}/pipermail/franken-dev-freifunk.net/attachments/20190412/7ac428aa/attachment.sig>


Mehr Informationen über die Mailingliste franken-dev