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

[Martin Kletzander]

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 at redhat.com>
> ---
>  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 at redhat.com
> https://www.redhat.com/mailman/listinfo/virt-tools-list
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/virt-tools-list/attachments/20140319/1449199c/attachment.sig>