| Message ID | 1520541066-4448-1-git-send-email-freifunk@adrianschmutzler.de |
|---|---|
| State | Accepted |
| Headers | show |
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
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. Reviewed-by: Fabian Bläse <fabian@blaese.de> Fabian > On 8. Mar 2018, at 21:31, Adrian Schmutzler <freifunk@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@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@freifunk.net > http://lists.freifunk.net/mailman/listinfo/franken-dev-freifunk.net
Hi Am 13. Juni 2018 10:45:18 MESZ schrieb "Fabian Bläse" <fabian@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@blaese.de> > >Fabian > >> On 8. Mar 2018, at 21:31, Adrian Schmutzler ><freifunk@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@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@freifunk.net >> http://lists.freifunk.net/mailman/listinfo/franken-dev-freifunk.net
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. Fabian > On 13. Jun 2018, at 10:49, Tim Niemeyer <tim@tn-x.org> wrote: > > Hi > > Am 13. Juni 2018 10:45:18 MESZ schrieb "Fabian Bläse" <fabian@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@blaese.de> >> >> Fabian >> >>> On 8. Mar 2018, at 21:31, Adrian Schmutzler >> <freifunk@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@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@freifunk.net >>> http://lists.freifunk.net/mailman/listinfo/franken-dev-freifunk.net
Hi Am 13. Juni 2018 10:55:32 MESZ schrieb "Fabian Bläse" <fabian@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@tn-x.org> wrote: >> >> Hi >> >> Am 13. Juni 2018 10:45:18 MESZ schrieb "Fabian Bläse" ><fabian@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@blaese.de> >>> >>> Fabian >>> >>>> On 8. Mar 2018, at 21:31, Adrian Schmutzler >>> <freifunk@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@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@freifunk.net >>>> http://lists.freifunk.net/mailman/listinfo/franken-dev-freifunk.net
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@freifunk.net] On Behalf > Of Tim Niemeyer > Sent: Mittwoch, 13. Juni 2018 11:01 > To: Fabian Bläse <fabian@blaese.de> > Cc: Adrian Schmutzler <freifunk@adrianschmutzler.de>; franken-dev > <franken-dev@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@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@tn-x.org> wrote: > >> > >> Hi > >> > >> Am 13. Juni 2018 10:45:18 MESZ schrieb "Fabian Bläse" > ><fabian@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@blaese.de> > >>> > >>> Fabian > >>> > >>>> On 8. Mar 2018, at 21:31, Adrian Schmutzler > >>> <freifunk@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@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@freifunk.net > >>>> http://lists.freifunk.net/mailman/listinfo/franken-dev-freifunk.net
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@freifunk.net] On > > Behalf > > Of Tim Niemeyer > > Sent: Mittwoch, 13. Juni 2018 11:01 > > To: Fabian Bläse <fabian@blaese.de> > > Cc: Adrian Schmutzler <freifunk@adrianschmutzler.de>; franken-dev > > <franken-dev@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@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@tn-x.org> wrote: > > > > > > > > Hi > > > > > > > > Am 13. Juni 2018 10:45:18 MESZ schrieb "Fabian Bläse" > > > > > > <fabian@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@blaese.de> > > > > > > > > > > Fabian > > > > > > > > > > > On 8. Mar 2018, at 21:31, Adrian Schmutzler > > > > > > > > > > <freifunk@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@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@freifunk.net > > > > > > http://lists.freifunk.net/mailman/listinfo/franken-dev-frei > > > > > > funk.net > >
(Schon vor ein paar Tagen) Applied. From: Fabian Bläse [mailto:fabian@blaese.de] Sent: Mittwoch, 13. Juni 2018 10:45 To: Adrian Schmutzler <freifunk@adrianschmutzler.de> Cc: franken-dev <franken-dev@freifunk.net> Subject: Re: [PATCH] fff-hoods: Do not run configurehood and nodewatcher at the same time 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. Reviewed-by: Fabian Bläse <fabian@blaese.de <mailto:fabian@blaese.de> > Fabian > On 8. Mar 2018, at 21:31, Adrian Schmutzler <freifunk@adrianschmutzler.de <mailto:freifunk@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@adrianschmutzler.de <mailto:freifunk@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@freifunk.net <mailto:franken-dev@freifunk.net> > http://lists.freifunk.net/mailman/listinfo/franken-dev-freifunk.net
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@adrianschmutzler.de> --- src/packages/fff/fff-hoods/files/usr/lib/micron.d/fff-hoods | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)