[libvirt] [PATCH] virsh: Print error message if argument parsing fails for cmdNodesuspend
Michal Privoznik
mprivozn at redhat.com
Tue Dec 13 14:31:38 UTC 2011
On 13.12.2011 14:53, Peter Krempa wrote:
> If parsing of arguments failed, virsh did silently exit returning and
> error state, but not specifying the possible problem.
>
> * tools/virsh: cmdNodesuspend: - error handling added
> ---
> tools/virsh.c | 12 +++++++++---
> 1 files changed, 9 insertions(+), 3 deletions(-)
>
> diff --git a/tools/virsh.c b/tools/virsh.c
> index d58b827..ebda248 100644
> --- a/tools/virsh.c
> +++ b/tools/virsh.c
> @@ -5270,14 +5270,20 @@ cmdNodeSuspend(vshControl *ctl, const vshCmd *cmd)
> if (!vshConnectionUsability(ctl, ctl->conn))
> return false;
>
> - if (vshCommandOptString(cmd, "target", &target) < 0)
> + if (vshCommandOptString(cmd, "target", &target) < 0) {
> + vshError(ctl, _("Invalid suspend target argument"));
I'd do s/suspend // in all strings you're adding.
ACK with that nit fixed.
Michal
More information about the libvir-list
mailing list