Re: [PATCH 7/8] virsh: Introduce virshSecretUUIDCompleter

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

 




On 01/12/2018 09:37 AM, Michal Privoznik wrote:
> This is a slight change from previous patches since virSecret
> does not have a name only UUID strings.
> 
> Signed-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx>
> ---
>  tools/virsh-completer.c | 46 ++++++++++++++++++++++++++++++++++++++++++++++
>  tools/virsh-completer.h |  4 ++++
>  tools/virsh-secret.c    | 15 ++++++++++-----
>  3 files changed, 60 insertions(+), 5 deletions(-)
> 

Reviewed-by: John Ferlan <jferlan@xxxxxxxxxx>

John


[...]

>  };
> @@ -667,7 +671,8 @@ static const vshCmdInfo info_secret_event[] = {
>  static const vshCmdOptDef opts_secret_event[] = {
>      {.name = "secret",
>       .type = VSH_OT_STRING,
> -     .help = N_("filter by secret name or uuid")
> +     .help = N_("filter by secret name or uuid"),

Wonder what a secret name is? ;-)  I know separate issue...  This and
the virsh.pod should be adjusted.

> +     .completer = virshSecretUUIDCompleter,
>      },
>      {.name = "event",
>       .type = VSH_OT_STRING,
> 

--
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