Skip to content

Commit e1624d2

Browse files
Update ValidateScenario to cater for AV Set resource IDs different casing .psm1 (#146)
If the Resource ID of Availability sets are in different cases the script fails as it thinks they are different resources. This is due to the use of Get-unique being case sensitive. The fix is to convert the resource id to lower before running get-unique
1 parent a6bab55 commit e1624d2

File tree

1 file changed

+4
-2
lines changed
  • AzureBasicLoadBalancerUpgrade/module/AzureBasicLoadBalancerUpgrade/modules/ValidateScenario

1 file changed

+4
-2
lines changed

AzureBasicLoadBalancerUpgrade/module/AzureBasicLoadBalancerUpgrade/modules/ValidateScenario/ValidateScenario.psm1

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -736,7 +736,9 @@ Function Test-SupportedMultiLBScenario {
736736
ForEach ($config in $multiLBConfig) {
737737
$basicLBBackends += $config.BasicLoadBalancer.BackendAddressPools.BackendIpConfigurations.id | ForEach-Object { $_.split('/virtualMachines/')[0] }
738738
}
739-
$groupedBackends = $basicLBBackends | Sort-Object | Get-Unique
739+
740+
$groupedBackends = $basicLBBackends | ForEach-Object { $_.id.ToLower() } | Sort-Object | Get-Unique
741+
740742

741743
If ($groupedBackends.Count -gt 1) {
742744
log -Severity Error -Message "[Test-SupportedMultiLBScenario] The provided Basic Load Balancers do not share backend pool members (more than one backend VMSS found: '$($groupedBackends)'). Using -multiLBConfig when backend is not shared adds risk and complexity in recovery." -terminateOnError
@@ -807,4 +809,4 @@ Function Test-SupportedMultiLBScenario {
807809
log -Message "[Test-SupportedMultiLBScenario] Multi-LB configuration is valid for migration"
808810
}
809811

810-
Export-ModuleMember -Function Test-SupportedMigrationScenario, _GetScenarioBackendType, Test-SupportedMultiLBScenario
812+
Export-ModuleMember -Function Test-SupportedMigrationScenario, _GetScenarioBackendType, Test-SupportedMultiLBScenario

0 commit comments

Comments
 (0)