Re: [PATCH] virsh: pool-list: introduce --uuid

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

 



On Thu, Jan 05, 2017 at 11:56:21AM +0800, Chen Hanxiao wrote:
> From: Chen Hanxiao <chenhanxiao@xxxxxxxxx>
> 
> 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

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]
  Powered by Linux