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(a)gmail.com> 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