freifunk-franken
/
firmware
9
7
Fork 13
Code Issues 48 Pull Requests 7 Releases 17 Wiki Activity

fff-network: remove iwconfig #328

Closed
rohammer wants to merge 1 commits from rohammer/firmware:nodewatcher-remove-iwconfig into master
pull from: rohammer/firmware:nodewatcher-remove-iwconfig
merge into: freifunk-franken:master
Conversation 14 Commits 1 Files Changed 1 +1 -8
rohammer commented 2024-03-25 22:48:46 +01:00
Member
Copy Link

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>
rohammer added 1 commit 2024-03-25 22:48:47 +01:00
ci/woodpecker/pr/woodpecker Pipeline is pending Details
ad0544ffb1 fff-network: remove iwconfig 
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
Author
Member
Copy Link

das Monitoring mag das Leerzeichen nicht.

das Monitoring mag das Leerzeichen nicht.
adschm commented 2024-03-29 18:28:18 +01:00
Owner
Copy Link

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
Owner
Copy Link

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.

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
Owner
Copy Link

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
Owner
Copy Link

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
Owner
Copy Link

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
Author
Member
Copy Link

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
Owner
Copy Link

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
Owner
Copy Link

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
Author
Member
Copy Link

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
Author
Member
Copy Link

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
Owner
Copy Link

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. :-)

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. :-)
rohammer commented 2024-04-01 21:25:32 +02:00
Author
Member
Copy Link

Du kannst das gerne noch ändern. Und ich freue mich wieder was gelernt zu haben!

Du kannst das gerne noch ändern. Und ich freue mich wieder was gelernt zu haben!
fbl commented 2024-04-01 23:54:53 +02:00
Owner
Copy Link

Applied, danke!

Applied, danke!
fbl closed this pull request 2024-04-01 23:54:53 +02:00
Some checks are pending
ci/woodpecker/pr/woodpecker Pipeline is pending
Details
ci/woodpecker/pull_request_closed/woodpecker Pipeline is pending
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
fbl
Labels
Clear labels   
RFC

   
RFT

   
WIP

   
blocked

   
bsp

touching bsp files or adding devices

  
bug

Something is not working

  
build/scripts/tools

touching buildscript or other tools

  
duplicate

This issue or pull request already exists

  
feature

New feature

  
fixed

Issue that got resolved

  
layer3

only for layer3 FW

  
mantis

Imported from mantis

  
more details required

Reported information is insufficient to debug the issue

  
needs changes

   
node

only for node FW

  
packages/fff

touching our packages

  
rejected

Rejected or won't fix

  
security

   
trivial

simple non-functional changes that are mostly matter of taste

  
upstream

touching upstream version, build or feed patches

No Label
RFC
RFT
WIP
blocked
bsp
bug
build/scripts/tools
duplicate
feature
fixed
layer3
mantis
more details required
needs changes
node
packages/fff
rejected
security
trivial
upstream
Milestone
Clear milestone
No items
No Milestone
20240401-beta
Assignees
Clear assignees
No Assignees
fbl
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: freifunk-franken/firmware#328
No description provided.
Delete Branch "rohammer/firmware:nodewatcher-remove-iwconfig"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?