Merge juju-bmc:update-snapcraft-plugs-naming into juju-bmc:master

Proposed by Jose Guedez
Status: Merged
Approved by: Jose Guedez
Approved revision: 88bdfdae0c2cebf101c510322c023691b24c4e06
Merged at revision: 51f726be644756cde50c5b1917dde6a5178cfaca
Proposed branch: juju-bmc:update-snapcraft-plugs-naming
Merge into: juju-bmc:master
Diff against target: 32 lines (+7/-5)
1 file modified
snap/snapcraft.yaml (+7/-5)
Reviewer Review Type Date Requested Status
Joe Guo (community) Approve
Review via email: mp+385444@code.launchpad.net

Commit message

update snapcraft.yaml plugs per suggestions from snap review

To post a comment you must log in.
Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

This merge proposal is being monitored by mergebot. Change the status to Approved to merge.

Revision history for this message
Jose Guedez (jfguedez) wrote :

Changes to address comments during snap review - https://forum.snapcraft.io/t/juju-bmc-request-to-use-personal-files/18065

Already tested in a couple clouds.

Revision history for this message
Joe Guo (guoqiao) wrote :

LGTM, +1.

review: Approve
Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

Change successfully merged at revision 51f726be644756cde50c5b1917dde6a5178cfaca

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/snap/snapcraft.yaml b/snap/snapcraft.yaml
index 0522094..e58e4d0 100644
--- a/snap/snapcraft.yaml
+++ b/snap/snapcraft.yaml
@@ -11,20 +11,22 @@ grade: devel # must be 'stable' to release into candidate/stable channels
11confinement: strict11confinement: strict
1212
13plugs:13plugs:
14 config-files:14 dot-maascli-db:
15 interface: personal-files15 interface: personal-files
16 read:
17 - $HOME/.maascli.db
18 - $HOME/.local/share/juju
19 write:16 write:
20 - $HOME/.maascli.db17 - $HOME/.maascli.db
18 dot-local-share-juju:
19 interface: personal-files
20 read:
21 - $HOME/.local/share/juju
2122
22apps:23apps:
23 juju-bmc:24 juju-bmc:
24 command: bin/juju-bmc25 command: bin/juju-bmc
25 plugs:26 plugs:
26 - network27 - network
27 - config-files28 - dot-maascli-db
29 - dot-local-share-juju
28 - ssh-keys30 - ssh-keys
2931
30parts:32parts:

Subscribers

People subscribed via source and target branches

to all changes: