[PATCH] fff-hoods: Do not run configurehood and nodewatcher at the same time

Tim Niemeyer tim at tn-x.org
Mi Jun 13 10:49:19 CEST 2018


Hi

Am 13. Juni 2018 10:45:18 MESZ schrieb "Fabian Bläse" <fabian at blaese.de>:
>Hübsch wäre es, sollte aber nicht dazu “missbraucht” werden,
>Nebenläufigkeitsprobleme zu “lösen”.
>Ist aber soweit ich das sehe nicht der Fall.

Also ist es unnötig.

Laut der Beschreibung soll es ja genau so ein Problem lösen.

Tim

>
>Reviewed-by: Fabian Bläse <fabian at blaese.de>
>
>Fabian
>
>> On 8. Mar 2018, at 21:31, Adrian Schmutzler
><freifunk at adrianschmutzler.de> wrote:
>> 
>> nodewatcher and configurehood have both been run with */5 in micrond.
>> 
>> This may lead to nodewatcher accessing information just being
>> changed by configurehood. To prevent this, we just change the
>> start of configurehood, as this has no known disadvantages.
>> 
>> A change of the nodewatcher start would also solve the problem,
>> but nodewatcher is synchronized to the alfred-master and the
>> Monitoring.
>> 
>> Fixes #96.
>> 
>> Signed-off-by: Adrian Schmutzler <freifunk at adrianschmutzler.de>
>> ---
>> src/packages/fff/fff-hoods/files/usr/lib/micron.d/fff-hoods | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>> 
>> 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..c2f56bb 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
>> +3-59/5 * * * * [ -f /tmp/started ] && /usr/sbin/configurehood
>> --
>> 2.7.4
>> 
>> --
>> franken-dev mailing list
>> franken-dev at freifunk.net
>> http://lists.freifunk.net/mailman/listinfo/franken-dev-freifunk.net


Mehr Informationen über die Mailingliste franken-dev