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

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

 



On Tue, Jun 07, 2016 at 01:29:19PM -0400, Cole Robinson wrote:
> On 06/07/2016 07:58 AM, Pavel Hrdina wrote:
> > Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1305210
> > 
> > Signed-off-by: Pavel Hrdina <phrdina@xxxxxxxxxx>
> > ---
> >  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."))
> > +
> 
> Maybe change that to 'The import path must...'
> 
> ACK

Thanks, pushed.

Pavel

_______________________________________________
virt-tools-list mailing list
virt-tools-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/virt-tools-list



[Index of Archives]     [Linux Virtualization]     [KVM Development]     [CentOS Virtualization]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]     [Video 4 Linux]

  Powered by Linux