add "Preferred" property to ConnectionContext interface
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Canonical System Image |
High
|
John McAleely | |||
| libqofono (Ubuntu) |
High
|
Jonas G. Drange | |||
| network-manager (Ubuntu) |
High
|
Tony Espy | |||
| network-manager (Ubuntu RTM) |
High
|
Tony Espy | |||
| nuntium (Ubuntu) |
High
|
Alfonso Sanchez-Beato | |||
| nuntium (Ubuntu RTM) |
High
|
Alfonso Sanchez-Beato | |||
| ofono (Ubuntu) |
High
|
Alfonso Sanchez-Beato | |||
| ofono (Ubuntu RTM) |
High
|
Alfonso Sanchez-Beato | |||
| ubuntu-system-settings (Ubuntu) |
High
|
Jonas G. Drange | |||
| ubuntu-system-settings (Ubuntu RTM) |
High
|
Jonas G. Drange | |||
Bug Description
To implement system-settings APN editor reliably there has to be a Tag field with is an arbitrary non user-visible string that can be set for any Context.
We also need org.ofono.
Related branches
- Ricardo Salveti: Approve on 2015-04-07
- Alfonso Sanchez-Beato: Approve on 2015-04-06
- PS Jenkins bot: Approve (continuous-integration) on 2015-04-01
-
Diff: 1314 lines (+686/-110)16 files modifieddebian/changelog (+19/-0)
doc/connman-api.txt (+8/-0)
drivers/mtkmodem/mtkrequest.h (+2/-2)
drivers/mtkmodem/mtkunsol.c (+35/-0)
drivers/mtkmodem/mtkunsol.h (+7/-0)
drivers/rilmodem/gprs-context.c (+9/-3)
gril/grilunsol.c (+75/-17)
include/modem.h (+5/-0)
plugins/bluetooth.c (+10/-0)
plugins/mtk.c (+326/-75)
plugins/ubuntu-apndb.c (+9/-10)
src/common.c (+1/-1)
src/gprs.c (+104/-1)
src/modem.c (+8/-0)
unit/test-common.c (+1/-1)
unit/test-mtkunsol.c (+67/-0)
- Ken VanDine: Approve on 2015-07-22
- PS Jenkins bot: Needs Fixing (continuous-integration) on 2015-07-22
- Alfonso Sanchez-Beato: Pending (mms/nuntium interactions) requested 2015-06-04
- Tony Espy: Pending (sanity) requested 2015-06-04
- Matthew Paul Thomas: Pending (design) requested 2015-06-04
-
Diff: 3490 lines (+2067/-1143)16 files modifiedplugins/cellular/CMakeLists.txt (+3/-4)
plugins/cellular/Components/CMakeLists.txt (+3/-0)
plugins/cellular/Components/KeyboardRectangle.qml (+74/-0)
plugins/cellular/Components/LabelTextField.qml (+54/-0)
plugins/cellular/Components/StandardAnimation.qml (+22/-0)
plugins/cellular/PageApnEditor.qml (+394/-346)
plugins/cellular/PageChooseApn.qml (+333/-535)
plugins/cellular/apn_editor.js (+192/-0)
plugins/cellular/apn_manager.js (+588/-60)
plugins/cellular/ofonoactivator.cpp (+0/-132)
plugins/cellular/ofonoactivator.h (+0/-42)
plugins/cellular/plugin.cpp (+0/-2)
tests/autopilot/ubuntu_system_settings/__init__.py (+147/-1)
tests/autopilot/ubuntu_system_settings/tests/__init__.py (+17/-21)
tests/autopilot/ubuntu_system_settings/tests/ofono.py (+99/-0)
tests/autopilot/ubuntu_system_settings/tests/test_cellular.py (+141/-0)
- Alfonso Sanchez-Beato: Needs Fixing on 2015-07-06
-
Diff: 613 lines (+549/-36)4 files modifieddebian/changelog (+9/-0)
debian/patches/0001-wwan-add-support-for-using-oFono-as-a-modem-manager.patch (+0/-36)
debian/patches/lp1361864-add-ofono-preferred-contexts.patch (+539/-0)
debian/patches/series (+1/-0)
- Mathieu Trudel-Lapierre: Approve on 2015-07-21
-
Diff: 686 lines (+251/-166)3 files modifieddebian/changelog (+3/-0)
debian/patches/0001-wwan-add-support-for-using-oFono-as-a-modem-manager.patch (+18/-63)
debian/patches/add_ofono_settings_support.patch (+230/-103)
- PS Jenkins bot: Needs Fixing (continuous-integration) on 2015-07-31
- Ken VanDine: Approve on 2015-07-27
-
Diff: 3487 lines (+2067/-1140)16 files modifiedplugins/cellular/CMakeLists.txt (+3/-4)
plugins/cellular/Components/CMakeLists.txt (+3/-0)
plugins/cellular/Components/KeyboardRectangle.qml (+74/-0)
plugins/cellular/Components/LabelTextField.qml (+54/-0)
plugins/cellular/Components/StandardAnimation.qml (+22/-0)
plugins/cellular/PageApnEditor.qml (+394/-346)
plugins/cellular/PageChooseApn.qml (+333/-535)
plugins/cellular/apn_editor.js (+192/-0)
plugins/cellular/apn_manager.js (+588/-60)
plugins/cellular/ofonoactivator.cpp (+0/-132)
plugins/cellular/ofonoactivator.h (+0/-42)
plugins/cellular/plugin.cpp (+0/-2)
tests/autopilot/ubuntu_system_settings/__init__.py (+147/-1)
tests/autopilot/ubuntu_system_settings/tests/__init__.py (+17/-18)
tests/autopilot/ubuntu_system_settings/tests/ofono.py (+99/-0)
tests/autopilot/ubuntu_system_settings/tests/test_cellular.py (+141/-0)
| Changed in ofono (Ubuntu): | |
| importance: | Undecided → Wishlist |
| Changed in ofono (Ubuntu): | |
| assignee: | nobody → Antti Kaijanmäki (kaijanmaki) |
| Tony Espy (awe) wrote : | #2 |
Moved status to Invalid as this isn't something I'm convinced we need.
| Changed in ofono (Ubuntu): | |
| status: | Incomplete → Invalid |
| Changed in ofono (Ubuntu): | |
| status: | Invalid → Confirmed |
| Tony Espy (awe) wrote : | #3 |
Changed back to Incomplete, as this came up again during a meeting this morning to discuss the current state of the APN editor.
I think what's needed is some way to indicate that an APN/context is the preferred APN for a Internet and/or MMS.
I'm not quite sure why a generic tag was proposed, as I think a simple boolean 'Preferred' property would suffice. Doing this also means we wouldn't need to add a new method either.
Note, NetworkManager currently reads the ofono gprs contexts directly from ofono's settings file ( /var/lib/
Also, if we were to implement this, NM's ofono code would need to change to honor the 'Preferred' APN, and ignore the other APNs. Nuntium would also have to change to honor this new property as well.
| Changed in ofono (Ubuntu): | |
| assignee: | Antti Kaijanmäki (kaijanmaki) → Alfonso Sanchez-Beato (alfonsosanchezbeato) |
| importance: | Wishlist → High |
| Changed in network-manager (Ubuntu): | |
| importance: | Undecided → High |
| Changed in nuntium (Ubuntu): | |
| importance: | Undecided → High |
| Changed in nuntium (Ubuntu): | |
| status: | New → Confirmed |
| Changed in network-manager (Ubuntu): | |
| status: | New → Confirmed |
| assignee: | nobody → Tony Espy (awe) |
| tags: | added: apn |
This brach
https:/
implements a per-context "Preferred" property that can be used freely by the ofono clients to express preference on context activation.
| Changed in ofono (Ubuntu): | |
| status: | Confirmed → In Progress |
| Changed in ubuntu-system-settings (Ubuntu): | |
| status: | New → Confirmed |
| Changed in ofono (Ubuntu): | |
| status: | In Progress → Fix Committed |
| Jonas G. Drange (jonas-drange) wrote : | #5 |
Can we use this tag for MMS as well? Since we cannot query nuntium directly (as per [1]).
[1] https:/
@Jonas, yes, it can be used for MMS too. There is in fact no restriction on how this flag is used on the ofono side. Whatever needs to be enforced is something between system-settings and NetworkManager. For ofono, it is just a property that can be read and written from outside, and it is not used internally.
| no longer affects: | libqofono (Ubuntu) |
| Launchpad Janitor (janitor) wrote : | #7 |
This bug was fixed in the package ofono - 1.12.bzr6892+
---------------
ofono (1.12.bzr6892+
[ Alfonso Sanchez-Beato ]
* ubuntu-apndb.c, src/common.c, test-common.c: provision IA APNs
LTE modems may require a new IA APN for non-GPRS operation,
this change allows a new IA APN type to be provisioned along
with Internet and MMS APNs.
* gril/grilunsol.c: LTE signal strength fix (LP: #1433867)
* ubuntu-apndb.c: load APNS w/out explict type (LP: #1437200)
* mtkodem, plugins/mtk.c, unit/tesk-
This change causes the firmware on specific MTK-based phone to be
dynamically switched/reset based on SIM type and roaming conditions.
* include, plugins/mtk.c, src/modem.c, gprs.c: set data for just one slot (LP: #1413672)
Make sure ConnectionManag
case the modem is of type dual SIM stand-by.
* doc, gprs: add 'Preferred' property to GPRS contexts (LP: #1361864)
-- CI Train Bot <email address hidden> Tue, 07 Apr 2015 06:50:28 +0000
| Changed in ofono (Ubuntu): | |
| status: | Fix Committed → Fix Released |
| Changed in libqofono (Ubuntu): | |
| importance: | Undecided → High |
| status: | New → Confirmed |
| assignee: | nobody → Jonas G. Drange (jonas-drange) |
| Changed in libqofono (Ubuntu): | |
| status: | Confirmed → In Progress |
| Changed in network-manager (Ubuntu): | |
| status: | Confirmed → In Progress |
| Changed in canonical-devices-system-image: | |
| status: | New → Confirmed |
| Changed in libqofono (Ubuntu): | |
| status: | In Progress → Fix Released |
| Changed in ubuntu-system-settings (Ubuntu): | |
| status: | Confirmed → In Progress |
| assignee: | nobody → Jonas G. Drange (jonas-drange) |
| summary: |
- add "Tag" property to ConnectionContext interface + add "Preferred" property to ConnectionContext interface |
Changes for nuntium in
| Changed in nuntium (Ubuntu): | |
| assignee: | nobody → Alfonso Sanchez-Beato (alfonsosanchezbeato) |
| status: | Confirmed → In Progress |
| Changed in canonical-devices-system-image: | |
| importance: | Undecided → High |
| milestone: | none → ww22-2015 |
| tags: | added: connectivity |
| Changed in nuntium (Ubuntu): | |
| status: | In Progress → Fix Released |
| Changed in canonical-devices-system-image: | |
| milestone: | ww22-2015 → ww24-2015 |
| Changed in ofono (Ubuntu RTM): | |
| status: | New → In Progress |
| importance: | Undecided → High |
| assignee: | nobody → Alfonso Sanchez-Beato (alfonsosanchezbeato) |
| Changed in ofono (Ubuntu): | |
| status: | Fix Released → In Progress |
| Changed in nuntium (Ubuntu): | |
| status: | Fix Released → In Progress |
| Changed in nuntium (Ubuntu RTM): | |
| assignee: | nobody → Alfonso Sanchez-Beato (alfonsosanchezbeato) |
| status: | New → In Progress |
| Tony Espy (awe) wrote : | #9 |
At a meeting this week, we decided that all of the components that may activate and/or use a context, should set the 'Preferred' flag for the context if it's successfully activated and/or used ( eg. a context is used by ofono for the INITIAL_ATTACH APN request; if this succeeds, ofono will set the context as 'Preferred' ).
As such, I've re-opened the bug tasks for the affected packages ( network-manager, nuntium, and ofono itself ), and made sure we have Ubuntu (RTM) tasks for the same to track their release as updates.
| Changed in ubuntu-system-settings (Ubuntu RTM): | |
| status: | New → In Progress |
| assignee: | nobody → jgdx (jgdx) |
| assignee: | jgdx (jgdx) → Jonas G. Drange (jonas-drange) |
| Changed in network-manager (Ubuntu RTM): | |
| importance: | Undecided → High |
| status: | New → In Progress |
| Changed in nuntium (Ubuntu RTM): | |
| importance: | Undecided → High |
| Changed in ubuntu-system-settings (Ubuntu): | |
| importance: | Undecided → High |
| Changed in ubuntu-system-settings (Ubuntu RTM): | |
| importance: | Undecided → High |
| Changed in network-manager (Ubuntu RTM): | |
| assignee: | nobody → Tony Espy (awe) |
| Changed in canonical-devices-system-image: | |
| milestone: | ww24-2015 → ww28-2015 |
| Launchpad Janitor (janitor) wrote : | #10 |
This bug was fixed in the package ofono - 1.12.bzr6900+
---------------
ofono (1.12.bzr6900+
[Tony Espy]
* test/rilmodem/sim: fix testing scripts for arale (LP: #1457775)
* test: reverse meaning of list-modems -p (LP: #1438715)
[ Alfonso Sanchez-Beato ]
* gril: cleanup traces
* src/gprs.c: set preferred for the used IA APN (LP: #1361864)
* rilmodem, mtkmodem, plugins/ril.c, plugins/mtk.c: retry when a
context deactivation request has finished with an error
* build, include, plugins/
src/gprs.c: resolve MMS proxy/MMSC host name (LP: #1417976)
[ Ratchanan Srirattanamet ]
* build, rilmodem, qcommsimmodem, plugins/ril.c,
src/
-- CI Train Bot <email address hidden> Thu, 02 Jul 2015 17:04:18 +0000
| Changed in ofono (Ubuntu): | |
| status: | In Progress → Fix Released |
| Łukasz Zemczak (sil2100) wrote : | #11 |
This bug was fixed in the package ofono 1.12.bzr6900+
---------------
ofono (1.12.bzr6900+
[Tony Espy]
* test/rilmodem/sim: fix testing scripts for arale (LP: #1457775)
* test: reverse meaning of list-modems -p (LP: #1438715)
[ Alfonso Sanchez-Beato ]
* gril: cleanup traces
* src/gprs.c: set preferred for the used IA APN (LP: #1361864)
* rilmodem, mtkmodem, plugins/ril.c, plugins/mtk.c: retry when a
context deactivation request has finished with an error
* build, include, plugins/
src/gprs.c: resolve MMS proxy/MMSC host name (LP: #1417976)
[ Ratchanan Srirattanamet ]
* build, rilmodem, qcommsimmodem, plugins/ril.c,
src/
-- CI Train Bot <email address hidden> Thu, 02 Jul 2015 17:04:18 +0000
| Changed in ofono (Ubuntu RTM): | |
| status: | In Progress → Fix Released |
| Changed in canonical-devices-system-image: | |
| milestone: | ww28-2015 → ww34-2015 |
| Changed in nuntium (Ubuntu): | |
| status: | In Progress → Fix Released |
| Changed in nuntium (Ubuntu RTM): | |
| status: | In Progress → Fix Released |
| Changed in canonical-devices-system-image: | |
| assignee: | nobody → John McAleely (john.mcaleely) |
| Tony Espy (awe) wrote : | #12 |
network-manager version 0.9.10.
| Changed in network-manager (Ubuntu RTM): | |
| status: | In Progress → Fix Released |
| no longer affects: | ubuntu-system-settings (Ubuntu Vivid) |
| Changed in canonical-devices-system-image: | |
| status: | Confirmed → In Progress |
| Łukasz Zemczak (sil2100) wrote : | #13 |
This bug was fixed in the package ubuntu-
---------------
ubuntu-
[ Ken VanDine ]
* [cellular/apn] new APN panel made for the new 'Preferred' property,
as well as the 'ResetContext' method. (LP: #1435360, #1361864,
#1376714, #1457676, #1424149, #1442646, #1388222)
ubuntu-
* No change upload, build with GCC 5.
ubuntu-
[ Christopher Lee ]
* Update AP helper so it doesn't attempt to use an unreachable value
when scrolling.
[ Ken VanDine ]
* Added a sleep to give set_forward_on_busy time to finish
* Use wait_select_single in the wifi dialog and bump the sleep to 1
second
* We need to explicitly stop dbus in tearDownClass, fixes lingering
dbus processes
ubuntu-
[ CI Train Bot ]
* New rebuild forced.
[ Manuel de la Pena ]
* Recompile with the new udm to avoid ABI issues.
ubuntu-
[ CI Train Bot ]
* Resync trunk.
[ Ken VanDine ]
* Use TextField.write to properly use the OSK Added helper to dismiss
OSK
[ jonas-drange ]
* [tests] avoids setting Interfaces property after the
Cellular/
ubuntu-
[ CI Train Bot ]
* Resync trunk.
[ Ken VanDine ]
* [wifi] Fixed handling of loading extended attributes from the
MenuModel (LP: #1472505)
[ handsome_feng<email address hidden> ]
* Add flickable in time/date page. (LP: #1469076)
[ jonas-drange ]
* [phone] make services insensitive when there are no services as well
as adding autopilot tests for sim services. (LP: #1470779)
ubuntu-
[ CI Train Bot ]
* New rebuild forced.
[ jonas-drange ]
* Lets the Wi-Fi panel respond to an SSID provided by dispatcher by
launching a "Connect to" dialogue. (LP: #1241986)
ubuntu-
[ CI Train Bot ]
* New rebuild forced.
[ Joerg Berroth ]
* Added support for 802-1x wireless network configurations. (LP:
#1241986)
ubuntu-
[ CI Train Bot ]
* New rebuild forced.
[ Ken VanDine ]
* Test searching for WiFi instead of Sound, online-accounts now has a
soundcloud account type (LP: #1468029)
[ Matthew Paul Thomas ]
* Changes "Install & Restart" to "Restart & Install" and "Not Now" to
"Cancel". lp:1359344 (LP: #1359344)
[ jonas-drange ]
* implement CFB, CFNRy and CFNRc as well as autopilot tests (LP:
#1463828)
ubuntu-
* debian/control:
- don't have an alternative depends on system-
it's d...
| Changed in ubuntu-system-settings (Ubuntu RTM): | |
| status: | In Progress → Fix Released |
| Changed in canonical-devices-system-image: | |
| status: | In Progress → Fix Released |
| Launchpad Janitor (janitor) wrote : | #14 |
This bug was fixed in the package network-manager - 0.9.10.0-4ubuntu19
---------------
network-manager (0.9.10.
[ Tony Espy ]
* debian/
that added APN, USERNAME and PASSWORD to NM_SETTING_GSM object.
NM doesn't actually need access to these settings, and USERNAME/
PASSWORD can cause issues with NM's secrets needed logic.
* debian/
debian/
debian/
debian/
to NMModemOfono's modem_state handling. Added get/set_
methods to NMSettingsConne
ofono connections to 30s. Finally added a 5s delay to NM_POLICY's activation
logic triggered when a modem device is disconnected. This allows modem time to
settle and NM to process the resulting DBus state changes. (LP: #1461593)
* debian/
debian/
debian/
debian/
changes collectively fix flight-mode on arale ( and other devices ), due to
some fundemental race conditions in the ofono logic. (LP: #1445080, #1440917)
* debian/
debian/
gprs-context 'Preferred' property. (LP: #1361864)
[ Mathieu Trudel-Lapierre ]
* d/p/0002-
Re-add schedule_scan() call after we get the ScanDone signal from the
supplicant. Otherwise we'd do one scan on startup and never scan again.
(LP: #1445134)
-- Mathieu Trudel-Lapierre <email address hidden> Wed, 05 Aug 2015 12:17:28 -0400
| Changed in network-manager (Ubuntu): | |
| status: | In Progress → Fix Released |
| Changed in ubuntu-system-settings (Ubuntu): | |
| status: | In Progress → Fix Released |
| status: | Fix Released → Fix Committed |
| Launchpad Janitor (janitor) wrote : | #15 |
This bug was fixed in the package ubuntu-
---------------
ubuntu-
[ Ken VanDine ]
* dismiss_osk after entering text
-- CI Train Bot <email address hidden> Fri, 14 Aug 2015 19:59:01 +0000
| Changed in ubuntu-system-settings (Ubuntu): | |
| status: | Fix Committed → Fix Released |


We've gone this path before... and met with incredible resistance from upstream ( and so did Jolla ) to this idea, so we went with the smarts for which context is "preferred" delegated to Network Manager and nuntium.
If you still feel strongly about this, we can revisit during the sprint.