Add switch configuration migration for dsa devices #168

fbl · 2021-09-03T00:38:20+02:00

fbl commented

2021-09-03 00:38:20 +02:00

No description provided.

fbl added the

RFC

WIP

labels 2021-09-03 00:38:20 +02:00

fbl added 1 commit 2021-09-03 00:38:20 +02:00

d7aa8c70a2 layer3-config: Bump required config_version

Signed-off-by: Fabian Bläse <fabian@blaese.de>

rohammer approved these changes 2021-09-03 10:30:49 +02:00

rohammer left a comment

Hi, schaut gut aus!

Reviewed-by: Robert Langhammer <rlanghammer@web.de>

Hi, schaut gut aus! ``` Reviewed-by: Robert Langhammer <rlanghammer@web.de> ```

ChristianD commented

2021-09-03 10:44:55 +02:00

Ich würde gerne die 05- im Filename nochmal kurz überlegen. Aktuell haben wir schon 2 05er uci-defaults Scripte:

-rw-r--r-- 1 root root 49 Jun 14 00:02 01-iproute
-rw-r--r-- 1 root root 1.2K Jun 14 00:02 02-network-rules
-rw-r--r-- 1 root root 1.1K Jun 14 00:02 05-config-system-migration
-rw-r--r-- 1 root root 744 Jun 14 00:02 05-wireguard-rules
-rw-r--r-- 1 root root 395 Jun 14 00:02 10_migrate-shadow
-rw-r--r-- 1 root root 369 Jun 14 00:02 12_network-generate-ula
-rw-r--r-- 1 root root 179 Jun 14 00:02 13_fix-group-user

Es wird ja vermutlich irgendwann eine Migration von Version 2 auf Version 3 stattfinden, das müsste dann nach diesem Script passieren (dann würde auch eine Migration von 1 auf 3 funktioneren da ja dann zuerst die 1 -> 2 läuft und danach 2 -> 3). D.h. gerade diese 05er wird dann immer voller.

Aktuell haben wir 01 02 05 05 10 12 [...] (siehe oben) als uci-defaults Dateien. Ich würde daher vorschlagen das auf 07 zu legen und wenn man dann on 2 -> 3 macht 07a zu verwenden, 07b für 3 -> 4 usw.

Ansonsten sieht es gut aus, ich habe mir jetzt nicht wirklich die Umschreibsachen angeguckt das hat ja Robert schon gemacht, mir gings rein um die Logik der Versionen.

Wenn wir uns beim Filenamen noch einig werden:

Reviewed-by: Christian Dresel <freifunk@dresel.systems>

Ich würde gerne die 05- im Filename nochmal kurz überlegen. Aktuell haben wir schon 2 05er uci-defaults Scripte: -rw-r--r-- 1 root root 49 Jun 14 00:02 01-iproute -rw-r--r-- 1 root root 1.2K Jun 14 00:02 02-network-rules -rw-r--r-- 1 root root 1.1K Jun 14 00:02 05-config-system-migration -rw-r--r-- 1 root root 744 Jun 14 00:02 05-wireguard-rules -rw-r--r-- 1 root root 395 Jun 14 00:02 10_migrate-shadow -rw-r--r-- 1 root root 369 Jun 14 00:02 12_network-generate-ula -rw-r--r-- 1 root root 179 Jun 14 00:02 13_fix-group-user Es wird ja vermutlich irgendwann eine Migration von Version 2 auf Version 3 stattfinden, das müsste dann nach diesem Script passieren (dann würde auch eine Migration von 1 auf 3 funktioneren da ja dann zuerst die 1 -> 2 läuft und danach 2 -> 3). D.h. gerade diese 05er wird dann immer voller. Aktuell haben wir 01 02 05 05 10 12 [...] (siehe oben) als uci-defaults Dateien. Ich würde daher vorschlagen das auf 07 zu legen und wenn man dann on 2 -> 3 macht 07a zu verwenden, 07b für 3 -> 4 usw. Ansonsten sieht es gut aus, ich habe mir jetzt nicht wirklich die Umschreibsachen angeguckt das hat ja Robert schon gemacht, mir gings rein um die Logik der Versionen. Wenn wir uns beim Filenamen noch einig werden: ``` Reviewed-by: Christian Dresel <freifunk@dresel.systems> ```

fbl commented

2021-09-03 11:15:08 +02:00

Wenn wir uns beim Filenamen noch einig werden:

Da werden wir uns bestimmt einig, denn dazu hab ich mir noch quasi überhaupt gar keine Gedanken gemacht. Auch deshalb ist das Ding noch "WIP" ;-)

> Wenn wir uns beim Filenamen noch einig werden: Da werden wir uns bestimmt einig, denn dazu hab ich mir noch quasi überhaupt gar keine Gedanken gemacht. Auch deshalb ist das Ding noch "WIP" ;-)

rohammer commented

2021-09-04 13:47:50 +02:00

Das mit den Nummern weitsichtig zu planen hat noch nie funktioniert. Ich erinnere nur an init. Es ist wirklich vertane Mühe, sich darüber länger Gedanken zu machen. Es kommt immer anders. Das scheint ein Naturgesetz zu sein.
Die Frage lautet also: funktioniert es? - ja - dann lassen.

Das mit den Nummern weitsichtig zu planen hat noch nie funktioniert. Ich erinnere nur an init. Es ist wirklich vertane Mühe, sich darüber länger Gedanken zu machen. Es kommt immer anders. Das scheint ein Naturgesetz zu sein. Die Frage lautet also: funktioniert es? - ja - dann lassen.

fbl commented

2021-09-04 16:29:29 +02:00

Abgesehen von ein bisschen Prototyping ist das bisher komplett ungetestet und die Ziffer war bisher auch nur Platzhalter. Also zumindest kurz prüfen muss man das noch.

fbl force-pushed dsamigration from 9b3f9954c0 to 8ffb376457

2021-10-23 23:59:03 +02:00

Compare

fbl commented

2021-10-23 23:59:05 +02:00

Changes: