Merge pull request #10 from dodinh/master

For rm commands, supply path instead of hardcoding one location
walkamongus · Aug 29, 2016 · 8d1a3ad · 8d1a3ad
2 parents 5d1b56d + a0f83ec
commit 8d1a3ad
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/manifests/sssd/config.pp b/manifests/sssd/config.pp
@@ -16,7 +16,8 @@
     }
 
     exec { 'force_config_cache_rebuild':
-      command     => "/usr/bin/rm -f ${::realmd::sssd_config_cache_file}",
+      path        => '/usr/bin:/usr/sbin:/bin',
+      command     => "rm -f ${::realmd::sssd_config_cache_file}",
       refreshonly => true,
     }
   }

diff --git a/spec/classes/sssd__config_spec.rb b/spec/classes/sssd__config_spec.rb
@@ -44,7 +44,8 @@
 
           it do
             should contain_exec('force_config_cache_rebuild').with({
-              'command'     => '/usr/bin/rm -f /var/lib/sss/db/config.ldb',
+              'path'        => '/usr/bin:/usr/sbin:/bin',
+              'command'     => 'rm -f /var/lib/sss/db/config.ldb',
               'refreshonly' => true,
             })
           end