[Ovirt-devel] Re: [PATCH server] remove selinux disable, selinux is handled in the installer configurator now, required libvirt, but not disable it

Joey Boggs jboggs at redhat.com
Wed Mar 4 16:04:12 UTC 2009


Joey Boggs wrote:
> This removes an unnecessary selinux disable in the installer, the configuration now handles selinux and sets to permissive. Also libvirt was being disabled, we should not care if it's running or not. dnsmasq still works with it enabled.
>
> ---
>  installer/modules/ovirt/manifests/ovirt.pp |    8 +-------
>  1 files changed, 1 insertions(+), 7 deletions(-)
>
> diff --git a/installer/modules/ovirt/manifests/ovirt.pp b/installer/modules/ovirt/manifests/ovirt.pp
> index 960f4db..e177015 100644
> --- a/installer/modules/ovirt/manifests/ovirt.pp
> +++ b/installer/modules/ovirt/manifests/ovirt.pp
> @@ -109,11 +109,6 @@ class ovirt::setup {
>  		notify => Service[qpidd]
>  	}
>  
> -	exec { "disable_selinux" :
> -		command => "/usr/sbin/lokkit --selinux=disabled",
> -        require => Package["ovirt-server"]
> -	}
> -
>  	service {"httpd" :
>                  enable => true,
>                  require => Package[httpd],
> @@ -121,8 +116,7 @@ class ovirt::setup {
>          }
>  
>  	service {"libvirt" :
> -                enable => false,
> -                require => Package[libvirt],
> +                require => Package[libvirt]
>          }
>  
>          service {"ovirt-host-browser" :
>   
pushed




More information about the ovirt-devel mailing list