Merge ~sylvain-pineau/plainbox-provider-resource:remove_n_wireless_sta_resource into plainbox-provider-resource:master

Proposed by Sylvain Pineau
Status: Merged
Approved by: Sylvain Pineau
Approved revision: a8c8fe4560ab397a19e0f50c2c201b0353871a98
Merged at revision: af590fc3147a7a2169495721b875f3147b9f2656
Proposed branch: ~sylvain-pineau/plainbox-provider-resource:remove_n_wireless_sta_resource
Merge into: plainbox-provider-resource:master
Diff against target: 27 lines (+1/-7)
1 file modified
jobs/resource.pxu (+1/-7)
Reviewer Review Type Date Requested Status
Sylvain Pineau (community) Approve
Review via email: mp+342061@code.launchpad.net

Description of the change

Remove N detection since using iwconfig is not future proof.

Nota: We already assume N is always supported i nthe wireless connection jobs

To post a comment you must log in.
Revision history for this message
Sylvain Pineau (sylvain-pineau) wrote :

self-approved

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/jobs/resource.pxu b/jobs/resource.pxu
index dbcfd1f..d443754 100644
--- a/jobs/resource.pxu
+++ b/jobs/resource.pxu
@@ -58,11 +58,6 @@ unit: packaging meta-data
58os-id: debian58os-id: debian
59Depends: python3-requests-unixsocket59Depends: python3-requests-unixsocket
6060
61# This is for wireless_sta_protocol
62unit: packaging meta-data
63os-id: debian
64Depends: wireless-tools
65
66id: cpuinfo61id: cpuinfo
67estimated_duration: 0.3762estimated_duration: 0.37
68plugin: resource63plugin: resource
@@ -277,9 +272,8 @@ id: wireless_sta_protocol
277plugin: resource272plugin: resource
278_summary: Resource job to identify Wi-Fi STA supported protocols273_summary: Resource job to identify Wi-Fi STA supported protocols
279_description:274_description:
280 Job listing STA supported protocols (802.11n, 802.11ac) per interfaces.275 Job listing STA supported 802.11 protocols per interfaces.
281command:276command:
282 for i in `iw dev | grep -oP 'Interface\s+\K\w+'`; do iwconfig $i | grep -q '802.11abgn' && echo $i"_n: supported" || echo $i"_n: unsupported"; done
283 for i in `iw dev | grep -oP 'Interface\s+\K\w+'`; do iw phy phy$(iw dev $i info | grep -oP 'wiphy\s+\K\d+') info | grep -q 'VHT' && echo $i"_ac: supported" || echo $i"_ac: unsupported"; done277 for i in `iw dev | grep -oP 'Interface\s+\K\w+'`; do iw phy phy$(iw dev $i info | grep -oP 'wiphy\s+\K\d+') info | grep -q 'VHT' && echo $i"_ac: supported" || echo $i"_ac: unsupported"; done
284estimated_duration: 0.5278estimated_duration: 0.5
285flags: preserve-locale279flags: preserve-locale

Subscribers

People subscribed via source and target branches