Docker Container restart iobroker per script

[fastfoot]

@ofbeqnpolkkl6mby5e13 sagte in restart iobroker per script?:

Nach dem Start des Skripts wurde sofort neu gestartet. Normal?

Nein, nicht normal. Das wird eigentlich durch die Variable firstRun verhindert, jedenfalls beim Erstellen der Datenpunkte passierte das und dann habe ich das eingebaut und es hat funktioniert.

Passiert das bei jedem Neustart des Skripts oder war das eine einmalige Sache? Falls Letzteres passiert es beim Erstellen des Restart-DP und sollte durch die Variable verhindert werden. Gerade getestet und funktioniert wie beschrieben. Falls das bei dir anders ist, kannst du das on() in einen Timeout packen, dann sollte er auch nicht beim Erstellen gleich triggern.

setTimeout(()=>{
    on({id: idRestart, change:'any'},async () => {
        if(dbg) log('Restart')
        restart(containerName);
    })
},2000)

[oFbEQnpoLKKl6mbY5e13]

@fastfoot
War eine einmalige Sache.

[oFbEQnpoLKKl6mbY5e13]

@fastfoot
Was müsste man tun, wenn man jetzt noch einen anderen Container neu starten möchte?

[fastfoot]

@ofbeqnpolkkl6mby5e13 sagte in restart iobroker per script?:

@fastfoot
Was müsste man tun, wenn man jetzt noch einen anderen Container neu starten möchte?

Die Variable containerName aus einem DP auslesen, anstatt fix festzulegen. Dann muss aber auch das const durch let ersetzt werden. Beim Erstellen des DP als default den Standardnamen angeben und die Zuweisung nach Aufruf von createDatapoints(). In der VIS dann setzen mit Buttons oder mit einem Select von MDW oder was es da sonst noch gibt

[oFbEQnpoLKKl6mbY5e13]

@fastfoot sagte in restart iobroker per script?:

Die Variable containerName aus einem DP auslesen, anstatt fix festzulegen.

Natürlich!

Dann muss aber auch das const durch let ersetzt werden. Beim Erstellen des DP als default den Standardnamen angeben und die Zuweisung nach Aufruf von createDatapoints(). In der VIS dann setzen mit Buttons oder mit einem Select von MDW oder was es da sonst noch gibt

Okay, danke!

[fastfoot]

@fastfoot sagte in Docker Container restart iobroker per script:

nun ja, von den Infos bin ich nicht soo sehr begeistert, aber immerhin. Ich frage ja aber auch nur die Containerinfos ab. Der Restart funktioniert einwandfrei, bin gespannt ob das auf anderen QNAP-Systemen auch so klappt. Ich habe die in den Dokus genannte curl Methode mit axios umgesetzt, weshalb das in der JS-Instanz eingetragen werden muss. Die Datenpunkte für die Tabelle und den Restart werden automatisch unter 0_userdata.0.QNAP angelegt. Vor dem Start sind User und Passwort sowie die IP Adresse und der Port der QNAP einzutragen. Happy Restart!

Mit dem neuesten Update 2.4 der QNAP Containerstation hat es die verwendete REST API zerschossen, wer also das Skript hier nutzt, sollte nicht updaten und auf einen Fix warten, kann mir nicht vorstellen dass es diesen nicht geben wird. Wer ausserdem weiss wie man die Containerstation downgraden kann, bitte mal melden

[oFbEQnpoLKKl6mbY5e13]

@fastfoot
Danke für die Info.

[fastfoot]

Das Skript läuft jetzt wieder, die URL hatte sich geändert. Somit kann ich endlich die neueste Container-Station APP nutzten.

[bug77]

bei mir funktioniert das Script leider nicht.

Nachdem ich den Link im Script geändert habe in

http://${server}:${port}/containerstation/api/v1/login

funktioniert die Abfrage.

Wenn ich aber Restart mache, passiert nichts, auch im Log steht nichts.

Habe User / Pass / IP / Containername (name oder Abbbild name?), FakeID bleibt xxx?,

[fastfoot]

@bug77 stimmt denn der Containername mit Zeile 17 überein?

[bug77]

@fastfoot ja habe sowohl den Container Namen, als auch die Container ID probiert.
Er liest alles ein, sehe alle Container als html, aber auf restart wird nicht reagiert

[fastfoot]

@bug77 sagte in Docker Container restart iobroker per script:

Nachdem ich den Link im Script geändert habe in

jetzt muss da ein Bindestrich rein

[fastfoot]

@bug77 sagte in Docker Container restart iobroker per script:

@fastfoot ja habe sowohl den Container Namen, als auch die Container ID probiert.
Er liest alles ein, sehe alle Container als html, aber auf restart wird nicht reagiert

gerade getestet, läuft. Rufe die Restart Funktion mal ohne Button auf: restart(containerName);

[bug77]

@fastfoot wie mach ich das?

[fastfoot]

@bug77 sagte in Docker Container restart iobroker per script:

@fastfoot wie mach ich das?

na zB in der letzten Zeile des Skripts eintragen.
schedule('30 13 * * *', () => restart(containerName)) Das macht den Restart um 13:30Uhr. Wenn erfolgreich auskommentieren, dann liegt es am Button dass es vorher nicht geklappt hat.

[bug77]

@fastfoot ok damit funktioniert es. Also liegt es am Button? Und nun?

[fastfoot]

@bug77 sagte in Docker Container restart iobroker per script:

@fastfoot ok damit funktioniert es. Also liegt es am Button? Und nun?

zeig mal das RAW Objekt deines Button

[bug77]

@fastfoot

{
 "from": "system.adapter.javascript.0",
 "user": "system.user.admin",
 "ts": 1631647986751,
 "common": {
   "name": "Container Restart",
   "type": "boolean",
   "role": "button",
   "desc": "von Skript erstellt",
   "read": true,
   "write": true,
   "def": false
 },
 "native": {},
 "acl": {
   "object": 1636,
   "owner": "system.user.admin",
   "ownerGroup": "system.group.administrator",
   "state": 1636
 },
 "_id": "0_userdata.0.Netzwerk.QNAP.Restart",
 "type": "state"
}

[fastfoot]

@bug77 die Definition ist nicht ganz korrekt, funktioniert aber genau so bei mir. Hast du denn den Kommentar in Zeile 31 entfernt? //restart(containerName);

[bug77]

@fastfoot said in Docker Container restart iobroker per script:

//restart(containerName);

ne habe nichts entfernt.

Jetzt habe ich die // weggemacht und es tut auf knopfdruck.
Das hatte ich aber nirgends gelesen, das man das machen muss.

Super, danke es tut jetzt