Merge ~morphis/snappy-hwe-snaps/+git/network-manager:f/fix-alias-setup into ~snappy-hwe-team/snappy-hwe-snaps/+git/network-manager:master

Proposed by Simon Fels
Status: Merged
Approved by: Konrad Zapałowicz
Approved revision: 5b125d7b699be3dddea19ea41ee3552c83d70f6f
Merged at revision: e6a2c7606bf7599d5d5bdce3448140f3943418db
Proposed branch: ~morphis/snappy-hwe-snaps/+git/network-manager:f/fix-alias-setup
Merge into: ~snappy-hwe-team/snappy-hwe-snaps/+git/network-manager:master
Diff against target: 41 lines (+8/-5)
2 files modified
snapcraft.yaml (+2/-4)
tests/main/aliases/task.yaml (+6/-1)
Reviewer Review Type Date Requested Status
Konrad Zapałowicz (community) code Approve
System Enablement Bot continuous-integration Approve
Review via email: mp+323972@code.launchpad.net

Description of the change

tests: fix alias setup for newer snapd versions

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: Approve (continuous-integration)
Revision history for this message
Konrad Zapałowicz (kzapalowicz) wrote :

ack

review: Approve (code)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/snapcraft.yaml b/snapcraft.yaml
2index 9e0dcc3..9bf87e2 100644
3--- a/snapcraft.yaml
4+++ b/snapcraft.yaml
5@@ -25,8 +25,7 @@ apps:
6 nmcli:
7 command: usr/bin/nmcli
8 plugs: [nmcli]
9- aliases:
10- - nmcli
11+ aliases: [nmcli]
12 networkmanager:
13 command: bin/networkmanager
14 daemon: simple
15@@ -39,8 +38,7 @@ apps:
16 # within the systemd unit file instead for example. Until this
17 # is implemented we keep the alias disabled.
18 #
19- # aliases:
20- # - NetworkManager
21+ # aliases: [NetworkManager]
22
23 parts:
24 hooks:
25diff --git a/tests/main/aliases/task.yaml b/tests/main/aliases/task.yaml
26index e2eff9d..f3fd040 100644
27--- a/tests/main/aliases/task.yaml
28+++ b/tests/main/aliases/task.yaml
29@@ -5,7 +5,12 @@ execute: |
30 # have them approved from the store in our snap-declaration assertion.
31 test ! -e /snap/bin/nmcli
32
33- snap alias network-manager nmcli
34+ snapd_version=$(snap version | awk '/^snapd / {print $2; exit}')
35+ target=$SNAP_NAME.nmcli
36+ if dpkg --compare-versions $snapd_version lt 2.25 ; then
37+ target=$SNAP_NAME
38+ fi
39+ snap alias $target nmcli
40
41 test -e /snap/bin/nmcli
42

Subscribers

People subscribed via source and target branches