Merge ~andreserl/maas:snap_confinement into maas:master

Proposed by Andres Rodriguez
Status: Merged
Approved by: Andres Rodriguez
Approved revision: a71081f6ec5ec9a2b9e9e490c4cce78eb8860a76
Merge reported by: MAAS Lander
Merged at revision: not available
Proposed branch: ~andreserl/maas:snap_confinement
Merge into: maas:master
Diff against target: 13 lines (+1/-1)
1 file modified
snap/snapcraft.yaml (+1/-1)
Reviewer Review Type Date Requested Status
Blake Rouse (community) Approve
Review via email: mp+333976@code.launchpad.net

Commit message

Change snap confinement to strict.

To post a comment you must log in.
Revision history for this message
Blake Rouse (blake-rouse) wrote :

Just as note this will allow users to install with

snap install --edge

Without --devmode, meaning the snap won't work. Same applies anyway when in --stable and --beta, just wanted to note that before landing.

review: Approve
Revision history for this message
Andres Rodriguez (andreserl) wrote :

agreed, i think in the docs we should highlight that --devmode is needed.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/snap/snapcraft.yaml b/snap/snapcraft.yaml
2index 0873d81..69d628b 100644
3--- a/snap/snapcraft.yaml
4+++ b/snap/snapcraft.yaml
5@@ -5,7 +5,7 @@ summary: Metal as a Service
6 description: |
7 Total automation of your physical servers for amazing data center operational efficiency.
8 grade: devel
9-confinement: devmode
10+confinement: strict
11
12 apps:
13 supervisor:

Subscribers

People subscribed via source and target branches