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(a)redhat.com +1-801-349-2682
Libvirt virtualization library http://libvirt.org
Attachments:
- signature.asc (application/pgp-signature — 619 bytes)