fff-network: remove iwconfig #328

rohammer · 2024-03-25T22:48:46+01:00

rohammer commented

2024-03-25 22:48:46 +01:00

iwconfig is upstream scheduled for removal and no longer working.
It is only used once to get the txpower for the nodewatcher.

Fixes: #327

Signed-off-by: Robert Langhammer rlanghammer@web.de

iwconfig is upstream scheduled for removal and no longer working. It is only used once to get the txpower for the nodewatcher. Fixes: #327 Signed-off-by: Robert Langhammer <rlanghammer@web.de>

fbl requested review from fbl 2024-03-25 23:10:13 +01:00

fbl added this to the 20240401-beta milestone 2024-03-25 23:10:18 +01:00

fbl self-assigned this 2024-03-25 23:10:21 +01:00

rohammer force-pushed nodewatcher-remove-iwconfig from ad0544ffb1 to b113e8e1d6

2024-03-26 14:18:53 +01:00

Compare

rohammer commented

2024-03-26 14:22:06 +01:00

das Monitoring mag das Leerzeichen nicht.

adschm commented

2024-03-29 18:28:18 +01:00

Testet vielleicht mal bei einigen Geräten, ob der Wert tatsächlich gleich ist.

Irgendwo gab es da Unterschiede bei der Berücksichtigung des antenna_gain, ich weiß aber nicht mehr, ob das hier wirklich relevant ist.

Testet vielleicht mal bei einigen Geräten, ob der Wert tatsächlich gleich ist. Irgendwo gab es da Unterschiede bei der Berücksichtigung des antenna_gain, ich weiß aber nicht mehr, ob das hier wirklich relevant ist.

fbl commented

2024-04-01 12:36:28 +02:00

Wir müssen außerdem prüfen welche Werte vom Monitoring verarbeitet werden. Denn auch wenn die Werte redundant vorhanden sind, heißen sie ja anders.

fbl commented

2024-04-01 12:42:13 +02:00

Das Monitoring wertet bereits jetzt schon nur die iw-Werte aus - mit Ausnahme der wlan_tx_power.

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

Das Monitoring wertet bereits jetzt schon nur die `iw`-Werte aus - mit Ausnahme der `wlan_tx_power`. `Reviewed-by: Fabian Bläse <fabian@blaese.de>`

fbl commented

2024-04-01 12:57:28 +02:00

Vorher: <wlan_tx_power>20 dBm</wlan_tx_power>
Nachher: <wlan_tx_power>20.00dBm</wlan_tx_power>

Mindestens die Leerstelle sollten wir noch fixen, besser aber auch die .00.

Vorher: <wlan_tx_power>20 dBm</wlan_tx_power> Nachher: <wlan_tx_power>20.00dBm</wlan_tx_power> Mindestens die Leerstelle sollten wir noch fixen, besser aber auch die `.00`.

fbl commented

2024-04-01 13:18:01 +02:00

Testet vielleicht mal bei einigen Geräten, ob der Wert tatsächlich gleich ist.

Wenn das Gerät zusätzliche tx power offsets hat (z.B. bei der NanoStation Loco M2 wegen zusätzlichem Verstärker), dann unterscheiden sich iw dev und iwinfo um genau den Wert des TX power offsets.

> Testet vielleicht mal bei einigen Geräten, ob der Wert tatsächlich gleich ist. Wenn das Gerät zusätzliche tx power offsets hat (z.B. bei der NanoStation Loco M2 wegen zusätzlichem Verstärker), dann unterscheiden sich `iw dev` und `iwinfo` um genau den Wert des TX power offsets.

rohammer commented

2024-04-01 14:00:48 +02:00

Testet vielleicht mal bei einigen Geräten, ob der Wert tatsächlich gleich ist.

Wenn das Gerät zusätzliche tx power offsets hat (z.B. bei der NanoStation Loco M2 wegen zusätzlichem Verstärker), dann unterscheiden sich iw dev und iwinfo um genau den Wert des TX power offsets.

Bei meiner loco M2 ist es gleich. Jeweils 11dBm.

> > Testet vielleicht mal bei einigen Geräten, ob der Wert tatsächlich gleich ist. > > Wenn das Gerät zusätzliche tx power offsets hat (z.B. bei der NanoStation Loco M2 wegen zusätzlichem Verstärker), dann unterscheiden sich `iw dev` und `iwinfo` um genau den Wert des TX power offsets. Bei meiner loco M2 ist es gleich. Jeweils 11dBm.

fbl commented

2024-04-01 14:10:42 +02:00

Das ist komisch. Was steht beim TX power offset im iwinfo?

Das ist komisch. Was steht beim `TX power offset` im `iwinfo`?

adschm commented

2024-04-01 14:33:10 +02:00

Testet vielleicht mal bei einigen Geräten, ob der Wert tatsächlich gleich ist.

Wenn das Gerät zusätzliche tx power offsets hat (z.B. bei der NanoStation Loco M2 wegen zusätzlichem Verstärker), dann unterscheiden sich iw dev und iwinfo um genau den Wert des TX power offsets.

Bei der Picostation war das glaube ich so.

> > Testet vielleicht mal bei einigen Geräten, ob der Wert tatsächlich gleich ist. > > Wenn das Gerät zusätzliche tx power offsets hat (z.B. bei der NanoStation Loco M2 wegen zusätzlichem Verstärker), dann unterscheiden sich `iw dev` und `iwinfo` um genau den Wert des TX power offsets. Bei der Picostation war das glaube ich so.

rohammer commented

2024-04-01 16:46:47 +02:00

Das ist komisch. Was steht beim TX power offset im iwinfo?

iwinfo gibt es da nicht. Ich habe mit iwconfig verglichen. Da kommt ja aktuell auch der Wert fürs monitoring her.

> Das ist komisch. Was steht beim `TX power offset` im `iwinfo`? iwinfo gibt es da nicht. Ich habe mit iwconfig verglichen. Da kommt ja aktuell auch der Wert fürs monitoring her.

rohammer force-pushed nodewatcher-remove-iwconfig from b113e8e1d6 to eeaa89b441

2024-04-01 16:54:12 +02:00

Compare

rohammer commented

2024-04-01 16:57:12 +02:00

Vorher: <wlan_tx_power>20 dBm</wlan_tx_power>
Nachher: <wlan_tx_power>20.00dBm</wlan_tx_power>

Mindestens die Leerstelle sollten wir noch fixen, besser aber auch die .00.

Punkt und Leerzeichen verträgt das monitoring nicht.
Ohne Punkt mit Leerzeichen geht.

> Vorher: <wlan_tx_power>20 dBm</wlan_tx_power> > Nachher: <wlan_tx_power>20.00dBm</wlan_tx_power> > > Mindestens die Leerstelle sollten wir noch fixen, besser aber auch die `.00`. Punkt und Leerzeichen verträgt das monitoring nicht. Ohne Punkt mit Leerzeichen geht.

fbl commented

2024-04-01 18:53:48 +02:00

Ist garantiert, dass auf einen Punkt immer zwei Zeichen folgen? Ich hätte hier zur Sicherheit einfach alle Zeichen nach dem ersten Punkt verworfen, aber vielleicht bin ich da auch ein wenig paranoid. Alternativ könnte die Zahl auch mit printf "%.0f" parsen und "sauber" wieder ausgeben. Aber das soll den Patch jetzt nicht aufhalten, ich nehme ihn auch so. :-)