The Puppet Labs Issue Tracker has Moved: https://tickets.puppetlabs.com

This issue tracker is now in read-only archive mode and automatic ticket export has been disabled. Redmine users will need to create a new JIRA account to file tickets using https://tickets.puppetlabs.com. See the following page for information on filing tickets with JIRA:

Bug #18215

processorcount counting CPU cores on Solaris

Added by Anonymous over 3 years ago. Updated over 2 years ago.

Status:ClosedStart date:
Priority:NormalDue date:
Assignee:-% Done:

0%

Category:solaris
Target version:1.7.4
Keywords: Affected Facter version:
Branch:https://github.com/puppetlabs/facter/pull/438

We've Moved!

Ticket tracking is now hosted in JIRA: https://tickets.puppetlabs.com


Description

The processorcount fact on Solaris is counting CPU cores –

myhost# facter |grep proc
physicalprocessorcount => 1
processor0 => SPARC64-VII
processor1 => SPARC64-VII
processor2 => SPARC64-VII
processor3 => SPARC64-VII
processor4 => SPARC64-VII
processor5 => SPARC64-VII
processor6 => SPARC64-VII
processor7 => SPARC64-VII
processorcount => 4
  • but on linux is counting CPU threads –
myotherhost# facter |grep proc
physicalprocessorcount => 1
processor0 => Intel(R) Core(TM) i7-2600 CPU @ 3.40GHz
processor1 => Intel(R) Core(TM) i7-2600 CPU @ 3.40GHz
processor2 => Intel(R) Core(TM) i7-2600 CPU @ 3.40GHz
processor3 => Intel(R) Core(TM) i7-2600 CPU @ 3.40GHz
processor4 => Intel(R) Core(TM) i7-2600 CPU @ 3.40GHz
processor5 => Intel(R) Core(TM) i7-2600 CPU @ 3.40GHz
processor6 => Intel(R) Core(TM) i7-2600 CPU @ 3.40GHz
processor7 => Intel(R) Core(TM) i7-2600 CPU @ 3.40GHz
processorcount => 8

This is a single 4-core CPU with 8 threads similar to the Solaris example above.

A patch here should also provide a processorcorecount fact that preserves the existing behaviour of processorcount on Solaris.

See discussions in puppet users https://groups.google.com/forum/?fromgroups=&pli=1#!topic/puppet-users/rOj9OszhlQM

And puppet developers https://groups.google.com/forum/?fromgroups=&pli=1#!topic/puppet-dev/payc4ToNcGQ

History

#1 Updated by Adrien Thebo almost 3 years ago

  • Status changed from Unreviewed to In Topic Branch Pending Review
  • Assignee deleted (Anonymous)
  • Branch set to https://github.com/puppetlabs/facter/pull/438

#2 Updated by Anonymous almost 3 years ago

  • Status changed from In Topic Branch Pending Review to Merged - Pending Release

#3 Updated by Josh Cooper over 2 years ago

This was backported to stable in commit a6d12a5 to be released in 1.7.4

#4 Updated by Josh Cooper over 2 years ago

  • Target version set to 1.7.4

#5 Updated by Melissa Stone over 2 years ago

  • Status changed from Merged - Pending Release to Closed

Release in Facter 1.7.4-rc1

Also available in: Atom PDF