Re: [virt-manager PATCH v2] details: Fix showing the firmware type in case of firmware auto selection

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

 



On 1/14/20 9:34 PM, Lin Ma wrote:
> For a shutoff VM, If user select uefi firmware auto selection, e.g.
> <os firmware='efi'>
> ...
> </os>
> 
> Its firmware information is set to 'BIOS' in details, This is incorrect.
> This fixes it.
> 
> Signed-off-by: Lin Ma <lma@xxxxxxxx>
> ---
>  virtManager/details/details.py | 7 +++++--
>  1 file changed, 5 insertions(+), 2 deletions(-)
> 
> diff --git a/virtManager/details/details.py b/virtManager/details/details.py
> index dc42e35f..339924ff 100644
> --- a/virtManager/details/details.py
> +++ b/virtManager/details/details.py
> @@ -2027,8 +2027,11 @@ class vmmDetails(vmmGObjectUI):
>  
>          # Firmware
>          domcaps = self.vm.get_domain_capabilities()
> -        firmware = domcaps.label_for_firmware_path(
> -            self.vm.get_xmlobj().os.loader)
> +        if self.vm.get_xmlobj().is_uefi():
> +            firmware = 'UEFI'
> +        else:
> +            firmware = domcaps.label_for_firmware_path(
> +                self.vm.get_xmlobj().os.loader)
>          if self.widget("overview-firmware").is_visible():
>              uiutil.set_list_selection(
>                  self.widget("overview-firmware"), firmware)
> 

Reviewed-by: Cole Robinson <crobinso@xxxxxxxxxx>

and pushed

Thanks,
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