On Thu, May 2, 2013 at 11:06 AM, Helga Velroyen <[email protected]> wrote:
> Currently, when creating an instance with file storage fails,
> the error message is rather general. This patch adds a hint
> on what to do as a possible fix.
>
> Signed-off-by: Helga Velroyen <[email protected]>
> ---
>  lib/bdev.py | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/lib/bdev.py b/lib/bdev.py
> index 84fa49d..9e93f56 100644
> --- a/lib/bdev.py
> +++ b/lib/bdev.py
> @@ -184,7 +184,9 @@ def _CheckFileStoragePath(path, allowed):
>        break
>    else:
>      raise errors.FileStoragePathError("Path '%s' is not acceptable for file"
> -                                      " storage" % path)
> +                                      " storage. A possible fix might be to 
> add"
> +                                      " it to 
> /etc/ganeti/file-storage-paths."
> +                                      % path)

Add "on all nodes", then LGTM

Thanks,

Guido

Reply via email to