[libvirt] [PATCH] virsh: Check wether found volume is member of the specified storage pool
Peter Krempa
pkrempa at redhat.com
Mon Jun 2 09:05:18 UTC 2014
On 05/30/14 16:28, Martin Kletzander wrote:
> On Fri, May 30, 2014 at 03:39:36PM +0200, Peter Krempa wrote:
>> When looking up storage volumes virsh uses multiple lookup steps. Some
>> of the steps don't require a pool name specified. This resulted into a
>> possibility that a volume would be part of a different pool than the
>> user specified:
>>
>> Let's have a /var/lib/libvirt/images/test.qcow image in the 'default'
>> pool and a second pool 'emptypool':
>>
>> Currently we'd return:
>> $ virsh vol-info --pool emptypool /var/lib/libvirt/images/test.qcow
>> Name: test.qcow
>> Type: file
>> Capacity: 100.00 MiB
>> Allocation: 212.00 KiB
>>
>
> I believed that the --pool parameter for vol-info (and *some* others)
> was only a hint in case you had more volumes with the same name
> (specifying absolute path with a pool doesn't make any sense). That
> would mean the BZ is notabug actually, but let's assume such users
> exist...
>
>> After the fix:
>> $ tools/virsh vol-info --pool emptypool /var/lib/libvirt/images/test.qcow
>> error: Requested volume '/var/lib/libvirt/images/test.qcow' found in a
>> different pool (default) than specified
>>
>
> I'd say this is rather noisy. How about changing it to ...'%s' not
> found in pool '%s'... or "is not in pool '%s'?
>
> ACK after release with or without the change mentioned above.
I went with your shorter wording and fixed the typo in the subject
pointed out by Jan. Pushed now.
Thanks.
Peter
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 901 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20140602/418111d3/attachment-0001.sig>
More information about the libvir-list
mailing list