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

Submitted by Robert Langhammer on Sept. 10, 2018, 7:48 p.m.

Details

Message ID 20180910194820.16700-3-rlanghammer@web.de
State Superseded
Headers show

Commit Message

Robert Langhammer Sept. 10, 2018, 7:48 p.m. 
and prevent running configurehood multiple times

Signed-off-by: Robert Langhammer <rlanghammer@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

Patch hide | download patch | download mbox 

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

Comments

Tim Niemeyer Sept. 10, 2018, 8:04 p.m. 
Am Montag, den 10.09.2018, 21:48 +0200 schrieb Robert Langhammer:
> and prevent running configurehood multiple times
> 
> Signed-off-by: Robert Langhammer <rlanghammer@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

Lieber sowas nehmen. Damit gibt es nur eine Abhängigkeit auf "lock" und
nicht auf "pidof", "basename", "wc" und "test" (auch wenn manches im
ash fest drin ist.

+lockfile="/var/lock/${0##*/}.lock"
+if ! lock -n "$lockfile"; then
+       echo "Only one instance of $0 allowed."
+       exit 1
+fi
+trap "lock -u \"$lockfile\"" INT TERM EXIT
+

Tim

> +
>  . /usr/share/libubox/jshn.sh
>  . /lib/functions/fff/keyxchange
>  . /lib/functions/fff/network
Robert Langhammer Sept. 10, 2018, 8:18 p.m. 
Am 10.09.2018 um 22:04 schrieb Tim Niemeyer:
> Am Montag, den 10.09.2018, 21:48 +0200 schrieb Robert Langhammer:
>> and prevent running configurehood multiple times
>>
>> Signed-off-by: Robert Langhammer <rlanghammer@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
> Lieber sowas nehmen. Damit gibt es nur eine Abhängigkeit auf "lock" und
> nicht auf "pidof", "basename", "wc" und "test" (auch wenn manches im
> ash fest drin ist.
Wenn "basename", "wc" und "test" fehlen, braucht man die Kiste nicht
mehr einzuschalten. Wie das mit pidof ist, weiss ich grad nicht. Das
gibt sich also nichts, nur mehr Code.

Robert
>
> +lockfile="/var/lock/${0##*/}.lock"
> +if ! lock -n "$lockfile"; then
> +       echo "Only one instance of $0 allowed."
> +       exit 1
> +fi
> +trap "lock -u \"$lockfile\"" INT TERM EXIT
> +
>
> Tim
>
>> +
>>  . /usr/share/libubox/jshn.sh
>>  . /lib/functions/fff/keyxchange
>>  . /lib/functions/fff/network
Tim Niemeyer Sept. 10, 2018, 8:24 p.m. 
Moin Robert

Am Montag, den 10.09.2018, 22:18 +0200 schrieb robert:
> 
> Am 10.09.2018 um 22:04 schrieb Tim Niemeyer:
> > Am Montag, den 10.09.2018, 21:48 +0200 schrieb Robert Langhammer:
> > > and prevent running configurehood multiple times
> > > 
> > > Signed-off-by: Robert Langhammer <rlanghammer@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
> > 
> > Lieber sowas nehmen. Damit gibt es nur eine Abhängigkeit auf "lock"
> > und
> > nicht auf "pidof", "basename", "wc" und "test" (auch wenn manches
> > im
> > ash fest drin ist.
> 
> Wenn "basename", "wc" und "test" fehlen, braucht man die Kiste nicht
> mehr einzuschalten. Wie das mit pidof ist, weiss ich grad nicht. Das
> gibt sich also nichts, nur mehr Code.

Naja.. Aber du rechnest da rum und machst größen vergleiche.. Das muss
doch total Fehleranfällig sein.. Und wenn man ein link auf
configurehood und das startet funktioniert es nicht.

Vielleicht sollten wir das einfach nicht unter sbin packen sondern
irgendwo anders.. Immerhin ist es nicht dafür gedacht manuell
ausgeführt zu werden.

Tim

> 
> Robert
> > 
> > +lockfile="/var/lock/${0##*/}.lock"
> > +if ! lock -n "$lockfile"; then
> > +       echo "Only one instance of $0 allowed."
> > +       exit 1
> > +fi
> > +trap "lock -u \"$lockfile\"" INT TERM EXIT
> > +
> > 
> > Tim
> > 
> > > +
> > >  . /usr/share/libubox/jshn.sh
> > >  . /lib/functions/fff/keyxchange
> > >  . /lib/functions/fff/network
> 
>
Robert Langhammer Sept. 10, 2018, 8:32 p.m. 
Am 10.09.2018 um 22:24 schrieb Tim Niemeyer:
> Moin Robert
>
> Am Montag, den 10.09.2018, 22:18 +0200 schrieb robert:
>> Am 10.09.2018 um 22:04 schrieb Tim Niemeyer:
>>> Am Montag, den 10.09.2018, 21:48 +0200 schrieb Robert Langhammer:
>>>> and prevent running configurehood multiple times
>>>>
>>>> Signed-off-by: Robert Langhammer <rlanghammer@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
>>> Lieber sowas nehmen. Damit gibt es nur eine Abhängigkeit auf "lock"
>>> und
>>> nicht auf "pidof", "basename", "wc" und "test" (auch wenn manches
>>> im
>>> ash fest drin ist.
>> Wenn "basename", "wc" und "test" fehlen, braucht man die Kiste nicht
>> mehr einzuschalten. Wie das mit pidof ist, weiss ich grad nicht. Das
>> gibt sich also nichts, nur mehr Code.
> Naja.. Aber du rechnest da rum und machst größen vergleiche.. Das muss
> doch total Fehleranfällig sein.. Und wenn man ein link auf
> configurehood und das startet funktioniert es nicht.
Alles klar, ein Lock ist der richtige Weg.
>
> Vielleicht sollten wir das einfach nicht unter sbin packen sondern
> irgendwo anders.. Immerhin ist es nicht dafür gedacht manuell
> ausgeführt zu werden.
>
> Tim
>
>> Robert
>>> +lockfile="/var/lock/${0##*/}.lock"
>>> +if ! lock -n "$lockfile"; then
>>> +       echo "Only one instance of $0 allowed."
>>> +       exit 1
>>> +fi
>>> +trap "lock -u \"$lockfile\"" INT TERM EXIT
>>> +
>>>
>>> Tim
>>>
>>>> +
>>>>  . /usr/share/libubox/jshn.sh
>>>>  . /lib/functions/fff/keyxchange
>>>>  . /lib/functions/fff/network
Adrian Schmutzler Sept. 14, 2018, 9:44 p.m. 
Hallo Robert,

der micrond startet mit START=50.

Wenn wir configurehood erst danach ausführen, können wir nicht auf
/tmp/started verzichten.

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.

Auch hier fehlt das Entfernen aus rc.local.

Grüße

Adrian

> -----Original Message-----
> From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf
> Of Robert Langhammer
> Sent: Montag, 10. September 2018 21:48
> To: franken-dev@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@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
Robert Langhammer Sept. 15, 2018, 7:01 a.m. 
Hi Adrian,


Am 14.09.2018 um 23:44 schrieb mail@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.
>
> 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.
>
> Auch hier fehlt das Entfernen aus rc.local.
>
> Grüße
>
> Adrian
>
>> -----Original Message-----
>> From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf
>> Of Robert Langhammer
>> Sent: Montag, 10. September 2018 21:48
>> To: franken-dev@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@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
>
Adrian Schmutzler Sept. 15, 2018, 11:52 a.m. 
Hallo Robert,

siehe unten.

> -----Original Message-----
> From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf
> Of robert
> Sent: Samstag, 15. September 2018 09:02
> To: franken-dev@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@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!

> >
> > 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:

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.

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@freifunk.net] On
> Behalf
> >> Of Robert Langhammer
> >> Sent: Montag, 10. September 2018 21:48
> >> To: franken-dev@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@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
> >
>
Robert Langhammer Sept. 15, 2018, 10:39 p.m. 
Hallo,

Am 15.09.2018 um 13:52 schrieb mail@adrianschmutzler.de:
> Hallo Robert,
>
> siehe unten.
>
>> -----Original Message-----
>> From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf
>> Of robert
>> Sent: Samstag, 15. September 2018 09:02
>> To: franken-dev@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@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@freifunk.net] On
>> Behalf
>>>> Of Robert Langhammer
>>>> Sent: Montag, 10. September 2018 21:48
>>>> To: franken-dev@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@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
>
Adrian Schmutzler Sept. 27, 2018, 8:21 p.m. 
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@freifunk.net] On Behalf
> Of robert
> Sent: Sonntag, 16. September 2018 00:39
> To: franken-dev@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@adrianschmutzler.de:
> > Hallo Robert,
> >
> > siehe unten.
> >
> >> -----Original Message-----
> >> From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On
> Behalf
> >> Of robert
> >> Sent: Samstag, 15. September 2018 09:02
> >> To: franken-dev@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@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@freifunk.net] On
> >> Behalf
> >>>> Of Robert Langhammer
> >>>> Sent: Montag, 10. September 2018 21:48
> >>>> To: franken-dev@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@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
> >
>
Robert Langhammer Sept. 27, 2018, 9:21 p.m. 
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@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@freifunk.net] On Behalf
>> Of robert
>> Sent: Sonntag, 16. September 2018 00:39
>> To: franken-dev@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@adrianschmutzler.de:
>>> Hallo Robert,
>>>
>>> siehe unten.
>>>
>>>> -----Original Message-----
>>>> From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On
>> Behalf
>>>> Of robert
>>>> Sent: Samstag, 15. September 2018 09:02
>>>> To: franken-dev@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@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@freifunk.net] On
>>>> Behalf
>>>>>> Of Robert Langhammer
>>>>>> Sent: Montag, 10. September 2018 21:48
>>>>>> To: franken-dev@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@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
>
Adrian Schmutzler Sept. 30, 2018, 3:10 p.m. 
Hallo Robert,

> >
> > 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 )

Wir haben auch beim nodewatcher das Problem, dass sich manchmal irgendein WiFi-Tool/-Treiber aufhängt.

Dann wird der nodewatcher nicht beendet, sondern hängt in der entsprechenden Zeile (das gleiche, was dann auch configurehood mehrfach parallel startet). Hier macht es Sinn, den nodewatcher kein weiteres Mal zu starten, sodass es keine zusätzliche Aufrufe gibt, die auf das Wifi-Tool feuern.

Das kann aber natürlich alles in einen separaten Patch.

Grüße

Adrian
Adrian Schmutzler Sept. 30, 2018, 3:13 p.m. 
Hallo Robert,

> > 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.

Dieser Patch hat die Zeit von 5-7 auf 2-3 Minuten verkürzt. Aber für den Nutzer (gerade beim ersten Flashen) ist es hilfreich, wenn der Router durch Aktivität der LEDs zeigt, dass er etwas tut.

Grüße

Adrian
Adrian Schmutzler April 12, 2019, 10:55 a.m. 
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@freifunk.net] On Behalf Of
> robert
> Sent: Donnerstag, 27. September 2018 23:22
> To: franken-dev@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@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@freifunk.net] On Behalf
> >> Of robert
> >> Sent: Sonntag, 16. September 2018 00:39
> >> To: franken-dev@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@adrianschmutzler.de:
> >>> Hallo Robert,
> >>>
> >>> siehe unten.
> >>>
> >>>> -----Original Message-----
> >>>> From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On
> >> Behalf
> >>>> Of robert
> >>>> Sent: Samstag, 15. September 2018 09:02
> >>>> To: franken-dev@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@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@freifunk.net] On
> >>>> Behalf
> >>>>>> Of Robert Langhammer
> >>>>>> Sent: Montag, 10. September 2018 21:48
> >>>>>> To: franken-dev@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@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
> >
>
Adrian Schmutzler April 15, 2019, 1:51 p.m. 
Hallo Robert und Rest,

 

ich habe gerade im Patchwork aufgeräumt und die diversen Patches zum Thema /tmp/started, procd statt rc.local etc. als Superseded markiert. Dort ging es teilweise um die Bootreihenfolge, teilweise um /tmp/started, und teilweise um das Locking. Das war inzwischen alles recht durcheinander.

 

Ich fände es erstrebenswert, wenn Robert eine v2 von diesem Patchset schickt. Dabei würde ich

1. das configurenetwork erstmal im rc.local lassen (incl. sleep 3), und es dann später mit meiner (oder einer anderen ordentlichen) Lösung loswerden (Patch 1/4 fiele weg).

2. Das configurehood in einen init.d stecken wie in diesem Patch (locking haben wir ja schon). Ich würde START=96 vorschlagen: rc.local ist START=95, dann läuft es sicher nach configurenetwork, aber nicht viel später. Ich selbst verwende in meiner FW START=98. Später, wenn wir den Boot anderweitig aufgeräumt haben, braucht man dann nur noch die START= ändern.

3. Was geht beim nodewatcher eigtl. kaputt, wenn er vor alfred läuft? Eigentlich kann er nur am Schluss nicht senden und wirft nen Fehler? Ich diesem Fall würde ich mit dem Fehler leben (ist ja im Prinzip die richtige Info, brauchen wir nicht künstlich schaffen) und die Zeile mit /tmp/started dort entfernen. (Patch 3/4 nur Zeile entfernen, kein Check dazu)

4. /tmp/started brauchen wir im Moment noch, damit configurehood (micrond mit START=50) nicht gelegentlich vor configurenetwork (rc.local mit START=95) startet. Ich würde im Moment daher auch erstmal auf Patch 4/4 verzichten. Sobald wir in der Lage sind, configurehood vor START=50 zu verlegen, kann /tmp/started weg. (Ich habe schon mal versucht, configurehood vorzuziehen, das klappt im Moment aber noch nicht ordentlich)

 

Wenn du willst, fände ich es daher schön, wenn du die genannten zwei v2-Patches schicken würdest und wir das schon Mal so weit vom Tisch kriegen.

 

https://pw.freifunk-franken.de/patch/930/

https://pw.freifunk-franken.de/patch/931/

https://pw.freifunk-franken.de/patch/928/

https://pw.freifunk-franken.de/patch/929/

 

Beste Grüße

 

Adrian

 

From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf Of Adrian Schmutzler
Sent: Freitag, 12. April 2019 12:55
To: 'robert' <rlanghammer@web.de>; franken-dev@freifunk.net
Subject: RE: [PATCH 2/4] fff-hoods: move start of configurehood from rc.local to init.d

 

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@freifunk.net] On Behalf Of 
> robert 
> Sent: Donnerstag, 27. September 2018 23:22 
> To: franken-dev@freifunk.net <mailto:franken-dev@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@adrianschmutzler.de <mailto:mail@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@freifunk.net] On Behalf 
> >> Of robert 
> >> Sent: Sonntag, 16. September 2018 00:39 
> >> To: franken-dev@freifunk.net <mailto:franken-dev@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@adrianschmutzler.de <mailto:mail@adrianschmutzler.de> : 
> >>> Hallo Robert, 
> >>> 
> >>> siehe unten. 
> >>> 
> >>>> -----Original Message----- 
> >>>> From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On 
> >> Behalf 
> >>>> Of robert 
> >>>> Sent: Samstag, 15. September 2018 09:02 
> >>>> To: franken-dev@freifunk.net <mailto:franken-dev@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@adrianschmutzler.de <mailto:mail@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@freifunk.net] On 
> >>>> Behalf 
> >>>>>> Of Robert Langhammer 
> >>>>>> Sent: Montag, 10. September 2018 21:48 
> >>>>>> To: franken-dev@freifunk.net <mailto:franken-dev@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@web.de <mailto:rlanghammer@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 
> > 
>