← Back to team overview

curtin-dev team mailing list archive

Re: [Merge] ~legovini/curtin:curtainer-mask-zfs-services into curtin:master

 


Diff comments:

> diff --git a/tools/curtainer b/tools/curtainer
> index 466d719..34ccfb6 100755
> --- a/tools/curtainer
> +++ b/tools/curtainer
> @@ -161,6 +161,14 @@ main() {
>  
>      inside "$name" which eatmydata >/dev/null || eatmydata=""
>  
> +    # curtin depends on zfsutils-linux via probert-storage, but zfsutils-linux
> +    # can't be installed in an unprivileged container as it fails to start
> +    # the zfs-mount and zfs-share services as /dev/zfs is missing. We do
> +    # not actually need ZFS to work in the container, so the problem can be
> +    # worked around by masking the services before the package is installed.
> +    inside "$name" systemctl mask zfs-mount

I can add a couple of `|| fail "msg"`, let's see if this actually works in Jenkins first.

> +    inside "$name" systemctl mask zfs-share
> +
>      if $proposed; then
>          mirror=$(inside $name awk '$1 == "deb" { print $2; exit(0); }' \
>              /etc/apt/sources.list) &&


-- 
https://code.launchpad.net/~legovini/curtin/+git/curtin/+merge/385820
Your team curtin developers is subscribed to branch curtin:master.


References