virsh: cmdDomdisplayReload: Require option name for --type

As this command was introduced in this release add the flag requiring to
pass optionname.

This is needed to actually disallow positional parsing of the value
despite documenting that the flag name is required.

Signed-off-by: Peter Krempa <pkrempa@redhat.com>
Reviewed-by: Ján Tomko <jtomko@redhat.com>

diff --git a/docs/manpages/virsh.rst b/docs/manpages/virsh.rst
index 115b802c45daf1483e06aba77a90a12f6ec484a0..28d430bf92f67e1f21b565605381092aae220b8f 100644 (file)
--- a/docs/manpages/virsh.rst
+++ b/docs/manpages/virsh.rst
@@ -1790,7 +1790,7 @@ domdisplay-reload
 
 ::
 
-    domdisplay-reload <domain> [[--type] <type>]
+    domdisplay-reload <domain> [--type <type>]
 
 Reload the domain's graphical display. This reloads its TLS certificates
 without restarting the domain. ``type`` can be any constant from the

diff --git a/tools/virsh-domain.c b/tools/virsh-domain.c
index 35809a866b254f9da7544a03c09d7d8b1bef40d8..cd378286605463999d4e904a0a15746f580ae72b 100644 (file)
--- a/tools/virsh-domain.c
+++ b/tools/virsh-domain.c
@@ -13290,6 +13290,7 @@ static const vshCmdOptDef opts_domdisplay_reload[] = {
     VIRSH_COMMON_OPT_DOMAIN_FULL(0),
     {.name = "type",
      .type = VSH_OT_INT,
+     .flags = VSH_OFLAG_REQ_OPT,
      .help = N_("graphics display type")
     },
     {.name = NULL}