Issue #17156 has been updated by Andrew  Parker.

Strangely enough, `puppet doc` should already be running in the master mode: 
<https://github.com/puppetlabs/puppet/blob/master/lib/puppet/application/doc.rb#L4>.
 However, on the line right above that it specifies that it shouldn't parse 
configs. Down near the bottom of the file it tries to do some fancy footwork to 
parse things in its own special way. I have no idea what this all achieves in 
the end.

What you are saying makes perfect sense, but I don't fully understand what it 
is doing right now so I'm not clear how far off it currently is. It does look 
like it never looks at the libdir, but instead only really cares about the 
modulepath.
----------------------------------------
Bug #17156: Regression: Can't restrict the plugins loaded by puppet doc with 
--modulepath in 3.0.1
https://projects.puppetlabs.com/issues/17156#change-75000

Author: Nick Fagerlund
Status: Accepted
Priority: Normal
Assignee: 
Category: settings
Target version: 3.x
Affected Puppet version: 3.0.1
Keywords: regression settings
Branch: 


The docs team uses `puppet doc --reference` to generate certain docs for the 
site, and we have to use `--modulepath /tmp/nothing` or something to exclude 
non-core resource types and functions from our personal modules. This worked 
through 3.0.0 and broke in 3.0.1. 

The changed behavior:

* In 3.0.0, puppet doc would ONLY load plugins from the modulepath; the 
contents of the agent's libdir had no effect, and changing the modulepath 
swapped out the entire set of plugins available.
* In 3.0.1, puppet doc loads plugins from both the modulepath AND the libdir. 
So if puppet agent has ever run, puppet doc will keep loading plugins until you 
delete the libdir. 

Git bisect tracks the change to commit:144bb76a7e825a221fd1cb5a20728c8cf5c03264 
-- "(#16637) Fix default confdir and vardir with puppet master as normal user." 

The command to test this with is `puppet doc --reference function --modulepath 
/tmp/nothing` -- make sure you have something like puppetlabs-stdlib in your 
modulepath when testing, so that the function reference gets nice and messy.


-- 
You have received this notification because you have either subscribed to it, 
or are involved in it.
To change your notification preferences, please click here: 
http://projects.puppetlabs.com/my/account

-- 
You received this message because you are subscribed to the Google Groups 
"Puppet Bugs" 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-bugs?hl=en.

Reply via email to