Re: [virt-manager PATCH] details: do not show MAC address if not set

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

 




> -----Original Message-----
> From: virt-tools-list-bounces@xxxxxxxxxx
> [mailto:virt-tools-list-bounces@xxxxxxxxxx] On Behalf Of Giuseppe Scrivano
> Sent: Friday, November 14, 2014 4:51 PM
> To: virt-tools-list@xxxxxxxxxx
> Subject:  [virt-manager PATCH] details: do not show MAC address
> if not set
> 
> Fixes this exception:
> Traceback (most recent call last):
>   File "virt-manager/virtManager/create.py", line 1780, in finish
>     self.customize(guest)
>   File "virt-manager/virtManager/create.py", line 1812, in customize
>     self.config_window = vmmDetails(virtinst_guest, self.topwin)
>   File "virt-manager/virtManager/details.py", line 560, in __init__
>     self.populate_hw_list()
>   File "virt-manager/virtManager/details.py", line 3165, in populate_hw_list
>     self.repopulate_hw_list()
>   File "virt-manager/virtManager/details.py", line 3220, in repopulate_hw_list
>     update_hwlist(HW_LIST_TYPE_NIC, dev)
>   File "virt-manager/virtManager/details.py", line 3195, in update_hwlist
>     label = _label_for_device(dev)
>   File "virt-manager/virtManager/details.py", line 188, in _label_for_device
>     return "NIC %s" % dev.macaddr[-9:]
> TypeError: 'NoneType' object has no attribute '__getitem__'
> 
> Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1164123
> 
> Signed-off-by: Giuseppe Scrivano <gscrivan@xxxxxxxxxx>
> ---

ACK.

Thanks,
- Chen

_______________________________________________
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