[PATCH] fff-hoods: remove dependency to /tmp/started

[Tim Niemeyer]

Am Mittwoch, den 13.06.2018, 13:37 +0200 schrieb Adrian Schmutzler:
> Hallo Fabian/Tim,
> 
> bitte meinen alten Kommentar beachten:
> 
> > weiß nicht mehr, ob wir das schon diskutiert haben, aber:
> > Das Locking verhindert, dass das File zweimal gleichzeitig läuft.
> > Micrond wird aber mit S50 gestartet, configurenetwork mit S95.
> > D.h. es kann ohne /tmp/started dazu kommen (wenn Zeitfenster blöd),
> > dass
> > durch micrond configurehood VOR configurenetwork ausführt, was
> > definitiv
> > nicht gut ist.
> > Dies betrifft nur das Entfernen der /tmp/started Bedingung, das
> > Lock selbst
> > ist davon ja nicht betroffen.
> 
> Ich habe kein Problem mit dem Locking, ich würde nur das /tmp/started
> trotzdem im micrond belassen, dann kann gar nichts mehr passieren!
> 
> Solange das /tmp/started entfernt wird, ist das daher für mich NAK.
Naja, Ziel des Patches ist ja dieses unglaublich furchtbare
/tmp/started _endlich_ mal los zu werden.

Wir brauchen einfach ne sauberere Architektur, sonst schießen wir uns
immer öfter selbst ins Knie!

Tim

> 
> Grüße
> 
> Adrian
> 
> 
> > -----Original Message-----
> > From: franken-dev [mailto:franken-dev-bounces at freifunk.net] On
> > Behalf
> > Of Fabian Bläse
> > Sent: Mittwoch, 13. Juni 2018 09:16
> > To: Tim Niemeyer <tim at tn-x.org>
> > Cc: franken-dev <franken-dev at freifunk.net>
> > Subject: Re: [PATCH] fff-hoods: remove dependency to /tmp/started
> > 
> > Ich wollte an dieser Stelle nochmal an dieses Patch erinnern.
> > Imo will man das unbedingt noch im Release haben.
> > 
> > Soweit ich das mit dem trap richtig überblicke, müsste das Lockfile
> > auch
> > immer sauber gelöscht werden.
> > 
> > Grade auch gesehen, dass noch was fehlt:
> > Reviewed-by: Fabian Bläse <fabian at blaese.de>
> > 
> > > On 20. Jan 2018, at 23:03, Tim Niemeyer <tim at tn-x.org> wrote:
> > > 
> > > The dependency was only to avoid duplicated script instances.
> > > Therefore, this patch implements a solution without the need to
> > > depend
> > > on the /tmp/started file.
> > > 
> > > Signed-off-by: Tim Niemeyer <tim at tn-x.org>
> > > ---
> > > 
> > > src/packages/fff/fff-hoods/Makefile                         | 2
> > > +-
> > > src/packages/fff/fff-hoods/files/usr/lib/micron.d/fff-hoods | 2
> > > +-
> > > src/packages/fff/fff-hoods/files/usr/sbin/configurehood     | 7
> > > +++++++
> > > 3 files changed, 9 insertions(+), 2 deletions(-)
> > > 
> > > diff --git a/src/packages/fff/fff-hoods/Makefile
> > > b/src/packages/fff/fff-hoods/Makefile
> > > index 11ab6d1..1a616ff 100644
> > > --- a/src/packages/fff/fff-hoods/Makefile
> > > +++ b/src/packages/fff/fff-hoods/Makefile
> > > @@ -1,7 +1,7 @@
> > > include $(TOPDIR)/rules.mk
> > > 
> > > PKG_NAME:=fff-hoods
> > > -PKG_VERSION:=0.0.1
> > > +PKG_VERSION:=2
> > > PKG_RELEASE:=1
> > > 
> > > PKG_BUILD_DIR:=$(BUILD_DIR)/$(PKG_NAME)
> > > 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 2aaa245..e14995b 100755
> > > --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> > > +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> > > @@ -1,5 +1,12 @@
> > > #!/bin/sh
> > > 
> > > +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
> > > +
> > > . /usr/share/libubox/jshn.sh
> > > . /lib/functions/fff/keyxchange
> > > . /lib/functions/fff/network
> > > --
> > > 2.11.0
> > > 
> > > --
> > > franken-dev mailing list
> > > franken-dev at freifunk.net
> > > http://lists.freifunk.net/mailman/listinfo/franken-dev-freifunk.n
> > > et
> 
> 
-------------- 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/af822876/attachment.sig>