Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update README.md and include automatic restore to iobroker_startup.sh #56

Merged
merged 6 commits into from
Jan 4, 2020

Conversation

duffbeer2000
Copy link
Contributor

Correct the default value of LC_ALL

Correct the default value of LC_ALL
Integration of automatic restore of an iobroker backup file. It detects if /opt/iobroker contains a single file and if so if it contains a "backup.json" and restores it.
@duffbeer2000 duffbeer2000 reopened this Dec 27, 2019
@duffbeer2000 duffbeer2000 changed the title Update README.md Update README.md and include automatic restore to iobroker_startup.sh Dec 27, 2019
As https://docs.docker.com/engine/reference/builder/ says MAINTAINER instruction is deprecated. Switching to LABEL instruction.
apt-get upgrade -y shouldn't be used in a Dockerfile
backup folder is always empty in this if case so not necessary to empty it
@buanet
Copy link
Owner

buanet commented Dec 31, 2019

Hi,
ich gehe mit den Commits soweit mit, vielen Dank dafür!
Allerdings würde ich ungern einen automatischen Restore einbauen. Soweit ich mich erinnere dauert der Restore unter Umständen sehr lange (zumindest habe ich sowas mal im Forum gelesen). Das wäre für den Containerstart ungünstig.
Wie sieht es denn mit der Zuverlässigkeit/ Fehleranfälligkeit aus? Ich kann das leider nicht beurteilen da ich backup/ restore noch gar nicht verwendet habe. :)
Meine Idee wäre das Backup automatisch in das entsprechende Verzeichnis zu Verschieben und in der Logausgabe auf die manuelle Ausführung des Restore-Befehls hin zu weisen. Also in etwa: "Backupfile gefunden, zur Wiederherstellung folgende Schritte über die Kommandozeile ausführen...".
Wie sind deine Erfahrungen zum restore?

MfG,
André

@duffbeer2000
Copy link
Contributor Author

duffbeer2000 commented Jan 1, 2020 via email

@buanet
Copy link
Owner

buanet commented Jan 4, 2020

Hallo Christian,
dir auch noch ein frohes Neues...

Danke für die Erläuterungen. Ich habe mal ein bisschen mit dem Backup rum gespielt.
Werde auf jeden Fall die Logausgabe noch anpassen, damit wir dort einen Hinweis haben, dass ioBroker nach dem Restore noch die Adapter nachinstalliert...
Außerdem müssen wir schauen ob das mit der Prüfung auf eine einzige Datei funktioniert. Ich hatte das bei mir erhöhen müssen. Hatte den Fall, dass ich den Ordner mal über den FileManager der DS geleert habe und dann noch eine versteckte Datei drin war. Seit dem prüfe ich auf Dateien < 3 :) Vielleicht muss ich da nochmal die Bendingungen anpassen/ erweitern...

MfG,
André

@buanet buanet merged commit ce10e99 into buanet:dev Jan 4, 2020
@buanet
Copy link
Owner

buanet commented Jan 4, 2020

Hallo Christian,
schau mal: d5a1bda
Habe das ganze nochmal etwas aufgehübscht. Der dev-build läuft gerade. Vielleicht kann ich das später gleich noch testen....

MfG,
André

@duffbeer2000
Copy link
Contributor Author

duffbeer2000 commented Jan 6, 2020 via email

@buanet
Copy link
Owner

buanet commented Jan 6, 2020

Hallo Christian,
habe die Prüfung auf die Dateien nochmal angepasst. In meinem Test lief das einwandfrei.
Die Änderungen sind auch schon in der neuen v4.0.3beta drin. Wenn ich nichts negatives höre, dann mache ich daraus demnächst die neue 4.1 stable.

MfG,
André

@duffbeer2000
Copy link
Contributor Author

duffbeer2000 commented Jan 6, 2020 via email

@buanet
Copy link
Owner

buanet commented Jan 9, 2020

Hallo Christian,
ja, ich suche nur nach einem Backup wenn der Ordner sonst leer ist und keine Dateien von ioBroker gefunden werden. Die Funktion sollte m. E. nur greifen wenn es eine Neuinstallation ist.

Die MayListeners Meldung kenne ich nicht. Sieht nach js-controller aus. Hast du mal google befragt?

MfG,
André

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants