haskal
/
vagrant
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Do not prepend default path because it can override modules. 

It is common for Puppet to manage itself. If the puppet code you are
deploying pushes files to /etc/puppet/modules/, then prepending this
path can break deployment because it will override the module path if
the deployment code is changing. There is no good reason to include this
path. Puppet has built in defaults for this reason.
This commit is contained in:
James Shubin 2013-12-17 14:29:54 -05:00
parent edb59ebeff
commit d4c76d1bcf
1 changed files with 0 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
plugins/provisioners/puppet/provisioner/puppet.rb
View File

@ -98,9 +98,6 @@ module VagrantPlugins
options = [config.options].flatten options = [config.options].flatten
module_paths = @module_paths.map { |_, to| to } module_paths = @module_paths.map { |_, to| to }
if !@module_paths.empty? if !@module_paths.empty?
# Prepend the default module path
module_paths.unshift("/etc/puppet/modules")
# Add the command line switch to add the module path # Add the command line switch to add the module path
options << "--modulepath '#{module_paths.join(':')}'" options << "--modulepath '#{module_paths.join(':')}'"
end end