virsh: Apply empty completer to arguments where completion doesn't make sense

Few outstanding arguments were not marked with 'virshCompleteEmpty'
completer despite the fact that we can't provide any reasonable
suggestion, e.g. for the new description of a domain or for the launch
secret.

Signed-off-by: Peter Krempa <pkrempa@redhat.com>
Reviewed-by: Michal Privoznik <mprivozn@redhat.com>
This commit is contained in:
Peter Krempa committed 2025-05-21 15:49:36 +02:00
1 parent 0f430122d3
commit 811749a266
2 files changed
+7

No files matched your search

+6
View File
@@ -8696,6 +8696,7 @@ static const vshCmdOptDef opts_desc[] = {
{.name = "new-desc",
.type = VSH_OT_ARGV,
.positional = true,
.completer = vshCompleteEmpty,
.help = N_("message")
},
{.name = NULL}
@@ -9936,10 +9937,12 @@ static const vshCmdOptDef opts_domsetlaunchsecstate[] = {
VIRSH_COMMON_OPT_DOMAIN_FULL(0),
{.name = "secrethdr",
.type = VSH_OT_STRING,
.completer = vshCompleteEmpty,
.help = N_("path to file containing the secret header"),
},
{.name = "secret",
.type = VSH_OT_STRING,
.completer = vshCompleteEmpty,
.help = N_("path to file containing the secret"),
},
{.name = "set-address",
@@ -10119,6 +10122,7 @@ static const vshCmdOptDef opts_qemu_monitor_command[] = {
.type = VSH_OT_ARGV,
.positional = true,
.required = true,
.completer = vshCompleteEmpty,
.help = N_("command")
},
{.name = NULL}
@@ -10503,6 +10507,7 @@ static const vshCmdOptDef opts_qemu_agent_command[] = {
.type = VSH_OT_ARGV,
.positional = true,
.required = true,
.completer = vshCompleteEmpty,
.help = N_("command")
},
{.name = NULL}
@@ -10590,6 +10595,7 @@ static const vshCmdOptDef opts_lxc_enter_namespace[] = {
.type = VSH_OT_ARGV,
.positional = true,
.required = true,
.completer = vshCompleteEmpty,
.help = N_("command to run")
},
{.name = NULL}
+1
View File
@@ -325,6 +325,7 @@ static const vshCmdOptDef opts_network_desc[] = {
{.name = "new-desc",
.type = VSH_OT_ARGV,
.positional = true,
.completer = vshCompleteEmpty,
.help = N_("message")
},
{.name = NULL}