Merge ~paride/curtin:curtainer-mask-zfs-services into curtin:master

Proposed by Paride Legovini
Status: Merged
Merged at revision: 14ce7d41155e8672f513d41504f21c13ba690abd
Proposed branch: ~paride/curtin:curtainer-mask-zfs-services
Merge into: curtin:master
Diff against target: 15 lines (+2/-2)
1 file modified
tools/curtainer (+2/-2)
Reviewer Review Type Date Requested Status
Richard Harding (community) Approve
Server Team CI bot continuous-integration Approve
Review via email: mp+385912@code.launchpad.net

Commit message

curtainer: fail is masking of zfs-mount or zfs-share fails

To post a comment you must log in.
Revision history for this message
Server Team CI bot (server-team-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Richard Harding (rharding) wrote :

TY

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/tools/curtainer b/tools/curtainer
2index 34ccfb6..a6fdc1b 100755
3--- a/tools/curtainer
4+++ b/tools/curtainer
5@@ -166,8 +166,8 @@ main() {
6 # the zfs-mount and zfs-share services as /dev/zfs is missing. We do
7 # not actually need ZFS to work in the container, so the problem can be
8 # worked around by masking the services before the package is installed.
9- inside "$name" systemctl mask zfs-mount
10- inside "$name" systemctl mask zfs-share
11+ inside "$name" systemctl mask zfs-mount || fail "failed to mask zfs-mount"
12+ inside "$name" systemctl mask zfs-share || fail "failed to mask zfs-share"
13
14 if $proposed; then
15 mirror=$(inside $name awk '$1 == "deb" { print $2; exit(0); }' \

Subscribers

People subscribed via source and target branches