[libvirt] [PATCH] virsh: Add --name and --description options to snapshot-create

Eric Blake eblake at redhat.com
Mon Apr 11 17:36:00 UTC 2011


On 04/10/2011 05:31 AM, Matthias Bolte wrote:
> This options are shortcuts to set name and description of a snapshot.
> 
> Suggested by Elias Probst
> ---
> 
> I'm not sure if this is be best approach. In case of the vol-* commands there
> is vol-create that takes and XML file and vol-create-as that takes a set of
> arguments.
> 
> So, should there actually be a snapshot-create-as to takes --name and
> --description options?

For consistency, I would argue that we don't have any other commands
that modify direct xml input, and so we shouldn't be starting that now.
 I agree that a snapshot-create-as that takes options (and builds xml
under the hood) would match with existing virsh paradigms better than
making the existing snapshot-create command modify input xml on the fly.
 In other words, I'm afraid that your concern was unfortunately valid,
and that we should NACK this patch in favor of one that adds a new
command instead.

-- 
Eric Blake   eblake at redhat.com    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 619 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20110411/be2326d8/attachment-0001.sig>


More information about the libvir-list mailing list