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
1diff --git a/jobs/resource.pxu b/jobs/resource.pxu
2index dbcfd1f..d443754 100644
3--- a/jobs/resource.pxu
4+++ b/jobs/resource.pxu
5@@ -58,11 +58,6 @@ unit: packaging meta-data
6 os-id: debian
7 Depends: python3-requests-unixsocket
8
9-# This is for wireless_sta_protocol
10-unit: packaging meta-data
11-os-id: debian
12-Depends: wireless-tools
13-
14 id: cpuinfo
15 estimated_duration: 0.37
16 plugin: resource
17@@ -277,9 +272,8 @@ id: wireless_sta_protocol
18 plugin: resource
19 _summary: Resource job to identify Wi-Fi STA supported protocols
20 _description:
21- Job listing STA supported protocols (802.11n, 802.11ac) per interfaces.
22+ Job listing STA supported 802.11 protocols per interfaces.
23 command:
24- 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
25 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
26 estimated_duration: 0.5
27 flags: preserve-locale

Subscribers

People subscribed via source and target branches