Issue #7996 has been updated by Adrien Thebo. Status changed from In Topic Branch Pending Merge to Merged - Pending Release
Merged in commit:498d78fdc683f13c80c575a8732b461dc86508b6 In the case that these facts are run on a system that doesn't have the necessary binaries, the fact will fail resolution and will return nil, so solaris 8 and 9 should be fine as-is. Thanks for the heads up on that point though. ---------------------------------------- Feature #7996: Solaris 10/11 fact for physicalprocessorcount and ProcessorCount https://projects.puppetlabs.com/issues/7996 Author: Merritt Krakowitzer Status: Merged - Pending Release Priority: Normal Assignee: Adrien Thebo Category: library Target version: 1.6.x Keywords: Branch: https://github.com/adrienthebo/facter/tree/ticket/master/7996-add_solaris_processor_facts Affected Facter version: As soon as I figure out how to commit this :) <pre> if Facter.value(:kernel) == "SunOS" Facter.add("ProcessorCount") do confine :operatingsystem => :solaris confine :virtual => :physical setcode do Facter::Util::Resolution.exec("/usr/bin/kstat cpu_info | /usr/bin/grep core_id | /usr/bin/uniq | /usr/bin/wc -l") end end end if Facter.value(:kernel) == "SunOS" Facter.add("physicalprocessorcount") do confine :operatingsystem => :solaris confine :virtual => :physical setcode do Facter::Util::Resolution.exec("/usr/sbin/psrinfo -p") end end end </pre> -- 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.
