diff options
| author | Micah <micah@leap.se> | 2016-07-12 16:46:08 -0400 | 
|---|---|---|
| committer | Micah <micah@leap.se> | 2016-07-12 16:46:08 -0400 | 
| commit | 40ea2656f072e23bbbccd22c39fb29a36390fa3a (patch) | |
| tree | 48d3b759d30e95a25577548b46df37db0cd0d5d3 /puppet/modules/stdlib/README_DEVELOPER.markdown | |
| parent | 95374aacb857ed35c2fdfe6be7c0bfab86653963 (diff) | |
git subrepo clone https://leap.se/git/puppet_stdlib puppet/modules/stdlib
subrepo:
  subdir:   "puppet/modules/stdlib"
  merged:   "7112363"
upstream:
  origin:   "https://leap.se/git/puppet_stdlib"
  branch:   "master"
  commit:   "7112363"
git-subrepo:
  version:  "0.3.0"
  origin:   "https://github.com/ingydotnet/git-subrepo"
  commit:   "1e79595"
Change-Id: I032e3e7c2984bf53b717373df495c039bb6f41b3
Diffstat (limited to 'puppet/modules/stdlib/README_DEVELOPER.markdown')
| -rw-r--r-- | puppet/modules/stdlib/README_DEVELOPER.markdown | 35 | 
1 files changed, 35 insertions, 0 deletions
| diff --git a/puppet/modules/stdlib/README_DEVELOPER.markdown b/puppet/modules/stdlib/README_DEVELOPER.markdown new file mode 100644 index 00000000..04349ed7 --- /dev/null +++ b/puppet/modules/stdlib/README_DEVELOPER.markdown @@ -0,0 +1,35 @@ +Puppet Specific Facts +===================== + +Facter is meant to stand alone and apart from Puppet.  However, Facter often +runs inside Puppet and all custom facts included in the stdlib module will +almost always be evaluated in the context of Puppet and Facter working +together. + +Still, we don't want to write custom facts that blow up in the users face if +Puppet is not loaded in memory.  This is often the case if the user runs +`facter` without also supplying the `--puppet` flag. + +Ah! But Jeff, the custom fact won't be in the `$LOAD_PATH` unless the user +supplies `--facter`! You might say... + +Not (always) true I say!  If the user happens to have a CWD of +`<modulepath>/stdlib/lib` then the facts will automatically be evaluated and +blow up. + +In any event, it's pretty easy to write a fact that has no value if Puppet is +not loaded.  Simply do it like this: + +    Facter.add(:node_vardir) do +      setcode do +        # This will be nil if Puppet is not available. +        Facter::Util::PuppetSettings.with_puppet do +          Puppet[:vardir] +        end +      end +    end + +The `Facter::Util::PuppetSettings.with_puppet` method accepts a block and +yields to it only if the Puppet library is loaded.  If the Puppet library is +not loaded, then the method silently returns `nil` which Facter interprets as +an undefined fact value.  The net effect is that the fact won't be set. | 
