At 2017-01-05 17:18:14, "Erik Skultety" <eskultet@xxxxxxxxxx> wrote: >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'. > As what we did in the other virsh command, such as 'list`, We could did as: 1) virsh # pool-list --name default root 2) virsh # pool-list --all --uuid 45fdb1f3-402d-44c0-b2c0-a7be61fea6c7 eb397f25-9a44-459d-80df-330ca8f65ba8 3) virsh # pool-list --all --uuid --name Name State Autostart UUID --------------------------------------------------------------------------------- default active yes 45fdb1f3-402d-44c0-b2c0-a7be61fea6c7 root active yes eb397f25-9a44-459d-80df-330ca8f65ba8 v2 will come soon. Regards, - Chen -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list
