On Mon, Jul 03, 2023 at 16:53:28 -0600, Matt Low wrote:
The current virtStorageBackendZFSCheckPool checks for the existence
of a
path under /dev/zvol/ to determine if the pool is active. ZFS does not
create a path under /dev/zvol/ if no ZFS volumes have been created under
a particular dataset, thus, empty ZFS storage pools are deactivated
whenever checkPool is called on them (as noted in referenced issue).
This commit changes virStorageBackendZFSCheckPool so that the 'zfs list'
command is used to explicitly check for the existence a dataset
specified by the pool's def->source.name.
Resolves:
https://gitlab.com/libvirt/libvirt/-/issues/221
Signed-off-by: Matt Low <matt(a)mlow.ca>
---
src/storage/storage_backend_zfs.c | 20 +++++++++++++++++---
1 file changed, 17 insertions(+), 3 deletions(-)
Reviewed-by: Peter Krempa <pkrempa(a)redhat.com>
I'll push it shortly.