+1
On Oct 7, 2010 5:12 PM, "Markus Roberts" <[email protected]> wrote:
> 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]<puppet-dev%[email protected]>
.
> For more options, visit this group at
http://groups.google.com/group/puppet-dev?hl=en.
>

-- 
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.

Reply via email to