Re: [PATCH virt-manager 2/2] virt-manager: disable CD/DVDROM passtrough on RHEL

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

 



On Wed, Mar 19, 2014 at 04:36:03PM +0100, Giuseppe Scrivano wrote:
> Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1072610
>
> Signed-off-by: Giuseppe Scrivano <gscrivan@xxxxxxxxxx>
> ---
>  virtManager/choosecd.py | 6 +++++-
>  virtManager/create.py   | 5 ++++-
>  2 files changed, 9 insertions(+), 2 deletions(-)
>
> diff --git a/virtManager/choosecd.py b/virtManager/choosecd.py
> index aa01ad2..8ce0b55 100644
> --- a/virtManager/choosecd.py
> +++ b/virtManager/choosecd.py
> @@ -97,7 +97,11 @@ class vmmChooseCD(vmmGObjectUI):
>
>      def reset_state(self):
>          self.mediacombo.reset_state()
> -        use_cdrom = (self.mediacombo.has_media())
> +
> +        disable_local_cdrom = self.vm.stable_defaults()
> +        self.widget("physical-media").set_sensitive(not disable_local_cdrom)
> +
> +        use_cdrom = (self.mediacombo.has_media()) and not disable_local_cdrom
>
>          self.widget("physical-media").set_active(use_cdrom)
>          self.widget("iso-image").set_active(not use_cdrom)
> diff --git a/virtManager/create.py b/virtManager/create.py
> index 757d700..9adafc3 100644
> --- a/virtManager/create.py
> +++ b/virtManager/create.py
> @@ -550,9 +550,12 @@ class vmmCreate(vmmGObjectUI):
>          cdrom_option.set_active(self.mediacombo.has_media())
>          iso_option.set_active(not self.mediacombo.has_media())
>
> +        disable_local_cdrom = self._stable_defaults()
> +        cdrom_option.set_sensitive(not disable_local_cdrom)
> +
>          # Only allow ISO option for remote VM
>          is_local = not self.conn.is_remote()
> -        if not is_local:
> +        if not is_local or disable_local_cdrom:

I haven't checked whether it's used on more places but this bit is
adding some obscurity similar to what I wanted to fix with
_stable_defaults (local = not remote; if not local...) :)

>              iso_option.set_active(True)
>
>          self.toggle_local_cdrom(cdrom_option)
> --
> 1.8.5.3
>
> _______________________________________________
> virt-tools-list mailing list
> virt-tools-list@xxxxxxxxxx
> https://www.redhat.com/mailman/listinfo/virt-tools-list

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