[RFC] fff-network: Add migration for Archer C7 WAN ports #93

fbl · 2021-01-27T17:49:38+01:00

fbl commented

2021-01-27 17:49:38 +01:00

With 8d66bdf the port configuration of TP-Link Archer C7 has been
changed to a single-interface configuration.

This had unforseen side effects on upgraded devices. Because WANDEV
is evaluated from the updated network.* file, the port configuration
of the switch is evaluated from the update-safe network.config, which
is now incompatible with the updated interface configuration.

Therefore, a migration script has to be added, which updates the port
configuration in network.config to the new single-interface network
configuration.

Fixes: #60
Signed-off-by: Fabian Bläse fabian@blaese.de

With 8d66bdf the port configuration of TP-Link Archer C7 has been changed to a single-interface configuration. This had unforseen side effects on upgraded devices. Because WANDEV is evaluated from the updated network.* file, the port configuration of the switch is evaluated from the update-safe network.config, which is now incompatible with the updated interface configuration. Therefore, a migration script has to be added, which updates the port configuration in network.config to the new single-interface network configuration. Fixes: #60 Signed-off-by: Fabian Bläse <fabian@blaese.de>

fbl added this to the 20210211-beta milestone 2021-01-27 17:49:38 +01:00

fbl added the

bug

node

RFC

labels 2021-01-27 17:49:38 +01:00

fbl force-pushed c7 from cc703feb52 to 9e9ac44617

2021-01-27 17:50:11 +01:00

Compare

adschm added the

packages/fff

label 2021-01-27 19:23:43 +01:00

adschm commented

2021-01-27 20:07:51 +01:00

Klingt konzeptionell plausibel, nur die SED Syntax ist mir nicht direkt geläufig...

rohammer approved these changes 2021-01-27 20:50:55 +01:00

rohammer commented

2021-01-27 20:51:00 +01:00

sed syntax is per se nicht gelaeufig. Wird auch nach 35 Jahren an der Linuxkonsole nicht besser. Aber passt schon. Ist ein wenig wie bei awk mit dem regex zum Zeilen raussuchen. Dann wird in-place ersetzt.

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

sed syntax is per se nicht gelaeufig. Wird auch nach 35 Jahren an der Linuxkonsole nicht besser. Aber passt schon. Ist ein wenig wie bei awk mit dem regex zum Zeilen raussuchen. Dann wird in-place ersetzt. ``` Reviewed-by: Robert Langhammer <rlanghammer@web.de> ```

fbl commented

2021-01-30 23:17:30 +01:00

Der sed ist maximal beschissen und es gibt bestimmt tausend Randfälle die passieren könnten, wenn man diese Konfiguration auf bestimmte Weise verändert hat. Darauf sollten wir dann auf der Update-Seite unbedingt hinweisen.

Für die "normalen" Nutzer, die jetzt also entweder den Stand vor 20200501 oder den Stand 20200501 haben, sollte es aber auf jeden Fall klappen. Und für die meisten "gewöhnlichen" Veränderungen an WAN_PORTS ebenfalls.

Der sed ist maximal beschissen und es gibt bestimmt tausend Randfälle die passieren könnten, wenn man diese Konfiguration auf bestimmte Weise verändert hat. Darauf sollten wir dann auf der Update-Seite unbedingt hinweisen. Für die "normalen" Nutzer, die jetzt also entweder den Stand vor 20200501 oder den Stand 20200501 haben, sollte es aber auf jeden Fall klappen. Und für die meisten "gewöhnlichen" Veränderungen an WAN_PORTS ebenfalls.

adschm commented

2021-01-31 14:12:07 +01:00

Wird gemergt.

adschm closed this pull request

2021-01-31 14:12:07 +01:00

fbl deleted branch c7

2021-01-31 14:55:27 +01:00