canonical / checkbox

Checkbox
https://checkbox.readthedocs.io
GNU General Public License v3.0
30 stars 44 forks source link

Wrong Netplan config causes ax/ac/bg/n wpa cases failed #1303

Closed baconYao closed 1 week ago

baconYao commented 1 week ago

Bug Description

The wrong netplan config used in Checkbox causes WPA cases failed.

Wrong config like below [1]:

# This is the network config written by checkbox
network:
  version: 2
  wifis:
    wlp1s0:
      access-points:
        cert-ac-wpa-tel-l4:
          auth:
            password: insecure
      dhcp4: true
      nameservers: {}

Add key-management: psk in auth can solve this issue.

[1]: Submission: https://certification.canonical.com/hardware/202307-31859/submission/375752/test-results/?term=_np

To Reproduce

  1. Install Checkbox Snap on UC22 image
  2. Run wireless-automated test plan

Environment

Relevant log output

No response

Additional context

No response

syncronize-issues-to-jira[bot] commented 1 week ago

Thank you for reporting us your feedback!

The internal ticket has been created: https://warthogs.atlassian.net/browse/CHECKBOX-1471.

This message was autogenerated

baconYao commented 1 week ago

The default configuration of Netplan generated by console-conf is like below and it's working

# This is the network config written by 'console-conf'
network:
  ethernets:
    eth0:
      dhcp4: true
  version: 2
  wifis:
    wlp1s0:
      access-points:
        cert-ac-wpa-tel-l4:
          password: insecure
      dhcp4: true
zongminl commented 1 week ago

Will be fixed by #1301

p-gentili commented 1 week ago

To add to the above, we went from (270):

# This is the network config written by checkbox
network:
  version: 2
  wifis:
    wlan0:
      access-points:
        cert-ac-wpa-tel-l3-01: {password: insecure}
      addresses: []
      dhcp4: True
      nameservers: {}

to (312):

# This is the network config written by checkbox
network:
  version: 2
  wifis:
    wlan0:
      access-points:
        cert-ac-wpa-tel-l3-01:
          auth:
            password: insecure
      dhcp4: true
      nameservers: {}
jocave commented 1 week ago

Change for the above: https://github.com/canonical/checkbox/commit/a506108725a61357557f2944628054e4cfcdbc90

Hook25 commented 1 week ago

Fixed dae49f5