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

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

 



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

- Cole

_______________________________________________
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