Joey Boggs
2009-Feb-25 20:48 UTC
[Ovirt-devel] [PATCH server] remove selinux disable, selinux is handled in the installer configurator now, required libvirt, but not disable it
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
Jason Guiditta
2009-Mar-04 15:54 UTC
[Ovirt-devel] [PATCH server] remove selinux disable, selinux is handled in the installer configurator now, required libvirt, but not disable it
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>
Joey Boggs
2009-Mar-04 16:04 UTC
[Ovirt-devel] Re: [PATCH server] remove selinux disable, selinux is handled in the installer configurator now, required libvirt, but not disable it
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