Re: [libvirt] [PATCH] virsh: pool-list: introduce --uuid
On Thu, Jan 05, 2017 at 11:56:21AM +0800, Chen Hanxiao wrote:
From: Chen Hanxiao <chenhanxiao(a)gmail.com>
This patch will introduce option [--uuid].
If specified, UUID of pools will be printed out.
virsh # pool-list --all
Name State Autostart
-------------------------------------------
default active yes
root active yes
virsh # pool-list --all --uuid
Name State Autostart UUID
---------------------------------------------------------------------------------
default active yes 45fdb1f3-402d-44c0-b2c0-a7be61fea6c7
root active yes eb397f25-9a44-459d-80df-330ca8f65ba8
The whole point of '--uuid' and '--name' options for the existing list
commands
is to improve shell scripting with virsh, i.e. avoid having to parse the format
of the table in order to extract some info and/or to call
<prefix>uuid command in a loop for every entry in the table. With your proposal
one would still have to parse the table contents which is in contrast in what
I've just mentioned. Also, this would introduce some inconsistency with how we
currently treat those options with other commands. So this patch will have to
be reworked and while at it, you could also introduce the '--name' option along
with '--uuid'.
Erik