curtin-dev team mailing list archive

[Olivier Gayot <mp+490155@xxxxxxxxxxxxxxxxxx>]

Thanks for noticing the issue regarding zfsroot! Is my assumption correct that we don't support encryption with zfsroot for now?

Diff comments:

> diff --git a/curtin/block/zfs.py b/curtin/block/zfs.py
> index bb78138..9f4eb8c 100644
> --- a/curtin/block/zfs.py
> +++ b/curtin/block/zfs.py
> @@ -462,6 +462,8 @@ def get_zpool_from_config(cfg):
>      sconfig = cfg['storage']['config']
>      for item in sconfig:
>          if item['type'] == 'zpool':
> +            if only_encrypted and item.get('encryption_style') is None:
> +                continue
>              zpools.append(item['pool'])
>          elif item['type'] == 'format':

Nice catch, thanks! updated

>              if item['fstype'] == 'zfsroot':


-- 
https://code.launchpad.net/~ogayot/curtin/+git/curtin/+merge/490155
Your team curtin developers is requested to review the proposed merge of ~ogayot/curtin:encrypted-zpool-requires-cryptsetup into curtin:master.