Merge pull request AutomatedLab#1034 from astavitsky/fix/processorcount

raandree · web-flow · commit ed3b3f1a0be2 · 2020-12-05T19:34:58.000+01:00
Fix processor count on multi-processor systems
diff --git a/AutomatedLabDefinition/AutomatedLabDefinition.psm1 b/AutomatedLabDefinition/AutomatedLabDefinition.psm1
@@ -2733,7 +2733,7 @@ function Add-LabMachineDefinition
                 }
                 else
                 {
-                    (Get-CimInstance -Namespace Root\CIMv2 -Class win32_processor).NumberOfLogicalProcessors 
+                    (Get-CimInstance -Namespace Root\CIMv2 -Class win32_processor | Measure-Object NumberOfLogicalProcessors -Sum).Sum
                 }
             }
             if ($script:processors -ge 2)
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -22,6 +22,7 @@
 - Fixes issue with DSC Pull Server role validation so that SQL2016 and newer is supported
 - Fixes an issue where SQL2019 sample databases were not installed correctly (#992)
 - Old types in Add / Get-Certificate2 threw many errors, fixed that.
+- Fixed processor count on multi-processor systems.
 
 ## 5.22.0 - 2020-07-10
 

Original file line number	Diff line number	Diff line change
`@@ -2733,7 +2733,7 @@ function Add-LabMachineDefinition`
`2733`	`2733`	`}`
`2734`	`2734`	`else`
`2735`	`2735`	`{`
`2736`		`- (Get-CimInstance -Namespace Root\CIMv2 -Class win32_processor).NumberOfLogicalProcessors`
	`2736`	`+ (Get-CimInstance -Namespace Root\CIMv2 -Class win32_processor \| Measure-Object NumberOfLogicalProcessors -Sum).Sum`
`2737`	`2737`	`}`
`2738`	`2738`	`}`
`2739`	`2739`	`if ($script:processors -ge 2)`