Re: [libvirt] [PATCH 1.2.8] storage: zfs: fix double listing of new volumes

2014-08-29 Thread Roman Bogorodskiy
John Ferlan wrote: On 08/27/2014 05:02 AM, Roman Bogorodskiy wrote: Currently, after calling commands to create a new volumes, virStorageBackendZFSCreateVol calls virStorageBackendZFSFindVols that calls virStorageBackendZFSParseVol. virStorageBackendZFSParseVol checks if a

Re: [libvirt] [PATCH 1.2.8] storage: zfs: fix double listing of new volumes

2014-08-28 Thread John Ferlan
On 08/27/2014 05:02 AM, Roman Bogorodskiy wrote: Currently, after calling commands to create a new volumes, virStorageBackendZFSCreateVol calls virStorageBackendZFSFindVols that calls virStorageBackendZFSParseVol. virStorageBackendZFSParseVol checks if a volume already exists by trying to

[libvirt] [PATCH 1.2.8] storage: zfs: fix double listing of new volumes

2014-08-27 Thread Roman Bogorodskiy
Currently, after calling commands to create a new volumes, virStorageBackendZFSCreateVol calls virStorageBackendZFSFindVols that calls virStorageBackendZFSParseVol. virStorageBackendZFSParseVol checks if a volume already exists by trying to get it using virStorageVolDefFindByName. For a just