Jira (PUP-4359) puppetversion fact is missing

[Eric Thompson (JIRA)]

Title: Message Title

Eric Thompson updated an issue   Puppet / PUP-4359 puppetversion fact is missing Change By: Eric Thompson Executing puppet with cfacter and stdlib generates an error every time puppet runs: {noformat} [root@jaiuwwlivqmy8by ~]# puppet --version 4.0.0-rc1 [root@jaiuwwlivqmy8by ~]# facter --version 2.4.2 [root@jaiuwwlivqmy8by ~]# cfacter --version 0.4.0 (commit f4eecdfa9ff69c801bfedba3fc0fd5285e86b3c5) [root@jaiuwwlivqmy8by ~]# facter puppetversion 4.0.0-rc1 [root@jaiuwwlivqmy8by ~]# cfacter puppetversion [root@jaiuwwlivqmy8by ~]# time puppet agent -t --server $(facter fqdn) --cfacter --trace Info: Retrieving pluginfacts Info: Retrieving plugin Info: Loading facts Error: Facter: error while resolving custom fact "pe_version": undefined method `match' for nil:NilClass backtrace: /opt/puppetlabs/puppet/cache/lib/facter/pe_version.rb:13:in `block (2 levels) in <top (required)>' /opt/puppetlabs/puppet/cache/lib/facter/pe_version.rb:20:in `call' /opt/puppetlabs/puppet/cache/lib/facter/pe_version.rb:20:in `value' /opt/puppetlabs/puppet/cache/lib/facter/pe_version.rb:20:in `block (2 levels) in <top (required)>' /opt/puppetlabs/puppet/lib/ruby/vendor_ruby/puppet/indirector/facts/facter.rb:27:in `call' /opt/puppetlabs/puppet/lib/ruby/vendor_ruby/puppet/indirector/facts/facter.rb:27:in `to_hash' /opt/puppetlabs/puppet/lib/ruby/vendor_ruby/puppet/indirector/facts/facter.rb:27:in `find' /opt/puppetlabs/puppet/lib/ruby/vendor_ruby/puppet/indirector/indirection.rb:194:in `find' /opt/puppetlabs/puppet/lib/ruby/vendor_ruby/puppet/configurer/fact_handler.rb:15:in `find_facts' /opt/puppetlabs/puppet/lib/ruby/vendor_ruby/puppet/configurer/fact_handler.rb:31:in `facts_for_uploading' /opt/puppetlabs/puppet/lib/ruby/vendor_ruby/puppet/configurer.rb:97:in `get_facts' {noformat} The error occurs because cfacter doesn't define the {{puppetversion}} fact, which causes stdlib issues: {code:ruby}     pe_ver = Facter.value("puppetversion").match(/Puppet Enterprise (\d+\.\d+\.\d+)/) {code} It makes sense for cfacter to not generate the {{puppetversion}} fact when run standalone (because we don't want a cyclic dependency). But I would think puppet would inject its {{puppetversion}} into facter (whichever implementation is being used), so that the fact is present during fact resolution and later compilation. So arguably this is a puppet issue, but may affect the facter/cfacter interface? risk assessment: high probability: high (PE requires) severity: high test layer:  acceptance  unit Add Comment   This message was sent by Atlassian JIRA (v6.3.15#6346-sha1:dbc023d) -- You received this message because you are subscribed to the Google Groups "Puppet Bugs" group. To unsubscribe from this group and stop receiving emails from it, send an email to puppet-bugs+unsubscr...@googlegroups.com. To post to this group, send email to puppet-bugs@googlegroups.com. Visit this group at http://groups.google.com/group/puppet-bugs. For more options, visit https://groups.google.com/d/optout.