Description of problem: in /usr/share/man/man3/Sys::Virt::StoragePool.3pm.gz for list_all_volumes, it list my @volumes = $dom->list_all_volumes($flags) Return a list of all storage volumes associated with this domain. The elements in the returned list are instances of the Sys::Virt::StorageVol class. The $flags parameter can be used to filter the list of return storage volumes. But it real test, list_all_volumes belongs to pool object but not domain, and it list all volumes in pool but not in domain. And now the flag only can be set to 0, so the doc need to be updated Also for my @nets = $pool->list_volumes() the return parameter better to set to @volumes but not @nets Version-Release number of selected component (if applicable): perl-Sys-Virt-0.10.2-4.el6.x86_64 How reproducible: 100% Steps to Reproduce: 1. 2. 3. Actual results: Expected results: Additional info:
commit 1bc198f982e7749661ccf9071c5ee6669ba7e09b Author: Daniel P. Berrange <berrange> Date: Tue Dec 11 10:57:02 2012 +0000 Fix misc typos in POD docs for StoragePool.pm Signed-off-by: Daniel P. Berrange <berrange>
Verify pass on perl-Sys-Virt-0.10.2-5.el6.x86_64 check doc my @vols = $pool->list_volumes() Return a list of all volumes in the storage pool. The elements in the returned list are instances of the Sys::Virt::StorageVol class. This method requires O(n) RPC calls, so the "list_all_volumes" method is recommended as a more efficient alternative. my @volumes = $pool->list_all_volumes($flags) Return a list of all storage volumes associated with this pool. The elements in the returned list are instances of the Sys::Virt::StorageVol class. The $flags parameter can be used to filter the list of return storage volumes.
Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory, and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. http://rhn.redhat.com/errata/RHBA-2013-0377.html