On 06/15/2014 03:48 AM, Dongsheng Yang wrote:
> We can not attach a disk with mode of both readonly and shareable.

Correct, because those are orthogonal modes.  You can't be in both modes
at the same time.

> Before:
>       # virsh attach-disk --domain jeos --source attach.img --target vdb 
> --persistent --mode readonly --mode shareable
>       error: option --mode already seen
>       # virsh attach-disk --domain jeos --source attach.img --target vdb 
> --persistent --mode readonly --shareable
>       error: option --mode already seen
>       # virsh attach-disk --domain jeos --source attach.img --target vdb 
> --persistent --mode readonly,shareable
>       error: No support for readonly,shareable in command 'attach-disk'
> After:
>       # virsh attach-disk --domain jeos --source attach.img --target vdb 
> --persistent --mode readonly,shareable
>       Disk attached successfully

I'm inclined to NACK this patch.  Pick one or the other, you can't have
both.


-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

--
libvir-list mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to