Merge ~alfonsosanchezbeato/snappy-hwe-snaps/+git/modem-manager:add-proxies into ~snappy-hwe-team/snappy-hwe-snaps/+git/modem-manager:master

Proposed by Alfonso Sanchez-Beato
Status: Merged
Approved by: Simon Fels
Approved revision: b1f57b020dca0e95a813f87bc4edd5fadb3028a1
Merged at revision: 0d4017f706313565e34e87d9d72cc3cd423c6c98
Proposed branch: ~alfonsosanchezbeato/snappy-hwe-snaps/+git/modem-manager:add-proxies
Merge into: ~snappy-hwe-team/snappy-hwe-snaps/+git/modem-manager:master
Diff against target: 78 lines (+19/-2)
4 files modified
run-tests.sh (+1/-1)
snapcraft.yaml (+2/-0)
tests/lib/prepare-all.sh (+9/-0)
tests/lib/prepare.sh (+7/-1)
Reviewer Review Type Date Requested Status
Simon Fels Approve
System Enablement Bot continuous-integration Approve
Review via email: mp+313779@code.launchpad.net

Description of the change

Add mbim and qmi proxies to the snap

To post a comment you must log in.
Revision history for this message
System Enablement Bot (system-enablement-ci-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
System Enablement Bot (system-enablement-ci-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
System Enablement Bot (system-enablement-ci-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
System Enablement Bot (system-enablement-ci-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
System Enablement Bot (system-enablement-ci-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Simon Fels (morphis) :
review: Needs Fixing
Revision history for this message
System Enablement Bot (system-enablement-ci-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Alfonso Sanchez-Beato (alfonsosanchezbeato) wrote :

Removed version bump.

<<
What causes this to be started? Can this be tested without having a correct modem available or faked somehow?
>>

It is started when a modem with mbim support is detected. There is no infrastructure for faking this easily, so a real modem is needed for testing.

Revision history for this message
System Enablement Bot (system-enablement-ci-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
System Enablement Bot (system-enablement-ci-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
System Enablement Bot (system-enablement-ci-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
System Enablement Bot (system-enablement-ci-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
System Enablement Bot (system-enablement-ci-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
System Enablement Bot (system-enablement-ci-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Simon Fels (morphis) wrote :

As CI wont pass anymore now (see https://code.launchpad.net/~morphis/snappy-hwe-snaps/+git/network-manager/+merge/314223) we need the same fix for modem-manager. Can you propose one with the same change?

Also if we can't create an automatic integration test we need a manual plainbox test. Can you propose one and link here?

Revision history for this message
System Enablement Bot (system-enablement-ci-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
System Enablement Bot (system-enablement-ci-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Simon Fels (morphis) wrote :

LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/run-tests.sh b/run-tests.sh
2index f2e9079..b4ce91b 100755
3--- a/run-tests.sh
4+++ b/run-tests.sh
5@@ -17,7 +17,7 @@
6 set -ex
7
8 image_name=ubuntu-core-16.img
9-channel=candidate
10+channel=stable
11 spread_opts=
12 force_new_image=0
13 test_from_channel=0
14diff --git a/snapcraft.yaml b/snapcraft.yaml
15index f8dfd22..5e4800c 100644
16--- a/snapcraft.yaml
17+++ b/snapcraft.yaml
18@@ -49,6 +49,7 @@ parts:
19 filesets:
20 wanted:
21 - lib/libmbim-glib.so*
22+ - usr/lib/libmbim/mbim-proxy
23
24 snap:
25 - $wanted
26@@ -72,6 +73,7 @@ parts:
27 filesets:
28 wanted:
29 - lib/libqmi-glib.so*
30+ - usr/lib/libqmi/qmi-proxy
31
32 snap:
33 - $wanted
34diff --git a/tests/lib/prepare-all.sh b/tests/lib/prepare-all.sh
35index 38bfb5b..1755933 100644
36--- a/tests/lib/prepare-all.sh
37+++ b/tests/lib/prepare-all.sh
38@@ -11,7 +11,16 @@ snap install --devmode --beta classic
39 cat <<-EOF > /home/test/build-snap.sh
40 #!/bin/sh
41 set -ex
42+
43+# FIXME: Enable propose for now until problems with conflicting systemd
44+# packages between the Ubuntu Core image ppa and the archive are fixed.
45+echo "deb http://archive.ubuntu.com/ubuntu/ xenial-proposed restricted main universe" > /etc/apt/sources.list.d/ubuntu-proposed.list
46+
47+# Ensure we have the latest updates installed as the core snap
48+# may be a bit out of date.
49 apt update
50+apt full-upgrade -y --force-yes
51+
52 apt install -y --force-yes snapcraft
53 cd /home/modem-manager
54 snapcraft clean
55diff --git a/tests/lib/prepare.sh b/tests/lib/prepare.sh
56index 8956644..aa3c329 100644
57--- a/tests/lib/prepare.sh
58+++ b/tests/lib/prepare.sh
59@@ -29,12 +29,18 @@ if [ -n "$SNAP_CHANNEL" ] ; then
60 snap install --$SNAP_CHANNEL modem-manager
61 fi
62 else
63+ # Install first from store to avoid error when performing the connection
64+ snap install modem-manager
65 # Install prebuilt modem-manager snap
66 snap install --dangerous /home/modem-manager/modem-manager_*_amd64.snap
67 # As we have a snap which we build locally its unasserted and therefore
68 # we don't have any snap-declarations in place and need to manually
69 # connect all plugs.
70- sudo snap connect modem-manager:mmcli modem-manager:service
71+ # NOTE This step is not needed (and will produce an error) because we
72+ # have made an initial installation for the store. When snapd finally
73+ # allows somehow connections of locally installed snaps this can be put
74+ # in place again, and the installation from the store removed.
75+ ##sudo snap connect modem-manager:mmcli modem-manager:service
76 fi
77
78 # Snapshot of the current snapd state for a later restore

Subscribers

People subscribed via source and target branches