Skip to content

Commit

Permalink
Merge pull request #121 from mitchellen/master
Browse files Browse the repository at this point in the history
added VRA installation parameter for multiple vCPUs
  • Loading branch information
recklessop authored Feb 25, 2022
2 parents a2d724a + 4b1506f commit ccc4118
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 1 deletion.
2 changes: 1 addition & 1 deletion Tests/Public/Install-ZertoVra.Tests.ps1
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ Describe $global:function -Tag 'Unit', 'Source', 'Built' {

Context "$global:function::Parameter Unit Tests" {
It "$global:function should have exactly 24 parameters defined" {
(Get-Command $global:function).Parameters.Count | Should -Be 24
(Get-Command $global:function).Parameters.Count | Should -Be 25
}

$ParameterTestCases = @(
Expand Down
6 changes: 6 additions & 0 deletions ZertoApiWrapper/Public/Install-ZertoVra.ps1
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,11 @@ function Install-ZertoVra {
)]
[ValidateRange(1, 16)]
[int]$memoryInGB = 3,
[Parameter(
HelpMessage = "Initial number of CPUs to assign to the VRA. Default is 1, Minimum is 1, Maximum is 4"
)]
[ValidateRange(1, 4)]
[int]$NumOfCpus = 1,
[Parameter(
HelpMessage = "Bandwidth group to assign to the VRA. If unspecified will assign to the 'default_group'"
)]
Expand Down Expand Up @@ -136,6 +141,7 @@ function Install-ZertoVra {
}
$vraBasic['HostIdentifier'] = $hostIdentifier.toString()
$vraBasic['MemoryInGB'] = $memoryInGB
$vraBasic['NumOfCpus'] = $NumOfCpus
$vraBasic['NetworkIdentifier'] = $networkIdentifier.toString()
$vraBasic['UsePublicKeyInsteadOfCredentials'] = $true
$vraBasicNetwork = [ordered]@{ }
Expand Down

0 comments on commit ccc4118

Please sign in to comment.