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

Tim Niemeyer tim at tn-x.org
Mi Jun 13 21:41:12 CEST 2018


Am Mittwoch, den 13.06.2018, 14:05 +0200 schrieb Adrian Schmutzler:
> Man kann ja beides kombinieren. In jedem Fall hat es keinen Nachteil,
> die Gleichzeitigkeit aufzuheben.
Ja, kannst von mir aus mit dem Random Dings machen.

Diese komische "Zeit" im Cron verwirrt am Ende nur.

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

Tim

> 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 blaes
> > e.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-frei
> > > > > > funk.net
> 
> 
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 488 bytes
Beschreibung: This is a digitally signed message part
URL         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20180613/25c71cab/attachment.sig>


Mehr Informationen über die Mailingliste franken-dev