Merge ~alfonsosanchezbeato/snappy-hwe-snaps/+git/modem-manager:add-proxies into ~snappy-hwe-team/snappy-hwe-snaps/+git/modem-manager:master
- Git
- lp:~alfonsosanchezbeato/snappy-hwe-snaps/+git/modem-manager
- add-proxies
- Merge into master
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) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Simon Fels | Approve | ||
System Enablement Bot | continuous-integration | Approve | |
Review via email: mp+313779@code.launchpad.net |
Commit message
Description of the change
Add mbim and qmi proxies to the snap
System Enablement Bot (system-enablement-ci-bot) wrote : | # |
System Enablement Bot (system-enablement-ci-bot) wrote : | # |
FAILED: Continuous integration, rev:ea110b16ab3
https:/
Executed test runs:
FAILURE: https:/
None: https:/
Click here to trigger a rebuild:
https:/
System Enablement Bot (system-enablement-ci-bot) wrote : | # |
FAILED: Continuous integration, rev:7413efaee7b
https:/
Executed test runs:
FAILURE: https:/
None: https:/
Click here to trigger a rebuild:
https:/
System Enablement Bot (system-enablement-ci-bot) wrote : | # |
FAILED: Continuous integration, rev:7413efaee7b
https:/
Executed test runs:
FAILURE: https:/
None: https:/
Click here to trigger a rebuild:
https:/
System Enablement Bot (system-enablement-ci-bot) wrote : | # |
FAILED: Continuous integration, rev:ac64a1e7a1d
https:/
Executed test runs:
FAILURE: https:/
None: https:/
Click here to trigger a rebuild:
https:/
Simon Fels (morphis) : | # |
System Enablement Bot (system-enablement-ci-bot) wrote : | # |
FAILED: Continuous integration, rev:ac64a1e7a1d
https:/
Executed test runs:
FAILURE: https:/
None: https:/
Click here to trigger a rebuild:
https:/
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.
System Enablement Bot (system-enablement-ci-bot) wrote : | # |
FAILED: Continuous integration, rev:8a90a1c61a9
https:/
Executed test runs:
FAILURE: https:/
None: https:/
Click here to trigger a rebuild:
https:/
System Enablement Bot (system-enablement-ci-bot) wrote : | # |
FAILED: Continuous integration, rev:8a90a1c61a9
https:/
Executed test runs:
FAILURE: https:/
None: https:/
Click here to trigger a rebuild:
https:/
System Enablement Bot (system-enablement-ci-bot) wrote : | # |
FAILED: Continuous integration, rev:833509cd1ab
https:/
Executed test runs:
FAILURE: https:/
None: https:/
Click here to trigger a rebuild:
https:/
System Enablement Bot (system-enablement-ci-bot) wrote : | # |
FAILED: Continuous integration, rev:833509cd1ab
https:/
Executed test runs:
FAILURE: https:/
None: https:/
Click here to trigger a rebuild:
https:/
System Enablement Bot (system-enablement-ci-bot) wrote : | # |
PASSED: Continuous integration, rev:a60f8daafb5
https:/
Executed test runs:
SUCCESS: https:/
None: https:/
Click here to trigger a rebuild:
https:/
System Enablement Bot (system-enablement-ci-bot) wrote : | # |
PASSED: Continuous integration, rev:a60f8daafb5
https:/
Executed test runs:
SUCCESS: https:/
None: https:/
Click here to trigger a rebuild:
https:/
Simon Fels (morphis) wrote : | # |
As CI wont pass anymore now (see https:/
Also if we can't create an automatic integration test we need a manual plainbox test. Can you propose one and link here?
System Enablement Bot (system-enablement-ci-bot) wrote : | # |
FAILED: Continuous integration, rev:b1f57b020dc
https:/
Executed test runs:
FAILURE: https:/
None: https:/
Click here to trigger a rebuild:
https:/
System Enablement Bot (system-enablement-ci-bot) wrote : | # |
PASSED: Continuous integration, rev:b1f57b020dc
https:/
Executed test runs:
SUCCESS: https:/
None: https:/
Click here to trigger a rebuild:
https:/
Preview Diff
1 | diff --git a/run-tests.sh b/run-tests.sh |
2 | index 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 |
14 | diff --git a/snapcraft.yaml b/snapcraft.yaml |
15 | index 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 |
34 | diff --git a/tests/lib/prepare-all.sh b/tests/lib/prepare-all.sh |
35 | index 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 |
55 | diff --git a/tests/lib/prepare.sh b/tests/lib/prepare.sh |
56 | index 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 |
FAILED: Continuous integration, rev:ea110b16ab3 ca1f3eded8f770e ec688b6fd00553 /jenkins. canonical. com/system- enablement/ job/generic- build-snap/ 540/ /jenkins. canonical. com/system- enablement/ job/generic- run-snap- spread- tests/277/ console /jenkins. canonical. com/system- enablement/ job/generic- update- snap-mp/ 448/console
https:/
Executed test runs:
FAILURE: https:/
None: https:/
Click here to trigger a rebuild: /jenkins. canonical. com/system- enablement/ job/generic- build-snap/ 540/rebuild
https:/