virsh: Add coredump format completion to dump command

Signed-off-by: Lin Ma <lma@suse.com>
Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
Reviewed-by: Michal Privoznik <mprivozn@redhat.com>

diff --git a/tools/virsh-completer-domain.c b/tools/virsh-completer-domain.c
index 1f72b856b4f89f1e0b769ccd63dbd93b64c6ea37..20d503ff09c83b400c1655034de55bfc5e0390a2 100644 (file)
--- a/tools/virsh-completer-domain.c
+++ b/tools/virsh-completer-domain.c
@@ -940,3 +940,22 @@ virshDomainFSMountpointsCompleter(vshControl *ctl,
     virshDomainFree(dom);
     return ret;
 }
+
+
+char **
+virshDomainCoreDumpFormatCompleter(vshControl *ctl G_GNUC_UNUSED,
+                                   const vshCmd *cmd G_GNUC_UNUSED,
+                                   unsigned int flags)
+{
+    char **ret = NULL;
+    size_t i;
+
+    virCheckFlags(0, NULL);
+
+    ret = g_new0(char *, VIR_DOMAIN_CORE_DUMP_FORMAT_LAST + 1);
+
+    for (i = 0; i < VIR_DOMAIN_CORE_DUMP_FORMAT_LAST; i++)
+        ret[i] = g_strdup(virDomainCoreDumpFormatTypeToString(i));
+
+    return ret;
+}

diff --git a/tools/virsh-completer-domain.h b/tools/virsh-completer-domain.h
index ef242d0c688f868aa1c67ac5ed0d8e909213d6f0..4e16d845145190a1a57940743dc26d4ef8c06783 100644 (file)
--- a/tools/virsh-completer-domain.h
+++ b/tools/virsh-completer-domain.h
@@ -122,3 +122,8 @@ char ** virshKeycodeNameCompleter(vshControl *ctl,
 char ** virshDomainFSMountpointsCompleter(vshControl *ctl,
                                           const vshCmd *cmd,
                                           unsigned int flags);
+
+char **
+virshDomainCoreDumpFormatCompleter(vshControl *ctl,
+                                   const vshCmd *cmd,
+                                   unsigned int flags);

diff --git a/tools/virsh-domain.c b/tools/virsh-domain.c
index 78276dd8479a1cd503661a10dd77c88369d8ab96..ecad3a5e5dcd31e2d58bc3b42dc84ef26fd96680 100644 (file)
--- a/tools/virsh-domain.c
+++ b/tools/virsh-domain.c
@@ -5405,6 +5405,8 @@ static const vshCmdOptDef opts_dump[] = {
     },
     {.name = "format",
      .type = VSH_OT_STRING,
+     .flags = VSH_OFLAG_NONE,
+     .completer = virshDomainCoreDumpFormatCompleter,
      .help = N_("specify the format of memory-only dump")
     },
     {.name = NULL}