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

[Adrian Schmutzler]

Man kann ja beides kombinieren. In jedem Fall hat es keinen Nachteil, die Gleichzeitigkeit aufzuheben.

Nebenbei werden die generierten Daten besser interpretierbar, z.B. ist die Zahl der Prozesse im Monitoring nach dem Patch viel glatter, was halt schöner zum debuggen ist.

Grüße

Adrian

> -----Original Message-----
> From: franken-dev [mailto:franken-dev-bounces at freifunk.net] On Behalf
> Of Tim Niemeyer
> Sent: Mittwoch, 13. Juni 2018 11:01
> To: Fabian Bläse <fabian at blaese.de>
> Cc: Adrian Schmutzler <freifunk at adrianschmutzler.de>; franken-dev
> <franken-dev at freifunk.net>
> Subject: Re: [PATCH] fff-hoods: Do not run configurehood and nodewatcher
> at the same time
> 
> Hi
> 
> Am 13. Juni 2018 10:55:32 MESZ schrieb "Fabian Bläse" <fabian at blaese.de>:
> >Hallo Tim,
> >
> >dann ist vielleicht die Beschreibung unpassend. Ich sehe das nicht als
> >Problem, wenn sich Daten während des Auslesens ändern, sonst müsste
> man
> >das anders fixen.
> >
> >Ich würde das aber trotzdem aufnehmen, um die ganzen Skripte zeitlich
> >ein wenig voneinander zu trennen, um ggf. den Router zu entlasten.
> >Schaden würde selbiges nämlich nicht.
> 
> Ja, aber nicht so. Die Pakete sollen unabhängig sein. Also wenn dann das
> random ding dafür verwenden.
> 
> Tim
> >
> >Fabian
> >
> >> On 13. Jun 2018, at 10:49, Tim Niemeyer <tim at tn-x.org> wrote:
> >>
> >> 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