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 check

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 > tryi

[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 crea