[Ovirt-devel] [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 Feb 25 20:48:48 UTC 2009


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




More information about the ovirt-devel mailing list