-
-
Notifications
You must be signed in to change notification settings - Fork 717
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
CLI: add database flag; improve systemd db check #17993
base: master
Are you sure you want to change the base?
Conversation
M.E. ist das unerwartet oder zumindest nicht Unix-like. Warum sollte ich als Anwender die Service-DB nutzen wenn ich eine User-lokale Installation habe? |
Weil es die maximale Ausnahme ist, dass evcc in einem Multi-User-Umgebung (mehrere Instanzen) genutzt wird. Das ist ja weiter möglich, man muss dann halt explizit den Pfad angeben oder nicht auch noch evcc als systemd laufen haben. |
Solange es gleichberechtigte User- und Systeminstallationen gibt wäre es jedenfalls falsch, einfach die andere/falsche DB zu nehmen.
ist auch nicht komplizierter als
|
Wir nehmen nicht einfach eine andere DB. Wir benutzen die auf systemebene vorhandene DB, die durch die Installation als Systemdienst (hat ja der Nutzer selbst entlang der Anleitung gemacht) erzeugt wurde. Was wäre denn dein alternativer Vorschlag? Das jetzige Verhalten, wo wir einfach "still" eine weitere Datenbank anlegen führt ja regelmäßig zu Verwirrung.
Sehe ich in der Tat komplett anders. Das eine ist mir via Help und CLI Docs zugänglich. Für das andere muss ich den ENV + YAML Merge Mechanismus von Viper verstehen. Bei der config-Datei machen wir das doch genau so. |
Wenn ich evcc als Nutzer starte dann ist auch diese DB zu verwenden, es sei denn ich überstimme das. Genauso wie der Serviceuser die Service-DB nehmen muss. "Keine explizite DB" für den non-Service user ist die user DB, nicht die Service-DB.
M.E. ist das UNIX-Standard. Das mag verwirrend sein, entspricht aber den gängigen Konventionen. Stillschweigend die Konventionen zu übergehen ist keine gute Lösung. |
fixes #17939
--database
flag:evcc --database /tmp/evcc.db
/var/lib/evcc/evcc.db
) if it exists and no explicit db is setsudo
hint if service database is not writableTODO: