nein, dazu ist es nie gekommen. Liegt auch daran, dass DP mit dem python script dreamboxctl vieles davon abdeckt, wenn auch nicht so umfachreich





Wenn ich das richtig verstehe, lässt du per cronjob das dreamciplus neu starten, was natürlich keine Rücksicht darauf nimmt ob eine Aufnahme läuft oder nicht

Im Grunde müsste man das in ein script packen und abfragen ob eine Aufnahme läuft - falls ja den restart verschieben

Man könnte mit dreamboctl recordings mit "Boardmitteln" in einer Schleife abfragen ob eine Aufnahme/Stream läuft, falls ja den den dreamciplus restart für z.B. für 10 Minuten verschieben und dann erneut versuchen bis der Rückgabewert des commands 0 ist.


Der Rückgabewert 0 bedeutet, das weder eine Aufnahme noch ein Stream über das Webinterface läuft - da macht dreamboxctl (leider) keinen Unterschied. D.h. der Rückgabewert 1 bedeutet, dass entweder eine Aufnahme läuft oder ein Webinterface Stream aktiv ist.

Wenn man nur die laufenden Aufnahmen ohne aktive Streams zählen will, kann man dass über einen Webinterface command im Terminal oder per script abfragen - dafür muss aber das Paket curl installiert sein:


die aktiven Webinterface Stream Sessions erhählt man, wenn man die streamproxy prozesse zählt


Dann einfach script in einer Schleife den Rückgabewert der Variable(n) $RECORDINGS (AND $STREAMS) abfragen und erst dann dreamciplus neu starten, wenn $RECORDINGS (und gegebenenfalls $STREAMS) den Wert 0 hat bzw. haben.

nice

Und funktioniert es wie gewünscht?

ah, ok

auf den ersten Blick sieht es gut aus, ich kann es mir ja mal genauer unter die Lupe nehmen
Du kannst es ja selbst testen, indem du Aufnahmen, Streams startest und dann das script manuell startest und statt dem restart eine echo ausgibst, das beschreibt was ausgeführt werden würde

wenn du dann noch das sleep auf z.B. 20 sekunden runterschraubst kannst du laufende Aufnahmen/Streams beenden und testen was dann das script ausspuckt. So mache ich das auch immer.


Was mir aufgefallen ist:
das script läuft wegen dem sleep 600 u.U. im stundenlangen Dauerloop wenn eine Aufnahme oder ein Stream läuft

Vielleicht wäre es besser das script in einem subprozess mit einem vorgestellten sleep 600 erneut zu starten und das laufende script sauber zu beenden

Aber das ist vermutlich Geschmackssache


Man könnte das auch noch aufwändiger dafür zielgenauer scripten, indem man die Endzeit der laufenden Aufnahme ausliest und mit der aktuellen Zeit vergleicht. Dann kann man die sleep time errechnen:

endtime - currenttime = sleep time

das gilt dann aber nur für Timer bzw. Sofort Aufnahmen aber nicht für Streams, da weiß man ja nicht wie lange die aktiv sind.

Dann könnte man noch eine Prüfung einbauen, ob eine weitere Aufnahme vor der endtime der laufenden Aufnahme startet ...

das wird vermutlich nicht funktionieren, da sind einige Fehler eingebaut bzw. wird einiges vermischt

if (( STREAMS > 0 && RECORDINGS == 0))

• die Variablen werden nicht korrekt angegeben, müsste $STREAMS und $RECORDINGS heissen
  
• if verlangt [ und kein (, das if [ bedingung ] entspricht einem if test bedingung, dann kann man die [] weglassen. Wichtig ist auch immer ein Leerzeichen nach [ und ein Leerzeichen vor dem ], sonst wird ein Fehler ausgespuckt bzw. funktioniert die Bedingungsprüfung nicht
  
• ">" bzw. "<" ist kein gültiger Vergleichsoperator für (ba)sh Bedingungen, dafür verwendet man -gt (greater than), -lt (less then), -ge (greater or equal), -le (less or equal) usw.
  
• das && passt hier auch nicht rein, das entspricht einem if bedingung = true; then do
  
  hier wäre -a korrekt, dass entspricht einem AND, also wenn Bedingung A und Bedingung B zutrifft
  -o enspricht einem OR - also wenn entweder Bedingung A oder Bedingung B zutrifft
  
  siehe z.B. Bedingungsprüfungen
  

das müsste also nach deinem Konzept in etwa so mit korrekter Syntax aussehen (ungetestet):

if [ $STREAMS -gt 0 -a $RECORDINGS = 0 ]; then

Da ist aber vermutlich ein logischer Fehler enthalten:
Das bewirkt, dass dreamciplus nur dann neu gestartet wird, wenn keine Aufnahme aber jedoch mindest ein Stream läuft! Wenn aber weder eine Aufnahme noch ein Stream läuft wird dreamciplus NICHT neu gestartet! Das ist vermutlich nicht der Wunsch.

Wenn dir die laufenden Streams egal sind, lass die Prüfung einfach weg, das macht es übersichtlicher

kann ein Moderator die Beitrage ab Post #8 oder #9 in einen eigenen Thread verschieben? Das hat mit diesem Thread nur am Rande zu tun

Titel vielleicht: "dreamciplus automatisiert neustarten" oder dergleichen

diesen Beitrag kann man dann gerne löschen

Ich nehme alles wieder zurück, ich hab grad etwas dazugelernt

Diese Art der Bedingungsprüfung bzw. Vergleich von Zahlen in einem Shell Script kannte ich noch nicht, aber das funktioniert ja wirklich




Ich hätte da eine Frage zum logischen Aufbau:

du prüfst mit dreamboxctl recordings ob eine Aufnahme läuft, die Ausgabe ist dann 0, 1, 2 usw.

Dann kommt die Bedingungsprüfung: Wenn keine Aufnahme läuft dann ist restart true, wenn eine läuft dann prüfst du über meinen Webinterface Command ein 2. Mal ob eine Aufnahme läuft.

Wozu die zweite Prüfung?

ah verstehe

ich habe das mal auf meiner dm820 getestet - werden nur die Vergleichsoperatoren > und < und == aktzeptiert, => und =< spuckt einen Fehler aus, allerdings scheint die Prüfung dennoch zu funktionieren.

Und weniger ist oft mehr

Ich suche gerade nach einer "einfachen" Lösung die Endzeit einer laufenden Aufnahme auszulesen, mal sehen

Was neben deiner vereinfachten Bedingungsprüfung bzw. vergleichen von Zahlen unter anderen Umständen nicht funktionieren wird ist deine Verwendung bzw. Aufruf der Varablen. Beim Aufruf muss man der gesetzten Variable ein "$" vorstellen, sonst sucht die shell nach einem programm oder einer funktion


also


und nicht



zweiteres würde nur "a" ausgeben

also besser statt
if (( currentTime >= stoptime ))
muss das heissen
if (( $currentTime >= $stoptime ))

statt
if (( hasRecording == 0 ))
dann
if (( $hasRecording == 0 ))

und statt
if (( STREAMS > 0 && RECORDINGS == 0))
dann
if (( $STREAMS > 0 && $RECORDINGS == 0))



Was auch immer empfehlenswert ist: veränderbare Variablen am Beginn des Scripts auf 0 oder auf leer zu setzen

Ich verwende für die einfachere Lesbarkeit immer nur Großbuchstaben für VARIABLEN

ich habe mir das mal genauer angesehen

man kann über die http://localhost/web/timerlist die Endzeit einer bzw der laufenden Aufnahmen auslesen und so aus der spätesten Timer Endzeit - akueller Zeit die sleep Zeit erreichnen

man kann auch so prüfen ob eine Timer Aufnahme vor der letzten aktellen Aufnahmeendzeit startet und so einfach die Entzeit der nächsten Aufnahme als Berechnungsgrundlage verwendet. Ausser es handelt sich um einen Umschalt Timer - auch das kann man auslesen (<e2justplay>1</e2justplay> = Umschalttimer JA, <e2justplay>0</e2justplay> = Umschalttimer NEIN)


Aber dann wird das Script ganz schön aufgeblasen und sollte den Code von Anfang an aufgeräumt halten und mit Funktionen arbeiten

Zum Verständnis:

mit curl -s -X POST "http://localhost/web/timerlist?&sessionid=$SESSIONID"|grep e2state


wird der e2state alle Timer Einträge ausgelesen


<e2state> kann vier Werte haben


0 = Aufnahme oder Umschalttimer in der Zukunft
1 = Aufnahme oder Umschaltimer in Vorbereitung - der Wert wechselt ca. 20 Sekunden vor Timerbeginn den Wert von 0 auf 1
2 = laufende Aufname
3 = erledigte Aufnahme oder Umschalttimer


Wenn man nun nach einem <e2state> Wert filtert UND sich 20 Zeilen vor und 5 Zeilen nach dem <e2state> ausgeben lässt hat man alle Informationen zur diesem bzw. diesen Timer Einträgen und kann Beginn, Endzeit, Service Referenz, Service Name, Umschalttimer ja/nein uvm. auslesen und damit rechnen oder restart auf true oder false setzen

Falls mehrere Timer ausgegeben werden ist an oberster Stelle immer der nächste Timer

Super, jetzt haben sich unsere Beträge wg. 2 Minunte wieder mal überschnitten ....

sorry

in welchem Post (#) ist deine neue Version?


ich habe meine Version hier wieder rausgenommen

ah gefunden,

wg. der Nachvollziehbarkeit das nächste man am Ende anhängen, sonst wird das später für einen der das liest nicht mehr nachvollziebar

ja, das sieht jetzt schon gut aus

Aber dieser Part stimmt noch immer nicht, da ist noch immer der logische Fehler enthalten



Hier willst du dreamciplus neu starten, weil RECORDINGS den Wert 0 hat

aber deine If Abfrage restartet dreamciplus nur dann, wenn $STREAM einen Wert größer 0 UND $RECORDINGS genau den Wert 0 hat

Wenn aber beides den Wert 0 hat - was ja best case ist - wird dreamciplus NICHT neu gestartet

das lässt sich einfach testen:

Die Bedingung müsste also lauten:

if (( $RECORDINGS == 0 ))
oder
if [ $RECORDINGS = 0 ];




ob ein stream läuft ist dir ja lt. letzten posts egal

ja, aber dann ist die zweite Abfrage reduntant bzw. doppelt gemoppelt und führt unter extremst unwahrscheinlichen Umständen sogar zu einem Fehlverhalten:





Wenn die if Abfrage if (( $HASRECORDING == 0 )) bereits zutrifft - also $RECORDINGS den Wert 0 hat, wird der else part nicht mehr berücksichtigt und das Script geht bei if [ $RESTART ] weiter


Wenn die if Abfrage if (( $HASRECORDING == 0 )) nicht zutrifft - also $RECORDINGS den Wert ungleich 0 hat, dann kommt der else part zum Tragen.

Dann fragst du noch einmal mit einem anderen Befehl RECORDINGS ab, obwohl du bereits weisst, dass dieser größer als 0 ist

Dann prüfst du auch noch die Anzahl der Streams und baust dann eine weitere Bedingung ein.


Die wird dir aber nur dann den Rückgabewert RESTART=true liefern, wenn der sehr, sehr, sehr unwahrschleinliche = nahezu unmögliche Fall eintritt, dass RECORDINGS im Millisekundenbereich den Wert von >0 auf 0 ändert und ein Stream läuft ( $STREAMS >0 ).

Und wenn dann aber kein Webinterface stream läuft, wird dreamciplus dennoch NICHT gestartet, obwohl der best case eingetreten ist:

STREAM=0
RECORDINGS=0



imho würde das mit deinen Vorgaben (Webinterface Streams werden nicht berücksichtig) völlig reichen