Skip to content

Commit

Permalink
many: lessen the use of core-support
Browse files Browse the repository at this point in the history
We would like to remove core-support but cannot do that at this stage.
As a second best thing prepare snapd for its eventual removal from the
core snap (the plug side).

This patch prepares snapd master for (eventual) removal of the
core-support plug from snapd.

Signed-off-by: Zygmunt Krynicki <zygmunt.krynicki@canonical.com>
  • Loading branch information
zyga committed Jul 3, 2018
1 parent f569912 commit 39f348e
Show file tree
Hide file tree
Showing 8 changed files with 12 additions and 54 deletions.
3 changes: 0 additions & 3 deletions tests/main/classic-ubuntu-core-transition-auth/task.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -68,6 +68,3 @@ execute: |
echo "ubuntu-core still installed, transition failed"
exit 1
fi
echo "Ensure interfaces are connected"
snap interfaces -i core-support | MATCH ":core-support.*core:core-support-plug"
3 changes: 0 additions & 3 deletions tests/main/classic-ubuntu-core-transition-two-cores/task.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -73,6 +73,3 @@ execute: |
exit 1
fi
snap interfaces -i network | MATCH ":network.*test-snapd-python-webserver"
echo "Ensure interfaces are connected"
snap interfaces -i core-support | MATCH ":core-support.*core:core-support-plug"
3 changes: 0 additions & 3 deletions tests/main/classic-ubuntu-core-transition/task.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -118,9 +118,6 @@ execute: |
echo "Ensure the webserver is working after a snap restart"
curl http://localhost | MATCH "XKCD rocks"
echo "Ensure interfaces are connected"
snap interfaces -i core-support | MATCH ":core-support.*core:core-support-plug"
echo "Ensure snap set core works"
snap set core system.power-key-action=ignore
if [ "$(snap get core system.power-key-action)" != "ignore" ]; then
Expand Down
7 changes: 0 additions & 7 deletions tests/main/interfaces-many/task.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -93,13 +93,6 @@ execute: |
CONNECTED_PATTERN="$slot_iface +$CONSUMER_SNAP"
DISCONNECTED_PATTERN="$slot_iface +-"
# The core-support plug is connected by the core snap, so account for
# that
if [ "$plug_iface" = "$CONSUMER_SNAP:core-support" ]; then
CONNECTED_PATTERN="$slot_iface +core:core-support-plug,$CONSUMER_SNAP"
DISCONNECTED_PATTERN="$slot_iface +core:core-support-plug$"
fi
# Skip any interfaces that core doesn't ship
if ! snap interfaces | MATCH "$slot_iface +" ; then
echo "$slot_iface not present, skipping"
Expand Down
6 changes: 0 additions & 6 deletions tests/main/snap-interface/snap-interface-core-support.yaml

This file was deleted.

6 changes: 6 additions & 0 deletions tests/main/snap-interface/snap-interface-network.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
name: network
summary: allows access to the network
plugs:
- network-consumer
slots:
- system
9 changes: 6 additions & 3 deletions tests/main/snap-interface/task.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,13 @@ summary: Check that "snap interface" works as expected
details: |
The "snap interface" command displays a listing of used interfaces
execute: |
snap interface | MATCH 'core-support\s+ special permissions for the core snap'
# shellcheck source=tests/lib/snaps.sh
. "$TESTSLIB/snaps.sh"
install_local network-consumer
snap interface | MATCH 'network\s+ allows access to the network'
snap interface --all | MATCH 'classic-support\s+ special permissions for the classic snap'
snap interface core-support > out.yaml
diff -u out.yaml snap-interface-core-support.yaml
snap interface network > out.yaml
diff -u out.yaml snap-interface-network.yaml
restore: |
rm -f out.yaml
29 changes: 0 additions & 29 deletions tests/regression/lp-1721518/task.yaml

This file was deleted.

0 comments on commit 39f348e

Please sign in to comment.