Skip to content

Commit 8ccbb68

Browse files
author
David Moreau Simard
committed
Fix boolean that always returned true for selinux fact
Facter returns 'false' if selinux is not enabled. Puppet evaluates it as true as it is not empty. Change-Id: I2f1d9a348fd981292aeef177b57f08e1b7ef2732 Closes-bug: #1328547 (cherry picked from commit 6d35b1e)
1 parent 03c465f commit 8ccbb68

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

manifests/storage/mount.pp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@
6262
# would be possible as context is. But, as there already is
6363
# chown command we'll just restorecon on selinux enabled
6464
# systems :(
65-
if ($::selinux) {
65+
if ($::selinux == 'true') {
6666
exec { "restorecon_mount_${name}":
6767
command => "restorecon ${mnt_base_dir}/${name}",
6868
path => ['/usr/sbin', '/bin'],

spec/defines/swift_storage_mount_spec.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@
4848

4949
let :facts do
5050
{
51-
:selinux => true,
51+
:selinux => 'true',
5252
}
5353
end
5454

0 commit comments

Comments
 (0)