Volumes that don't suport SELinux should be considered in_sync so they don't generate spurious change notice.
Patch from Darrell Fuhriman Signed-off-by: Markus Roberts <[email protected]> --- lib/puppet/type/file/selcontext.rb | 8 ++++++-- 1 files changed, 6 insertions(+), 2 deletions(-) diff --git a/lib/puppet/type/file/selcontext.rb b/lib/puppet/type/file/selcontext.rb index a33c6a0..df36d6b 100644 --- a/lib/puppet/type/file/selcontext.rb +++ b/lib/puppet/type/file/selcontext.rb @@ -43,9 +43,13 @@ module Puppet def insync?(value) if not selinux_support? debug("SELinux bindings not found. Ignoring parameter.") - return true + true + elsif not selinux_label_support?(@resource[:path]) + debug("SELinux not available for this filesystem. Ignoring parameter.") + true + else + super end - super end def sync -- 1.7.0.4 -- You received this message because you are subscribed to the Google Groups "Puppet Developers" 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-dev?hl=en.
