[PATCH] storage: zfs: Use 'zfs list' to check pool status

[Michal Prívozník]

On 7/17/23 10:58, Peter Krempa wrote:
> 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 at mlow.ca>
>> ---
>>  src/storage/storage_backend_zfs.c | 20 +++++++++++++++++---
>>  1 file changed, 17 insertions(+), 3 deletions(-)
> 
> Reviewed-by: Peter Krempa <pkrempa at redhat.com>
> 
> I'll push it shortly.
> 

Pushed now.

Michal