Merge lp:~jonas-drange/ubuntu-system-settings/hotspots-binding into lp:ubuntu-system-settings
| Status: | Superseded |
|---|---|
| Proposed branch: | lp:~jonas-drange/ubuntu-system-settings/hotspots-binding |
| Merge into: | lp:ubuntu-system-settings |
| Diff against target: |
3076 lines (+2449/-276) 16 files modified
plugins/cellular/CMakeLists.txt (+1/-0) plugins/cellular/Common.qml (+97/-0) plugins/cellular/Components/MultiSim.qml (+7/-2) plugins/cellular/Components/SingleSim.qml (+6/-2) plugins/cellular/Hotspot.qml (+39/-22) plugins/cellular/HotspotSetup.qml (+301/-70) plugins/cellular/hotspotmanager.cpp (+619/-159) plugins/cellular/hotspotmanager.h (+147/-19) plugins/system-update/system_update.cpp (+7/-0) plugins/system-update/system_update.h (+2/-0) plugins/system-update/update_manager.h (+4/-0) tests/autopilot/ubuntu_system_settings/__init__.py (+132/-0) tests/autopilot/ubuntu_system_settings/tests/__init__.py (+62/-1) tests/autopilot/ubuntu_system_settings/tests/networkmanager.py (+902/-0) tests/autopilot/ubuntu_system_settings/tests/test_cellular.py (+122/-1) tests/plugins/system-update/fakesystemupdate.h (+1/-0) |
| To merge this branch: | bzr merge lp:~jonas-drange/ubuntu-system-settings/hotspots-binding |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Ken VanDine | 2015-03-09 | Needs Fixing on 2015-05-15 | |
| PS Jenkins bot | continuous-integration | Approve on 2015-05-13 | |
| Mathieu Trudel-Lapierre | Abstain on 2015-03-19 | ||
| Matthew Paul Thomas | design | 2015-03-10 | Pending |
|
Review via email:
|
|||
This proposal has been superseded by a proposal from 2015-05-18.
Commit Message
[cellular] ap support in binding and rewriting the API to make it easier to interact with
Description of the Change
Changes
This modifies the current hotspotmanager binding to do AP and adhoc hotspots, but AP by default. It also mimics [1] in a lot of ways.
So, roughly it
* adds means to talk to URFKill and wpa_supplicant, to respectively soft block/unblock Wi-Fi, and change the interface driver if necessary (which it is on all hybris devices),
* uses NetworkManager events to be less racy,
* and checks for failures and reports them to the user.
The UI has been changed to use the new binding, albeit still hidden due to bug 1426923 bug 1429314 bug 1421671 and bug 1427358.
How to test
If libhybris device (krillin):
1. Edit /etc/dbus-
2. Confirm you can call $ gdbus call --system -d fi.w1.wpa_
All devices:
1. Edit /usr/share/
2. Change 'Exec' line to be Exec=env USS_SHOW_ALL_UI=1 system-settings %u or do $ initctl set-env USS_SHOW_ALL_UI=1
Checklist
* Is your branch in sync with latest trunk (e.g. bzr pull lp:trunk -> no changes)
Yes
* Did you build your software in a clean sbuild/pbuilder chroot or ppa?
Yes
* Did you build your software in a clean sbuild/pbuilder armhf chroot or ppa?
Yes
* Has your component "TestPlan” been executed successfully on emulator, N4?
Yes (Krillin, Arale), not N4 due to bug from above.
* Has a 5 minute exploratory testing run been executed on N4?
No, see above.
* If you changed the packaging (debian), did you subscribe a core-dev to this MP?
N/A
* If you changed the UI, did you subscribe the design-reviewers to this MP?
Yes (mpt)
* What components might get impacted by your changes?
Cellular for Single and Multi-SIM
* Have you requested review by the teams of these owning components?
N/A
[1] http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1339
http://
Executed test runs:
UNSTABLE: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
PASSED: Continuous integration, rev:1354
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
PASSED: Continuous integration, rev:1359
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
| Ken VanDine (ken-vandine) wrote : | # |
See a couple comments and questions inline.
| Jonas G. Drange (jonas-drange) wrote : | # |
Information given, but sat MP back to WIP to work on what Ken pointed out.
| PS Jenkins bot (ps-jenkins) wrote : | # |
PASSED: Continuous integration, rev:1362
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
PASSED: Continuous integration, rev:1363
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
| Ken VanDine (ken-vandine) wrote : | # |
In my testing I found that the ssidField which needs to be 8 or more characters only becomes valid when you enter the 9th character or the field loses focus. This MP is more about the backend it needs and the UI is still hidden. Not sure we want to block on that. However the hotspot I created on my mako appeared to work, but I couldn't make a successful connection to it from my laptop.
| Mathieu Trudel-Lapierre (cyphermox) wrote : | # |
I only reviewed the backend code but it looks fine to me; see one minor inline comment about wifi security.
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1364
http://
Executed test runs:
UNSTABLE: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1365
http://
Executed test runs:
UNSTABLE: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
| Ken VanDine (ken-vandine) wrote : | # |
Looks good, I pushed a fix to show the UI and disable predictive text in the password to another branch
| PS Jenkins bot (ps-jenkins) wrote : | # |
PASSED: Continuous integration, rev:1368
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
- 1369. By Jonas G. Drange on 2015-05-13
-
style
- 1370. By Jonas G. Drange on 2015-05-13
-
skip if wifi disabled
| PS Jenkins bot (ps-jenkins) wrote : | # |
PASSED: Continuous integration, rev:1370
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
| Ken VanDine (ken-vandine) wrote : | # |
Please fix the indenting of the .cpp and .h files, you used 2 spaces instead of 4. Also see the comments inline in the diff.
- 1371. By Jonas G. Drange on 2015-05-15
-
start addressing ken's comments
- 1372. By Jonas G. Drange on 2015-05-18
-
four tests, fixing setup and teardown
- 1373. By Jonas G. Drange on 2015-05-18
-
fixing some test imports, indent now 4 spaces
- 1374. By Jonas G. Drange on 2015-05-18
-
merge trunk
- 1375. By Jonas G. Drange on 2015-05-18
-
merge prereq branch
- 1376. By Jonas G. Drange on 2015-05-18
-
hide if mako
- 1377. By Jonas G. Drange on 2015-05-18
-
add comment to import
- 1378. By Jonas G. Drange on 2015-05-18
-
hiding completely on mako
- 1379. By Jonas G. Drange on 2015-05-22
-
unhide for testing
- 1380. By Jonas G. Drange on 2015-05-22
-
remove enabledshim
- 1381. By Jonas G. Drange on 2015-05-22
-
using usc serverpropertys
ynchroniser - 1382. By Jonas G. Drange on 2015-05-22
-
foo
- 1383. By Jonas G. Drange on 2015-05-26
-
let the checkbox render before starting hotspot
- 1384. By Jonas G. Drange on 2015-05-26
-
disable back button so as to ensure that the wifi is soft unblocked
- 1385. By Jonas G. Drange on 2015-07-16
-
sync trunk
- 1386. By Jonas G. Drange on 2015-07-23
-
sync with trunk
- 1387. By Jonas G. Drange on 2015-07-31
-
sync with trunk


FAILED: Continuous integration, rev:1337 jenkins. qa.ubuntu. com/job/ ubuntu- system- settings- ci/2013/ jenkins. qa.ubuntu. com/job/ generic- deb-autopilot- vivid-touch/ 1695 jenkins. qa.ubuntu. com/job/ ubuntu- system- settings- vivid-i386- ci/283 jenkins. qa.ubuntu. com/job/ generic- deb-autopilot- runner- vivid-mako/ 1503 jenkins. qa.ubuntu. com/job/ generic- mediumtests- builder- vivid-armhf/ 1693 jenkins. qa.ubuntu. com/job/ generic- mediumtests- builder- vivid-armhf/ 1693/artifact/ work/output/ *zip*/output. zip s-jenkins. ubuntu- ci:8080/ job/touch- flash-device/ 18664
http://
Executed test runs:
UNSTABLE: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild: s-jenkins. ubuntu- ci:8080/ job/ubuntu- system- settings- ci/2013/ rebuild
http://