Issue #9054 has been updated by Sean Millichamp.
Franz, I have pushed what I have to https://github.com/seanmil/puppet/tree/selinux/bug/9054_selboolean_fix I took a stab at putting some tests together for it, but my RSpec-fu is too weak for the amount of time I have/had available. I have a feeling that the selboolean type could be rewritten a lot cleaner, but for now I think the fixes in the above branch should address the reported issue. It has been a number of months since I originally hacked on it, but I did some limited testing on systems with SELinux installed but disabled and it seemed to address the issue. If the original reporters want to give the fix a spin that would be great. Thanks! ---------------------------------------- Bug #9054: selboolean reports error if selinux is not enabled https://projects.puppetlabs.com/issues/9054 Author: Darrell Fuhriman Status: Code Insufficient Priority: Normal Assignee: Sean Millichamp Category: SELinux Target version: Affected Puppet version: Keywords: Branch: If an selboolean is defined for a class, but is executing on a host that does not have SELinux enabled, an error is reported. e.g.: Execution of '/usr/sbin/getsebool httpd_can_network_connect' returned 1: /usr/sbin/getsebool: SELinux is disabled It should not be considered a failure, but perhaps a warning if you try to set/get an SELinux boolean on a host that does not have selinux enabled. Before setting/getting the boolean, a quick call to getenforce could be made, and a warning issued if it returns "Disabled". -- You have received this notification because you have either subscribed to it, or are involved in it. To change your notification preferences, please click here: http://projects.puppetlabs.com/my/account -- You received this message because you are subscribed to the Google Groups "Puppet Bugs" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/puppet-bugs?hl=en.
