[virt-tools-list] [virt-manager PATCH] create: report an error if storage doesn't exists for import installation

Cole Robinson crobinso at redhat.com
Tue Jun 7 15:10:02 UTC 2016


On 06/07/2016 10:21 AM, Pavel Hrdina wrote:
> Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1305210
> 
> Signed-off-by: Pavel Hrdina <phrdina at redhat.com>
> ---
>  virtManager/create.py | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/virtManager/create.py b/virtManager/create.py
> index f4f9d05..d03e949 100644
> --- a/virtManager/create.py
> +++ b/virtManager/create.py
> @@ -1778,6 +1778,12 @@ class vmmCreate(vmmGObjectUI):
>                  return self.err.val_err(
>                                  _("A storage path to import is required."))
>  
> +            if not virtinst.VirtualDisk.path_definitely_exists(
> +                                                self.conn.get_backend(),
> +                                                import_path):
> +                return self.err.val_err(_("A storage path must point to "
> +                                          "an existing storage."))
> +
>          elif instmethod == INSTALL_PAGE_CONTAINER_APP:
>              instclass = virtinst.ContainerInstaller
>  
> 

Looks like this patch was resent, and the intended 1/2 patch was missed?

Thanks,
Cole




More information about the virt-tools-list mailing list