Merge ~jocave/plainbox-provider-checkbox:wireless-nm-conn-backup into plainbox-provider-checkbox:master
Status: | Merged |
---|---|
Approved by: | Jonathan Cave |
Approved revision: | 1b78eb86ff91861c9f527762afcbdcb4bff6d98c |
Merged at revision: | 928732fb124a96929fbab342dd40a8c5d2ca703e |
Proposed branch: | ~jocave/plainbox-provider-checkbox:wireless-nm-conn-backup |
Merge into: | plainbox-provider-checkbox:master |
Diff against target: |
208 lines (+127/-0) 3 files modified
bin/wifi_nmcli_backup.py (+79/-0) units/wireless/jobs.pxu (+28/-0) units/wireless/test-plan.pxu (+20/-0) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Sylvain Pineau (community) | Approve | ||
Review via email: mp+362872@code.launchpad.net |
Description of the change
Bug #1765350 requested that the wireless connection jobs restore any saved NetworkManager connection information upon completion of the test process.
The jobs removed any stored configurations to prevent any unintended re-connections to an AP that is not the one specified in the test procedure.
This MR creates new templated jobs to bracket the cert test plans that copy the NetworkManager configuration files to a predictable location prior and then restore and delete them afterwards. The files are root readable only and reload the configuration also requires root privileges.
Tested on my development machine. My stored APs were still present after running wireless-
LGTM, +1