Merge ~ack/maas:1976194-snap-maas-common-dir into maas:master

Proposed by Alberto Donato
Status: Merged
Approved by: Alberto Donato
Approved revision: f83f3471c317bddd46a4bdcbed447c28c884719b
Merge reported by: MAAS Lander
Merged at revision: not available
Proposed branch: ~ack/maas:1976194-snap-maas-common-dir
Merge into: maas:master
Diff against target: 12 lines (+1/-0)
1 file modified
snap/hooks/install (+1/-0)
Reviewer Review Type Date Requested Status
MAAS Lander Approve
Anton Troyanov Approve
Review via email: mp+423603@code.launchpad.net

Commit message

LP:1976194 ensure the $SNAP_COMMON/maas directory is always created

To post a comment you must log in.
Revision history for this message
MAAS Lander (maas-lander) wrote :

UNIT TESTS
-b 1976194-snap-maas-common-dir lp:~ack/maas/+git/maas into -b master lp:~maas-committers/maas

STATUS: FAILED
LOG: http://maas-ci.internal:8080/job/maas/job/branch-tester/12711/console
COMMIT: f83f3471c317bddd46a4bdcbed447c28c884719b

review: Needs Fixing
Revision history for this message
Alberto Donato (ack) wrote :

jenkins: !test

Revision history for this message
Anton Troyanov (troyanov) :
review: Approve
Revision history for this message
MAAS Lander (maas-lander) wrote :

UNIT TESTS
-b 1976194-snap-maas-common-dir lp:~ack/maas/+git/maas into -b master lp:~maas-committers/maas

STATUS: SUCCESS
COMMIT: f83f3471c317bddd46a4bdcbed447c28c884719b

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/snap/hooks/install b/snap/hooks/install
2index 73e0b03..fad804e 100755
3--- a/snap/hooks/install
4+++ b/snap/hooks/install
5@@ -3,6 +3,7 @@
6 # GNU Affero General Public License version 3 (see the file LICENSE).
7
8 mkdir -p \
9+ "$SNAP_COMMON/maas" \
10 "$SNAP_DATA/preseeds" \
11 "$SNAP_DATA/root" \
12 "$SNAP_DATA/supervisord"

Subscribers

People subscribed via source and target branches