feat: Backup current database, if package is upgraded

[starwarsfan]

If package is upgraded, backup the current database to

`/var/backups/evcc/` 

by following the naming schema

`evcc.db.<evcc-version>.<timestamp>.bak`

So one would have automatically a db backup with every upgrade. Please let me know where to put this into the documentation!

[sourcery-ai]

Hey there - I've reviewed your changes and they look great!

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.}

[mucki12]

Bin kein Code Experte und kann die Aufbewahrungsfrist bzw. das Vorhalten wie vieler Backup Versionen nicht erkennen.

Wird ja wenn dann hoffentlich nur optional sein - ich benötige das aufgrund vorhandener Backups zumindest nicht.

[starwarsfan]

Wird ja wenn dann hoffentlich nur optional sein - ich benötige das aufgrund vorhandener Backups zumindest nicht.

Guter Punkt, danke!

Optional ist das nicht, da man darauf bei der Paketinstallation nicht wirklich Einfluss hat. Was schlägst Du vor, wieviele Versionen liegen bleiben sollten? Da man ja insbesondere im Problemfall durchaus etwas "herumspringen" kann, würde ich acht oder zehn Versionen liegen lassen. Meinungen?

[mucki12]

Nun ja, da müssen andere Meinungen kommen.
Meine Meinung nach exakt 0 - da ich ja selbstverständlich Backups habe.

[StefanSchoof]

Mein /var/backups hat ein Recht einheitlich 7 Dateien (einmal 0 und 1 bis zu 6 .gz).

Das sieht so einheitlich aus, da Frage ich mich ob es ein tool oder Befehl in den Debian packaging tool chain gibt die sich darum kümmert.

[starwarsfan]

Das sieht so einheitlich aus, da Frage ich mich ob es ein tool oder Befehl in den Debian packaging tool chain gibt die sich darum kümmert.

Das macht vermutlich logrotate. Ich werde mir mal die Konfig dazu anschauen, dann kann man das ganz elegant mit Bordmitteln lösen.

[naltatis]

FYI: The avg. db size will grow with upcoming versions when we start to introduce metrics (#23185). So I wouldn't recommend to keep many versions around. Especially on size-constrained devices.

[RenatusRo]

@starwarsfan FYI:
yourip:7070/#/issue
und dann auf 'Configuration UI Show details' clicken gibt einen schoenen json dump der settings.
mit #23771 war ich aber abgeblitzt - vllt. wird das jetzt wieder interessant :-)

[mucki12]

Landen die Backups dann bei einer Home Assistant Installation in /addon_configs/49686a9f_evcc?

[starwarsfan]

Landen die Backups dann bei einer Home Assistant Installation in /addon_configs/49686a9f_evcc?

Mit HA hat das nichts zu tun. Hier geht es um das (Debian-)Paket, welches via Paketmanager installiert wird.

[starwarsfan]

FYI: The avg. db size will grow with upcoming versions when we start to introduce metrics (#23185). So I wouldn't recommend to keep many versions around. Especially on size-constrained devices.

Understand. I'm thinking about some kind of configuration option to define the amount of backups to keep. But I have no idea at the moment, how to get that configured value during package removal. Any ideas?