[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

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



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 redhat com    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]