Issue #5410 has been updated by Garrett Honeycutt.
Here is another use case where facter does the wrong thing by returning zero.
Created a simple custom fact with the following code
<pre>
require 'facter'
Facter.add("role") do
setcode do
%x{cat /etc/role}.chomp
end
end
</pre>
but did not create the /etc/role file
<pre>
# facter role
cat: /etc/role: No such file or directory
cat: /etc/role: No such file or directory
# echo $?
0
</pre>
----------------------------------------
Bug #5410: facter does not use return codes
https://projects.puppetlabs.com/issues/5410
Author: Garrett Honeycutt
Status: Needs design decision
Priority: Normal
Assignee: Paul Nasrat
Category: binary
Target version:
Keywords:
Branch:
It appears that facter always returns zero
Reproduction:
<pre>
# facter operatingsystem
CentOS
# echo $?
0
# facter thisfactdoesnotexist
# echo $?
0
</pre>
Supposing that the fact 'thisfactdoesnotexist' is not a valid fact name, the
return code should not be zero.
--
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.
