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

[Jason Guiditta]

ACK, works for me.

On Wed, Feb 25, 2009 at 3:48 PM, Joey Boggs <jboggs at redhat.com> 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" :
> --
> 1.6.0.6
>
> _______________________________________________
> Ovirt-devel mailing list
> Ovirt-devel at redhat.com
> https://www.redhat.com/mailman/listinfo/ovirt-devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/ovirt-devel/attachments/20090304/79bbfeb4/attachment.htm>