Re: [virt-manager PATCH] addhardware: generate target only if not customizing

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

 



On Tue, Feb 18, 2014 at 09:41:06AM -0500, Cole Robinson wrote:
> On 02/18/2014 07:27 AM, Martin Kletzander wrote:
> > Commit 078e1a4d0503d98884b5b61df83021941bf32e8d changed the fact that
> > disk target was generated only when adding new disk, not when current
> > disk is being customized, so fix that back.
> >
> > Signed-off-by: Martin Kletzander <mkletzan@xxxxxxxxxx>
> > ---
> >  virtManager/addhardware.py | 6 ++++--
> >  1 file changed, 4 insertions(+), 2 deletions(-)
> >
> > diff --git a/virtManager/addhardware.py b/virtManager/addhardware.py
> > index ecb5fc8..b1f6251 100644
> > --- a/virtManager/addhardware.py
> > +++ b/virtManager/addhardware.py
> > @@ -1471,20 +1471,22 @@ class vmmAddHardware(vmmGObjectUI):
> >              return disk
> >
> >          try:
> > +            used = []
> >              disk.bus = bus
> >              if cache:
> >                  disk.driver_cache = cache
> >
> >              # Generate target
> >              if not self.is_customize_dialog:
> > -                used = []
> >                  disks = (self.vm.get_disk_devices() +
> >                           self.vm.get_disk_devices(inactive=True))
> >                  for d in disks:
> >                      used.append(d.target)
> >
> >              prefer_ctrl = self._set_disk_controller(disk, controller_model, disks)
> > -            disk.generate_target(used, prefer_ctrl)
> > +
> > +            if not self.is_customize_dialog:
> > +                disk.generate_target(used, prefer_ctrl)
> >
> >          except Exception, e:
> >              return self.err.val_err(_("Storage parameter error."), e)
> >
>
> ACK
>

Thanks, pushed.

Martin

Attachment: signature.asc
Description: Digital signature

_______________________________________________
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