Skip to content

(16468) creates a fact for every Puppet config setting#315

Closed
ghoneycutt wants to merge 1 commit intopuppetlabs:masterfrom
ghoneycutt:tickets/master/16468_facts_for_all_puppet_config_settings
Closed

(16468) creates a fact for every Puppet config setting#315
ghoneycutt wants to merge 1 commit intopuppetlabs:masterfrom
ghoneycutt:tickets/master/16468_facts_for_all_puppet_config_settings

Conversation

@ghoneycutt
Copy link

This creates a fact prepended with 'puppet_config_' for every Puppet
configuration setting.

This is useful for templating, as well as conditional logic based on
these settings.

** Craig Dunn authored this with me.

This creates a fact prepended with 'puppet_config_' for every Puppet
configuration setting.

This is useful for templating, as well as conditional logic based on
these settings.

** Craig Dunn authored this with me.
@zaphod42
Copy link

What happens if puppet isn't installed? It seems like something like this should be in a module and then accessed with facter -p.

@jeffmccune
Copy link

Next actions are to re-work this as a Puppet module instead of a change to Facter. Facter does not, and should not, have a dependency on Puppet, however, Puppet does and should have a dependency on Facter.

I'm going to go ahead and close this pull request for the time being. Please re-open this pull request once the next actions are addressed, new information is available, or you have a question related to this pull request. We've become aware of difficulties re-opening pull requests, in the event you cannot please mention jeffmccune or adrienthebo with an @ sign in front and we'll re-open this pull request.

Closing the pull request doesn't mean we don't consider this change valuable, just that there are things that need to be addressed before it can be merged. If you have any questions or concerns, please don't hesitate to ping us in #puppet-dev on irc.freenode.net.

@jeffmccune jeffmccune closed this Mar 4, 2013
whopper pushed a commit to whopper/facter that referenced this pull request Mar 18, 2015
(CFACT-230) Fix architecture fact resolution on certain Linux distros.
florindragos pushed a commit that referenced this pull request Jun 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants