Parallel MOF combilation

gaelcolas / DscInfraSample

A Control Repo for a DSC Infrastructure as Code

MIT License

47 stars 22 forks source link

Parallel MOF combilation #16

Closed raandree closed 6 years ago

raandree commented 6 years ago

Adding support for parallel MOF compilation. The number if jobs can be defined with parameter 'MofCompilationTaskCount'. If this parameter is not used, standard serial build is used
- Starting more than 2 processes loading the same modules can cause issues. This is why the 'RandomWait' parameter is required in parallel build to let the new processes wait to a random number of seconds.
Adding support for parallel Meta MOF compilation.
Throwing if the compilation does not succeed so InvokeBuild get the failure status
Removing 'FilterNode' and 'FilteredNode' references as it is not used and redundant with 'Filter'.
Simple code reorg and code style alignments

The MOF build still works successful.

gaelcolas commented 6 years ago

Failed build? Flying today, won't have time to look.

raandree commented 6 years ago

Working now, there was still a case sensitivity issue with the roles in the config files.

raandree commented 6 years ago

@gaelcolas, any comments on this PR?

raandree commented 6 years ago

Right...

raandree commented 6 years ago

This took quite some time:

PSDesiredStateConfiguration\node : Specified argument was out of the range of valid values.
Parameter name: times
At C:\Git\Aldi DSC\Operations\RootConfiguration.ps1:16 char:5
+     node $ConfigurationData.AllNodes.NodeName {
+     ~~~~
    + CategoryInfo          : OperationStopped: (:) [PSDesiredStateConfiguration\node], ArgumentOutOfRangeException
    + FullyQualifiedErrorId : System.ArgumentOutOfRangeException,PSDesiredStateConfiguration\node

The error was actually caused in RootConfiguration.ps1::51 if "55 -$okMessage.Length" results in a negative number. Fix is on the way...

raandree commented 6 years ago

Ping...

gaelcolas commented 6 years ago

All good and tested (finally). Merged