Re: [PATCH] storage: Fix error message on vol-create-from failure

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

 



Sorry, please ignore this. The title should be 'virsh: ' not 'storage: '.
I'll post another one soon.

  ozaki-r

On Mon, Oct 24, 2011 at 8:18 PM, Ryota Ozaki <ozaki.ryota@xxxxxxxxx> wrote:
> If vol-create-from is failed due to 'input volume not found',
> virsh outputs like this:
>  $ sudo virsh vol-create-from testpool test-vol.xml test.img
>  error: failed to get vol 'test.img', specifying --pool might help
>  error: Storage volume not found: no storage vol with matching path
> However, '--pool' is incorrect because it is already specified as
> second argument ('testpool' in this case). It should be "--inputpool".
>
> The patch fixes this by using pooloptname, which will be "inputpool"
> in this case and "pool" in other cases, as error message.
> ---
>  tools/virsh.c |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tools/virsh.c b/tools/virsh.c
> index 72344f0..5544a41 100644
> --- a/tools/virsh.c
> +++ b/tools/virsh.c
> @@ -15216,8 +15216,8 @@ vshCommandOptVolBy(vshControl *ctl, const vshCmd *cmd,
>         if (pool)
>             vshError(ctl, _("failed to get vol '%s'"), n);
>         else
> -            vshError(ctl, _("failed to get vol '%s', specifying --pool "
> -                            "might help"), n);
> +            vshError(ctl, _("failed to get vol '%s', specifying --%s "
> +                            "might help"), n, pooloptname);
>     }
>
>     if (pool)
> --
> 1.7.4.1
>
>

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