Re: [PATCH v2 0/7] storage:dir: ploop volumes support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



08.02.2016 16:04, Olga Krishtal пишет:
In-Reply-To:

v2:
- fixed memory leak
- chenged the return value of all helper functions to 0/-1.
Now check for success is smth like that: vir****Ploop() < 0
- fixed some identation issues.


Overall looks good, except minor issues in some patches. Also, I wonder if we should add some check to ensure ploop binary existence? This would make user experience much better. For instance, instead of unconditionally returning virStorageBackendCreatePloop in case of ploop volumes in virStorageBackendGetBuildVolFromFunction we could check for ploop external tool and report an error message that it wasn't detected and it is required to be installed for PLOOP storage pools.

Maxim Nestratov

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]