[libvirt] [PATCH] virsh: add new vol-pool command
Eric Blake
eblake at redhat.com
Wed Jun 9 00:54:43 UTC 2010
On 06/08/2010 08:06 AM, Justin Clift wrote:
> This patch adds a new "vol-pool" command to virsh, to round out the
> identifier conversion functions for volumes in virsh. Now it is
> possible to work with volumes when starting from just a volume key
> or volume path.
When resending, it helps to use 'git send-email
--subject-prefix=PATCHv2' to call attention to the fact that it is a resend.
> ---
Also, if you list what changed between v1 and v2, it makes subsequent
reviews easier for anyone who read the first version. Listing it after
the --- output by git means that it is not part of the git commit
message but is part of the email (here, I use git send-email --annotate
to pull up an editor on everything I'm about to send, and type after the
---); but several others on this list find it easier to track their v2,
v3, ... changes directly in the commit message. I kind of like the
sideband effect of doing it only in the email (a year from now, when
reading 'git log', I don't care how many tries it took you to get to
what was actually committed, if the intermediate tries weren't also
committed), but tracking the changes with 'git commit --amend' is also
nice, so a compromise is to keep your interdiff changelog in the commit
messages locally, then manually move the --- up a few lines when doing
the 'git send-email' so that someone else doing 'git am' doesn't replay
the history.
> tools/virsh.c | 48 ++++++++++++++++++++++++++++++++++++++++++++++++
> 1 files changed, 48 insertions(+), 0 deletions(-)
Given your separate email with the virsh.pod patches for this and
several other vol-* commands at once,
ACK.
--
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/20100608/c4724604/attachment-0001.sig>
More information about the libvir-list
mailing list