Prevent multiple start of configurehood and nodewatcher

[Adrian Schmutzler]

Gut, das kürzt die Sache ab.

 

Dann schicke ich demnächst einen Patch mit B.

 

Wegen des lock-file-Namens bin ich unentschieden.

 

Grüße

 

Adrian

 

From: franken-dev [mailto:franken-dev-bounces at freifunk.net] On Behalf Of robert
Sent: Dienstag, 30. Oktober 2018 18:22
To: franken-dev at freifunk.net
Subject: Re: Prevent multiple start of configurehood and nodewatcher

 

Hi Adrian,

B ist richtig. Man könnte noch überlegen, ob man das Lockfile hart codet und nicht aus $0 generiert. Hilft gegen unterschiedliche Dateinamen, Links.

A ist Mist. Ich darf das sagen, weil es von mir stammt ;-) Tim hat auch erklärt warum.

Robert

Am 30.10.18 um 17:34 schrieb Adrian Schmutzler:

	Hallo,

	 

	wie Fabian angeregt hat, sollten wir uns nochmals mit der Frage beschäftigen, ein mehrfaches Ausführen von configurehood zu vermeiden (und meiner Meinung nach auch des nodewatcher).

	 

	Ich möchte dies gerne von der Debatte um /tmp/started trennen.

	 

	Vorgeschlagen sind:

	 

	A)

	if [ $( pidof $( basename $0 )  | wc -w ) -gt 2 ]; then

	                echo "Only one instance of $0 allowed."

	                exit 1

	fi

	 

	B)

	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

	 

	 

	Ich war so frei, und habe bei A das echo und den exit code ergänzt.

	 

	Ich würde mich über Rückmeldung/Einschätzungen/Abstimmungen freuen und würde dann je nach Präferenz JEWEILS einen Patch für configurehood und nodewatcher mit der favorisierten Variante schicken.

	Bei A sollte man auch nochmal durchdenken, ob das mit string vs. numeric so passt.

	 

	Je nachdem wie lange der Rest dauert wäre ich auch durchaus noch daran interessiert, das mit in das beta-Release zu bauen.

	 

	Grüße

	 

	Adrian
-------------- nächster Teil --------------
Ein Dateianhang mit HTML-Daten wurde abgetrennt...
URL: <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20181030/2cd33b43/attachment.html>
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : nicht verfügbar
Dateityp    : application/pgp-signature
Dateigröße  : 834 bytes
Beschreibung: nicht verfügbar
URL         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20181030/2cd33b43/attachment.sig>