Cole Robinson <[email protected]> writes:

> We expose a simple combobox with two entries: BIOS, and UEFI. The
> UEFI option is only selectable if 1) libvirt supports the necessary
> domcapabilities bits, 2) it detects that qemu supports the necessary
> command line options, and 3) libvirt detects a UEFI binary on the
> host that maps to a known template via qemu.conf
>
> If those conditions aren't met, we disable the UEFI option, and show
> a small warning icon with an explanatory tooltip.
>
> The option can only be changed via New VM->Customize Before Install.
> For existing x86 VMs, it's a readonly label.
> ---
>  ui/details.ui          | 84 
> ++++++++++++++++++++++++++++++++++++++++++++++----
>  virtManager/details.py | 72 ++++++++++++++++++++++++++++++++++++++++++-
>  virtManager/domain.py  | 32 ++++++++++++++++++-
>  virtinst/support.py    |  3 ++
>  4 files changed, 183 insertions(+), 8 deletions(-)

one problem that I've found is that I cannot delete a VM with UEFI as I
get this error:

libvirtError: Requested operation is not valid: cannot delete inactive domain 
with nvram

since virt-manager first stops a running VM and then undefine it.


also, since we use VIR_DOMAIN_UNDEFINE_NVRAM (commit
ead9d3f56dfb32411dac586e6d43dca23d2377f0), should we revert?


commit e5d2059853a472273b91cdefd224381978249b65
Author: Giuseppe Scrivano <[email protected]>
Date:   Thu Sep 11 17:13:31 2014 +0200

    virt-manager: delete nvram file on VM deletion

IMHO, it will make more sense to not use VIR_DOMAIN_UNDEFINE_NVRAM and
use the same mechanism we use for other files virt-manager deletes by
itself.  What do you think?

Regards,
Giuseppe

_______________________________________________
virt-tools-list mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/virt-tools-list

Reply via email to