Add support for TP-Link Archer C7 v4 #278

sethur · 2023-02-20T13:46:16+01:00

sethur commented

2023-02-20 13:46:16 +01:00

First-time contributor

Simple configuration update to enable firmware for TP-Link Archer C7 v4 routers.

Currently running on https://monitoring.freifunk-franken.de/mac/0c8063dcf936, though admittedly only basic functionality test has been done. If more specific testing is desired before a merge, please let me know which tests should be performed.

Simple configuration update to enable firmware for TP-Link Archer C7 v4 routers. Currently running on https://monitoring.freifunk-franken.de/mac/0c8063dcf936, though admittedly only basic functionality test has been done. If more specific testing is desired before a merge, please let me know which tests should be performed.

fbl approved these changes 2023-02-20 17:06:00 +01:00

fbl left a comment

Sieht gut aus. Hast du beide Varianten getestet oder nur node?
In jedem Fall kannst du für den Test noch ein Tested-by: Name <email@domain.tld> ergänzen, entweder im Commit, oder als Kommentar (dann ergänze ich das beim mergen).

Passt bei der node-Variante die Reihenfolge der Ports im Web-UI?
Andernfalls muss noch die PORTORDER Variable in der network.* Datei ergänzt werden.

Kann die auf dem Label aufgedruckte MAC-Adresse mit folgenden Befehlen korrekt ausgegeben werden?

. /lib/functions.sh
. /lib/functions/system.sh
get_mac_label

Andernfalls müsste noch die ROUTERMAC Variable in der network.* Datei ergänzt werden.

Sind die MAC-Adressen aller Batman-Interfaces (w2mesh, w5mesh, eth0.3) unterschiedlich?

Abgesehen davon:
Reviewed-by: Fabian Bläse <fabian@blaese.de>

Sieht gut aus. Hast du beide Varianten getestet oder nur node? In jedem Fall kannst du für den Test noch ein `Tested-by: Name <email@domain.tld>` ergänzen, entweder im Commit, oder als Kommentar (dann ergänze ich das beim mergen). Passt bei der node-Variante die Reihenfolge der Ports im Web-UI? Andernfalls muss noch die PORTORDER Variable in der network.* Datei ergänzt werden. Kann die auf dem Label aufgedruckte MAC-Adresse mit folgenden Befehlen korrekt ausgegeben werden? ``` . /lib/functions.sh . /lib/functions/system.sh get_mac_label ``` Andernfalls müsste noch die ROUTERMAC Variable in der network.* Datei ergänzt werden. Sind die MAC-Adressen aller Batman-Interfaces (w2mesh, w5mesh, eth0.3) unterschiedlich? Abgesehen davon: `Reviewed-by: Fabian Bläse <fabian@blaese.de>`

fbl added the

bsp

label 2023-02-20 17:08:39 +01:00

fbl added this to the 202303-beta milestone 2023-02-20 17:08:45 +01:00

fbl self-assigned this 2023-02-20 17:08:49 +01:00

sethur force-pushed master from 9413262018 to 3bf61f9d9c

2023-02-25 20:33:33 +01:00

Compare

sethur commented

2023-02-25 20:34:25 +01:00

First-time contributor

Sieht gut aus. Hast du beide Varianten getestet oder nur node?

Test ist beschränkt auf die node-variante, Layer3 ist komplett ungetestet.

In jedem Fall kannst du für den Test noch ein Tested-by: Name <email@domain.tld> ergänzen, entweder im Commit, oder als Kommentar (dann ergänze ich das beim mergen).

Ist im commit, PR wurde geupdated :)

Passt bei der node-Variante die Reihenfolge der Ports im Web-UI?
Andernfalls muss noch die PORTORDER Variable in der network.* Datei ergänzt werden.

Reihenfolge sieht gut aus.

Kann die auf dem Label aufgedruckte MAC-Adresse mit folgenden Befehlen korrekt ausgegeben werden?
. /lib/functions.sh
. /lib/functions/system.sh
get_mac_label
Andernfalls müsste noch die ROUTERMAC Variable in der network.* Datei ergänzt werden.

Ja, es wird die korrekte MAC ausgegeben.

Sind die MAC-Adressen aller Batman-Interfaces (w2mesh, w5mesh, eth0.3) unterschiedlich?

Jawoll, eth im ersten byte, die meshes im letzen:

> Sieht gut aus. Hast du beide Varianten getestet oder nur node? Test ist beschränkt auf die node-variante, Layer3 ist komplett ungetestet. > In jedem Fall kannst du für den Test noch ein `Tested-by: Name <email@domain.tld>` ergänzen, entweder im Commit, oder als Kommentar (dann ergänze ich das beim mergen). Ist im commit, PR wurde geupdated :) > Passt bei der node-Variante die Reihenfolge der Ports im Web-UI? > Andernfalls muss noch die PORTORDER Variable in der network.* Datei ergänzt werden. Reihenfolge sieht gut aus. > Kann die auf dem Label aufgedruckte MAC-Adresse mit folgenden Befehlen korrekt ausgegeben werden? > ``` > . /lib/functions.sh > . /lib/functions/system.sh > get_mac_label > ``` > Andernfalls müsste noch die ROUTERMAC Variable in der network.* Datei ergänzt werden. Ja, es wird die korrekte MAC ausgegeben. > Sind die MAC-Adressen aller Batman-Interfaces (w2mesh, w5mesh, eth0.3) unterschiedlich? Jawoll, eth im ersten byte, die meshes im letzen: ![image](/attachments/ff4a02e6-017a-4104-a244-1198ff858744)

image.png

70 KiB

fbl commented

2023-02-25 22:04:38 +01:00

Danke! Ist gemerged.

fbl closed this pull request

2023-02-25 22:04:38 +01:00

ci/woodpecker/pr/woodpecker Pipeline is pending

Details