[libvirt] [PATCH] virsh: introduce name-uuid for list command

[Chen Hanxiao]

At 2016-06-15 18:12:33, "Ján Tomko" <jtomko at redhat.com> wrote:
>On Wed, Jun 15, 2016 at 05:36:05PM +0800, Chen Hanxiao wrote:
>>From: Chen Hanxiao <chenhanxiao at gmail.com>
>>
>>This patch will show both name and UUID of domain:
>>
>> Id    Name                           State      UUID
>> ---------------------------------------------------------------------------------------
>>  3     f23                            running    918f1dd6-b19f-412b-ba17-d113bad89af8
>>
>>Signed-off-by: Chen Hanxiao <chenhanxiao at gmail.com>
>>---
>> tools/virsh-domain-monitor.c | 27 ++++++++++++++++++++++++---
>> 1 file changed, 24 insertions(+), 3 deletions(-)
>>
>>diff --git a/tools/virsh-domain-monitor.c b/tools/virsh-domain-monitor.c
>>index 0a93949..32e4ea4 100644
>>--- a/tools/virsh-domain-monitor.c
>>+++ b/tools/virsh-domain-monitor.c
>>@@ -1844,10 +1849,10 @@ cmdList(vshControl *ctl, const vshCmd *cmd)
>>     FILTER("state-shutoff", VIR_CONNECT_LIST_DOMAINS_SHUTOFF);
>>     FILTER("state-other",   VIR_CONNECT_LIST_DOMAINS_OTHER);
>>
>>-    if (optTable + optName + optUUID > 1) {
>>+    if (optTable + optName + optUUID + optNameUuid > 1) {
>>         vshError(ctl, "%s",
>>-                 _("Only one argument from --table, --name and --uuid "
>>-                   "may be specified."));
>>+                 _("Only one argument from --table, --name, --uuid "
>>+                   "and --name-uuid may be specified."));
>
>Allowing both --name and --uuid for the same functionality would be
>nicer, is there a reason why we can't do that?
>

--uuid show UUIDs without title.
Guessing it's used for scripts?

Regards,
- Chen

>Jan
>>         return false;
>>     }
>>
>
>--
>libvir-list mailing list
>libvir-list at redhat.com
>https://www.redhat.com/mailman/listinfo/libvir-list