[virt-tools-list] [virt-manager PATCH] details: disable config disk with readonly and shareable at the same time

Chen Hanxiao chen_han_xiao at 126.com
Tue Jan 28 15:16:42 UTC 2014


At 2014-01-28 22:21:26,"Cole Robinson" <crobinso at redhat.com> wrote:

>On 01/28/2014 02:09 AM, Chen Hanxiao wrote:
>> From: Chen Hanxiao <chenhanxiao at cn.fujitsu.com>
>> 
>> Currently we could config disk with readonly and
>> shareable at the same time, which is meaningless.
>> virsh had already discouraged users doing this.
>> This patch will disable users to config both
>> readonly and shareable at the same time by UI.
>> 
>
>How does 'virsh' complain? If libvirt throws an error about this, I'd rather
>just let it complain and show that error to the user, than reproduce their
>error check, since this is a fairly minor corner case.
>

virsh could not do this since commit:

f919cf691735535dedc66a2cae244350ebb6c5e5
virsh # attach-disk aa 1.img sdd --shareable --mode=readonly --print-xml
error: option --mode already seen
If throwing errors is not acceptable, do we have some better metod to avoid this?
Thanks
-Chen
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/virt-tools-list/attachments/20140128/b4589169/attachment.htm>


More information about the virt-tools-list mailing list