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
Config UI: add missing config sections / partial yaml inputs (BC) #13319
base: master
Are you sure you want to change the base?
Conversation
@naltatis ich brauche mal eine Idee. ModbusProxy ist eine Liste. Wie wollen wir hier mit Updates umgehen:
Meine Tendenz wΓ€re per Element ΓΌber numerischen Index. Passt das? |
@andig Im ersten Schritt (dieser PR) wΓΌrde ich dir nur einen Yaml Block geben und keine RΓΌcksicht auf die innere Struktur nehmen. HeiΓt, ein Update hat immer ein komplettes Neu-initialisieren zur Folge. Nichts auf Item-Ebene. Gleiches gilt fΓΌr Messaging. Da ist die Struktur ja noch komplizierter, weil Messages und Services (Liste). Das ist natΓΌrlich nicht schΓΆn und wir mΓΌssen das frΓΌher oder spΓ€ter sauber als Struktur/Liste ausmodellieren. Aber ich sehe das hier erstmal als einen pragmatischen Zwischenschritt. |
Passt, das ist hΓ€sslich aber einfach. |
@naltatis die existierenden Site APIs, z.B: Vorschlag:
Wenn das passt mΓΌssen wir noch die Settings initial migrieren und dann nur noch aus der DB lesen. |
cmd/root.go
Outdated
@@ -192,7 +190,7 @@ func runRoot(cmd *cobra.Command, args []string) { | |||
|
|||
// setup database | |||
if err == nil && conf.Influx.URL != "" { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@andig ich musste den !== ""
Check wieder reverten. Ohne den stoppt das Publishing und die UI bekommt keine Werte Updates. Dadurch sind auch die e2e Tests kaputt gegangen.
WIP: configure, restart, fail, fix, restart, success cycle π startup.error.mp4 |
Sieht super aus! Mit der error class solltest du auch direkt mqtt rot highlighten kΓΆnnen? |
Wo finde ich die? Ich hab nur was in 'fatal' gesehen. |
Aber jetzt: {
"result": {
"auth": {
"vehicles": {}
},
"fatal": {
"class": "mqtt",
"error": "failed configuring mqtt: error connecting: network Error : dial tcp: lookup foo.bar: no such host"
},
"influx": {},
"interval": 3,
"loadpoints": [],
"mqtt": {
"topic": "evcc"
},
"network": {
"schema": "http",
"host": "evcc.local",
"port": 7070
}
}
} Falls es ein Fehler eines GerΓ€tes ist kommt auch der GerΓ€tename als |
Den brauchen wir aber- sonst funktioniert die Migration des Settings nicht. Mir ist nicht klar, was da stoppen sollte? Evtl. vorherige Fehlmigration? Aber auch dann sollte nix stoppen! |
Ne, das passiert auch bei einer neuen Installation (siehe E2E tests). Du kannst das ja einfach reproduzieren. |
Hab ich kein Problem. Welches konkrete Peoblem hast du denn? |
@naltatis ich sehe kein Problem nach revert:
|
Replaces #11739
Goal
Later steps (seperate PRs)
BC
Todos
remove all ns conversions from UI@naltatis